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/bc_svc.c | |
parent | c4ded8d9771c8e8d5d7a202d58af9c70591dd675 (diff) |
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) | |||
53 | if (IS_ERR(task)) | 53 | if (IS_ERR(task)) |
54 | ret = PTR_ERR(task); | 54 | ret = PTR_ERR(task); |
55 | else { | 55 | else { |
56 | BUG_ON(atomic_read(&task->tk_count) != 1); | 56 | WARN_ON_ONCE(atomic_read(&task->tk_count) != 1); |
57 | ret = task->tk_status; | 57 | ret = task->tk_status; |
58 | rpc_put_task(task); | 58 | rpc_put_task(task); |
59 | } | 59 | } |