diff options
author | Weston Andros Adamson <dros@netapp.com> | 2012-10-23 16:43:35 +0200 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-11-04 20:43:41 +0100 |
commit | 4c9c52e479f493ccfc3f513e870912115b49058f (patch) | |
tree | 36cefe2821fcd4224a63e3f116d861a934502231 /net/sunrpc/bc_svc.c | |
parent | SUNRPC: remove BUG_ON from xprt_destroy_backchannel (diff) | |
download | linux-4c9c52e479f493ccfc3f513e870912115b49058f.tar.xz linux-4c9c52e479f493ccfc3f513e870912115b49058f.zip |
SUNRPC: remove BUG_ON from bc_send
Replace BUG_ON() with WARN_ON_ONCE(). The error condition is a simple
ref counting sanity check and the following code will not free anything
until final put.
Signed-off-by: Weston Andros Adamson <dros@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc/bc_svc.c')
-rw-r--r-- | net/sunrpc/bc_svc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sunrpc/bc_svc.c b/net/sunrpc/bc_svc.c index 0b2eb388cbda..15c7a8a1c24f 100644 --- a/net/sunrpc/bc_svc.c +++ b/net/sunrpc/bc_svc.c @@ -53,7 +53,7 @@ int bc_send(struct rpc_rqst *req) if (IS_ERR(task)) ret = PTR_ERR(task); else { - BUG_ON(atomic_read(&task->tk_count) != 1); + WARN_ON_ONCE(atomic_read(&task->tk_count) != 1); ret = task->tk_status; rpc_put_task(task); } |