Re: [dpdk-dev] [PATCH v3 00/11] Add Marvell CNXK mempool driver

[Jerin Jacob <jerinjacobk@gmail.com>]

On Tue, Apr 6, 2021 at 8:41 PM Ashwin Sekhar T K <asekhar@marvell.com> wrote:
>
> This patchset adds the mempool/cnxk driver which provides the support for the
> integrated mempool device found in Marvell CN10K SoC.
>
> The code includes mempool driver functionality for Marvell CN9K SoC as well,
> but right now it is not enabled. The future plan is to deprecate existing
> mempool/octeontx2 driver once the 'CNXK' drivers are feature complete for
> Marvell CN9K SoC.
>
> Depends-on: series-16131 ("Add Marvell CNXK common driver")
>
> v3:
>  - Change batch op data initialization to plt init callback.
>  - Reserve a memzone for batch op data.
>  - Handle batch op data initialization in secondary process.

1) http://patches.dpdk.org/project/dpdk/patch/20210406151115.1889455-12-asekhar@marvell.com/
shows some CI issues, Could you check, Is this valid or not?
2) Common code series has added the following [1] section, Could you
add mempool driver update as the second bullet.

Rest looks good to me.

[1]
diff --git a/doc/guides/rel_notes/release_21_05.rst
b/doc/guides/rel_notes/release_21_05.rst
index 2ffeb92..cc6e53e 100644
--- a/doc/guides/rel_notes/release_21_05.rst
+++ b/doc/guides/rel_notes/release_21_05.rst
@@ -65,6 +65,15 @@  New Features
       representor=[[c#]pf#]sf# sf[0,2-1023] /* 1023 SFs.                     */
       representor=[c#]pf#      c2pf[0,1]    /* 2 PFs on controller 2.        */

+* **Added support for Marvell CN10K SoC drivers.**
+
+  Added Marvell CN10K SoC support. Marvell CN10K SoC are based on Octeon 10
+  family of ARM64 processors with ARM Neoverse N2 core with accelerators for
+  packet processing, timers, cryptography, etc.
+
+  * Added common/cnxk driver consisting of common API to be used by
+    net, crypto and event PMD's.


>
> Ashwin Sekhar T K (11):
>   mempool/cnxk: add build infra and doc
>   mempool/cnxk: add device probe/remove
>   mempool/cnxk: add generic ops
>   mempool/cnxk: register plt init callback
>   mempool/cnxk: add cn9k mempool ops
>   mempool/cnxk: add cn9k optimized mempool enqueue/dequeue
>   mempool/cnxk: add cn10k mempool ops
>   mempool/cnxk: add batch op init
>   mempool/cnxk: add cn10k batch enqueue op
>   mempool/cnxk: add cn10k get count op
>   mempool/cnxk: add cn10k batch dequeue op
>
>  MAINTAINERS                              |   6 +
>  doc/guides/mempool/cnxk.rst              |  91 +++++++
>  doc/guides/mempool/index.rst             |   1 +
>  doc/guides/platform/cnxk.rst             |   3 +
>  drivers/mempool/cnxk/cn10k_mempool_ops.c | 319 +++++++++++++++++++++++
>  drivers/mempool/cnxk/cn9k_mempool_ops.c  |  89 +++++++
>  drivers/mempool/cnxk/cnxk_mempool.c      | 202 ++++++++++++++
>  drivers/mempool/cnxk/cnxk_mempool.h      |  28 ++
>  drivers/mempool/cnxk/cnxk_mempool_ops.c  | 191 ++++++++++++++
>  drivers/mempool/cnxk/meson.build         |  16 ++
>  drivers/mempool/cnxk/version.map         |   3 +
>  drivers/mempool/meson.build              |   3 +-
>  12 files changed, 951 insertions(+), 1 deletion(-)
>  create mode 100644 doc/guides/mempool/cnxk.rst
>  create mode 100644 drivers/mempool/cnxk/cn10k_mempool_ops.c
>  create mode 100644 drivers/mempool/cnxk/cn9k_mempool_ops.c
>  create mode 100644 drivers/mempool/cnxk/cnxk_mempool.c
>  create mode 100644 drivers/mempool/cnxk/cnxk_mempool.h
>  create mode 100644 drivers/mempool/cnxk/cnxk_mempool_ops.c
>  create mode 100644 drivers/mempool/cnxk/meson.build
>  create mode 100644 drivers/mempool/cnxk/version.map
>
> --
> 2.31.0
>