runc-shim: remove misleading comment

laurazard · laurazard · commit 760935e5211d · 2024-09-06T11:01:58.000+01:00
It's not true that `s.mu` needs to be held when calling `handleProcessExit`, and indeed hasn't been the case for a while – see 892dc54. Signed-off-by: Laura Brehm <laurabrehm@hey.com> (cherry picked from commit 7f3bf99) Signed-off-by: Laura Brehm <laurabrehm@hey.com>
diff --git a/runtime/v2/runc/task/service.go b/runtime/v2/runc/task/service.go
@@ -706,7 +706,6 @@ func (s *service) send(evt interface{}) {
 	s.events <- evt
 }
 
-// s.mu must be locked when calling handleProcessExit
 func (s *service) handleProcessExit(e runcC.Exit, c *runc.Container, p process.Process) {
 	if ip, ok := p.(*process.Init); ok {
 		// Ensure all children are killed

Original file line number	Diff line number	Diff line change
`@@ -706,7 +706,6 @@ func (s *service) send(evt interface{}) {`
`706`	`706`	`s.events <- evt`
`707`	`707`	`}`
`708`	`708`
`709`		`-// s.mu must be locked when calling handleProcessExit`
`710`	`709`	`func (s service) handleProcessExit(e runcC.Exit, c runc.Container, p process.Process) {`
`711`	`710`	`if ip, ok := p.(*process.Init); ok {`
`712`	`711`	`// Ensure all children are killed`