> -----Original Message-----
> From: Akhil Goyal <gak...@marvell.com>
> Sent: Saturday, July 17, 2021 3:44 AM
> To: Shiri Kuzin <shi...@nvidia.com>; dev@dpdk.org
> Cc: Matan Azrad <ma...@nvidia.com>; Suanming Mou
> <suanmi...@nvidia.com>; david.march...@redhat.com
> Subject: RE: [EXT] [PATCH v8 03/16] crypto/mlx5: add session operations
>
> > Sessions are used in symmetric transformations in order to prepare
> > objects and data for packet processing stage.
> >
> > A mlx5 session includes iv_offset, pointer to mlx5_crypto_dek struct,
> > bsf_size, bsf_p_type, block size index, encryption_order and
> > encryption standard.
> >
> > Implement the next session operations:
> > mlx5_crypto_sym_session_get_size- returns the size of the mlx5
> > session struct.
> > mlx5_crypto_sym_session_configure- prepares the DEK hash-list
> > and saves all the session data.
> > mlx5_crypto_sym_session_clear - destroys the DEK hash-list.
> >
> > Signed-off-by: Shiri Kuzin <shi...@nvidia.com>
> > Acked-by: Matan Azrad <ma...@nvidia.com>
> > ---
> > doc/guides/cryptodevs/features/mlx5.ini | 5 +
> > doc/guides/cryptodevs/mlx5.rst | 10 ++
>
> Documentation update is done in 3/16 which means the feature is complete.
> But basic operations and queue pair initialization is done later.
> This is not good. Please move this patch after QP additions.
> Please take reference of other drivers for sequence of patches.
I'm not sure what was discussed before for suitable documentation places, but
as you suggested here and in other patches, can we be aligned to put all this
documentation related pieces to the (13/16) final PMD patch?
>
> > drivers/crypto/mlx5/mlx5_crypto.c | 172 +++++++++++++++++++++++-
> > 3 files changed, 182 insertions(+), 5 deletions(-)
> >
