The branch main has been updated by kib:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=2f7ce2326f9486e90cda2f0129331e1ab3694252

commit 2f7ce2326f9486e90cda2f0129331e1ab3694252
Author:     Hans Petter Selasky <[email protected]>
AuthorDate: 2021-06-16 13:01:27 +0000
Commit:     Konstantin Belousov <[email protected]>
CommitDate: 2021-07-12 09:34:39 +0000

    mlx5en: Add missing error case when creating VXLAN flow tables.
    
    While at it remove redundant checks for IFCAP_VXLAN_HWCSUM flag.
    
    MFC after:      1 week
    Reviewed by:    kib
    Sponsored by:   Mellanox Technologies // NVIDIA Networking
---
 sys/dev/mlx5/mlx5_en/mlx5_en_flow_table.c | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/sys/dev/mlx5/mlx5_en/mlx5_en_flow_table.c 
b/sys/dev/mlx5/mlx5_en/mlx5_en_flow_table.c
index 6be95550c713..cdfe19d910d1 100644
--- a/sys/dev/mlx5/mlx5_en/mlx5_en_flow_table.c
+++ b/sys/dev/mlx5/mlx5_en/mlx5_en_flow_table.c
@@ -2252,34 +2252,28 @@ mlx5e_open_flow_table(struct mlx5e_priv *priv)
                err = mlx5e_create_main_flow_table(priv, true);
                if (err)
                        goto err_destroy_main_flow_table;
-       }
 
-       if ((priv->ifp->if_capenable & IFCAP_VXLAN_HWCSUM) != 0) {
                err = mlx5e_create_inner_rss_flow_table(priv);
                if (err)
                        goto err_destroy_main_vxlan_flow_table;
-       }
 
-       if ((priv->ifp->if_capenable & IFCAP_VXLAN_HWCSUM) != 0) {
                err = mlx5e_add_vxlan_catchall_rule(priv);
                if (err != 0)
                        goto err_destroy_inner_rss_flow_table;
-       }
 
-       if ((priv->ifp->if_capenable & IFCAP_VXLAN_HWCSUM) != 0) {
                err = mlx5e_add_main_vxlan_rules(priv);
                if (err != 0)
-                       goto err_destroy_inner_rss_flow_table;
+                       goto err_destroy_vxlan_catchall_rule;
        }
 
        return (0);
 
+err_destroy_vxlan_catchall_rule:
+       mlx5e_del_vxlan_catchall_rule(priv);
 err_destroy_inner_rss_flow_table:
-       if ((priv->ifp->if_capenable & IFCAP_VXLAN_HWCSUM) != 0)
-               mlx5e_destroy_inner_rss_flow_table(priv);
+       mlx5e_destroy_inner_rss_flow_table(priv);
 err_destroy_main_vxlan_flow_table:
-       if ((priv->ifp->if_capenable & IFCAP_VXLAN_HWCSUM) != 0)
-               mlx5e_destroy_main_vxlan_flow_table(priv);
+       mlx5e_destroy_main_vxlan_flow_table(priv);
 err_destroy_main_flow_table:
        mlx5e_destroy_main_flow_table(priv);
 err_destroy_vxlan_flow_table:
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main
To unsubscribe, send any mail to "[email protected]"

Reply via email to