Hi Pablo, > -----Original Message----- > From: De Lara Guarch, Pablo > Sent: Friday, January 26, 2018 9:04 AM > To: akhil.go...@nxp.com; hemant.agra...@nxp.com; Doherty, Declan > <declan.dohe...@intel.com>; > jerin.ja...@intel.com; Trahe, Fiona <fiona.tr...@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 > Cc: dev@dpdk.org; De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com> > 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 <pablo.de.lara.gua...@intel.com> > --- > 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 Regards deprecation of max_nb_sessions from both structs: Acked-by: Fiona Trahe <fiona.tr...@intel.com>
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
