Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f30dfbba authored by Weston Andros Adamson's avatar Weston Andros Adamson Committed by Trond Myklebust
Browse files

SUNRPC: remove two BUG_ON asserts 



Replace two BUG_ON() calls checking the RPC_BC_PA_IN_USE flag with
WARN_ON_ONCE().

Signed-off-by: default avatarWeston Andros Adamson <dros@netapp.com>
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent 749386e9

net/sunrpc/backchannel_rqst.c

+2 −2
Original line number Diff line number Diff line
@@ -59,7 +59,7 @@ static void xprt_free_allocation(struct rpc_rqst *req) 
	struct xdr_buf *xbufp;



	dprintk("RPC:        free allocations for req= %p\n", req);

	BUG_ON(test_bit(RPC_BC_PA_IN_USE, &req->rq_bc_pa_state));

	WARN_ON_ONCE(test_bit(RPC_BC_PA_IN_USE, &req->rq_bc_pa_state));

	xbufp = &req->rq_private_buf;

	free_page((unsigned long)xbufp->head[0].iov_base);

	xbufp = &req->rq_snd_buf;
@@ -258,7 +258,7 @@ void xprt_free_bc_request(struct rpc_rqst *req) 
	dprintk("RPC:       free backchannel req=%p\n", req);



	smp_mb__before_clear_bit();

	BUG_ON(!test_bit(RPC_BC_PA_IN_USE, &req->rq_bc_pa_state));

	WARN_ON_ONCE(!test_bit(RPC_BC_PA_IN_USE, &req->rq_bc_pa_state));

	clear_bit(RPC_BC_PA_IN_USE, &req->rq_bc_pa_state);

	smp_mb__after_clear_bit();