diff options
| author | Jiri Pirko <[email protected]> | 2023-06-01 09:17:17 +0200 |
|---|---|---|
| committer | Saeed Mahameed <[email protected]> | 2023-06-23 12:27:35 -0700 |
| commit | 29e4c95faee52a9b7a4f1293cb92cd17a0b5fd91 (patch) | |
| tree | ace530c1432bb826bd7d1bde155afdeaa7f557cb | |
| parent | 899862b653d74ed5fc3a61cd5e14a88b824a71d7 (diff) | |
net/mlx5: Remove pointless vport lookup from mlx5_esw_check_port_type()
As xa_get_mark() returns false in case the entry is not present,
no need to redundantly check if vport is present. Remove the lookup.
Signed-off-by: Jiri Pirko <[email protected]>
Reviewed-by: Shay Drory <[email protected]>
Signed-off-by: Saeed Mahameed <[email protected]>
| -rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/eswitch.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/eswitch.c b/drivers/net/ethernet/mellanox/mlx5/core/eswitch.c index b4e465856127..faec7d7a4400 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/eswitch.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/eswitch.c @@ -1908,12 +1908,6 @@ int mlx5_eswitch_set_vport_mac(struct mlx5_eswitch *esw, static bool mlx5_esw_check_port_type(struct mlx5_eswitch *esw, u16 vport_num, xa_mark_t mark) { - struct mlx5_vport *vport; - - vport = mlx5_eswitch_get_vport(esw, vport_num); - if (IS_ERR(vport)) - return false; - return xa_get_mark(&esw->vports, vport_num, mark); } |