diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2011-11-28 16:55:59 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-11-28 16:55:59 -0500 |
| commit | a34815b96f9a21b3a2e2912dfd0d994acd2855e3 (patch) | |
| tree | e2e6ac428637059305d7c00274f0c034f0c82858 | |
| parent | 9b5a4d4f65e260a109eaeea8bbc8062a7c58b55e (diff) | |
| parent | 884a45d964dd395eda945842afff5e16bcaedf56 (diff) | |
Merge branch 'for-3.2-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup
* 'for-3.2-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup:
cgroup_freezer: fix freezing groups with stopped tasks
| -rw-r--r-- | kernel/cgroup_freezer.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/kernel/cgroup_freezer.c b/kernel/cgroup_freezer.c index 5e828a2ca8e6..213c0351dad8 100644 --- a/kernel/cgroup_freezer.c +++ b/kernel/cgroup_freezer.c | |||
| @@ -153,6 +153,13 @@ static void freezer_destroy(struct cgroup_subsys *ss, | |||
| 153 | kfree(cgroup_freezer(cgroup)); | 153 | kfree(cgroup_freezer(cgroup)); |
| 154 | } | 154 | } |
| 155 | 155 | ||
| 156 | /* task is frozen or will freeze immediately when next it gets woken */ | ||
| 157 | static bool is_task_frozen_enough(struct task_struct *task) | ||
| 158 | { | ||
| 159 | return frozen(task) || | ||
| 160 | (task_is_stopped_or_traced(task) && freezing(task)); | ||
| 161 | } | ||
| 162 | |||
| 156 | /* | 163 | /* |
| 157 | * The call to cgroup_lock() in the freezer.state write method prevents | 164 | * The call to cgroup_lock() in the freezer.state write method prevents |
| 158 | * a write to that file racing against an attach, and hence the | 165 | * a write to that file racing against an attach, and hence the |
| @@ -231,7 +238,7 @@ static void update_if_frozen(struct cgroup *cgroup, | |||
| 231 | cgroup_iter_start(cgroup, &it); | 238 | cgroup_iter_start(cgroup, &it); |
| 232 | while ((task = cgroup_iter_next(cgroup, &it))) { | 239 | while ((task = cgroup_iter_next(cgroup, &it))) { |
| 233 | ntotal++; | 240 | ntotal++; |
| 234 | if (frozen(task)) | 241 | if (is_task_frozen_enough(task)) |
| 235 | nfrozen++; | 242 | nfrozen++; |
| 236 | } | 243 | } |
| 237 | 244 | ||
| @@ -284,7 +291,7 @@ static int try_to_freeze_cgroup(struct cgroup *cgroup, struct freezer *freezer) | |||
| 284 | while ((task = cgroup_iter_next(cgroup, &it))) { | 291 | while ((task = cgroup_iter_next(cgroup, &it))) { |
| 285 | if (!freeze_task(task, true)) | 292 | if (!freeze_task(task, true)) |
| 286 | continue; | 293 | continue; |
| 287 | if (frozen(task)) | 294 | if (is_task_frozen_enough(task)) |
| 288 | continue; | 295 | continue; |
| 289 | if (!freezing(task) && !freezer_should_skip(task)) | 296 | if (!freezing(task) && !freezer_should_skip(task)) |
| 290 | num_cant_freeze_now++; | 297 | num_cant_freeze_now++; |
