StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix soong_ui file rlimits for go 1.21

Browse Source 
Go 1.21 modifies the file limit but restores the original when
execing subprocesses if it hasn't be overridden.  Call Setrlimit
even if it doesn't appear to be necessary so that the syscall package
considers it set.

Bug: 309895579
Test: m nothing
Change-Id: I4d0b27bac90a2a88bfc68a8491d54dc106e1ec13
This commit is contained in:
Colin Cross
2023-11-08 22:09:57 -08:00
parent f2fab8347d
commit 611a2fbdd7
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
cmd/soong_ui/main.go
View File

@@ -680,9 +680,11 @@ func setMaxFiles(ctx build.Context) {
}
ctx.Verbosef("Current file limits: %d soft, %d hard", limits.Cur, limits.Max)
if limits.Cur == limits.Max {
return
}
// Go 1.21 modifies the file limit but restores the original when
// execing subprocesses if it hasn't be overridden. Call Setrlimit
// here even if it doesn't appear to be necessary so that the
// syscall package considers it set.
limits.Cur = limits.Max
err = syscall.Setrlimit(syscall.RLIMIT_NOFILE, &limits)