diff options
author | James Bottomley <James.Bottomley@HansenPartnership.com> | 2016-10-16 16:51:31 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2016-10-16 16:51:31 -0400 |
commit | ebbb7594000eb266cac2bbcfdd93d829c7b0063f (patch) | |
tree | d070e95b27445f432d7a5ae67e718210fad7f723 | |
parent | 1001354ca34179f3db924eb66672442a173147dc (diff) | |
parent | 8a4236a2c7868768943a24dc7b1e2ff495836880 (diff) |
Merge remote-tracking branch 'mkp-scsi/4.9/scsi-fixes' into fixes
-rw-r--r-- | drivers/s390/scsi/zfcp_dbf.c | 2 | ||||
-rw-r--r-- | drivers/scsi/ipr.c | 3 | ||||
-rw-r--r-- | drivers/scsi/scsi_dh.c | 6 | ||||
-rw-r--r-- | drivers/scsi/scsi_scan.c | 6 |
4 files changed, 6 insertions, 11 deletions
diff --git a/drivers/s390/scsi/zfcp_dbf.c b/drivers/s390/scsi/zfcp_dbf.c index 637cf8973c9e..581001989937 100644 --- a/drivers/s390/scsi/zfcp_dbf.c +++ b/drivers/s390/scsi/zfcp_dbf.c | |||
@@ -384,7 +384,7 @@ void zfcp_dbf_san(char *tag, struct zfcp_dbf *dbf, | |||
384 | /* if (len > rec_len): | 384 | /* if (len > rec_len): |
385 | * dump data up to cap_len ignoring small duplicate in rec->payload | 385 | * dump data up to cap_len ignoring small duplicate in rec->payload |
386 | */ | 386 | */ |
387 | spin_lock_irqsave(&dbf->pay_lock, flags); | 387 | spin_lock(&dbf->pay_lock); |
388 | memset(payload, 0, sizeof(*payload)); | 388 | memset(payload, 0, sizeof(*payload)); |
389 | memcpy(payload->area, paytag, ZFCP_DBF_TAG_LEN); | 389 | memcpy(payload->area, paytag, ZFCP_DBF_TAG_LEN); |
390 | payload->fsf_req_id = req_id; | 390 | payload->fsf_req_id = req_id; |
diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c index a8762a3efeef..532474109624 100644 --- a/drivers/scsi/ipr.c +++ b/drivers/scsi/ipr.c | |||
@@ -2586,7 +2586,6 @@ static void ipr_process_error(struct ipr_cmnd *ipr_cmd) | |||
2586 | struct ipr_hostrcb *hostrcb = ipr_cmd->u.hostrcb; | 2586 | struct ipr_hostrcb *hostrcb = ipr_cmd->u.hostrcb; |
2587 | u32 ioasc = be32_to_cpu(ipr_cmd->s.ioasa.hdr.ioasc); | 2587 | u32 ioasc = be32_to_cpu(ipr_cmd->s.ioasa.hdr.ioasc); |
2588 | u32 fd_ioasc; | 2588 | u32 fd_ioasc; |
2589 | char *envp[] = { "ASYNC_ERR_LOG=1", NULL }; | ||
2590 | 2589 | ||
2591 | if (ioa_cfg->sis64) | 2590 | if (ioa_cfg->sis64) |
2592 | fd_ioasc = be32_to_cpu(hostrcb->hcam.u.error64.fd_ioasc); | 2591 | fd_ioasc = be32_to_cpu(hostrcb->hcam.u.error64.fd_ioasc); |
@@ -2607,8 +2606,8 @@ static void ipr_process_error(struct ipr_cmnd *ipr_cmd) | |||
2607 | } | 2606 | } |
2608 | 2607 | ||
2609 | list_add_tail(&hostrcb->queue, &ioa_cfg->hostrcb_report_q); | 2608 | list_add_tail(&hostrcb->queue, &ioa_cfg->hostrcb_report_q); |
2609 | schedule_work(&ioa_cfg->work_q); | ||
2610 | hostrcb = ipr_get_free_hostrcb(ioa_cfg); | 2610 | hostrcb = ipr_get_free_hostrcb(ioa_cfg); |
2611 | kobject_uevent_env(&ioa_cfg->host->shost_dev.kobj, KOBJ_CHANGE, envp); | ||
2612 | 2611 | ||
2613 | ipr_send_hcam(ioa_cfg, IPR_HCAM_CDB_OP_CODE_LOG_DATA, hostrcb); | 2612 | ipr_send_hcam(ioa_cfg, IPR_HCAM_CDB_OP_CODE_LOG_DATA, hostrcb); |
2614 | } | 2613 | } |
diff --git a/drivers/scsi/scsi_dh.c b/drivers/scsi/scsi_dh.c index 54d446c9f56e..b8d3b97b217a 100644 --- a/drivers/scsi/scsi_dh.c +++ b/drivers/scsi/scsi_dh.c | |||
@@ -36,9 +36,9 @@ struct scsi_dh_blist { | |||
36 | }; | 36 | }; |
37 | 37 | ||
38 | static const struct scsi_dh_blist scsi_dh_blist[] = { | 38 | static const struct scsi_dh_blist scsi_dh_blist[] = { |
39 | {"DGC", "RAID", "clariion" }, | 39 | {"DGC", "RAID", "emc" }, |
40 | {"DGC", "DISK", "clariion" }, | 40 | {"DGC", "DISK", "emc" }, |
41 | {"DGC", "VRAID", "clariion" }, | 41 | {"DGC", "VRAID", "emc" }, |
42 | 42 | ||
43 | {"COMPAQ", "MSA1000 VOLUME", "hp_sw" }, | 43 | {"COMPAQ", "MSA1000 VOLUME", "hp_sw" }, |
44 | {"COMPAQ", "HSV110", "hp_sw" }, | 44 | {"COMPAQ", "HSV110", "hp_sw" }, |
diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c index 212e98d940bc..6f7128f49c30 100644 --- a/drivers/scsi/scsi_scan.c +++ b/drivers/scsi/scsi_scan.c | |||
@@ -1307,7 +1307,6 @@ static void scsi_sequential_lun_scan(struct scsi_target *starget, | |||
1307 | static int scsi_report_lun_scan(struct scsi_target *starget, int bflags, | 1307 | static int scsi_report_lun_scan(struct scsi_target *starget, int bflags, |
1308 | enum scsi_scan_mode rescan) | 1308 | enum scsi_scan_mode rescan) |
1309 | { | 1309 | { |
1310 | char devname[64]; | ||
1311 | unsigned char scsi_cmd[MAX_COMMAND_SIZE]; | 1310 | unsigned char scsi_cmd[MAX_COMMAND_SIZE]; |
1312 | unsigned int length; | 1311 | unsigned int length; |
1313 | u64 lun; | 1312 | u64 lun; |
@@ -1349,9 +1348,6 @@ static int scsi_report_lun_scan(struct scsi_target *starget, int bflags, | |||
1349 | } | 1348 | } |
1350 | } | 1349 | } |
1351 | 1350 | ||
1352 | sprintf(devname, "host %d channel %d id %d", | ||
1353 | shost->host_no, sdev->channel, sdev->id); | ||
1354 | |||
1355 | /* | 1351 | /* |
1356 | * Allocate enough to hold the header (the same size as one scsi_lun) | 1352 | * Allocate enough to hold the header (the same size as one scsi_lun) |
1357 | * plus the number of luns we are requesting. 511 was the default | 1353 | * plus the number of luns we are requesting. 511 was the default |
@@ -1470,12 +1466,12 @@ retry: | |||
1470 | out_err: | 1466 | out_err: |
1471 | kfree(lun_data); | 1467 | kfree(lun_data); |
1472 | out: | 1468 | out: |
1473 | scsi_device_put(sdev); | ||
1474 | if (scsi_device_created(sdev)) | 1469 | if (scsi_device_created(sdev)) |
1475 | /* | 1470 | /* |
1476 | * the sdev we used didn't appear in the report luns scan | 1471 | * the sdev we used didn't appear in the report luns scan |
1477 | */ | 1472 | */ |
1478 | __scsi_remove_device(sdev); | 1473 | __scsi_remove_device(sdev); |
1474 | scsi_device_put(sdev); | ||
1479 | return ret; | 1475 | return ret; |
1480 | } | 1476 | } |
1481 | 1477 | ||