[dm-devel] [PATCH RFC] multipath-tools: *untested* use sysfs prio also for arrays with dual implicit/explicit alua support

[Martin Wilck]

Hi Xose,

do you have a specific use case where this helps / changes matters?

The idea of the current code is to use PRIO_SYSFS always, except for
RDAC arrays with implicit ALUA, where periodic RTPGs are necessary (see
7e2f46d). This patch would change matters only for RDAC arrays which
advertise both explicit and implicit ALUA support.

Maybe Steven can clarify if such RDAC arrays exist, and whether we
should use PRIO_SYSFS or PRIO_ALUA for them.

Martin


On Sat, 2020-07-25 at 00:34 +0200, Xose Vazquez Perez wrote:
> Cc: Martin Wilck <mwilck at suse.com>
> Cc: Benjamin Marzinski <bmarzins at redhat.com>
> Cc: Hannes Reinecke <hare at suse.de>
> Cc: DM-DEVEL ML <dm-devel at redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez at gmail.com>
> ---
> diff --git a/libmultipath/propsel.c b/libmultipath/propsel.c
> index 897e48ca..5a82234f 100644
> --- a/libmultipath/propsel.c
> +++ b/libmultipath/propsel.c
> @@ -595,7 +595,7 @@ detect_prio(struct config *conf, struct path *
> pp)
>   		tpgs = path_get_tpgs(pp);
>   		if (tpgs == TPGS_NONE)
>   			return;
> -		if ((tpgs == TPGS_EXPLICIT || !check_rdac(pp)) &&
> +		if ((tpgs == TPGS_EXPLICIT || tpgs == TPGS_BOTH ||
> !check_rdac(pp)) &&
>   		    sysfs_get_asymmetric_access_state(pp, buff, 512) >=
> 0)
>   			default_prio = PRIO_SYSFS;
>   		else
> 
> In short:
> 
> diff --git a/libmultipath/propsel.c b/libmultipath/propsel.c
> index 897e48ca..a9609a01 100644
> --- a/libmultipath/propsel.c
> +++ b/libmultipath/propsel.c
> @@ -595,7 +595,7 @@ detect_prio(struct config *conf, struct path *
> pp)
>   		tpgs = path_get_tpgs(pp);
>   		if (tpgs == TPGS_NONE)
>   			return;
> -		if ((tpgs == TPGS_EXPLICIT || !check_rdac(pp)) &&
> +		if ((tpgs != TPGS_IMPLICIT || !check_rdac(pp)) &&
>   		    sysfs_get_asymmetric_access_state(pp, buff, 512) >=
> 0)
>   			default_prio = PRIO_SYSFS;
>   		else
>