Hi Shally,

> -----Original Message-----
> From: Verma, Shally [mailto:shally.ve...@cavium.com]
> Sent: Tuesday, January 30, 2018 11:54 AM
> To: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; Akhil Goyal
> <akhil.go...@nxp.com>; Trahe, Fiona <fiona.tr...@intel.com>;
> hemant.agra...@nxp.com; Doherty, Declan <declan.dohe...@intel.com>;
> Griffin, John <john.grif...@intel.com>; Jain, Deepak K
> <deepak.k.j...@intel.com>; j...@semihalf.com; t...@semihalf.com;
> d...@marvell.com; nsams...@marvell.com; jianbo....@arm.com; Jacob,
> Jerin <jerin.jacobkollanukka...@cavium.com>; Athreya, Narayana Prasad
> <narayanaprasad.athr...@cavium.com>; Murthy, Nidadavolu
> <nidadavolu.mur...@cavium.com>
> Cc: dev@dpdk.org
> Subject: RE: [dpdk-dev] [PATCH] doc: announce ABI change for crypto info
> struct
> 
> 
> 
> >-----Original Message-----
> >From: De Lara Guarch, Pablo [mailto:pablo.de.lara.gua...@intel.com]
> >Sent: 30 January 2018 16:51
> >To: Verma, Shally <shally.ve...@cavium.com>; Akhil Goyal
> ><akhil.go...@nxp.com>; Trahe, Fiona <fiona.tr...@intel.com>;
> >hemant.agra...@nxp.com; Doherty, Declan <declan.dohe...@intel.com>;
> >Griffin, John <john.grif...@intel.com>; Jain, Deepak K
> ><deepak.k.j...@intel.com>; j...@semihalf.com; t...@semihalf.com;
> >d...@marvell.com; nsams...@marvell.com; jianbo....@arm.com; Jacob,
> >Jerin <jerin.jacobkollanukka...@cavium.com>; Athreya, Narayana Prasad
> ><narayanaprasad.athr...@cavium.com>; Murthy, Nidadavolu
> ><nidadavolu.mur...@cavium.com>
> >Cc: dev@dpdk.org
> >Subject: RE: [dpdk-dev] [PATCH] doc: announce ABI change for crypto
> >info struct
> >
> >Hi Shally/Ahkil,
> >
> >> -----Original Message-----
> >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Verma, Shally
> >> Sent: Tuesday, January 30, 2018 7:56 AM
> >> To: Akhil Goyal <akhil.go...@nxp.com>; De Lara Guarch, Pablo
> >> <pablo.de.lara.gua...@intel.com>; Trahe, Fiona
> >> <fiona.tr...@intel.com>; hemant.agra...@nxp.com; Doherty, Declan
> >> <declan.dohe...@intel.com>; Griffin, John <john.grif...@intel.com>;
> >> Jain, Deepak K <deepak.k.j...@intel.com>; j...@semihalf.com;
> >> t...@semihalf.com; d...@marvell.com; nsams...@marvell.com;
> >> jianbo....@arm.com; Jacob, Jerin
> >> <jerin.jacobkollanukka...@cavium.com>; Athreya, Narayana Prasad
> >> <narayanaprasad.athr...@cavium.com>; Murthy, Nidadavolu
> >> <nidadavolu.mur...@cavium.com>
> >> Cc: dev@dpdk.org
> >> Subject: Re: [dpdk-dev] [PATCH] doc: announce ABI change for crypto
> >> info struct
> >>
> >> I do see current cryptodev unit testcase (inside \test dir) uses
> >> info.sym.max_nb_sessions param for session mempool_create. So, such
> >> testcases change are also in proposal?
> >
> >Yes, for these tests, we can just define a macro in the tests, instead of
> using the info structure.
> 
> [Shally] Ok, then you mean applications will choose any random number
> during mempool_create and not dependent on device max_nb_sessions?

Yes, actually for the unit tests, even one session is enough.

> 
> >>
> >> Another point, we recently submitted an RFC patch on lib/cryptodev
> >> with asymmetric crypto support
> >> (https://dpdk.org/dev/patchwork/patch/34308/) which is awaiting
> >> review and these fields have role to play there.
> >> So, could this change be please viewed in conjunction with asym RFC?
> >
> >Do you need it for asymmetric? Anyway, this would remove the
> symmetric function and structures, not applicable for you.
> 
> [Shally] I would say addition of asym in lib/cryptodev is not entirely
> standalone, specifically for PMDs that can support both.
> My key concern are max_nb_sessions_per_qp and related
> qp_attach_sym/asym APIs which enable management of queue distribution
> among sym and asym in current proposal, specifically, for PMDs that can
> support both but have dedicated qp for each. Right now proposal is open
> for feedback and would prefer to be covered before sym related changes
> could be applied.

Actually, I have been thinking about this. Given the time we have until 18.02 
is out,
and that this is not urgent to be applied (this is just code cleanup),
I am postponing this until next release. 

My other reason is that the info structure has a rte_pci_device pointer which 
should be removed.
However, I believe it is better to leave it for next release and discuss it 
with other libraries which has this, like ethdev.

Thanks,
Pablo

Reply via email to