Re: [PATCH] multipath-tools: hwtable housekeeping

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Sep 25, 2024 at 01:55:22AM +0200, Xose Vazquez Perez wrote:
> Update some descriptions. And limit the Dorado regex,
> because "Dorado "is used as a generic brand by Huawei.
> 
> =====================    
> HUASY   Dorado2100
> HUAWEI  Dorado2100
> HUAWEI  Dorado2100 G2
> HUASY   Dorado5100
> HUAWEI  Dorado5100
> =====================    
> 
> Cc: Martin Wilck <mwilck@xxxxxxxx>
> Cc: Benjamin Marzinski <bmarzins@xxxxxxxxxx>
> Cc: Christophe Varoqui <christophe.varoqui@xxxxxxxxxxx>
> Cc: DM-DEVEL ML <dm-devel@xxxxxxxxxxxxxxx>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@xxxxxxxxx>

Reviewed-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx>

> ---
>  libmultipath/hwtable.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index c539a9b5..01d50303 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -819,7 +819,7 @@ static struct hwentry default_hw[] = {
>  	 */
>  	{
>  		/*
> -		 * ONTAP FAS/AFF Series
> +		 * ONTAP FAS/AFF/ASA Series
>  		 *
>  		 * Maintainer: Martin George <marting@xxxxxxxxxx>
>  		 */
> @@ -1099,7 +1099,7 @@ static struct hwentry default_hw[] = {
>  	 * Pure Storage
>  	 */
>  	{
> -		/* FlashArray */
> +		/* FlashArray family */
>  		.vendor        = "PURE",
>  		.product       = "FlashArray",
>  		.pgpolicy      = GROUP_BY_PRIO,
> @@ -1120,15 +1120,15 @@ static struct hwentry default_hw[] = {
>  	 * Huawei
>  	 */
>  	{
> -		/* All, except OceanStor V3-V6 */
> +		/* Older than OceanStor V3 */
>  		.vendor        = "^(HUAWEI|HUASY|HS)",
> -		.product       = "^(Dorado|HVS8|S[23568]|V1[568]|VIS6000)",
> +		.product       = "^(Dorado[25]|HVS8|S[23568]|V1[568]|VIS6000)",
>  		.pgpolicy      = GROUP_BY_PRIO,
>  		.pgfailback    = -FAILBACK_IMMEDIATE,
>  		.no_path_retry = 15,
>  	},
>  	{
> -		/* OceanStor V3-V6 */
> +		/* OceanStor V3 or better */
>  		.vendor        = "^(HUAWEI|AnyStor|Marstor|NETPOSA|SanM|SUGON|UDsafe)",
>  		.product       = "XSG1",
>  		.pgpolicy      = GROUP_BY_PRIO,
> -- 
> 2.46.1





[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux