RE: [PATCH 0/2] IPsec on Arm

["Zhang, Roy Fan" <roy.fan.zhang@intel.com>]

Hi Ruifeng,

We have no technical issues with these patches based on their current functionality. However, it is worth pointing out that we are planning some changes to the Intel® Multi-Buffer Crypto for IPSec library which will require API changes in the future. These changes are, but not limited to, to consolidate some of the crypto PMDs into a single PMD, which will simplify the code and reduce future maintenance effort. When these changes are made, your library will need to be updated too in order to take advantage of the consolidation. We can provide further details of the changes that we’re considering separately, so that you’re not taken by surprise when these change occurs.

We also want you to know, for any change we will make to these PMDs, we do not guarantee it works on ARM platform. We advise you to do necessary testing/verification in necessary testing/verification to the future patches for the PMDs based on ipsec-mb on your platform.

Regards,
Fan

> -----Original Message-----
> From: Ruifeng Wang <ruifeng.wang@arm.com>
> Sent: Wednesday, July 27, 2022 9:14 AM
> To: Zhang, Roy Fan <roy.fan.zhang@intel.com>; De Lara Guarch, Pablo
> <pablo.de.lara.guarch@intel.com>; Wang, Yipeng1
> <yipeng1.wang@intel.com>; Gobriel, Sameh <sameh.gobriel@intel.com>;
> Richardson, Bruce <bruce.richardson@intel.com>; Medvedkin, Vladimir
> <vladimir.medvedkin@intel.com>; gakhil@marvell.com
> Cc: dev@dpdk.org; honnappa.nagarahalli@arm.com; nd@arm.com; Ruifeng
> Wang <ruifeng.wang@arm.com>
> Subject: [PATCH 0/2] IPsec on Arm
> 
> This patch set integrates Arm port of ipsec_mb library.
> ZUC and SNOW3g PMDs are available on Arm platform with this integration.
> 
> This series depends on:
> http://patches.dpdk.org/project/dpdk/patch/20220630154123.2565439-1-
> asekhar@marvell.com/
> 
> Depends-on: patch-113578 ("crypto/ipsec_mb: enable support for arm64")
> 
> Ruifeng Wang (2):
>   crypto/ipsec_mb: remove redundant includes
>   crypto/ipsec_mb: enable IPsec on Arm platform
> 
>  app/test/test_cryptodev_hash_test_vectors.h  |  4 ++++
>  doc/guides/cryptodevs/snow3g.rst             | 14 ++++++++++----
>  doc/guides/cryptodevs/zuc.rst                | 14 ++++++++++----
>  drivers/common/qat/meson.build               |  6 +++++-
>  drivers/crypto/ipsec_mb/ipsec_mb_private.c   |  6 ++++++
>  drivers/crypto/ipsec_mb/ipsec_mb_private.h   |  4 ++++
>  drivers/crypto/ipsec_mb/meson.build          |  6 +++++-
>  drivers/crypto/ipsec_mb/pmd_aesni_gcm_priv.h |  2 --
>  drivers/crypto/ipsec_mb/pmd_aesni_mb_priv.h  |  2 --
>  drivers/crypto/qat/qat_sym_session.c         |  4 ++++
>  10 files changed, 48 insertions(+), 14 deletions(-)
> 
> --
> 2.25.1