----- Original Message -----
> From: "Hannes Reinecke" <h...@suse.de>
> To: "Martin K. Petersen" <martin.peter...@oracle.com>
> Cc: "Christoph Hellwig" <h...@lst.de>, "James Bottomley"
> <james.bottom...@hansenpartnership.com>,
> linux-scsi@vger.kernel.org, "Hannes Reinecke" <h...@suse.de>, "Hannes
> Reinecke" <h...@suse.com>
> Sent: Monday, September 12, 2016 4:20:53 AM
> Subject: [PATCH] scsi: do not print 'reservation conflict' for TEST UNIT READY
>
> SPC-2 and SPC-3 (or later) differ in the handling of reservation
> conflict for TEST UNIT READY. SPC-2 will return 'reservation conflict',
> whereas SPC-3 will return GOOD status.
> On a mixed system with both SPC-2 and SPC-3 targets one will
> see lots of 'reservation conflict' messages from the SPC-2 system but
> no messages from the SPC-3 system when eg multipath path checkers.
> These messages might confuse the unsuspecting user although in fact
> they just signal normal operation.
> So we should not be printing out 'reservation conflict' for
> TEST UNIT READY responses.
>
> Signed-off-by: Hannes Reinecke <h...@suse.com>
> ---
> drivers/scsi/scsi_error.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c
> index 106a6ad..3040fe5 100644
> --- a/drivers/scsi/scsi_error.c
> +++ b/drivers/scsi/scsi_error.c
> @@ -1923,8 +1923,9 @@ int scsi_decide_disposition(struct scsi_cmnd *scmd)
> return SUCCESS;
>
> case RESERVATION_CONFLICT:
> - sdev_printk(KERN_INFO, scmd->device,
> - "reservation conflict\n");
> + if (scmd->cmnd[0] != TEST_UNIT_READY)
> + sdev_printk(KERN_INFO, scmd->device,
> + "reservation conflict\n");
> set_host_byte(scmd, DID_NEXUS_FAILURE);
> return SUCCESS; /* causes immediate i/o error */
> default:
> --
> 1.8.5.6
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
Looks good to me.
Reviewed-by Laurence Oberman <lober...@redhat.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
