Re: [PATCH 19.11] net/mlx5: fix Altivec Rx

[Christian Ehrhardt <christian.ehrhardt@canonical.com>]

On Fri, Dec 3, 2021 at 9:59 AM Xueming Li <xuemingl@nvidia.com> wrote:
>
> [ upstream commit 68fa62924de4fed4c241db9b7fcb58e66f62a2f9 ]
>
> This patch fixes stale field reference.

Applied, thanks

> Fixes: a18ac6113331 ("net/mlx5: add metadata support to Rx datapath")
> Cc: stable@dpdk.org
>
> Signed-off-by: Xueming Li <xuemingl@nvidia.com>
> Acked-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
> ---
>  drivers/net/mlx5/mlx5_rxtx_vec_altivec.h | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h b/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h
> index acbc329674a..576bc627b7b 100644
> --- a/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h
> +++ b/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h
> @@ -808,10 +808,10 @@ rxq_burst_v(struct mlx5_rxq_data *rxq, struct rte_mbuf **pkts, uint16_t pkts_n,
>                         (vector unsigned short)cqe_tmp1, cqe_sel_mask1);
>                 cqe_tmp2 = (vector unsigned char)(vector unsigned long){
>                         *(__attribute__((__aligned__(8))) unsigned long *)
> -                       &cq[pos + p3].rsvd3[9], 0LL};
> +                       &cq[pos + p3].rsvd4[2], 0LL};
>                 cqe_tmp1 = (vector unsigned char)(vector unsigned long){
>                         *(__attribute__((__aligned__(8))) unsigned long *)
> -                       &cq[pos + p2].rsvd3[9], 0LL};
> +                       &cq[pos + p2].rsvd4[2], 0LL};
>                 cqes[3] = (vector unsigned char)
>                         vec_sel((vector unsigned short)cqes[3],
>                         (vector unsigned short)cqe_tmp2,
> @@ -871,10 +871,10 @@ rxq_burst_v(struct mlx5_rxq_data *rxq, struct rte_mbuf **pkts, uint16_t pkts_n,
>                         (vector unsigned short)cqe_tmp1, cqe_sel_mask1);
>                 cqe_tmp2 = (vector unsigned char)(vector unsigned long){
>                         *(__attribute__((__aligned__(8))) unsigned long *)
> -                       &cq[pos + p1].rsvd3[9], 0LL};
> +                       &cq[pos + p1].rsvd4[2], 0LL};
>                 cqe_tmp1 = (vector unsigned char)(vector unsigned long){
>                         *(__attribute__((__aligned__(8))) unsigned long *)
> -                       &cq[pos].rsvd3[9], 0LL};
> +                       &cq[pos].rsvd4[2], 0LL};
>                 cqes[1] = (vector unsigned char)
>                         vec_sel((vector unsigned short)cqes[1],
>                         (vector unsigned short)cqe_tmp2, cqe_sel_mask2);
> --
> 2.34.0
>


-- 
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd