[PATCH v3 0/6] crypto/security session framework rework

[Ruifeng Wang]

> -----Original Message-----
> From: Akhil Goyal <gakhil at marvell.com>
> Sent: Sunday, September 25, 2022 12:36 AM
> To: dev at dpdk.org
> Cc: thomas at monjalon.net; david.marchand at redhat.com; hemant.agrawal at nxp.com;
> vattunuru at marvell.com; ferruh.yigit at xilinx.com; andrew.rybchenko at oktetlabs.ru;
> konstantin.v.ananyev at yandex.ru; jiawenwu at trustnetic.com; yisen.zhuang at huawei.com;
> irusskikh at marvell.com; jerinj at marvell.com; adwivedi at marvell.com;
> maxime.coquelin at redhat.com; chandu at amd.com; Ruifeng Wang <Ruifeng.Wang at arm.com>; Ajit
> Khaparde (ajit.khaparde at broadcom.com) <ajit.khaparde at broadcom.com>; anoobj at marvell.com;
> pablo.de.lara.guarch at intel.com; matan at nvidia.com; g.singh at nxp.com; qiming.yang at intel.com;
> wenjun1.wu at intel.com; jianwang at trustnetic.com; jingjing.wu at intel.com;
> beilei.xing at intel.com; ndabilpuram at marvell.com; roy.fan.zhang at intel.com; Akhil Goyal
> <gakhil at marvell.com>
> Subject: [PATCH v3 0/6] crypto/security session framework rework
> 
> This patchset reworks the symmetric crypto and security session data structure to use a
> single virtual/physical contiguous buffer for symmetric crypto/security session and driver
> private data.
> In addition the session data structure is now private.
> The session is represented as an opaque pointer in the application.
> 
> With the change the session is no longer supported to be accessed by multiple device
> drivers. For the same reason rte_cryptodev_sym_session_init/clear APIs are deprecated as
> rte_cryptodev_sym_session_create/free will initialize and clear the driver specific data
> field.
> 
> The change was also submitted last year during DPDK 21.11 timeframe also[1], but was not
> applied due to lack of feedback from community. Please help in getting this cleanup merged
> in this cycle.
> 
> Now the similar work was already done for asymmetric crypto.
> This patchset is rebased over current tree and fixes all the issues reported so far.
> 
> Changes in v3:
> - Updated release notes
> - fixed checkpatch issues
> - renamed macro to get sess priv data to align with crypto macro
> - added acked-by/tested-by
> 

Hi Akhil,

Thanks for the changes.
Armv8 crypo PMD has build issue with this series.
I updated code based on your series to fix the build. And an issue found
in unit test is also fixed.
http://patches.dpdk.org/project/dpdk/cover/20220926100120.3980185-1-ruifeng.wang@arm.com/

Could you merge my changes into your series?

Thanks.