Skip to content

Commit a873d28

Browse files
author
Benjamin Poirier
committed
net/mlx5e: Add mqprio_rl cleanup and free in mlx5e_priv_cleanup()
JIRA: https://issues.redhat.com/browse/RHEL-45365 Upstream-status: v6.10-rc7 commit 1da839e Author: Jianbo Liu <jianbol@nvidia.com> Date: Thu Jun 27 21:02:38 2024 +0300 net/mlx5e: Add mqprio_rl cleanup and free in mlx5e_priv_cleanup() In the cited commit, mqprio_rl cleanup and free are mistakenly removed in mlx5e_priv_cleanup(), and it causes the leakage of host memory and firmware SCHEDULING_ELEMENT objects while changing eswitch mode. So, add them back. Fixes: 0bb7228 ("net/mlx5e: Fix mqprio_rl handling on devlink reload") Signed-off-by: Jianbo Liu <jianbol@nvidia.com> Reviewed-by: Dragos Tatulea <dtatulea@nvidia.com> Signed-off-by: Tariq Toukan <tariqt@nvidia.com> Signed-off-by: David S. Miller <davem@davemloft.net> Signed-off-by: Benjamin Poirier <bpoirier@redhat.com>
1 parent 14236ab commit a873d28

File tree

1 file changed

+5
-0
lines changed
  • drivers/net/ethernet/mellanox/mlx5/core

1 file changed

+5
-0
lines changed

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

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5701,6 +5701,11 @@ void mlx5e_priv_cleanup(struct mlx5e_priv *priv)
57015701
kfree(priv->htb_qos_sq_stats[i]);
57025702
kvfree(priv->htb_qos_sq_stats);
57035703

5704+
if (priv->mqprio_rl) {
5705+
mlx5e_mqprio_rl_cleanup(priv->mqprio_rl);
5706+
mlx5e_mqprio_rl_free(priv->mqprio_rl);
5707+
}
5708+
57045709
memset(priv, 0, sizeof(*priv));
57055710
}
57065711

0 commit comments

Comments
 (0)