Re: [net 04/14] net/mlx5: Disable eswitch before waiting for VF pages

[Piotr Raczynski <piotr.raczynski@intel.com>]

On Tue, Mar 14, 2023 at 10:49:30AM -0700, Saeed Mahameed wrote:
> From: Daniel Jurgens <danielj@nvidia.com>
> 
> The offending commit changed the ordering of moving to legacy mode and
> waiting for the VF pages. Moving to legacy mode is important in
> bluefield, because it sends the host driver into error state, and frees
> its pages. Without this transition we end up waiting 2 minutes for
> pages that aren't coming before carrying on with the unload process.
> 
> Fixes: f019679ea5f2 ("net/mlx5: E-switch, Remove dependency between sriov and eswitch mode")
> 
> Signed-off-by: Daniel Jurgens <danielj@nvidia.com>
> Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
> ---
>  drivers/net/ethernet/mellanox/mlx5/core/main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/mellanox/mlx5/core/main.c b/drivers/net/ethernet/mellanox/mlx5/core/main.c
> index 540840e80493..f36a3aa4b5c8 100644
> --- a/drivers/net/ethernet/mellanox/mlx5/core/main.c
> +++ b/drivers/net/ethernet/mellanox/mlx5/core/main.c
> @@ -1364,8 +1364,8 @@ static void mlx5_unload(struct mlx5_core_dev *dev)
>  {
>  	mlx5_devlink_traps_unregister(priv_to_devlink(dev));
>  	mlx5_sf_dev_table_destroy(dev);
> -	mlx5_sriov_detach(dev);
>  	mlx5_eswitch_disable(dev->priv.eswitch);
> +	mlx5_sriov_detach(dev);
>  	mlx5_lag_remove_mdev(dev);
>  	mlx5_ec_cleanup(dev);
>  	mlx5_sf_hw_table_destroy(dev);
> -- 
> 2.39.2
> 
Looks good, thanks.
Reviewed-by: Piotr Raczynski <piotr.raczynski@intel.com>