[dpdk-dev] [PATCH 3/4] net/mlx5: fix secondary process attach port Tx queue

Raslan Darawsheh rasland at nvidia.com
Mon Jan 25 09:30:58 CET 2021


Hi,

> -----Original Message-----
> From: Suanming Mou <suanmingm at nvidia.com>
> Sent: Sunday, January 24, 2021 1:02 PM
> To: Slava Ovsiienko <viacheslavo at nvidia.com>; Matan Azrad
> <matan at nvidia.com>
> Cc: Raslan Darawsheh <rasland at nvidia.com>; dev at dpdk.org;
> stable at dpdk.org
> Subject: [PATCH 3/4] net/mlx5: fix secondary process attach port Tx queue
> 
> Currently, the secondary process port UAR register mapping used by Tx
> queue is done during port initializing.
> 
> Unluckily, in port hot-plug case, the secondary process was requested
> to initialize the port when primary process did not complete the
> device configuration and the port Tx queue number is not configured
> yet. Hence, the secondary process getS the zero Tx queue number during
> probing, causing the UAR registers not be mapped in the correct
> fashion.
> 
> This commit checks the configured number of Tx queues in secondary
> process when the port start is requested. In case the Tx queue
> number mismatch found the UAR mapping is reinitialized accordingly.
> 
> Fixes: 2aac5b5d119f ("net/mlx5: sync stop/start with secondary process")
> cc: stable at dpdk.org
> 
> Signed-off-by: Suanming Mou <suanmingm at nvidia.com>
> Acked-by: Viacheslav Ovsiienko <viacheslavo at nvidia.com>
> ---
>  drivers/net/mlx5/linux/mlx5_mp_os.c | 19 +++++++++++++++++++
>  drivers/net/mlx5/mlx5.c             |  2 +-
>  drivers/net/mlx5/mlx5.h             |  6 +++++-
>  3 files changed, 25 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/mlx5/linux/mlx5_mp_os.c
> b/drivers/net/mlx5/linux/mlx5_mp_os.c
> index 08ade75..95372e2 100644
> --- a/drivers/net/mlx5/linux/mlx5_mp_os.c
> +++ b/drivers/net/mlx5/linux/mlx5_mp_os.c
[...]
> 
> -/* Per-process private structure. */
> +/*
> + * Per-process private structure.
> + * Caution, secondary pocess may rebuid the struct during port start.
> + */
Typo in
pocess-> process
rebuid-> rebuild
will fix during integration.
[...]
> 1.8.3.1

Kindest regards
Raslan Darawsheh


More information about the dev mailing list