[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