提交 4f455143 编写于 作者: M Mark Bloch 提交者: Saeed Mahameed

net/mlx5: Lag, don't use magic numbers for ports

Instead of using 1 & 2 as the ports numbers use an enum value.
Signed-off-by: NMark Bloch <mbloch@nvidia.com>
Reviewed-by: NMaor Gottlieb <maorg@nvidia.com>
Signed-off-by: NSaeed Mahameed <saeedm@nvidia.com>
上级 9278287e
...@@ -40,6 +40,11 @@ ...@@ -40,6 +40,11 @@
#include "lag.h" #include "lag.h"
#include "mp.h" #include "mp.h"
enum {
MLX5_LAG_EGRESS_PORT_1 = 1,
MLX5_LAG_EGRESS_PORT_2,
};
/* General purpose, use for short periods of time. /* General purpose, use for short periods of time.
* Beware of lock dependencies (preferably, no locks should be acquired * Beware of lock dependencies (preferably, no locks should be acquired
* under it). * under it).
...@@ -193,15 +198,15 @@ static void mlx5_infer_tx_affinity_mapping(struct lag_tracker *tracker, ...@@ -193,15 +198,15 @@ static void mlx5_infer_tx_affinity_mapping(struct lag_tracker *tracker,
p2en = tracker->netdev_state[MLX5_LAG_P2].tx_enabled && p2en = tracker->netdev_state[MLX5_LAG_P2].tx_enabled &&
tracker->netdev_state[MLX5_LAG_P2].link_up; tracker->netdev_state[MLX5_LAG_P2].link_up;
*port1 = 1; *port1 = MLX5_LAG_EGRESS_PORT_1;
*port2 = 2; *port2 = MLX5_LAG_EGRESS_PORT_2;
if ((!p1en && !p2en) || (p1en && p2en)) if ((!p1en && !p2en) || (p1en && p2en))
return; return;
if (p1en) if (p1en)
*port2 = 1; *port2 = MLX5_LAG_EGRESS_PORT_1;
else else
*port1 = 2; *port1 = MLX5_LAG_EGRESS_PORT_2;
} }
static int _mlx5_modify_lag(struct mlx5_lag *ldev, u8 v2p_port1, u8 v2p_port2) static int _mlx5_modify_lag(struct mlx5_lag *ldev, u8 v2p_port1, u8 v2p_port2)
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册