[PATCH v2 00/25] add the NFP vDPA PMD
Ferruh Yigit
ferruh.yigit at amd.com
Wed Oct 25 18:09:08 CEST 2023
On 10/24/2023 3:28 AM, Chaoyong He wrote:
> This patch series aims to add the NFP vDPA PMD, we also grab the common
> logic into the `drivers/common/nfp` directory.
>
> ---
> v2:
> * Grab more logic into the `drivers/common/nfp` directory.
> * Delete some logic which should be when moving logic.
> ---
>
> Chaoyong He (25):
> drivers: introduce the NFP common library
> net/nfp: make VF PMD using of NFP common module
> net/nfp: rename common module name
> net/nfp: rename ctrl module name
> net/nfp: extract the cap data field
> net/nfp: extract the qcp data field
> net/nfp: extract the ctrl BAR data field
> net/nfp: extract the ctrl data field
> net/nfp: change the parameter of APIs
> net/nfp: change the parameter of reconfig
> net/nfp: extract the MAC address data field
> net/nfp: rename parameter in related logic
> drivers: add the common ctrl module
> drivers: add the nfp common module
> drivers: move queue logic to common module
> drivers: move platform module to common library
> drivers: move device module to common library
> drivers/vdpa: introduce the NFP vDPA library
> drivers: add the basic framework of vDPA PMD
> vdpa/nfp: add the logic of remap PCI memory
> vdpa/nfp: add the hardware init logic
> drivers: add the datapath update logic
> vdpa/nfp: add the notify related logic
> vdpa/nfp: add nfp vDPA device operations
> doc: add the common and vDPA document
>
Overall pretty clean set, but there are a few minor issues, commented on
patches.
Also can you please address checkpatch warnings:
### [PATCH] drivers: add the datapath update logic
Warning in drivers/vdpa/nfp/nfp_vdpa.c:
Using __atomic_xxx built-ins, prefer rte_atomic_xxx
### [PATCH] vdpa/nfp: add the notify related logic
Warning in drivers/vdpa/nfp/nfp_vdpa.c:
Using pthread functions, prefer rte_thread
### [PATCH] vdpa/nfp: add nfp vDPA device operations
Warning in drivers/vdpa/nfp/nfp_vdpa.c:
Using __atomic_xxx built-ins, prefer rte_atomic_xxx
And some typos:
vdpa/nfp: add nfp vDPA device operations
opetation
More information about the dev
mailing list