Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 3fe3d819 authored by Or Gerlitz's avatar Or Gerlitz Committed by David S. Miller
Browse files

net/mlx5: E-Switch, Use the correct free() function 



We must use kvfree() for something that could have been allocated with vzalloc(),
do that.

Fixes: 5742df0f ('net/mlx5: E-Switch, Introduce VST vport ingress/egress ACLs')
Fixes: 86d722ad ('net/mlx5: Use flow steering infrastructure for mlx5_en')
Signed-off-by: default avatarOr Gerlitz <ogerlitz@mellanox.com>
Reported-by: default avatarIlya Lesokhin <ilyal@mellanox.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent bd02ef8e

drivers/net/ethernet/mellanox/mlx5/core/eswitch.c

+3 −3
Original line number Diff line number Diff line
@@ -529,7 +529,7 @@ static int esw_create_fdb_table(struct mlx5_eswitch *esw, int nvports) 
		}

	}



	kfree(flow_group_in);

	kvfree(flow_group_in);

	return err;

}
@@ -1097,7 +1097,7 @@ static void esw_vport_enable_egress_acl(struct mlx5_eswitch *esw, 
	vport->egress.drop_grp = drop_grp;

	vport->egress.allowed_vlans_grp = vlan_grp;

out:

	kfree(flow_group_in);

	kvfree(flow_group_in);

	if (err && !IS_ERR_OR_NULL(vlan_grp))

		mlx5_destroy_flow_group(vlan_grp);

	if (err && !IS_ERR_OR_NULL(acl))
@@ -1259,7 +1259,7 @@ static void esw_vport_enable_ingress_acl(struct mlx5_eswitch *esw, 
			mlx5_destroy_flow_table(vport->ingress.acl);

	}



	kfree(flow_group_in);

	kvfree(flow_group_in);

}



static void esw_vport_cleanup_ingress_rules(struct mlx5_eswitch *esw,