Re: [PATCH v1] net/cpfl: fix IO port forwarding issue

[Bruce Richardson <bruce.richardson@intel.com>]

On Fri, Nov 08, 2024 at 11:09:23AM +0000, Praveen Shetty wrote:
> Issue was reported in dpdk internal regression testing that
> forwarding the packets from IDPF VF to IO ports using the cpfl
> represented_port action was not working.
> This patch will fix this issue.
> 
> Fixes: 861261957684 ("net/cpfl: add checks for flow action types")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Praveen Shetty <praveen.shetty@intel.com>
> ---
>  drivers/net/cpfl/cpfl_flow_engine_fxp.c | 5 -----
>  1 file changed, 5 deletions(-)
> 

Following discussion with Praveen to clarify the issue and the fix, applied
with expanded commit-log message (below) to next-net-intel.

"""
    net/cpfl: fix forwarding to physical port
    
    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 (861261957684 ("net/cpfl: add checks for
    flow action types"))
    
    This patch removes the incorrect check to fix the issue.
"""

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Applied,
Thanks.