Skip to content

Commit

Permalink
fix comments for ClearRlimitNofileCache
Browse files Browse the repository at this point in the history
Signed-off-by: lifubang <[email protected]>
(cherry picked from commit a35f7d8)
Signed-off-by: lifubang <[email protected]>
  • Loading branch information
lifubang committed May 16, 2024
1 parent 99ceba9 commit ed13815
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions libcontainer/system/rlimit_go119.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,14 @@ import (
//go:linkname syscallOrigRlimitNofile syscall.origRlimitNofile
var syscallOrigRlimitNofile atomic.Pointer[syscall.Rlimit]

// As reported in issue #4195, the new version of go runtime(since 1.19)
// will cache rlimit-nofile. Before executing execve, the rlimit-nofile
// of the process will be restored with the cache. In runc, this will
// cause the rlimit-nofile setting by the parent process for the container
// to become invalid. It can be solved by clearing this cache. But
// unfortunately, go stdlib doesn't provide such function, so we need to
// link to the private var `origRlimitNofile` in package syscall to hack.
// ClearRlimitNofileCache is to clear go runtime's nofile rlimit cache.
func ClearRlimitNofileCache() {
// As reported in issue #4195, the new version of go runtime(since 1.19)
// will cache rlimit-nofile. Before executing execve, the rlimit-nofile
// of the process will be restored with the cache. In runc, this will
// cause the rlimit-nofile setting by the parent process for the container
// to become invalid. It can be solved by clearing this cache. But
// unfortunately, go stdlib doesn't provide such function, so we need to
// link to the private var `origRlimitNofile` in package syscall to hack.
syscallOrigRlimitNofile.Store(nil)
}

0 comments on commit ed13815

Please sign in to comment.