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/freezer.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/freezer.c')
-rw-r--r-- | kernel/freezer.c | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/kernel/freezer.c b/kernel/freezer.c index 389549f0a94e..2589a61de44c 100644 --- a/kernel/freezer.c +++ b/kernel/freezer.c | |||
@@ -100,20 +100,17 @@ static void fake_signal_wake_up(struct task_struct *p) | |||
100 | } | 100 | } |
101 | 101 | ||
102 | /** | 102 | /** |
103 | * freeze_task - send a freeze request to given task | 103 | * freeze_task - send a freeze request to given task |
104 | * @p: task to send the request to | 104 | * @p: task to send the request to |
105 | * @sig_only: if set, the request will only be sent if the task has the | ||
106 | * PF_FREEZER_NOSIG flag unset | ||
107 | * Return value: 'false', if @sig_only is set and the task has | ||
108 | * PF_FREEZER_NOSIG set or the task is frozen, 'true', otherwise | ||
109 | * | 105 | * |
110 | * The freeze request is sent by setting the tasks's TIF_FREEZE flag and | 106 | * If @p is freezing, the freeze request is sent by setting %TIF_FREEZE |
111 | * either sending a fake signal to it or waking it up, depending on whether | 107 | * flag and either sending a fake signal to it or waking it up, depending |
112 | * or not it has PF_FREEZER_NOSIG set. If @sig_only is set and the task | 108 | * on whether it has %PF_FREEZER_NOSIG set. |
113 | * has PF_FREEZER_NOSIG set (ie. it is a typical kernel thread), its | 109 | * |
114 | * TIF_FREEZE flag will not be set. | 110 | * RETURNS: |
111 | * %false, if @p is not freezing or already frozen; %true, otherwise | ||
115 | */ | 112 | */ |
116 | bool freeze_task(struct task_struct *p, bool sig_only) | 113 | bool freeze_task(struct task_struct *p) |
117 | { | 114 | { |
118 | unsigned long flags; | 115 | unsigned long flags; |
119 | 116 | ||