Re: [PATCH v4 08/15] vdpa/mlx5: add multi-thread management for configuration

[Maxime Coquelin <maxime.coquelin@redhat.com>]

On 6/18/22 11:02, Li Zhang wrote:
> The LM process includes a lot of objects creations and
> destructions in the source and the destination servers.
> As much as LM time increases, the packet drop of the VM increases.
> To improve LM time need to parallel the configurations for mlx5 FW.
> Add internal multi-thread management in the driver for it.
> 
> A new devarg defines the number of threads and their CPU.
> The management is shared between all the devices of the driver.
> Since the event_core also affects the datapath events thread,
> reduce the priority of the datapath event thread to
> allow fast configuration of the devices doing the LM.
> 
> Signed-off-by: Li Zhang <lizh@nvidia.com>
> Acked-by: Matan Azrad <matan@nvidia.com>
> ---
>   doc/guides/vdpadevs/mlx5.rst          |  11 +++
>   drivers/vdpa/mlx5/meson.build         |   1 +
>   drivers/vdpa/mlx5/mlx5_vdpa.c         |  41 ++++++++
>   drivers/vdpa/mlx5/mlx5_vdpa.h         |  36 +++++++
>   drivers/vdpa/mlx5/mlx5_vdpa_cthread.c | 129 ++++++++++++++++++++++++++
>   drivers/vdpa/mlx5/mlx5_vdpa_event.c   |   2 +-
>   drivers/vdpa/mlx5/mlx5_vdpa_virtq.c   |   8 +-
>   7 files changed, 223 insertions(+), 5 deletions(-)
>   create mode 100644 drivers/vdpa/mlx5/mlx5_vdpa_cthread.c
> 

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime