mbox series

[v4,0/3] use architecure independent macros

Message ID 20181220115645.37584-1-roy.fan.zhang@intel.com (mailing list archive)
Headers
Series use architecure independent macros |

Message

Fan Zhang Dec. 20, 2018, 11:56 a.m. UTC
  This patch updates the aesni_mb to use IMB_* arch independent macros to
reduce the code size and future maintaining effort.

In intel-ipsec-mb library 0.52 all supported algorithms now have the IMB_*
arch independent macros enabled. The macros help reducing the application's
code size and remove the burden of maintaining the support to different
architectures such as SSE and AVX*, etc.

This patch adds this support into AESNI-MB PMD. Meanwhile to keep
supporting the older version of intel-ipsec-mb library, the existing
rte_aesni_mb_pmd*.c are renamed to rte_aesni_mb_pmd*_compat.c and the
compiler will check the version number in /usr/include/inte-ipsec-mb.h and
decides which files to be compiled. For intel-ipsec-mb library 0.52 the
rte_aesni_mb_pmd*.c will be compiled. For the older version
rte_aesni_mb_pmd*_compat.c will be compiled.

It is planned to change the minimum intel-ipsec-mb support version to 0.52
in DPDK 19.05 release. By then all code intended for supporting older
version will be removed, including rte_aesni_mb_pmd*_compat.c.

Acked-by: Damian Nowak <damianx.nowak@intel.com>

v4:
- combined documentation patches into one.
- updated release note.

v3:
- patch split.
- fixed meson build bug.
- updated commit message.
- updated documentation.

v2:
- making the PMD compatible with both new intel-ipsec-mb version 0.52 and older
- fixed a bug

Fan Zhang (3):
  crypto/aesni_mb: rename files
  crypto/aesni_mb: use architecture independent macros
  doc: update documentation

 doc/guides/cryptodevs/aesni_mb.rst                 |    4 +-
 doc/guides/rel_notes/deprecation.rst               |    3 +
 doc/guides/rel_notes/release_19_02.rst             |    3 +
 drivers/crypto/aesni_mb/Makefile                   |   26 +-
 drivers/crypto/aesni_mb/meson.build                |   18 +-
 drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c         |  190 ++-
 drivers/crypto/aesni_mb/rte_aesni_mb_pmd_compat.c  | 1239 ++++++++++++++++++++
 drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c     |   86 +-
 .../crypto/aesni_mb/rte_aesni_mb_pmd_ops_compat.c  |  719 ++++++++++++
 drivers/crypto/aesni_mb/rte_aesni_mb_pmd_private.h |   52 +-
 10 files changed, 2169 insertions(+), 171 deletions(-)
 create mode 100644 drivers/crypto/aesni_mb/rte_aesni_mb_pmd_compat.c
 create mode 100644 drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops_compat.c
  

Comments

De Lara Guarch, Pablo Jan. 9, 2019, 10:09 p.m. UTC | #1
> -----Original Message-----
> From: Zhang, Roy Fan
> Sent: Thursday, December 20, 2018 11:57 AM
> To: dev@dpdk.org
> Cc: akhil.goyal@nxp.com; De Lara Guarch, Pablo
> <pablo.de.lara.guarch@intel.com>
> Subject: [PATCH v4 0/3] use architecure independent macros
> 
> This patch updates the aesni_mb to use IMB_* arch independent macros to
> reduce the code size and future maintaining effort.
> 
> In intel-ipsec-mb library 0.52 all supported algorithms now have the IMB_*
> arch independent macros enabled. The macros help reducing the
> application's code size and remove the burden of maintaining the support to
> different architectures such as SSE and AVX*, etc.
> 
> This patch adds this support into AESNI-MB PMD. Meanwhile to keep
> supporting the older version of intel-ipsec-mb library, the existing
> rte_aesni_mb_pmd*.c are renamed to rte_aesni_mb_pmd*_compat.c and
> the compiler will check the version number in /usr/include/inte-ipsec-mb.h
> and decides which files to be compiled. For intel-ipsec-mb library 0.52 the
> rte_aesni_mb_pmd*.c will be compiled. For the older version
> rte_aesni_mb_pmd*_compat.c will be compiled.
> 
> It is planned to change the minimum intel-ipsec-mb support version to 0.52
> in DPDK 19.05 release. By then all code intended for supporting older version
> will be removed, including rte_aesni_mb_pmd*_compat.c.
> 
> Acked-by: Damian Nowak <damianx.nowak@intel.com>

Series applied to dpdk-next-crypto.

Thanks,
Pablo