diff options
author | Weston Andros Adamson <dros@netapp.com> | 2012-10-23 10:43:47 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-11-04 14:43:42 -0500 |
commit | 2bd4eef87bc169f1baf5d1518ba939897cc32471 (patch) | |
tree | bbf350c3849229888a85481e2f390a40aad414ad /net/sunrpc/sched.c | |
parent | f50ad42837eb874c1a0cd7cca2001364b06f7ac4 (diff) |
SUNRPC: remove BUG_ONs checking RPC_IS_QUEUED
Replace two BUG_ON() calls with WARN_ON_ONCE() and early returns.
Signed-off-by: Weston Andros Adamson <dros@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc/sched.c')
-rw-r--r-- | net/sunrpc/sched.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/net/sunrpc/sched.c b/net/sunrpc/sched.c index e6db49699bce..69049179c280 100644 --- a/net/sunrpc/sched.c +++ b/net/sunrpc/sched.c | |||
@@ -133,7 +133,9 @@ static void __rpc_add_wait_queue(struct rpc_wait_queue *queue, | |||
133 | struct rpc_task *task, | 133 | struct rpc_task *task, |
134 | unsigned char queue_priority) | 134 | unsigned char queue_priority) |
135 | { | 135 | { |
136 | BUG_ON (RPC_IS_QUEUED(task)); | 136 | WARN_ON_ONCE(RPC_IS_QUEUED(task)); |
137 | if (RPC_IS_QUEUED(task)) | ||
138 | return; | ||
137 | 139 | ||
138 | if (RPC_IS_PRIORITY(queue)) | 140 | if (RPC_IS_PRIORITY(queue)) |
139 | __rpc_add_wait_queue_priority(queue, task, queue_priority); | 141 | __rpc_add_wait_queue_priority(queue, task, queue_priority); |
@@ -707,7 +709,9 @@ static void __rpc_execute(struct rpc_task *task) | |||
707 | dprintk("RPC: %5u __rpc_execute flags=0x%x\n", | 709 | dprintk("RPC: %5u __rpc_execute flags=0x%x\n", |
708 | task->tk_pid, task->tk_flags); | 710 | task->tk_pid, task->tk_flags); |
709 | 711 | ||
710 | BUG_ON(RPC_IS_QUEUED(task)); | 712 | WARN_ON_ONCE(RPC_IS_QUEUED(task)); |
713 | if (RPC_IS_QUEUED(task)) | ||
714 | return; | ||
711 | 715 | ||
712 | for (;;) { | 716 | for (;;) { |
713 | void (*do_action)(struct rpc_task *); | 717 | void (*do_action)(struct rpc_task *); |