> -----Original Message----- > From: Trahe, Fiona > Sent: Friday, January 26, 2018 10:45 AM > To: De Lara Guarch, Pablo <[email protected]>; > [email protected]; [email protected]; Doherty, Declan > <[email protected]>; [email protected]; Griffin, John > <[email protected]>; Jain, Deepak K <[email protected]>; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected] > Cc: [email protected]; Trahe, Fiona <[email protected]> > Subject: RE: [PATCH] doc: announce ABI change for crypto info struct > > Hi Pablo, > > > -----Original Message----- > > From: De Lara Guarch, Pablo > > Sent: Friday, January 26, 2018 9:04 AM > > To: [email protected]; [email protected]; Doherty, Declan > > <[email protected]>; [email protected]; Trahe, Fiona > > <[email protected]>; Griffin, John <[email protected]>; Jain, > > Deepak K <[email protected]>; [email protected]; > > [email protected]; [email protected]; [email protected]; > > [email protected] > > Cc: [email protected]; De Lara Guarch, Pablo > > <[email protected]> > > Subject: [PATCH] doc: announce ABI change for crypto info struct > > > > Since the API changes made in 17.08, the session mempool is not > > created anymore in each crypto device. > > Therefore, there is no need to have, in the cryptodev info structure, > > the maximum number of sessions supported per device and per queue > > pair. > > > > Signed-off-by: Pablo de Lara <[email protected]> > > --- > > doc/guides/rel_notes/deprecation.rst | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/doc/guides/rel_notes/deprecation.rst > > b/doc/guides/rel_notes/deprecation.rst > > index d59ad5988..5588ba7c1 100644 > > --- a/doc/guides/rel_notes/deprecation.rst > > +++ b/doc/guides/rel_notes/deprecation.rst > > @@ -59,3 +59,8 @@ Deprecation Notices > > be added between the producer and consumer structures. The size of > the > > structure and the offset of the fields will remain the same on > > platforms with 64B cache line, but will change on other platforms. > > + > > +* cryptodev: The structure ``sym``, including its fields > > +``max_nb_sessions`` > > + and ``max_nb_sessions_per_qp``, in structure > > +``rte_cryptodev_info``, > > + will be removed in 18.05, as these fields are not relevant anymore > > + since the session mempool is not internal in the crypto device > anymore. > > -- > [Fiona] max_nb_sessions must be also removed from struct > rte_cryptodev_pmd_init_params
Good point. Since this structure is internal, I guess we don't need a deprecation notice for it, but I will remove it in the patch for 18.05. > Regards deprecation of max_nb_sessions from both structs: > Acked-by: Fiona Trahe <[email protected]> > > If removing the max_nb_sessions_per_qp, then the following functions > should also be deprecated. > rte_cryptodev_queue_pair_attach_sym_session > rte_cryptodev_queue_pair_detach_sym_session > These and the max_nb_session_per_qp were added here at request of NXP: > http://dpdk.org/ml/archives/dev/2017-March/060740.html Akhil, do you agree on this change? Thanks, Pablo
