Skip to content

Commit

Permalink
net/cpfl: fix forwarding to physical port
Browse files Browse the repository at this point in the history
CPFL PMD should be able to support below traffic forwarding capabilities
based on the rte flow action types.

 1. Forwarding the traffic to the local CPFL vports using
    port_representor action.

 2. Forwarding the traffic to the physical IO ports using
    represented_port action.

 3. Forwarding the traffic to an IDPF VF using represented_port action.

The 2nd use case, forwarding the traffic to IO ports using
represented_port action, is not working due to the additional check
added in the previous patch (8612619 ("net/cpfl: add checks for
flow action types"))

This patch removes the incorrect check to fix the issue.

Fixes: 8612619 ("net/cpfl: add checks for flow action types")
Cc: [email protected]

Signed-off-by: Praveen Shetty <[email protected]>
Acked-by: Bruce Richardson <[email protected]>
  • Loading branch information
PraveenShettyINC authored and bruce-richardson committed Nov 11, 2024
1 parent 64998e4 commit 7b12182
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions drivers/net/cpfl/cpfl_flow_engine_fxp.c
Original file line number Diff line number Diff line change
Expand Up @@ -301,11 +301,6 @@ cpfl_fxp_parse_action(struct cpfl_itf *itf,
PMD_DRV_LOG(ERR, "Cannot use port_representor action for the represented_port");
goto err;
}
if (action_type == RTE_FLOW_ACTION_TYPE_REPRESENTED_PORT &&
dst_itf->type == CPFL_ITF_TYPE_VPORT) {
PMD_DRV_LOG(ERR, "Cannot use represented_port action for the local vport");
goto err;
}
if (is_vsi)
dev_id = cpfl_get_vsi_id(dst_itf);
else
Expand Down

0 comments on commit 7b12182

Please sign in to comment.