diff options
| author |   Weston Andros Adamson <dros@netapp.com> | 2012-10-23 10:43:35 -0400 |
|---|---|---|
| committer |   Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-11-04 14:43:41 -0500 |
| commit | 4c9c52e479f493ccfc3f513e870912115b49058f (patch) | |
| tree | 36cefe2821fcd4224a63e3f116d861a934502231 /net/sunrpc | |
| parent | SUNRPC: remove BUG_ON from xprt_destroy_backchannel (diff) | |
| download | linux-dev-4c9c52e479f493ccfc3f513e870912115b49058f.tar.xz linux-dev-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')
| -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); } |