aboutsummaryrefslogtreecommitdiffstats
path: root/net/sunrpc/sched.c
diff options
context:
space:
mode:
authorWeston Andros Adamson <dros@netapp.com>2012-10-23 10:43:47 -0400
committerTrond Myklebust <Trond.Myklebust@netapp.com>2012-11-04 14:43:42 -0500
commit2bd4eef87bc169f1baf5d1518ba939897cc32471 (patch)
treebbf350c3849229888a85481e2f390a40aad414ad /net/sunrpc/sched.c
parentf50ad42837eb874c1a0cd7cca2001364b06f7ac4 (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.c8
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 *);