aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBhanu Prakash Gollapudi <bprakash@broadcom.com>2011-05-27 14:47:26 -0400
committerJames Bottomley <JBottomley@Parallels.com>2011-06-29 11:58:04 -0400
commitb5a95fe7ef464a67fab6ff870aa740739e788f90 (patch)
treef9fe9ca0d8aeade956a16050f4ee1dd79f6ce170
parenta4dc08cecfdb3172be61227324a4d2c5fe2de9e5 (diff)
[SCSI] bnx2fc: scsi_dma_unmap() not invoked on IO completions
Do not set io_req->sc_cmd to NULL until bnx2fc_unmap_sg_list() is called to enable it to unmap the DMA mappings. Signed-off-by: Bhanu Prakash Gollapudi <bprakash@broadcom.com> Signed-off-by: James Bottomley <JBottomley@Parallels.com>
-rw-r--r--drivers/scsi/bnx2fc/bnx2fc_io.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/bnx2fc/bnx2fc_io.c b/drivers/scsi/bnx2fc/bnx2fc_io.c
index b5b5c346d779..454c72cdafbc 100644
--- a/drivers/scsi/bnx2fc/bnx2fc_io.c
+++ b/drivers/scsi/bnx2fc/bnx2fc_io.c
@@ -1734,7 +1734,6 @@ void bnx2fc_process_scsi_cmd_compl(struct bnx2fc_cmd *io_req,
1734 printk(KERN_ERR PFX "SCp.ptr is NULL\n"); 1734 printk(KERN_ERR PFX "SCp.ptr is NULL\n");
1735 return; 1735 return;
1736 } 1736 }
1737 io_req->sc_cmd = NULL;
1738 1737
1739 if (io_req->on_active_queue) { 1738 if (io_req->on_active_queue) {
1740 list_del_init(&io_req->link); 1739 list_del_init(&io_req->link);
@@ -1754,6 +1753,7 @@ void bnx2fc_process_scsi_cmd_compl(struct bnx2fc_cmd *io_req,
1754 } 1753 }
1755 1754
1756 bnx2fc_unmap_sg_list(io_req); 1755 bnx2fc_unmap_sg_list(io_req);
1756 io_req->sc_cmd = NULL;
1757 1757
1758 switch (io_req->fcp_status) { 1758 switch (io_req->fcp_status) {
1759 case FC_GOOD: 1759 case FC_GOOD: