diff options
author | Sven Wegener <sven.wegener@stealer.net> | 2008-08-14 03:55:22 +0000 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-08-20 11:15:26 -0700 |
commit | 1cc2f16f28ccfc4c0b477e3b0de7b89817876938 (patch) | |
tree | 0a252faed7300d25cf59e48b2559f2654a3e461f /include/asm-s390/cio.h | |
parent | b50504f77c63d836cd2ed62d3e675353970b4d58 (diff) | |
download | lwn-1cc2f16f28ccfc4c0b477e3b0de7b89817876938.tar.gz lwn-1cc2f16f28ccfc4c0b477e3b0de7b89817876938.zip |
ipvs: Fix possible deadlock in estimator code
commit 8ab19ea36c5c5340ff598e4d15fc084eb65671dc upstream
There is a slight chance for a deadlock in the estimator code. We can't call
del_timer_sync() while holding our lock, as the timer might be active and
spinning for the lock on another cpu. Work around this issue by using
try_to_del_timer_sync() and releasing the lock. We could actually delete the
timer outside of our lock, as the add and kill functions are only every called
from userspace via [gs]etsockopt() and are serialized by a mutex, but better
make this explicit.
Signed-off-by: Sven Wegener <sven.wegener@stealer.net>
Acked-by: Simon Horman <horms@verge.net.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'include/asm-s390/cio.h')
0 files changed, 0 insertions, 0 deletions