Re: [PATCH 2/2] Merge Dell MD3XXX hardware table entries

[Xose Vazquez Perez <xose.vazquez@gmail.com>]

Hannes Reinecke wrote(2015-03-30 6:23:23):

> Remove duplicate entries and merge MD36xxi/MD36xxf into a single
> one.
> 
> Signed-off-by: Hannes Reinecke <hare@suse.de>
> ---
>  libmultipath/hwtable.c | 53 ++++----------------------------------------------
>  1 file changed, 4 insertions(+), 49 deletions(-)
> 
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index fa4b20f..f83f3e7 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -728,7 +728,7 @@ static struct hwentry default_hw[] = {
>  		.prio_args     = NULL,
>  	},
>  	{
> -		/* DELL MD3000 */
> +		/* DELL MD3000/MD3000i */
>  		.vendor        = "DELL",
>  		.product       = "MD3000",
>  		.bl_product    = "Universal Xport",
> @@ -743,22 +743,7 @@ static struct hwentry default_hw[] = {
>  		.prio_args     = NULL,
>  	},
>  	{
> -		/* DELL MD3000i */
> -		.vendor        = "DELL",
> -		.product       = "MD3000i",
> -		.bl_product    = "Universal Xport",
> -		.features      = "2 pg_init_retries 50",
> -		.hwhandler     = "1 rdac",
> -		.pgpolicy      = GROUP_BY_PRIO,
> -		.pgfailback    = -FAILBACK_IMMEDIATE,
> -		.rr_weight     = RR_WEIGHT_NONE,
> -		.no_path_retry = 15,
> -		.checker_name  = RDAC,
> -		.prio_name     = PRIO_RDAC,
> -		.prio_args     = NULL,
> -	},
> -	{
> -		/* DELL MD32xx */
> +		/* DELL MD32xx/MD32xxi */
>  		.vendor        = "DELL",
>  		.product       = "MD32xx",
>  		.bl_product    = "Universal Xport",
> @@ -773,39 +758,9 @@ static struct hwentry default_hw[] = {
>  		.prio_args     = NULL,
>  	},
>  	{
> -		/* DELL MD32xxi */
> -		.vendor        = "DELL",
> -		.product       = "MD32xxi",
> -		.bl_product    = "Universal Xport",
> -		.features      = "2 pg_init_retries 50",
> -		.hwhandler     = "1 rdac",
> -		.pgpolicy      = GROUP_BY_PRIO,
> -		.pgfailback    = -FAILBACK_IMMEDIATE,
> -		.rr_weight     = RR_WEIGHT_NONE,
> -		.no_path_retry = 15,
> -		.checker_name  = RDAC,
> -		.prio_name     = PRIO_RDAC,
> -		.prio_args     = NULL,
> -	},
> -	{
> -		/* DELL MD36xxi */
> -		.vendor        = "DELL",
> -		.product       = "MD36xxi",
> -		.bl_product    = "Universal Xport",
> -		.features      = "2 pg_init_retries 50",
> -		.hwhandler     = "1 rdac",
> -		.pgpolicy      = GROUP_BY_PRIO,
> -		.pgfailback    = -FAILBACK_IMMEDIATE,
> -		.rr_weight     = RR_WEIGHT_NONE,
> -		.no_path_retry = 15,
> -		.checker_name  = RDAC,
> -		.prio_name     = PRIO_RDAC,
> -		.prio_args     = NULL,
> -	},
> -	{
> -		/* DELL MD36xxf */
> +		/* DELL MD36xxi/MD36xxf */
>  		.vendor        = "DELL",
> -		.product       = "MD36xxf",
> +		.product       = "MD36xx(i|f)",
>  		.bl_product    = "Universal Xport",
>  		.features      = "2 pg_init_retries 50",
>  		.hwhandler     = "1 rdac",
> -- 
> 1.8.4.5

This patch is missing in upstream git.