diff options
| author | Oleg Nesterov <[email protected]> | 2013-04-30 15:28:17 -0700 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2013-04-30 17:04:06 -0700 |
| commit | dc7ee2aac830e5423f41de87d50441f138f648da (patch) | |
| tree | 735a75c2d2455721dffed758632f07f62cbf2206 /scripts/stackusage | |
| parent | 079148b919d0c58b796f9ae98bdb53028dbcd5e7 (diff) | |
coredump: change wait_for_dump_helpers() to use wait_event_interruptible()
wait_for_dump_helpers() calls wake_up/kill_fasync from inside the
wait_event-like loop. This is not needed and in fact this is not
strictly correct, we can/should do this only once after we change
pipe->writers. We could even check if it becomes zero.
Change this code to use use wait_event_interruptible(), this can also
help to make this wait freezable.
With this patch we check pipe->readers without pipe_lock(), this is
fine. Once we see pipe->readers == 1 we know that the handler
decremented the counter, this is all we need.
Signed-off-by: Oleg Nesterov <[email protected]>
Acked-by: Mandeep Singh Baines <[email protected]>
Cc: Neil Horman <[email protected]>
Cc: "Rafael J. Wysocki" <[email protected]>
Cc: Tejun Heo <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'scripts/stackusage')
0 files changed, 0 insertions, 0 deletions