diff options
author | Jayamohan Kallickal <jayamohan.kallickal@emulex.com> | 2010-10-06 14:16:03 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-10-11 18:43:34 -0400 |
commit | af4c609c0d645f196b570c58dd4ee878ff3afd24 (patch) | |
tree | 10e1aadc138f2ecd0ea75840d7da08366a84bff3 /drivers/scsi | |
parent | 91446f060b9ae27875d38b00dc3c6394b8321bff (diff) |
[SCSI] be2iscsi: Remove premature free of cid
Remove unnecessary beiscsi_put_cid that was freeing up the cid while
in use
Signed-off-by: Jayamohan Kallickal <jayamohan.kallickal@emulex.com>
Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/be2iscsi/be_iscsi.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/scsi/be2iscsi/be_iscsi.c b/drivers/scsi/be2iscsi/be_iscsi.c index 7d4d2275573..8b897c80ec7 100644 --- a/drivers/scsi/be2iscsi/be_iscsi.c +++ b/drivers/scsi/be2iscsi/be_iscsi.c | |||
@@ -523,7 +523,6 @@ static int beiscsi_open_conn(struct iscsi_endpoint *ep, | |||
523 | if (beiscsi_ep->ep_cid > (phba->fw_config.iscsi_cid_start + | 523 | if (beiscsi_ep->ep_cid > (phba->fw_config.iscsi_cid_start + |
524 | phba->params.cxns_per_ctrl * 2)) { | 524 | phba->params.cxns_per_ctrl * 2)) { |
525 | SE_DEBUG(DBG_LVL_1, "Failed in allocate iscsi cid\n"); | 525 | SE_DEBUG(DBG_LVL_1, "Failed in allocate iscsi cid\n"); |
526 | beiscsi_put_cid(phba, beiscsi_ep->ep_cid); | ||
527 | goto free_ep; | 526 | goto free_ep; |
528 | } | 527 | } |
529 | 528 | ||
@@ -560,7 +559,6 @@ static int beiscsi_open_conn(struct iscsi_endpoint *ep, | |||
560 | SE_DEBUG(DBG_LVL_1, "mgmt_open_connection Failed" | 559 | SE_DEBUG(DBG_LVL_1, "mgmt_open_connection Failed" |
561 | " status = %d extd_status = %d\n", | 560 | " status = %d extd_status = %d\n", |
562 | status, extd_status); | 561 | status, extd_status); |
563 | beiscsi_put_cid(phba, beiscsi_ep->ep_cid); | ||
564 | free_mcc_tag(&phba->ctrl, tag); | 562 | free_mcc_tag(&phba->ctrl, tag); |
565 | pci_free_consistent(phba->ctrl.pdev, nonemb_cmd.size, | 563 | pci_free_consistent(phba->ctrl.pdev, nonemb_cmd.size, |
566 | nonemb_cmd.va, nonemb_cmd.dma); | 564 | nonemb_cmd.va, nonemb_cmd.dma); |
@@ -575,7 +573,6 @@ static int beiscsi_open_conn(struct iscsi_endpoint *ep, | |||
575 | beiscsi_ep->cid_vld = 1; | 573 | beiscsi_ep->cid_vld = 1; |
576 | SE_DEBUG(DBG_LVL_8, "mgmt_open_connection Success\n"); | 574 | SE_DEBUG(DBG_LVL_8, "mgmt_open_connection Success\n"); |
577 | } | 575 | } |
578 | beiscsi_put_cid(phba, beiscsi_ep->ep_cid); | ||
579 | pci_free_consistent(phba->ctrl.pdev, nonemb_cmd.size, | 576 | pci_free_consistent(phba->ctrl.pdev, nonemb_cmd.size, |
580 | nonemb_cmd.va, nonemb_cmd.dma); | 577 | nonemb_cmd.va, nonemb_cmd.dma); |
581 | return 0; | 578 | return 0; |