[dpdk-dev] [PATCH] cryptodev: extend api of asymmetric crypto by sessionless

Trahe, Fiona fiona.trahe at intel.com
Fri Jun 21 13:58:53 CEST 2019


Hi Akhil, Arek, Shally,

> -----Original Message-----
> From: Akhil Goyal [mailto:akhil.goyal at nxp.com]
> Sent: Thursday, June 20, 2019 3:17 PM
> To: Kusztal, ArkadiuszX <arkadiuszx.kusztal at intel.com>; dev at dpdk.org
> Cc: Trahe, Fiona <fiona.trahe at intel.com>; shally.verma at caviumnetworks.com
> Subject: RE: [PATCH] cryptodev: extend api of asymmetric crypto by sessionless
> >
> > Asymmetric cryptography algorithms may more likely use
> > sessionless API so there is need to extend API.
> >
> > Signed-off-by: Arek Kusztal <arkadiuszx.kusztal at intel.com>
> > ---
> Acked-by: Akhil Goyal <akhil.goyal at nxp.com>

[Fiona] The code is ok but I think a little more is needed.
As all PMDs don't support sessionless, this needs to be handled as an optional capability.
And in future some PMDs may only support SESSIONLESS and some only support WITH_SESSION.
So I propose adding 2 feature flags to the API
RTE_CRYPTODEV_FF_ASYM_WTH_SESSION
RTE_CRYPTODEV_FF_ASYM_SESSIONLESS
and including in this patch the PMD and UT changes to set and test the first flag.
We'll follow up with SESSIONLESS QAT implementation and UTs in a separate patchset.

Also documentation updates should go with this API patch, i.e.
 - update section 16.7.2 in the cryptodev programmers guide - and review that doc in case other sections need updating.
 - fix comment in rte_crypto.h under STATUS_INVALID_SESSION
 - release note





More information about the dev mailing list