diff options
author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-10-29 18:43:08 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-10-30 10:28:35 -0400 |
commit | 59612d187912750f416fbffe0c00bc0811c54ab5 (patch) | |
tree | 12ab7055f876fdd8b397e7efaa8d7b4207f30b00 /fs | |
parent | c511851de162e8ec03d62e7d7feecbdf590d881d (diff) |
Revert "select: use freezable blocking call"
This reverts commit 9745cdb36da8 (select: use freezable blocking call)
that triggers problems during resume from suspend to RAM on Paul Bolle's
32-bit x86 machines. Paul says:
Ever since I tried running (release candidates of) v3.11 on the two
working i686s I still have lying around I ran into issues on resuming
from suspend. Reverting 9745cdb36da8 (select: use freezable blocking
call) resolves those issues.
Resuming from suspend on i686 on (release candidates of) v3.11 and
later triggers issues like:
traps: systemd[1] general protection ip:b738e490 sp:bf882fc0 error:0 in libc-2.16.so[b731c000+1b0000]
and
traps: rtkit-daemon[552] general protection ip:804d6e5 sp:b6cb32f0 error:0 in rtkit-daemon[8048000+d000]
Once I hit the systemd error I can only get out of the mess that the
system is at that point by power cycling it.
Since we are reverting another freezer-related change causing similar
problems to happen, this one should be reverted as well.
References: https://lkml.org/lkml/2013/10/29/583
Reported-by: Paul Bolle <pebolle@tiscali.nl>
Fixes: 9745cdb36da8 (select: use freezable blocking call)
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Cc: 3.11+ <stable@vger.kernel.org> # 3.11+
Diffstat (limited to 'fs')
-rw-r--r-- | fs/select.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/select.c b/fs/select.c index 35d4adc749d9..dfd5cb18c012 100644 --- a/fs/select.c +++ b/fs/select.c | |||
@@ -238,8 +238,7 @@ int poll_schedule_timeout(struct poll_wqueues *pwq, int state, | |||
238 | 238 | ||
239 | set_current_state(state); | 239 | set_current_state(state); |
240 | if (!pwq->triggered) | 240 | if (!pwq->triggered) |
241 | rc = freezable_schedule_hrtimeout_range(expires, slack, | 241 | rc = schedule_hrtimeout_range(expires, slack, HRTIMER_MODE_ABS); |
242 | HRTIMER_MODE_ABS); | ||
243 | __set_current_state(TASK_RUNNING); | 242 | __set_current_state(TASK_RUNNING); |
244 | 243 | ||
245 | /* | 244 | /* |