diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2007-05-09 05:34:20 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-05-09 15:30:53 -0400 |
commit | b9aac8e0d32499217417ff0b494731811f185b18 (patch) | |
tree | 1cd79adb88431d3ebe6b9dd253b8ed1bbd4f9e07 | |
parent | 23b2e5991afde5af91a1a661d7f47ee56120759e (diff) |
worker_thread: don't play with signals
worker_thread() doesn't need to "Block and flush all signals", this was
already done by its caller, kthread().
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | kernel/workqueue.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 0611de815a8f..87693b37d017 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c | |||
@@ -290,18 +290,11 @@ static int worker_thread(void *__cwq) | |||
290 | struct cpu_workqueue_struct *cwq = __cwq; | 290 | struct cpu_workqueue_struct *cwq = __cwq; |
291 | DEFINE_WAIT(wait); | 291 | DEFINE_WAIT(wait); |
292 | struct k_sigaction sa; | 292 | struct k_sigaction sa; |
293 | sigset_t blocked; | ||
294 | 293 | ||
295 | if (!cwq->wq->freezeable) | 294 | if (!cwq->wq->freezeable) |
296 | current->flags |= PF_NOFREEZE; | 295 | current->flags |= PF_NOFREEZE; |
297 | 296 | ||
298 | set_user_nice(current, -5); | 297 | set_user_nice(current, -5); |
299 | |||
300 | /* Block and flush all signals */ | ||
301 | sigfillset(&blocked); | ||
302 | sigprocmask(SIG_BLOCK, &blocked, NULL); | ||
303 | flush_signals(current); | ||
304 | |||
305 | /* | 298 | /* |
306 | * We inherited MPOL_INTERLEAVE from the booting kernel. | 299 | * We inherited MPOL_INTERLEAVE from the booting kernel. |
307 | * Set MPOL_DEFAULT to insure node local allocations. | 300 | * Set MPOL_DEFAULT to insure node local allocations. |