aboutsummaryrefslogtreecommitdiffstats
path: root/kernel/power/process.c
diff options
context:
space:
mode:
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>2014-10-22 16:47:32 -0400
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2014-10-22 16:47:32 -0400
commit71be2114a5474a76edad95343d89b8731457fccd (patch)
tree7b8f80accc9354f39fabd5e63fd6194e808a4906 /kernel/power/process.c
parenta28e785a9f794ba32e603570ab52a262cf963489 (diff)
PM / freezer: Clean up code after recent fixes
Clean up the code in process.c after recent changes to get rid of unnecessary labels and goto statements. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'kernel/power/process.c')
-rw-r--r--kernel/power/process.c31
1 files changed, 16 insertions, 15 deletions
diff --git a/kernel/power/process.c b/kernel/power/process.c
index 7f0d4343af1b..5a6ec8678b9a 100644
--- a/kernel/power/process.c
+++ b/kernel/power/process.c
@@ -108,25 +108,27 @@ static int try_to_freeze_tasks(bool user_only)
108 return todo ? -EBUSY : 0; 108 return todo ? -EBUSY : 0;
109} 109}
110 110
111static bool __check_frozen_processes(void)
112{
113 struct task_struct *g, *p;
114
115 for_each_process_thread(g, p)
116 if (p != current && !freezer_should_skip(p) && !frozen(p))
117 return false;
118
119 return true;
120}
121
111/* 122/*
112 * Returns true if all freezable tasks (except for current) are frozen already 123 * Returns true if all freezable tasks (except for current) are frozen already
113 */ 124 */
114static bool check_frozen_processes(void) 125static bool check_frozen_processes(void)
115{ 126{
116 struct task_struct *g, *p; 127 bool ret;
117 bool ret = true;
118 128
119 read_lock(&tasklist_lock); 129 read_lock(&tasklist_lock);
120 for_each_process_thread(g, p) { 130 ret = __check_frozen_processes();
121 if (p != current && !freezer_should_skip(p) &&
122 !frozen(p)) {
123 ret = false;
124 goto done;
125 }
126 }
127done:
128 read_unlock(&tasklist_lock); 131 read_unlock(&tasklist_lock);
129
130 return ret; 132 return ret;
131} 133}
132 134
@@ -167,15 +169,14 @@ int freeze_processes(void)
167 * on the way out so we have to double check for race. 169 * on the way out so we have to double check for race.
168 */ 170 */
169 if (oom_kills_count() != oom_kills_saved && 171 if (oom_kills_count() != oom_kills_saved &&
170 !check_frozen_processes()) { 172 !check_frozen_processes()) {
171 __usermodehelper_set_disable_depth(UMH_ENABLED); 173 __usermodehelper_set_disable_depth(UMH_ENABLED);
172 printk("OOM in progress."); 174 printk("OOM in progress.");
173 error = -EBUSY; 175 error = -EBUSY;
174 goto done; 176 } else {
177 printk("done.");
175 } 178 }
176 printk("done.");
177 } 179 }
178done:
179 printk("\n"); 180 printk("\n");
180 BUG_ON(in_atomic()); 181 BUG_ON(in_atomic());
181 182