> 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-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
