> On Dec 16, 2015, at 8:57 AM, Sagi Grimberg <sagig@xxxxxxxxxxxxxxxxxx> wrote: > > >> +static void >> +__frwr_dma_unmap(struct rpcrdma_xprt *r_xprt, struct rpcrdma_mr_seg *seg, >> + int rc) >> +{ >> + struct ib_device *device = r_xprt->rx_ia.ri_device; >> + struct rpcrdma_mw *mw = seg->rl_mw; >> + int nsegs = seg->mr_nsegs; >> + >> + seg->rl_mw = NULL; >> + >> + while (nsegs--) >> + rpcrdma_unmap_one(device, seg++); > > Chuck, shouldn't this be replaced with ib_dma_unmap_sg? Looks like this was left over from before the conversion to use ib_dma_unmap_sg. I'll have a look. > Sorry for the late comment (Didn't find enough time to properly > review this...) -- 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