Commit 684f062c authored by Shay Drory's avatar Shay Drory Committed by Saeed Mahameed
Browse files

net/mlx5: group fdb cleanup to single function 



Currently, the allocation of fdb software objects are done is single
function, oppose to the cleanup of them.
Group the cleanup of fdb software objects to single function.

Signed-off-by: default avatarShay Drory <shayd@nvidia.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@nvidia.com>
parent 91707779

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

+10 −8
Original line number Diff line number Diff line
@@ -2866,6 +2866,14 @@ static int create_fdb_bypass(struct mlx5_flow_steering *steering) 
	return 0;

}



static void cleanup_fdb_root_ns(struct mlx5_flow_steering *steering)

{

	cleanup_root_ns(steering->fdb_root_ns);

	steering->fdb_root_ns = NULL;

	kfree(steering->fdb_sub_ns);

	steering->fdb_sub_ns = NULL;

}



static int init_fdb_root_ns(struct mlx5_flow_steering *steering)

{

	struct fs_prio *maj_prio;
@@ -2916,10 +2924,7 @@ static int init_fdb_root_ns(struct mlx5_flow_steering *steering) 
	return 0;



out_err:

	cleanup_root_ns(steering->fdb_root_ns);

	kfree(steering->fdb_sub_ns);

	steering->fdb_sub_ns = NULL;

	steering->fdb_root_ns = NULL;

	cleanup_fdb_root_ns(steering);

	return err;

}
@@ -3079,10 +3084,7 @@ void mlx5_fs_core_cleanup(struct mlx5_core_dev *dev) 
	struct mlx5_flow_steering *steering = dev->priv.steering;



	cleanup_root_ns(steering->root_ns);

	cleanup_root_ns(steering->fdb_root_ns);

	steering->fdb_root_ns = NULL;

	kfree(steering->fdb_sub_ns);

	steering->fdb_sub_ns = NULL;

	cleanup_fdb_root_ns(steering);

	cleanup_root_ns(steering->port_sel_root_ns);

	cleanup_root_ns(steering->sniffer_rx_root_ns);

	cleanup_root_ns(steering->sniffer_tx_root_ns);