diff options
author | Tejun Heo <tj@kernel.org> | 2011-11-21 15:32:26 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2011-11-21 15:32:26 -0500 |
commit | 839e3407d90a810318d17c17ceb3d5928a910704 (patch) | |
tree | 9b75e8f69cadba5d33537698931cf637a22d423b /kernel/power/process.c | |
parent | 37ad8aca94a1da2112a7c56151390914e80d1113 (diff) |
freezer: remove unused @sig_only from freeze_task()
After "freezer: make freezing() test freeze conditions in effect
instead of TIF_FREEZE", freezing() returns authoritative answer on
whether the current task should freeze or not and freeze_task()
doesn't need or use @sig_only. Remove it.
While at it, rewrite function comment for freeze_task() and rename
@sig_only to @user_only in try_to_freeze_tasks().
This patch doesn't cause any functional change.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Oleg Nesterov <oleg@redhat.com>
Diffstat (limited to 'kernel/power/process.c')
-rw-r--r-- | kernel/power/process.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/kernel/power/process.c b/kernel/power/process.c index 0beb51e1dec9..77274c9ba2f1 100644 --- a/kernel/power/process.c +++ b/kernel/power/process.c | |||
@@ -22,7 +22,7 @@ | |||
22 | */ | 22 | */ |
23 | #define TIMEOUT (20 * HZ) | 23 | #define TIMEOUT (20 * HZ) |
24 | 24 | ||
25 | static int try_to_freeze_tasks(bool sig_only) | 25 | static int try_to_freeze_tasks(bool user_only) |
26 | { | 26 | { |
27 | struct task_struct *g, *p; | 27 | struct task_struct *g, *p; |
28 | unsigned long end_time; | 28 | unsigned long end_time; |
@@ -37,14 +37,14 @@ static int try_to_freeze_tasks(bool sig_only) | |||
37 | 37 | ||
38 | end_time = jiffies + TIMEOUT; | 38 | end_time = jiffies + TIMEOUT; |
39 | 39 | ||
40 | if (!sig_only) | 40 | if (!user_only) |
41 | freeze_workqueues_begin(); | 41 | freeze_workqueues_begin(); |
42 | 42 | ||
43 | while (true) { | 43 | while (true) { |
44 | todo = 0; | 44 | todo = 0; |
45 | read_lock(&tasklist_lock); | 45 | read_lock(&tasklist_lock); |
46 | do_each_thread(g, p) { | 46 | do_each_thread(g, p) { |
47 | if (p == current || !freeze_task(p, sig_only)) | 47 | if (p == current || !freeze_task(p)) |
48 | continue; | 48 | continue; |
49 | 49 | ||
50 | /* | 50 | /* |
@@ -65,7 +65,7 @@ static int try_to_freeze_tasks(bool sig_only) | |||
65 | } while_each_thread(g, p); | 65 | } while_each_thread(g, p); |
66 | read_unlock(&tasklist_lock); | 66 | read_unlock(&tasklist_lock); |
67 | 67 | ||
68 | if (!sig_only) { | 68 | if (!user_only) { |
69 | wq_busy = freeze_workqueues_busy(); | 69 | wq_busy = freeze_workqueues_busy(); |
70 | todo += wq_busy; | 70 | todo += wq_busy; |
71 | } | 71 | } |