Hi Shally, > -----Original Message----- > From: Verma, Shally [mailto:[email protected]] > Sent: Friday, June 22, 2018 4:39 PM > To: De Lara Guarch, Pablo <[email protected]> > Cc: Trahe, Fiona <[email protected]>; [email protected]; > [email protected]; Athreya, Narayana Prasad > <[email protected]>; Sahu, Sunila > <[email protected]>; Gupta, Ashish <[email protected]> > Subject: RE: [PATCH v3 1/6] lib/cryptodev: add asymmetric algos in cryptodev > > Hi Pablo > > >-----Original Message----- > >From: De Lara Guarch, Pablo [mailto:[email protected]] > >Sent: 15 June 2018 14:10 > >To: Verma, Shally <[email protected]> > >Cc: Trahe, Fiona <[email protected]>; [email protected]; > >[email protected]; Athreya, Narayana Prasad > ><[email protected]>; Sahu, Sunila > ><[email protected]>; Gupta, Ashish <[email protected]> > >Subject: RE: [PATCH v3 1/6] lib/cryptodev: add asymmetric algos in > >cryptodev > > > //snip > > > > >... > > > >> +/** > >> + * Asymmetric Cryptographic Operation. > >> + * > >> + * Structure describing asymmetric crypto operation params. > >> + * > >> + */ > >> +struct rte_crypto_asym_op { > >> + struct rte_cryptodev_asym_session *session; > >> + /**< Handle for the initialised session context */ > >> + > > > >Looking at the xform structure, it looks like a chain of xforms is possible. > >Looking at this union, this case wouldn't be possible, as only one item from > >the > union can be set. > > [Shally] xforms, which support chaining, would need to have op_type in their > respective xform struct. > Example struct rte_crypto_dh_xform, where app can chain Deffie-hellman > public and/or shared secret compute and DSA sign compute. > > +struct rte_crypto_dh_xform { > + enum rte_crypto_asym_op_type type; > + /**< Setup xform for key generate or shared secret compute */ and DSA > +xforms struct > > test_cryptodev_asym illustrates how to setup chained dh+dsa ops.
Are you talking about test_dh_gen_kp? Because this is the only function where I see that there is a chain of xforms. In this case, both xforms are the same type (RTE_CRYPTO_ASYM_XFORM_DH), and the operation only sets parameters for rte_crypto_dh_op_param. I would expect that dh_op_param and dsa_op_param would need to be set, which couldn't be done. Thanks, Pablo > > Thanks > Shally > > > > > //snip
