> On Apr 26, 2016, at 4:16 PM, Sagi Grimberg <sagi@xxxxxxxxxxx> wrote: > > >> +static void >> +__frwr_reset_and_unmap(struct rpcrdma_xprt *r_xprt, struct rpcrdma_mw *mw) >> +{ >> + struct rpcrdma_ia *ia = &r_xprt->rx_ia; >> + struct rpcrdma_frmr *f = &mw->frmr; >> + int rc; >> + >> + rc = __frwr_reset_mr(ia, mw); >> + ib_dma_unmap_sg(ia->ri_device, f->fr_sg, f->fr_nents, f->fr_dir); >> + if (rc) >> + return; >> + >> + rpcrdma_put_mw(r_xprt, mw); >> +} >> + > > If this is not being called from other sites I'd say just > do it in __frwr_recovery_worker instead of having it just bouncing > the call... It will be called from another site in 16/18. -- 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