On 08/24/2012 04:08 AM, Bart Van Assche wrote:
> A quote from SPC-4: "While in the unavailable primary target port
> asymmetric access state, the device server shall support those of
> the following commands that it supports while in the active/optimized
> state: [ ... ] d) SET TARGET PORT GROUPS; [ ... ]". Hence enable
> sending STPG to a target port group that is in the unavailable state.
> 
> Signed-off-by: Bart Van Assche <[email protected]>
> Cc: Hannes Reinecke <[email protected]>
> Cc: Mike Christie <[email protected]>
> Cc: Mike Snitzer <[email protected]>
> Cc: Babu Moger <[email protected]>
> Cc: <[email protected]>
> ---
>  drivers/scsi/device_handler/scsi_dh_alua.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> 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:
> 

Hey, was this patch ok Hannes and James and others?

I see bb2c94a3a6ad37fe7d34f369295bcdf5387e264f was merged, but this one
was not.

I was thinking this might have got missed because it has the same patch
description as  bb2c94a3a6ad37fe7d34f369295bcdf5387e264f, so it is
confusion, but they are different patches. One fixes alua_rtpg and one
fixes alua_activate, but both are needed.
--
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