diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2006-09-08 09:47:15 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-08 10:22:50 -0700 |
commit | c5780e976e19faff345fcef4a01db87108b51a44 (patch) | |
tree | 1e75fc4cd3765beb575fb3196a477d97ed1905a4 /drivers/ide/ide-proc.c | |
parent | 3a459756810912d2c2bf188cef566af255936b4d (diff) | |
download | linux-3.10-c5780e976e19faff345fcef4a01db87108b51a44.tar.gz linux-3.10-c5780e976e19faff345fcef4a01db87108b51a44.tar.bz2 linux-3.10-c5780e976e19faff345fcef4a01db87108b51a44.zip |
[PATCH] Use the correct restart option for futex_lock_pi
The current implementation of futex_lock_pi returns -ERESTART_RESTARTBLOCK
in case that the lock operation has been interrupted by a signal. This
results in a return of -EINTR to userspace in case there is an handler for
the signal. This is wrong, because userspace expects that the lock
function does not return in any case of signal delivery.
This was not caught by my insufficient test case, but triggered a nasty
userspace problem in an high load application scenario. Unfortunately also
glibc does not check for this invalid return value.
Using -ERSTARTNOINTR makes sure, that the interrupted syscall is restarted.
The restart block related code can be safely removed, as the possible
timeout argument is an absolute time value.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Acked-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/ide/ide-proc.c')
0 files changed, 0 insertions, 0 deletions