IB/srp: Fix unmapping of fake scatterlist

The recently merged patch to create a fake scatterlist for non-SG SCSI
commands had a bug: the driver ended up doing dma_unmap_sg() on a
scatterlist scmnd->request_buffer rather than the fake scatter list it
created.  Fix this so that the driver unmaps the same thing it maps.

Signed-off-by: Roland Dreier <rolandd@cisco.com>

diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c
index 61924cc..fd8a95a 100644 (file)
--- a/drivers/infiniband/ulp/srp/ib_srp.c
+++ b/drivers/infiniband/ulp/srp/ib_srp.c
@@ -607,10 +607,10 @@ static void srp_unmap_data(struct scsi_cmnd *scmnd,
         */
        if (likely(scmnd->use_sg)) {
                nents = scmnd->use_sg;
-               scat  = (struct scatterlist *) scmnd->request_buffer;
+               scat  = scmnd->request_buffer;
        } else {
                nents = 1;
-               scat  = (struct scatterlist *) scmnd->request_buffer;
+               scat  = &req->fake_sg;
        }
 
        dma_unmap_sg(target->srp_host->dev->dma_device, scat, nents,