[dm-devel] [PATCH] multipath-tools: add VIOLIN arrays to hardware table

[Hannes Reinecke]

On 10/15/2016 01:50 PM, Xose Vazquez Perez wrote:
> Based on documentation provided by the manufacturer:
> https://drive.google.com/open?id=0B_B6YmEmO7cDQlMzc1BsaUxZRVU
>
> Cc: Christophe Varoqui <christophe.varoqui at opensvc.com>
> Cc: device-mapper development <dm-devel at redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez at gmail.com>
> ---
>  libmultipath/hwtable.c | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
>
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index aca46ab..e34032d 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -944,6 +944,32 @@ static struct hwentry default_hw[] = {
>  		.pgpolicy      = MULTIBUS,
>  		.no_path_retry = 12,
>  	},
> +	/*
> +	 * Violin Memory
> +	 */
> +	{
> +		/* V3000 / V6000 Series */
> +		.vendor        = "VIOLIN",
> +		.product       = "SAN ARRAY$",
> +		.pgpolicy      = GROUP_BY_SERIAL,
> +		.no_path_retry = 30,
> +	},
> +	{
> +		.vendor        = "VIOLIN",
> +		.product       = "SAN ARRAY ALUA",
> +		.hwhandler     = "1 alua",
> +		.pgpolicy      = GROUP_BY_PRIO,
> +		.pgfailback    = -FAILBACK_IMMEDIATE,
> +		.prio_name     = PRIO_ALUA,
> +		.no_path_retry = 30,
> +	},
> +	{
> +		/* FSP 7000 family */
> +		.vendor        = "VIOLIN",
> +		.product       = "CONCERTO ARRAY",
> +		.pgpolicy      = MULTIBUS,
> +		.no_path_retry = 30,
> +	},
>  #if 0
>  	/*
>  	 * Copy this TEMPLATE to add new hardware.
>
Where's the point of the first and the third entry?
They'll be identical to the default settings, no?

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare at suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)