Skip to content

Commit f2dd007

Browse files
committed
Merge tag 'scsi-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi
Pull SCSI fixes from James Bottomley: "Four fixes, all in drivers. These patches mosly fix error legs and exceptional conditions (scsi_dh_alua, qla2xxx). The lpfc fixes are for coding issues with lpfc features" * tag 'scsi-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi: scsi: lpfc: Correct BDE DMA address assignment for GEN_REQ_WQE scsi: lpfc: Fix split code for FLOGI on FCoE scsi: qla2xxx: Fix missed DMA unmap for aborted commands scsi: scsi_dh_alua: Properly handle the ALUA transitioning state
2 parents c3f5e69 + 7752662 commit f2dd007

File tree

4 files changed

+8
-6
lines changed

4 files changed

+8
-6
lines changed

drivers/scsi/device_handler/scsi_dh_alua.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1172,9 +1172,8 @@ static blk_status_t alua_prep_fn(struct scsi_device *sdev, struct request *req)
11721172
case SCSI_ACCESS_STATE_OPTIMAL:
11731173
case SCSI_ACCESS_STATE_ACTIVE:
11741174
case SCSI_ACCESS_STATE_LBA:
1175-
return BLK_STS_OK;
11761175
case SCSI_ACCESS_STATE_TRANSITIONING:
1177-
return BLK_STS_AGAIN;
1176+
return BLK_STS_OK;
11781177
default:
11791178
req->rq_flags |= RQF_QUIET;
11801179
return BLK_STS_IOERR;

drivers/scsi/lpfc/lpfc_els.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1330,7 +1330,7 @@ lpfc_issue_els_flogi(struct lpfc_vport *vport, struct lpfc_nodelist *ndlp,
13301330
if (bf_get(lpfc_sli_intf_if_type, &phba->sli4_hba.sli_intf) ==
13311331
LPFC_SLI_INTF_IF_TYPE_0) {
13321332
/* FLOGI needs to be 3 for WQE FCFI */
1333-
ct = ((SLI4_CT_FCFI >> 1) & 1) | (SLI4_CT_FCFI & 1);
1333+
ct = SLI4_CT_FCFI;
13341334
bf_set(wqe_ct, &wqe->els_req.wqe_com, ct);
13351335

13361336
/* Set the fcfi to the fcfi we registered with */

drivers/scsi/lpfc/lpfc_sli.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10720,10 +10720,10 @@ __lpfc_sli_prep_gen_req_s4(struct lpfc_iocbq *cmdiocbq, struct lpfc_dmabuf *bmp,
1072010720

1072110721
/* Words 0 - 2 */
1072210722
bde = (struct ulp_bde64_le *)&cmdwqe->generic.bde;
10723-
bde->addr_low = cpu_to_le32(putPaddrLow(bmp->phys));
10724-
bde->addr_high = cpu_to_le32(putPaddrHigh(bmp->phys));
10723+
bde->addr_low = bpl->addr_low;
10724+
bde->addr_high = bpl->addr_high;
1072510725
bde->type_size = cpu_to_le32(xmit_len);
10726-
bde->type_size |= cpu_to_le32(ULP_BDE64_TYPE_BLP_64);
10726+
bde->type_size |= cpu_to_le32(ULP_BDE64_TYPE_BDE_64);
1072710727

1072810728
/* Word 3 */
1072910729
cmdwqe->gen_req.request_payload_len = xmit_len;

drivers/scsi/qla2xxx/qla_target.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3826,6 +3826,9 @@ int qlt_abort_cmd(struct qla_tgt_cmd *cmd)
38263826

38273827
spin_lock_irqsave(&cmd->cmd_lock, flags);
38283828
if (cmd->aborted) {
3829+
if (cmd->sg_mapped)
3830+
qlt_unmap_sg(vha, cmd);
3831+
38293832
spin_unlock_irqrestore(&cmd->cmd_lock, flags);
38303833
/*
38313834
* It's normal to see 2 calls in this path:

0 commit comments

Comments
 (0)