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
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

Reply via email to