Commit 67b60518 authored by Johannes Berg's avatar Johannes Berg Committed by Paul Mackerras

[POWERPC] windfarm: Fix windfarm thread freezer interaction

When I fixed the windfarm freezer interaction first in commit
1ed2ddf3, an earlier patch than the one
I came up with after comments was committed. This has come back to haunt
us now because commit d5d8c597 changed
the freezer to no long send signals. Fix it by removing the windfarm
thread's signal logic and restoring the original try_to_freeze().

We could simply revert 1ed2ddf3 now
but I feel that the assertion that no signal is delivered to the
windfarm thread needs not be there.
Signed-off-by: default avatarJohannes Berg <johannes@sipsolutions.net>
Acked-by: default avatarRafael J. Wysocki <rjw@sisk.pl>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent a792e75d
...@@ -94,7 +94,9 @@ static int wf_thread_func(void *data) ...@@ -94,7 +94,9 @@ static int wf_thread_func(void *data)
DBG("wf: thread started\n"); DBG("wf: thread started\n");
set_freezable(); set_freezable();
while(!kthread_should_stop()) { while (!kthread_should_stop()) {
try_to_freeze();
if (time_after_eq(jiffies, next)) { if (time_after_eq(jiffies, next)) {
wf_notify(WF_EVENT_TICK, NULL); wf_notify(WF_EVENT_TICK, NULL);
if (wf_overtemp) { if (wf_overtemp) {
...@@ -116,12 +118,6 @@ static int wf_thread_func(void *data) ...@@ -116,12 +118,6 @@ static int wf_thread_func(void *data)
delay = next - jiffies; delay = next - jiffies;
if (delay <= HZ) if (delay <= HZ)
schedule_timeout_interruptible(delay); schedule_timeout_interruptible(delay);
/* there should be no non-suspend signal, but oh well */
if (signal_pending(current) && !try_to_freeze()) {
printk(KERN_WARNING "windfarm: thread got sigl !\n");
break;
}
} }
DBG("wf: thread stopped\n"); DBG("wf: thread stopped\n");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment