Re: [PATCH] scsi_dh_alua: skip RTPG for devices only supporting active/optimized

2017-12-13 Thread Hannes Reinecke
On 12/12/2017 07:25 PM, Bart Van Assche wrote: > On Tue, 2017-12-12 at 08:01 +0100, Hannes Reinecke wrote: >> On 12/12/2017 01:00 AM, Bart Van Assche wrote: >>> On Fri, 2017-12-08 at 11:14 +0100, Hannes Reinecke wrote: @@ -541,6 +544,20 @@ static int alua_rtpg(struct scsi_device *sdev, struct

Re: [PATCH] scsi_dh_alua: skip RTPG for devices only supporting active/optimized

2017-12-12 Thread Bart Van Assche
On Tue, 2017-12-12 at 08:01 +0100, Hannes Reinecke wrote: > On 12/12/2017 01:00 AM, Bart Van Assche wrote: > > On Fri, 2017-12-08 at 11:14 +0100, Hannes Reinecke wrote: > > > @@ -541,6 +544,20 @@ static int alua_rtpg(struct scsi_device *sdev, > > > struct alua_port_group *pg) > > > retval = subm

Re: [PATCH] scsi_dh_alua: skip RTPG for devices only supporting active/optimized

2017-12-11 Thread Hannes Reinecke
On 12/12/2017 01:00 AM, Bart Van Assche wrote: > On Fri, 2017-12-08 at 11:14 +0100, Hannes Reinecke wrote: >> @@ -541,6 +544,20 @@ static int alua_rtpg(struct scsi_device *sdev, struct >> alua_port_group *pg) >> retval = submit_rtpg(sdev, buff, bufflen, &sense_hdr, pg->flags); >> >> if

Re: [PATCH] scsi_dh_alua: skip RTPG for devices only supporting active/optimized

2017-12-11 Thread Bart Van Assche
On Fri, 2017-12-08 at 11:14 +0100, Hannes Reinecke wrote: > @@ -541,6 +544,20 @@ static int alua_rtpg(struct scsi_device *sdev, struct > alua_port_group *pg) > retval = submit_rtpg(sdev, buff, bufflen, &sense_hdr, pg->flags); > > if (retval) { > + /* > + * If

[PATCH] scsi_dh_alua: skip RTPG for devices only supporting active/optimized

2017-12-08 Thread Hannes Reinecke
From: Hannes Reinecke For hardware only supporting active/optimized there's no point in ever re-issuing RTPG as the only new state we can possibly read is active/optimized. This avoid spurious errors during path failover on such arrays. Signed-off-by: Hannes Reinecke --- drivers/scsi/device_ha