Re: [PATCH v10 00/13] preparation for the rte_flow offload of nfp PMD

[Ferruh Yigit <ferruh.yigit@amd.com>]

On 10/5/2022 12:34 PM, Ferruh Yigit wrote:
> On 9/26/2022 7:59 AM, Chaoyong He wrote:
>> This is the first patch series to add the support of rte_flow offload for
>> nfp PMD, includes:
>> Add the support of flower firmware application
>> Add the support of representor port
>> Add the flower service infrastructure
>> Add the cmsg interactive channels between pmd and fw
>>
>> * Changes since v9
>> - Remove the use of rte_eth_tx_burst()
>> - Remove the logics rely on OvS
>>
>> * 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 (13):
>>    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 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: add flower PF related routines
>>    net/nfp: move rxtx function to header file
>>    net/nfp: add flower PF rxtx logic
>>    net/nfp: add the representor port rxtx logic
>>
> 
> Series applied to dpdk-next-net/main, thanks.
> 

Ahh, this has a small conflict with other set [1], and other one is fix.
To keep fix patch backportable to stable trees, I will get that series 
first and apply this set on top of it, by force push to next-net.

Please highlight these kind of dependencies in advance, in commit log etc.

And can you please double check final code in next-net head?

[1]
https://patches.dpdk.org/project/dpdk/patch/20220922130314.694790-2-niklas.soderlund@corigine.com/
[1/2] net/nfp: fix return value of nfp card init function