> -----Original Message----- > From: Akhil Goyal <gak...@marvell.com> > Sent: Sunday, September 25, 2022 12:36 AM > To: dev@dpdk.org > Cc: tho...@monjalon.net; david.march...@redhat.com; hemant.agra...@nxp.com; > vattun...@marvell.com; ferruh.yi...@xilinx.com; andrew.rybche...@oktetlabs.ru; > konstantin.v.anan...@yandex.ru; jiawe...@trustnetic.com; > yisen.zhu...@huawei.com; > irussk...@marvell.com; jer...@marvell.com; adwiv...@marvell.com; > maxime.coque...@redhat.com; cha...@amd.com; Ruifeng Wang > <ruifeng.w...@arm.com>; Ajit > Khaparde (ajit.khapa...@broadcom.com) <ajit.khapa...@broadcom.com>; > ano...@marvell.com; > pablo.de.lara.gua...@intel.com; ma...@nvidia.com; g.si...@nxp.com; > qiming.y...@intel.com; > wenjun1...@intel.com; jianw...@trustnetic.com; jingjing...@intel.com; > beilei.x...@intel.com; ndabilpu...@marvell.com; roy.fan.zh...@intel.com; > Akhil Goyal > <gak...@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.w...@arm.com/ Could you merge my changes into your series? Thanks.
