[PATCH v9 00/12] preparation for the rte_flow offload of nfp PMD

Ferruh Yigit ferruh.yigit at xilinx.com
Tue Sep 20 16:56:18 CEST 2022


On 9/15/2022 11:44 AM, Chaoyong He wrote:
> * Changes since v8
> - Update the nfp.rst
> - Fix the 'app_hw' to 'app_fw'
> - Remove the ovs compatible header file
> - Remove the use of rte_eth_dev_configure()/rte_eth_rx_burst()/rte_eth_dev_start() API
> 
> * Changes since v7
> - Adjust the logics to make sure not break the pci probe process
> - Change 'app' to 'app_fw' in all logics to avoid confuse
> - Fix problem about log level
> 
> * Changes since v6
> - Fix the compile error
> 
> * Changes since v5
> - Compare integer with 0 explicitly
> - Change helper macro to function
> - Implement the dummy functions
> - Remove some unnecessary logics
> 
> * Changes since v4
> - Remove the unneeded '__rte_unused' attribute
> - Fixup a potential memory leak problem
> 
> * Changes since v3
> - Add the 'Depends-on' tag
> 
> * Changes since v2
> - Remove the use of rte_panic()
> 
> * Changes since v1
> - Fix the compile error
> 
> Depends-on: series-23707 ("Add support of NFP3800 chip and firmware with NFDk")
> 
> Chaoyong He (12):
>    net/nfp: move app specific attributes to own struct
>    net/nfp: simplify initialization and remove dead code
>    net/nfp: move app specific init logic to own function
>    net/nfp: add initial flower firmware support
>    net/nfp: add flower PF setup logic
>    net/nfp: add flower PF related routines
>    net/nfp: add flower ctrl VNIC related logics
>    net/nfp: move common rxtx function for flower use
>    net/nfp: add flower ctrl VNIC rxtx logic
>    net/nfp: add flower representor framework
>    net/nfp: move rxtx function to header file
>    net/nfp: add flower PF rxtx logic
> 

Hi Chaoyong,

Patchset looks good, except from two issues we have discussed before, 
those issues are:

* Creating a new ethdev just for driver-FW control communication
* Application (OvS) specific code in the driver

I commented them separately and cc'ed more folks, we can proceed when 
above are resolved.

Thanks,
ferruh



More information about the dev mailing list