OSDN Git Service

net/mlx5e: Remove redundant __func__ arg from fs_err() calls
authorRoi Dayan <roid@nvidia.com>
Thu, 30 Mar 2023 09:25:25 +0000 (12:25 +0300)
committerSaeed Mahameed <saeedm@nvidia.com>
Fri, 19 May 2023 17:50:29 +0000 (10:50 -0700)
fs_err() already logs the function name. remote the arg so the function
name will not be logged twice.

Signed-off-by: Roi Dayan <roid@nvidia.com>
Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
drivers/net/ethernet/mellanox/mlx5/core/en_fs.c

index 33bfe4d..934b0d5 100644 (file)
@@ -283,7 +283,7 @@ static int __mlx5e_add_vlan_rule(struct mlx5e_flow_steering *fs,
        if (IS_ERR(*rule_p)) {
                err = PTR_ERR(*rule_p);
                *rule_p = NULL;
-               fs_err(fs, "%s: add rule failed\n", __func__);
+               fs_err(fs, "add rule failed\n");
        }
 
        return err;
@@ -395,8 +395,7 @@ int mlx5e_add_vlan_trap(struct mlx5e_flow_steering *fs, int trap_id, int tir_num
        if (IS_ERR(rule)) {
                err = PTR_ERR(rule);
                fs->vlan->trap_rule = NULL;
-               fs_err(fs, "%s: add VLAN trap rule failed, err %d\n",
-                      __func__, err);
+               fs_err(fs, "add VLAN trap rule failed, err %d\n", err);
                return err;
        }
        fs->vlan->trap_rule = rule;
@@ -421,8 +420,7 @@ int mlx5e_add_mac_trap(struct mlx5e_flow_steering *fs, int trap_id, int tir_num)
        if (IS_ERR(rule)) {
                err = PTR_ERR(rule);
                fs->l2.trap_rule = NULL;
-               fs_err(fs, "%s: add MAC trap rule failed, err %d\n",
-                      __func__, err);
+               fs_err(fs, "add MAC trap rule failed, err %d\n", err);
                return err;
        }
        fs->l2.trap_rule = rule;
@@ -763,7 +761,7 @@ static int mlx5e_add_promisc_rule(struct mlx5e_flow_steering *fs)
        if (IS_ERR(*rule_p)) {
                err = PTR_ERR(*rule_p);
                *rule_p = NULL;
-               fs_err(fs, "%s: add promiscuous rule failed\n", __func__);
+               fs_err(fs, "add promiscuous rule failed\n");
        }
        kvfree(spec);
        return err;
@@ -995,7 +993,7 @@ static int mlx5e_add_l2_flow_rule(struct mlx5e_flow_steering *fs,
 
        ai->rule = mlx5_add_flow_rules(ft, spec, &flow_act, &dest, 1);
        if (IS_ERR(ai->rule)) {
-               fs_err(fs, "%s: add l2 rule(mac:%pM) failed\n", __func__, mv_dmac);
+               fs_err(fs, "add l2 rule(mac:%pM) failed\n", mv_dmac);
                err = PTR_ERR(ai->rule);
                ai->rule = NULL;
        }