Hi Bruce- > On Aug 22, 2016, at 2:57 PM, Chuck Lever <chuck.lever@xxxxxxxxxx> wrote: > > Using NFSv4.1 on RDMA should be safe, so broaden the new checks in > rpc_create(). > > WARN_ON_ONCE is used, matching most other WARN call sites in clnt.c. > > Fixes: 39a9beab5acb ("rpc: share one xps between all backchannels") > Fixes: d50039ea5ee6 ("nfsd4/rpc: move backchannel create logic...") > Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx> > --- Do you want to take this, or should it go through Trond/Anna ? > net/sunrpc/clnt.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/net/sunrpc/clnt.c b/net/sunrpc/clnt.c > index 7f79fb7..66f23b3 100644 > --- a/net/sunrpc/clnt.c > +++ b/net/sunrpc/clnt.c > @@ -453,7 +453,7 @@ static struct rpc_clnt *rpc_create_xprt(struct rpc_create_args *args, > struct rpc_xprt_switch *xps; > > if (args->bc_xprt && args->bc_xprt->xpt_bc_xps) { > - WARN_ON(args->protocol != XPRT_TRANSPORT_BC_TCP); > + WARN_ON_ONCE(!(args->protocol & XPRT_TRANSPORT_BC)); > xps = args->bc_xprt->xpt_bc_xps; > xprt_switch_get(xps); > } else { > @@ -520,7 +520,7 @@ struct rpc_clnt *rpc_create(struct rpc_create_args *args) > char servername[48]; > > if (args->bc_xprt) { > - WARN_ON(args->protocol != XPRT_TRANSPORT_BC_TCP); > + WARN_ON_ONCE(!(args->protocol & XPRT_TRANSPORT_BC)); > xprt = args->bc_xprt->xpt_bc_xprt; > if (xprt) { > xprt_get(xprt); > > -- > To unsubscribe from this list: send the line "unsubscribe linux-nfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Chuck Lever -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html