> On Dec 16, 2015, at 8:57 AM, Sagi Grimberg <sa...@dev.mellanox.co.il> 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to