Re: [PATCH] net/ice: fix segmentation fault Rx function

[Andre Muezerie <andremue@linux.microsoft.com>]

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: andremue@linux.microsoft.com
> 
> Signed-off-by: Vladimir Medvedkin <vladimir.medvedkin@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.