[dpdk-dev] [PATCH 4/8] crypto/octeontx2: add cryptodev sec capabilities

Tejasree Kondoj ktejasree at marvell.com
Thu Jul 2 12:32:01 CEST 2020


Hi Akhil,

Please see inline.

Thanks
Tejasree

> -----Original Message-----
> From: Akhil Goyal <akhil.goyal at nxp.com>
> Sent: Thursday, July 2, 2020 2:37 AM
> To: Tejasree Kondoj <ktejasree at marvell.com>; Radu Nicolau
> <radu.nicolau at intel.com>
> Cc: Narayana Prasad Raju Athreya <pathreya at marvell.com>; Anoob Joseph
> <anoobj at marvell.com>; Vamsi Krishna Attunuru <vattunuru at marvell.com>;
> dev at dpdk.org
> Subject: [EXT] RE: [PATCH 4/8] crypto/octeontx2: add cryptodev sec
> capabilities
> 
> External Email
> 
> ----------------------------------------------------------------------
> > +static const struct rte_cryptodev_capabilities *
> > +otx2_cpt_sec_caps_get(union cpt_eng_caps *hw_caps) {
> > +	SEC_CAPS_ADD(hw_caps, aes);
> > +
> > +	sec_caps_add(caps_end, RTE_DIM(caps_end));
> > +
> > +	return otx2_cpt_sec_caps;
> > +}
> SEC_CAPS_ADD should be called earlier when the security context is created
> Or where all other capabilities of the PMD are initialized.
> It should not be added when capabilities need to be retrieved.
> As of now you are supporting only AES-GCM, but in future if you add more
> algos, Then it would be difficult to manage.
[Tejasree]  We will initialize capabilities during probe and capabilities_get() would return pointer. Would that work?



More information about the dev mailing list