diff options
| author | Weston Andros Adamson <dros@netapp.com> | 2012-10-23 16:43:27 +0200 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-11-04 20:43:40 +0100 |
| commit | 9a6478f6cccbd15af30f02368e68b47390360485 (patch) | |
| tree | 9881821a86caa2e374ce95a242fb5e94dec7303d /net/sunrpc | |
| parent | SUNRPC: remove BUG_ON in __rpc_clnt_handle_event (diff) | |
| download | linux-9a6478f6cccbd15af30f02368e68b47390360485.tar.xz linux-9a6478f6cccbd15af30f02368e68b47390360485.zip | |
SUNRPC: remove BUG_ON from rpc_run_bc_task
Replace BUG_ON() with WARN_ON_ONCE() - rpc_run_bc_task calls rpc_init_task()
then increments the tk_count, so this is a simple sanity check that
if hit once would hit every time this code path is executed.
Signed-off-by: Weston Andros Adamson <dros@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc')
| -rw-r--r-- | net/sunrpc/clnt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sunrpc/clnt.c b/net/sunrpc/clnt.c index 245de1a208f4..32aea0b779c5 100644 --- a/net/sunrpc/clnt.c +++ b/net/sunrpc/clnt.c @@ -918,7 +918,7 @@ struct rpc_task *rpc_run_bc_task(struct rpc_rqst *req, task->tk_action = call_bc_transmit; atomic_inc(&task->tk_count); - BUG_ON(atomic_read(&task->tk_count) != 2); + WARN_ON_ONCE(atomic_read(&task->tk_count) != 2); rpc_execute(task); out: |