mbox series

[v4,00/11] bbdev PMD ACC100

Message ID 1599242047-58232-1-git-send-email-nicolas.chautru@intel.com (mailing list archive)
Headers
Series bbdev PMD ACC100 |

Message

Chautru, Nicolas Sept. 4, 2020, 5:53 p.m. UTC
  v4: an odd compilation error is reported for one CI variant
using "gcc latest" which looks to me like a false positive of
maybe-undeclared. 
http://mails.dpdk.org/archives/test-report/2020-August/148936.html
Still forcing a dummy declare to remove this CI warning 
I will check with ci@dpdk.org in parallel.  
v3: missed a change during rebase
v2: includes clean up from latest CI checks.

This set includes a new PMD for the accelerator
ACC100 for 4G+5G FEC in 20.11. 
Documentation is updated as well accordingly.
Existing unit tests are all still supported.


Nicolas Chautru (11):
  drivers/baseband: add PMD for ACC100
  baseband/acc100: add register definition file
  baseband/acc100: add info get function
  baseband/acc100: add queue configuration
  baseband/acc100: add LDPC processing functions
  baseband/acc100: add HARQ loopback support
  baseband/acc100: add support for 4G processing
  baseband/acc100: add interrupt support to PMD
  baseband/acc100: add debug function to validate input
  baseband/acc100: add configure function
  doc: update bbdev feature table

 app/test-bbdev/Makefile                            |    3 +
 app/test-bbdev/meson.build                         |    3 +
 app/test-bbdev/test_bbdev_perf.c                   |   72 +
 config/common_base                                 |    4 +
 doc/guides/bbdevs/acc100.rst                       |  233 +
 doc/guides/bbdevs/features/acc100.ini              |   14 +
 doc/guides/bbdevs/features/mbc.ini                 |   14 -
 doc/guides/bbdevs/index.rst                        |    1 +
 doc/guides/rel_notes/release_20_11.rst             |    6 +
 drivers/baseband/Makefile                          |    2 +
 drivers/baseband/acc100/Makefile                   |   28 +
 drivers/baseband/acc100/acc100_pf_enum.h           | 1068 +++++
 drivers/baseband/acc100/acc100_vf_enum.h           |   73 +
 drivers/baseband/acc100/meson.build                |    8 +
 drivers/baseband/acc100/rte_acc100_cfg.h           |  113 +
 drivers/baseband/acc100/rte_acc100_pmd.c           | 4684 ++++++++++++++++++++
 drivers/baseband/acc100/rte_acc100_pmd.h           |  593 +++
 .../acc100/rte_pmd_bbdev_acc100_version.map        |   10 +
 drivers/baseband/meson.build                       |    2 +-
 mk/rte.app.mk                                      |    1 +
 20 files changed, 6917 insertions(+), 15 deletions(-)
 create mode 100644 doc/guides/bbdevs/acc100.rst
 create mode 100644 doc/guides/bbdevs/features/acc100.ini
 delete mode 100644 doc/guides/bbdevs/features/mbc.ini
 create mode 100644 drivers/baseband/acc100/Makefile
 create mode 100644 drivers/baseband/acc100/acc100_pf_enum.h
 create mode 100644 drivers/baseband/acc100/acc100_vf_enum.h
 create mode 100644 drivers/baseband/acc100/meson.build
 create mode 100644 drivers/baseband/acc100/rte_acc100_cfg.h
 create mode 100644 drivers/baseband/acc100/rte_acc100_pmd.c
 create mode 100644 drivers/baseband/acc100/rte_acc100_pmd.h
 create mode 100644 drivers/baseband/acc100/rte_pmd_bbdev_acc100_version.map
  

Comments

Chautru, Nicolas Sept. 21, 2020, 2:36 p.m. UTC | #1
Hi Akhil, 
Just a heads up on this bbdev PMD which is ready and was reviewed for some time by the community.
There is one warning on patchwork but it can be ignored (one ack email sent with bad formatting). 
Thanks and best regards, 
Nic

