diff options
| author |   Trond Myklebust <trond.myklebust@primarydata.com> | 2015-07-22 17:05:32 -0400 |
|---|---|---|
| committer | Trond Myklebust <trond.myklebust@primarydata.com> | 2015-07-22 17:10:50 -0400 |
| commit | 1980bd4d829a87ccd21b949f8a11ff1b426f5b0b (patch) | |
| tree | e12601661f45afd32d41c54a5e43a5da45778b98 /net/sunrpc | |
| parent | SUNRPC: Fix a backchannel deadlock (diff) | |
| download | linux-dev-1980bd4d829a87ccd21b949f8a11ff1b426f5b0b.tar.xz linux-dev-1980bd4d829a87ccd21b949f8a11ff1b426f5b0b.zip | |
SUNRPC: xprt_complete_bc_request must also decrement the free slot count
Calling xprt_complete_bc_request() effectively causes the slot to be allocated,
so it needs to decrement the backchannel free slot count as well.
Fixes: 0d2a970d0ae5 ("SUNRPC: Fix a backchannel race")
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Diffstat (limited to 'net/sunrpc')
| -rw-r--r-- | net/sunrpc/backchannel_rqst.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sunrpc/backchannel_rqst.c b/net/sunrpc/backchannel_rqst.c index 5a3b50aec397..6255d141133b 100644 --- a/net/sunrpc/backchannel_rqst.c +++ b/net/sunrpc/backchannel_rqst.c @@ -336,7 +336,7 @@ void xprt_complete_bc_request(struct rpc_rqst *req, uint32_t copied) spin_lock(&xprt->bc_pa_lock); list_del(&req->rq_bc_pa_list); - xprt->bc_alloc_count--; + xprt_dec_alloc_count(xprt, 1); spin_unlock(&xprt->bc_pa_lock); req->rq_private_buf.len = copied; |