diff options
| author | Ingo Molnar <mingo@kernel.org> | 2013-12-17 09:27:08 -0500 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2013-12-17 09:27:08 -0500 |
| commit | bb799d3b980eb803ca2da4a4eefbd9308f8d988a (patch) | |
| tree | 69fbe0cd6d47b23a50f5e1d87bf7489532fae149 /net/tipc/handler.c | |
| parent | 919fc6e34831d1c2b58bfb5ae261dc3facc9b269 (diff) | |
| parent | 319e2e3f63c348a9b66db4667efa73178e18b17d (diff) | |
Merge tag 'v3.13-rc4' into core/locking
Merge Linux 3.13-rc4, to refresh this rather old tree with the latest fixes.
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'net/tipc/handler.c')
| -rw-r--r-- | net/tipc/handler.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/net/tipc/handler.c b/net/tipc/handler.c index b36f0fcd9bdf..e4bc8a296744 100644 --- a/net/tipc/handler.c +++ b/net/tipc/handler.c | |||
| @@ -56,12 +56,13 @@ unsigned int tipc_k_signal(Handler routine, unsigned long argument) | |||
| 56 | { | 56 | { |
| 57 | struct queue_item *item; | 57 | struct queue_item *item; |
| 58 | 58 | ||
| 59 | spin_lock_bh(&qitem_lock); | ||
| 59 | if (!handler_enabled) { | 60 | if (!handler_enabled) { |
| 60 | pr_err("Signal request ignored by handler\n"); | 61 | pr_err("Signal request ignored by handler\n"); |
| 62 | spin_unlock_bh(&qitem_lock); | ||
| 61 | return -ENOPROTOOPT; | 63 | return -ENOPROTOOPT; |
| 62 | } | 64 | } |
| 63 | 65 | ||
| 64 | spin_lock_bh(&qitem_lock); | ||
| 65 | item = kmem_cache_alloc(tipc_queue_item_cache, GFP_ATOMIC); | 66 | item = kmem_cache_alloc(tipc_queue_item_cache, GFP_ATOMIC); |
| 66 | if (!item) { | 67 | if (!item) { |
| 67 | pr_err("Signal queue out of memory\n"); | 68 | pr_err("Signal queue out of memory\n"); |
| @@ -112,10 +113,14 @@ void tipc_handler_stop(void) | |||
| 112 | struct list_head *l, *n; | 113 | struct list_head *l, *n; |
| 113 | struct queue_item *item; | 114 | struct queue_item *item; |
| 114 | 115 | ||
| 115 | if (!handler_enabled) | 116 | spin_lock_bh(&qitem_lock); |
| 117 | if (!handler_enabled) { | ||
| 118 | spin_unlock_bh(&qitem_lock); | ||
| 116 | return; | 119 | return; |
| 117 | 120 | } | |
| 118 | handler_enabled = 0; | 121 | handler_enabled = 0; |
| 122 | spin_unlock_bh(&qitem_lock); | ||
| 123 | |||
| 119 | tasklet_kill(&tipc_tasklet); | 124 | tasklet_kill(&tipc_tasklet); |
| 120 | 125 | ||
| 121 | spin_lock_bh(&qitem_lock); | 126 | spin_lock_bh(&qitem_lock); |
