diff options
author | gVisor bot <gvisor-bot@google.com> | 2019-08-07 20:03:42 +0000 |
---|---|---|
committer | gVisor bot <gvisor-bot@google.com> | 2019-08-07 20:03:42 +0000 |
commit | abdd9fb7c165e6aae38d2af1bd83657ae4988caf (patch) | |
tree | b833ad10163ea6fc3866f9f35a02995c28cd4682 /runsc | |
parent | e48a653bca635915220c4fb7b3102ef3b5b08888 (diff) | |
parent | 79cc4397fd99fbdd5c74ac5bb7804a463d7981d8 (diff) |
Merge 79cc4397 (automated)
Diffstat (limited to 'runsc')
-rw-r--r-- | runsc/container/container.go | 24 |
1 files changed, 5 insertions, 19 deletions
diff --git a/runsc/container/container.go b/runsc/container/container.go index 2a8453931..bbb364214 100644 --- a/runsc/container/container.go +++ b/runsc/container/container.go @@ -1176,30 +1176,16 @@ func (c *Container) adjustOOMScoreAdj(conf *boot.Config) error { return nil } - // Set oom_score_adj for the sandbox. + // Set the lowest of all containers oom_score_adj to the sandbox. if err := setOOMScoreAdj(c.Sandbox.Pid, lowScore); err != nil { return fmt.Errorf("setting oom_score_adj for sandbox %q: %v", c.Sandbox.ID, err) } - // Set the gofer's oom_score_adj to the minimum of -500 and the - // sandbox's oom_score_adj to better ensure that the sandbox is killed - // before the gofer. - // - // TODO(gvisor.dev/issue/601) Set oom_score_adj for the gofer to - // the same oom_score_adj as the sandbox. - goferScoreAdj := -500 - if lowScore < goferScoreAdj { - goferScoreAdj = lowScore + // Set container's oom_score_adj to the gofer since it is dedicated to the + // container, in case the gofer uses up too much memory. + if err := setOOMScoreAdj(c.GoferPid, *c.Spec.Process.OOMScoreAdj); err != nil { + return fmt.Errorf("setting gofer oom_score_adj for container %q: %v", c.ID, err) } - - // Set oom_score_adj for gofers for all containers in the sandbox. - for _, container := range containers { - err := setOOMScoreAdj(container.GoferPid, goferScoreAdj) - if err != nil { - return fmt.Errorf("setting oom_score_adj for container %q: %v", container.ID, err) - } - } - return nil } |