Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cc5fd15f authored Sep 12, 2019 by Alex Vesker Committed by Saeed Mahameed Sep 24, 2019

net/mlx5: DR, Remove redundant vport number from action



The vport number is part of the vport_cap, there is no reason
to store in a separate variable on the vport.

Fixes: 9db810ed ("net/mlx5: DR, Expose steering action functionality")
Signed-off-by: Alex Vesker <valex@mellanox.com>
Reviewed-by: Maor Gottlieb <maorg@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>

parent d32d7c52

drivers/net/ethernet/mellanox/mlx5/core/steering/dr_action.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -615,7 +615,7 @@ static int dr_action_handle_cs_recalc(struct mlx5dr_domain *dmn,
		* that recalculates the CS and forwards to the vport.
		*/
		ret = mlx5dr_domain_cache_get_recalc_cs_ft_addr(dest_action->vport.dmn,
		dest_action->vport.num,
		dest_action->vport.caps->num,
		final_icm_addr);
		if (ret) {
		mlx5dr_err(dmn, "Failed to get FW cs recalc flow table\n");
		@@ -744,7 +744,7 @@ int mlx5dr_actions_build_ste_arr(struct mlx5dr_matcher *matcher,
		dest_action = action;
		if (rx_rule) {
		/* Loopback on WIRE vport is not supported */
		if (action->vport.num == WIRE_PORT)
		if (action->vport.caps->num == WIRE_PORT)
		goto out_invalid_arg;

		attr.final_icm_addr = action->vport.caps->icm_address_rx;

drivers/net/ethernet/mellanox/mlx5/core/steering/dr_types.h

+0 −1

Original line number	Diff line number	Diff line
		@@ -745,7 +745,6 @@ struct mlx5dr_action {
		struct {
		struct mlx5dr_domain *dmn;
		struct mlx5dr_cmd_vport_cap *caps;
		u32 num;
		} vport;
		struct {
		u32 vlan_hdr; /* tpid_pcp_dei_vid */