On Thu, Jan 30, 2025 at 07:26:17PM +0000, Vladimir Medvedkin wrote:
> Fix a typo in avx512 rx function, where accidentally the _mm_load_si128
> argument was passed by value instead of a pointer.
>
> Fixes: 43fd3624fdfe ("drivers: replace GCC pragma with cast")
> Cc: andre...@linux.microsoft.com
>
> Signed-off-by: Vladimir Medvedkin <vladimir.medved...@intel.com>
> ---
> drivers/net/intel/ice/ice_rxtx_vec_avx512.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/intel/ice/ice_rxtx_vec_avx512.c
> b/drivers/net/intel/ice/ice_rxtx_vec_avx512.c
> index 681b31f61f..bd49be07c9 100644
> --- a/drivers/net/intel/ice/ice_rxtx_vec_avx512.c
> +++ b/drivers/net/intel/ice/ice_rxtx_vec_avx512.c
> @@ -474,7 +474,7 @@ _ice_recv_raw_pkts_vec_avx512(struct ice_rx_queue *rxq,
> (RTE_CAST_PTR(const __m128i *,
> &rxdp[7].wb.status_error1));
> rte_compiler_barrier();
> const __m128i raw_desc_bh6 = _mm_load_si128
> - (RTE_CAST_PTR(const __m128i *,
> rxdp[6].wb.status_error1));
> + (RTE_CAST_PTR(const __m128i *,
> &rxdp[6].wb.status_error1));
> rte_compiler_barrier();
> const __m128i raw_desc_bh5 = _mm_load_si128
> (RTE_CAST_PTR(const __m128i *,
> &rxdp[5].wb.status_error1));
> --
> 2.43.0
Thanks for catching and fixing this.
