Skip to content

Commit

Permalink
libctr: Reset the inherited cpu affinity
Browse files Browse the repository at this point in the history
Old kernels do that automatically, but new kernels remember
the affinity that was set before the cgroup move due to

https://lore.kernel.org/lkml/[email protected]

This is undesirable for containers, because they inherit
the systemd affinity when the should really move to the container
space cpus.

see https://issues.redhat.com/browse/OCPBUGS-15102 for more information

Signed-off-by: Martin Sivak <[email protected]>
Signed-off-by: Peter Hunt <[email protected]>
  • Loading branch information
haircommander committed Sep 28, 2023
1 parent 1614cab commit 22c6427
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions libcontainer/process_linux.go
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,14 @@ func (p *setnsProcess) start() (retErr error) {
}
}
}
// Reset the inherited cpu affinity. Old kernels do that automatically, but
// new kernels remember the affinity that was set before the cgroup move.
// This is undesirable, because it inherits the systemd affinity when the container
// should really move to the container space cpus.
if err = unix.SchedSetaffinity(p.pid(), &unix.CPUSet{}); err != nil && err != unix.EINVAL && err != unix.ENODEV {

Check failure on line 151 in libcontainer/process_linux.go

View workflow job for this annotation

GitHub Actions / lint

comparing with != will fail on wrapped errors. Use errors.Is to check for a specific error (errorlint)
return fmt.Errorf("error resetting pid %d affinity: %w", p.pid(), err)
}

if p.intelRdtPath != "" {
// if Intel RDT "resource control" filesystem path exists
_, err := os.Stat(p.intelRdtPath)
Expand Down Expand Up @@ -419,6 +427,14 @@ func (p *initProcess) start() (retErr error) {
if err := p.manager.Apply(p.pid()); err != nil {
return fmt.Errorf("unable to apply cgroup configuration: %w", err)
}

// Reset the inherited cpu affinity. Old kernels do that automatically, but
// new kernels remember the affinity that was set before the cgroup move.
// This is undesirable, because it inherits the systemd affinity when the container
// should really move to the container space cpus.
if err = unix.SchedSetaffinity(p.pid(), &unix.CPUSet{}); err != nil && err != unix.EINVAL && err != unix.ENODEV {

Check failure on line 435 in libcontainer/process_linux.go

View workflow job for this annotation

GitHub Actions / lint

comparing with != will fail on wrapped errors. Use errors.Is to check for a specific error (errorlint)
return fmt.Errorf("error resetting pid %d affinity: %w", p.pid(), err)
}
if p.intelRdtManager != nil {
if err := p.intelRdtManager.Apply(p.pid()); err != nil {
return fmt.Errorf("unable to apply Intel RDT configuration: %w", err)
Expand Down

0 comments on commit 22c6427

Please sign in to comment.