Re: [dpdk-dev] [PATCH 4/6] crypto/octeontx2: add asymmetric in enqueue/dequeue ops

[Anoob Joseph <anoobj@marvell.com>]

Hi Akhil,

PMD doesn't support sessionless with ASYM yet. Will make the suggested change in v2.

Thanks,
Anoob

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Akhil Goyal
> Sent: Tuesday, October 1, 2019 7:42 PM
> To: Anoob Joseph <anoobj@marvell.com>; Pablo de Lara
> <pablo.de.lara.guarch@intel.com>; Thomas Monjalon <thomas@monjalon.net>
> Cc: Sunila Sahu <ssahu@marvell.com>; Jerin Jacob Kollanukkaran
> <jerinj@marvell.com>; Narayana Prasad Raju Athreya
> <pathreya@marvell.com>; Fiona Trahe <fiona.trahe@intel.com>; Kanaka Durga
> Kotamarthy <kkotamarthy@marvell.com>; Shally Verma
> <shallyv@marvell.com>; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 4/6] crypto/octeontx2: add asymmetric in
> enqueue/dequeue ops
> 
> 
> Hi Anoob,
> 
> >  static __rte_always_inline int __hot
> >  otx2_cpt_enqueue_sym(struct otx2_cpt_qp *qp, struct rte_crypto_op *op,
> >  		     struct pending_queue *pend_q)
> > @@ -488,7 +560,9 @@ otx2_cpt_enqueue_burst(void *qptr, struct
> > rte_crypto_op **ops, uint16_t nb_ops)
> >
> >  	for (count = 0; count < nb_ops; count++) {
> >  		op = ops[count];
> > -		if (op->sess_type == RTE_CRYPTO_OP_WITH_SESSION)
> > +		if (op->type == RTE_CRYPTO_OP_TYPE_ASYMMETRIC)
> > +			ret = otx2_cpt_enqueue_asym(qp, op, pend_q);
> > +		else if (op->sess_type == RTE_CRYPTO_OP_WITH_SESSION)
> >  			ret = otx2_cpt_enqueue_sym(qp, op, pend_q);
> >  		else if (op->sess_type == RTE_CRYPTO_OP_SESSIONLESS)
> >  			ret = otx2_cpt_enqueue_sym_sessless(qp, op, pend_q);
> 
> Do you support session less in asym case? If not, can we add a check here.
>