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 <mwi...@suse.com>
> Cc: Benjamin Marzinski <bmarz...@redhat.com>
> Cc: Christophe Varoqui <christophe.varo...@opensvc.com>
> Cc: DM-DEVEL ML <dm-devel@lists.linux.dev>
> Signed-off-by: Xose Vazquez Perez <xose.vazq...@gmail.com>

Reviewed-by: Benjamin Marzinski <bmarz...@redhat.com>

> ---
>  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 <mart...@netapp.com>
>                */
> @@ -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


Reply via email to