Commit 13795553 authored by Parav Pandit's avatar Parav Pandit Committed by Saeed Mahameed
Browse files

net/mlx5: E-Switch Make cleanup sequence mirror of init 



Make cleanup sequence mirror of init sequence for cleaning up reps
and freeing vports.

Also when reps initialization fails, there is no need to perform reps
cleanup.

Signed-off-by: default avatarParav Pandit <parav@nvidia.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@nvidia.com>
parent 6308a5f0

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

+1 −2
Original line number Diff line number Diff line
@@ -1856,7 +1856,6 @@ int mlx5_eswitch_init(struct mlx5_core_dev *dev) 
abort:

	if (esw->work_queue)

		destroy_workqueue(esw->work_queue);

	esw_offloads_cleanup_reps(esw);

	kfree(esw->vports);

	kfree(esw);

	return err;
@@ -1871,7 +1870,6 @@ void mlx5_eswitch_cleanup(struct mlx5_eswitch *esw) 


	esw->dev->priv.eswitch = NULL;

	destroy_workqueue(esw->work_queue);

	esw_offloads_cleanup_reps(esw);

	mutex_destroy(&esw->state_lock);

	WARN_ON(!xa_empty(&esw->offloads.vhca_map));

	xa_destroy(&esw->offloads.vhca_map);
@@ -1879,6 +1877,7 @@ void mlx5_eswitch_cleanup(struct mlx5_eswitch *esw) 
	mlx5e_mod_hdr_tbl_destroy(&esw->offloads.mod_hdr);

	mutex_destroy(&esw->offloads.encap_tbl_lock);

	mutex_destroy(&esw->offloads.decap_tbl_lock);

	esw_offloads_cleanup_reps(esw);

	kfree(esw->vports);

	kfree(esw);

}