On Aug 24, 2012, at 4:08 AM, Bart Van Assche <[email protected]> wrote:
> diff --git a/drivers/scsi/device_handler/scsi_dh_alua.c 
> b/drivers/scsi/device_handler/scsi_dh_alua.c
> index 08d80a6..a05d469 100644
> --- a/drivers/scsi/device_handler/scsi_dh_alua.c
> +++ b/drivers/scsi/device_handler/scsi_dh_alua.c
> @@ -641,8 +641,7 @@ static int alua_rtpg(struct scsi_device *sdev, struct 
> alua_dh_data *h)
>               h->state = TPGS_STATE_STANDBY;
>               break;
>       case TPGS_STATE_OFFLINE:
> -     case TPGS_STATE_UNAVAILABLE:
> -             /* Path unusable for unavailable/offline */
> +             /* Path unusable */
>               err = SCSI_DH_DEV_OFFLINED;
>               break;
>       default:

We must not have tested unavailable state very much. 3rd bug in a short time. 
Looks ok to me.

Reviewed-by: Mike Christie <[email protected]>


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to