> -----Original Message----- > From: Akhil Goyal <[email protected]> > Sent: Sunday, September 25, 2022 12:36 AM > To: [email protected] > Cc: [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; Ruifeng Wang > <[email protected]>; Ajit > Khaparde ([email protected]) <[email protected]>; > [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > Akhil Goyal > <[email protected]> > 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/[email protected]/ Could you merge my changes into your series? Thanks.