> -----Original Message-----
> From: Chautru, Nicolas <nicolas.chautru@intel.com>
> Sent: Friday, September 4, 2020 10:54 AM
> To: dev@dpdk.org; akhil.goyal@nxp.com
> Cc: Richardson, Bruce <bruce.richardson@intel.com>; Xu, Rosen
> <rosen.xu@intel.com>; dave.burley@accelercomm.com;
> aidan.goddard@accelercomm.com; Yigit, Ferruh <ferruh.yigit@intel.com>; Liu,
> Tianjiao <tianjiao.liu@intel.com>; Chautru, Nicolas
> <nicolas.chautru@intel.com>
> Subject: [PATCH v4 00/11] bbdev PMD ACC100
> 
> v4: an odd compilation error is reported for one CI variant using "gcc latest"
> which looks to me like a false positive of maybe-undeclared.
> http://mails.dpdk.org/archives/test-report/2020-August/148936.html
> Still forcing a dummy declare to remove this CI warning I will check with
> ci@dpdk.org in parallel.
> v3: missed a change during rebase
> v2: includes clean up from latest CI checks.
> 
> This set includes a new PMD for the accelerator
> ACC100 for 4G+5G FEC in 20.11.
> Documentation is updated as well accordingly.
> Existing unit tests are all still supported.
> 
> 
> Nicolas Chautru (11):
>   drivers/baseband: add PMD for ACC100
>   baseband/acc100: add register definition file
>   baseband/acc100: add info get function
>   baseband/acc100: add queue configuration
>   baseband/acc100: add LDPC processing functions
>   baseband/acc100: add HARQ loopback support
>   baseband/acc100: add support for 4G processing
>   baseband/acc100: add interrupt support to PMD
>   baseband/acc100: add debug function to validate input
>   baseband/acc100: add configure function
>   doc: update bbdev feature table
> 
>  app/test-bbdev/Makefile                            |    3 +
>  app/test-bbdev/meson.build                         |    3 +
>  app/test-bbdev/test_bbdev_perf.c                   |   72 +
>  config/common_base                                 |    4 +
>  doc/guides/bbdevs/acc100.rst                       |  233 +
>  doc/guides/bbdevs/features/acc100.ini              |   14 +
>  doc/guides/bbdevs/features/mbc.ini                 |   14 -
>  doc/guides/bbdevs/index.rst                        |    1 +
>  doc/guides/rel_notes/release_20_11.rst             |    6 +
>  drivers/baseband/Makefile                          |    2 +
>  drivers/baseband/acc100/Makefile                   |   28 +
>  drivers/baseband/acc100/acc100_pf_enum.h           | 1068 +++++
>  drivers/baseband/acc100/acc100_vf_enum.h           |   73 +
>  drivers/baseband/acc100/meson.build                |    8 +
>  drivers/baseband/acc100/rte_acc100_cfg.h           |  113 +
>  drivers/baseband/acc100/rte_acc100_pmd.c           | 4684
> ++++++++++++++++++++
>  drivers/baseband/acc100/rte_acc100_pmd.h           |  593 +++
>  .../acc100/rte_pmd_bbdev_acc100_version.map        |   10 +
>  drivers/baseband/meson.build                       |    2 +-
>  mk/rte.app.mk                                      |    1 +
>  20 files changed, 6917 insertions(+), 15 deletions(-)  create mode 100644
> doc/guides/bbdevs/acc100.rst  create mode 100644
> doc/guides/bbdevs/features/acc100.ini
>  delete mode 100644 doc/guides/bbdevs/features/mbc.ini
>  create mode 100644 drivers/baseband/acc100/Makefile  create mode 100644
> drivers/baseband/acc100/acc100_pf_enum.h
>  create mode 100644 drivers/baseband/acc100/acc100_vf_enum.h
>  create mode 100644 drivers/baseband/acc100/meson.build
>  create mode 100644 drivers/baseband/acc100/rte_acc100_cfg.h
>  create mode 100644 drivers/baseband/acc100/rte_acc100_pmd.c
>  create mode 100644 drivers/baseband/acc100/rte_acc100_pmd.h
>  create mode 100644
> drivers/baseband/acc100/rte_pmd_bbdev_acc100_version.map
> 
> --
> 1.8.3.1
  
Akhil Goyal Sept. 22, 2020, 7:32 p.m. UTC | #2
Hi Nicolas,

> 
> Hi Akhil,
> Just a heads up on this bbdev PMD which is ready and was reviewed for some
> time by the community.
> There is one warning on patchwork but it can be ignored (one ack email sent
> with bad formatting).
> Thanks and best regards,
> Nic
There are changes in Makefiles, which are not required as all makefiles are removed
As we have moved to meson build.

Could you please update the series.

Thanks,
Akhil
  
Chautru, Nicolas Sept. 23, 2020, 2:21 a.m. UTC | #3
Hi Akhil,

> 
> Hi Nicolas,
> 
> >
> > Hi Akhil,
> > Just a heads up on this bbdev PMD which is ready and was reviewed for
> > some time by the community.
> > There is one warning on patchwork but it can be ignored (one ack email
> > sent with bad formatting).
> > Thanks and best regards,
> > Nic
> There are changes in Makefiles, which are not required as all makefiles are
> removed As we have moved to meson build.
> 
> Could you please update the series.
> 
> Thanks,
> Akhil

No problem Akhil. I just rebased base on latest on main and sent as a v6 (ignore the v5, I had left one old Makefile after rebase). 

Thanks,
Nic