mbox series

[v10,0/7] bbdev changes for 22.11

Message ID 20220930184605.47655-1-nicolas.chautru@intel.com (mailing list archive)
Headers show
Series bbdev changes for 22.11 | expand

Message

Nicolas Chautru Sept. 30, 2022, 6:45 p.m. UTC
v10: replacing the _PADDED_MAX enum to _SIZE_MAX macro based on suggestion from Ferruh/Maxime/Akhil. Thanks
v9: removing code snippet from documentation in 5/7 requested by Akhil. Thanks. 
v8: edit based on review by Akhil : typos, coding guidelines. No functional change. Thanks
v7: couple of typos in documentation spotted by Maxime. Thanks.
v6: added one comment in commit 2/7 suggested by Maxime.
v5: update base on review from Tom Rix. Number of typos reported and resolved,
removed the commit related to rw_lock for now, added a commit for
code clean up from review, resolved one rebase issue between 2 commits, used size of array for some bound check implementation. Thanks. 
v4: update to the last 2 commits to include function to print the queue status and a fix to the rte_lock within the wrong structure
v3: update to device status info to also use padded size for the related array.
Adding also 2 additionals commits to allow the API struc to expose more information related to queues corner cases/warning as well as an optional rw lock.
Hemant, Maxime, this is planned for DPDK 21.11 but would like review/ack early is possible to get this applied earlier and due to time off this summer.
Thanks
Nic


Nicolas Chautru (7):
  bbdev: allow operation type enum for growth
  bbdev: add device status info
  bbdev: add device info on queue topology
  drivers/baseband: update PMDs to expose queue per operation
  bbdev: add new operation for FFT processing
  bbdev: add queue related warning and status information
  bbdev: remove unnecessary if-check

 app/test-bbdev/test_bbdev.c                   |   2 +-
 app/test-bbdev/test_bbdev_perf.c              |   6 +-
 doc/guides/prog_guide/bbdev.rst               | 103 +++++++++++
 drivers/baseband/acc100/rte_acc100_pmd.c      |  30 ++--
 .../fpga_5gnr_fec/rte_fpga_5gnr_fec.c         |   9 +
 drivers/baseband/fpga_lte_fec/fpga_lte_fec.c  |   9 +
 drivers/baseband/la12xx/bbdev_la12xx.c        |  10 +-
 drivers/baseband/null/bbdev_null.c            |   1 +
 .../baseband/turbo_sw/bbdev_turbo_software.c  |  13 ++
 examples/bbdev_app/main.c                     |   2 +-
 lib/bbdev/rte_bbdev.c                         |  57 +++++-
 lib/bbdev/rte_bbdev.h                         | 156 +++++++++++++++-
 lib/bbdev/rte_bbdev_op.h                      | 167 ++++++++++++++++--
 lib/bbdev/version.map                         |  12 ++
 14 files changed, 542 insertions(+), 35 deletions(-)

Comments

Akhil Goyal Sept. 30, 2022, 8:38 p.m. UTC | #1
> v10: replacing the _PADDED_MAX enum to _SIZE_MAX macro based on
> suggestion from Ferruh/Maxime/Akhil. Thanks

Since we have a consensus.
Series Acked-by: Akhil Goyal <gakhil@marvell.com>

Applied to dpdk-next-crypto
Added release notes and removed deprecation notices.

Please fix the documentation as recommended as a separate patch.

Thanks

> v9: removing code snippet from documentation in 5/7 requested by Akhil.
> Thanks.
> v8: edit based on review by Akhil : typos, coding guidelines. No functional
> change. Thanks
> v7: couple of typos in documentation spotted by Maxime. Thanks.
> v6: added one comment in commit 2/7 suggested by Maxime.
> v5: update base on review from Tom Rix. Number of typos reported and
> resolved,
> removed the commit related to rw_lock for now, added a commit for
> code clean up from review, resolved one rebase issue between 2 commits, used
> size of array for some bound check implementation. Thanks.
> v4: update to the last 2 commits to include function to print the queue status
> and a fix to the rte_lock within the wrong structure
> v3: update to device status info to also use padded size for the related array.
> Adding also 2 additionals commits to allow the API struc to expose more
> information related to queues corner cases/warning as well as an optional rw
> lock.
> Hemant, Maxime, this is planned for DPDK 21.11 but would like review/ack early
> is possible to get this applied earlier and due to time off this summer.
> Thanks
> Nic
> 
> 
> Nicolas Chautru (7):
>   bbdev: allow operation type enum for growth
>   bbdev: add device status info
>   bbdev: add device info on queue topology
>   drivers/baseband: update PMDs to expose queue per operation
>   bbdev: add new operation for FFT processing
>   bbdev: add queue related warning and status information
>   bbdev: remove unnecessary if-check
> 
>  app/test-bbdev/test_bbdev.c                   |   2 +-
>  app/test-bbdev/test_bbdev_perf.c              |   6 +-
>  doc/guides/prog_guide/bbdev.rst               | 103 +++++++++++
>  drivers/baseband/acc100/rte_acc100_pmd.c      |  30 ++--
>  .../fpga_5gnr_fec/rte_fpga_5gnr_fec.c         |   9 +
>  drivers/baseband/fpga_lte_fec/fpga_lte_fec.c  |   9 +
>  drivers/baseband/la12xx/bbdev_la12xx.c        |  10 +-
>  drivers/baseband/null/bbdev_null.c            |   1 +
>  .../baseband/turbo_sw/bbdev_turbo_software.c  |  13 ++
>  examples/bbdev_app/main.c                     |   2 +-
>  lib/bbdev/rte_bbdev.c                         |  57 +++++-
>  lib/bbdev/rte_bbdev.h                         | 156 +++++++++++++++-
>  lib/bbdev/rte_bbdev_op.h                      | 167 ++++++++++++++++--
>  lib/bbdev/version.map                         |  12 ++
>  14 files changed, 542 insertions(+), 35 deletions(-)
> 
> --
> 2.37.1