Re: [PATCH 11/16] scsi_dh_alua: multipath failover fails with error 15
On 01/31/14 10:29, Hannes Reinecke wrote: When a path is already optimized multipath failover will fail with the message Could not failover device X:Y: Handler scsi_dh_alua Error 15 Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/device_handler/scsi_dh_alua.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/device_handler/scsi_dh_alua.c b/drivers/scsi/device_handler/scsi_dh_alua.c index a1c69bb..8ea35a9 100644 --- a/drivers/scsi/device_handler/scsi_dh_alua.c +++ b/drivers/scsi/device_handler/scsi_dh_alua.c @@ -851,6 +851,8 @@ static unsigned alua_stpg(struct scsi_device *sdev, struct alua_port_group *pg) return SCSI_DH_RETRY; } switch (pg-state) { + case TPGS_STATE_OPTIMIZED: + return SCSI_DH_OK; case TPGS_STATE_NONOPTIMIZED: if ((pg-flags ALUA_OPTIMIZE_STPG) (!pg-pref) @@ -865,10 +867,11 @@ static unsigned alua_stpg(struct scsi_device *sdev, struct alua_port_group *pg) break; case TPGS_STATE_TRANSITIONING: return SCSI_DH_RETRY; - break; default: + sdev_printk(KERN_INFO, sdev, + %s: stpg failed, unhandled TPGS state %d, + ALUA_DH_NAME, pg-state); return SCSI_DH_NOSYS; - break; } /* Set state to transitioning */ pg-state = TPGS_STATE_TRANSITIONING; (replying to an e-mail of last year) Hello Hannes, Our Q.A. team started to run into the issue that is fixed by this patch. Do you have the time to resend this patch series or would you rather prefer that I split out this patch, test it and post it with a Cc: stable tag ? Thanks, Bart. -- 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
Re: [PATCH 11/16] scsi_dh_alua: multipath failover fails with error 15
On 03/30/2015 02:30 PM, Bart Van Assche wrote: On 01/31/14 10:29, Hannes Reinecke wrote: When a path is already optimized multipath failover will fail with the message Could not failover device X:Y: Handler scsi_dh_alua Error 15 Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/device_handler/scsi_dh_alua.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/device_handler/scsi_dh_alua.c b/drivers/scsi/device_handler/scsi_dh_alua.c index a1c69bb..8ea35a9 100644 --- a/drivers/scsi/device_handler/scsi_dh_alua.c +++ b/drivers/scsi/device_handler/scsi_dh_alua.c @@ -851,6 +851,8 @@ static unsigned alua_stpg(struct scsi_device *sdev, struct alua_port_group *pg) return SCSI_DH_RETRY; } switch (pg-state) { +case TPGS_STATE_OPTIMIZED: +return SCSI_DH_OK; case TPGS_STATE_NONOPTIMIZED: if ((pg-flags ALUA_OPTIMIZE_STPG) (!pg-pref) @@ -865,10 +867,11 @@ static unsigned alua_stpg(struct scsi_device *sdev, struct alua_port_group *pg) break; case TPGS_STATE_TRANSITIONING: return SCSI_DH_RETRY; -break; default: +sdev_printk(KERN_INFO, sdev, +%s: stpg failed, unhandled TPGS state %d, +ALUA_DH_NAME, pg-state); return SCSI_DH_NOSYS; -break; } /* Set state to transitioning */ pg-state = TPGS_STATE_TRANSITIONING; (replying to an e-mail of last year) Hello Hannes, Our Q.A. team started to run into the issue that is fixed by this patch. Do you have the time to resend this patch series or would you rather prefer that I split out this patch, test it and post it with a Cc: stable tag ? Well, I'll have to repost the patch-series anyway; unfortunately it's being held off by some unresolved issues. Best I'll split if off into 'real' bugfixes and the actual scsi_dh_alua updates. Cheers, Hannes -- Dr. Hannes ReineckezSeries Storage h...@suse.de +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- 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
[PATCH 11/16] scsi_dh_alua: multipath failover fails with error 15
When a path is already optimized multipath failover will fail with the message Could not failover device X:Y: Handler scsi_dh_alua Error 15 Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/device_handler/scsi_dh_alua.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/device_handler/scsi_dh_alua.c b/drivers/scsi/device_handler/scsi_dh_alua.c index a1c69bb..8ea35a9 100644 --- a/drivers/scsi/device_handler/scsi_dh_alua.c +++ b/drivers/scsi/device_handler/scsi_dh_alua.c @@ -851,6 +851,8 @@ static unsigned alua_stpg(struct scsi_device *sdev, struct alua_port_group *pg) return SCSI_DH_RETRY; } switch (pg-state) { + case TPGS_STATE_OPTIMIZED: + return SCSI_DH_OK; case TPGS_STATE_NONOPTIMIZED: if ((pg-flags ALUA_OPTIMIZE_STPG) (!pg-pref) @@ -865,10 +867,11 @@ static unsigned alua_stpg(struct scsi_device *sdev, struct alua_port_group *pg) break; case TPGS_STATE_TRANSITIONING: return SCSI_DH_RETRY; - break; default: + sdev_printk(KERN_INFO, sdev, + %s: stpg failed, unhandled TPGS state %d, + ALUA_DH_NAME, pg-state); return SCSI_DH_NOSYS; - break; } /* Set state to transitioning */ pg-state = TPGS_STATE_TRANSITIONING; -- 1.7.12.4 -- 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
[PATCH 11/16] scsi_dh_alua: multipath failover fails with error 15
When a path is already optimized multipath failover will fail with the message Could not failover device X:Y: Handler scsi_dh_alua Error 15 Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/device_handler/scsi_dh_alua.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/device_handler/scsi_dh_alua.c b/drivers/scsi/device_handler/scsi_dh_alua.c index 51f4a31..dc77ce4 100644 --- a/drivers/scsi/device_handler/scsi_dh_alua.c +++ b/drivers/scsi/device_handler/scsi_dh_alua.c @@ -851,6 +851,8 @@ static unsigned alua_stpg(struct scsi_device *sdev, struct alua_port_group *pg) return SCSI_DH_RETRY; } switch (pg-state) { + case TPGS_STATE_OPTIMIZED: + return SCSI_DH_OK; case TPGS_STATE_NONOPTIMIZED: if ((pg-flags ALUA_OPTIMIZE_STPG) (!pg-pref) @@ -865,10 +867,11 @@ static unsigned alua_stpg(struct scsi_device *sdev, struct alua_port_group *pg) break; case TPGS_STATE_TRANSITIONING: return SCSI_DH_RETRY; - break; default: + sdev_printk(KERN_INFO, sdev, + %s: stpg failed, unhandled TPGS state %d, + ALUA_DH_NAME, pg-state); return SCSI_DH_NOSYS; - break; } /* Set state to transitioning */ pg-state = TPGS_STATE_TRANSITIONING; -- 1.7.12.4 -- 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