> -----Original Message-----
> From: Martin Weiser <[email protected]>
> Sent: Tuesday, August 8, 2023 7:40 PM
> To: [email protected]
> Cc: Martin Weiser <[email protected]>; Yang, Qiming
> <[email protected]>; Zhang, Qi Z <[email protected]>
> Subject: [PATCH v2] net/ice: write rx timestamp to the first mbuf segment in
> scattered rx
> 
> Previously, the rx timestamp was written to the last segment of the mbuf 
> chain,
> which was unexpected.

Added:

Fixes: 5543827fc6df ("net/ice: improve performance of Rx timestamp offload")
Cc: [email protected]> 

> Signed-off-by: Martin Weiser <[email protected]>
> ---
>  drivers/net/ice/ice_rxtx.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ice/ice_rxtx.c b/drivers/net/ice/ice_rxtx.c index
> 64c4486b4b..e07c6d1f15 100644
> --- a/drivers/net/ice/ice_rxtx.c
> +++ b/drivers/net/ice/ice_rxtx.c
> @@ -2150,7 +2150,7 @@ ice_recv_scattered_pkts(void *rx_queue,
>                       }
>                       rxq->hw_time_update = rte_get_timer_cycles() /
>                                            (rte_get_timer_hz() / 1000);
> -                     *RTE_MBUF_DYNFIELD(rxm,
> +                     *RTE_MBUF_DYNFIELD(first_seg,
>                                          (ice_timestamp_dynfield_offset),
>                                          rte_mbuf_timestamp_t *) = ts_ns;
>                       pkt_flags |= ice_timestamp_dynflag;
> --
> 2.34.1

Thanks for capture this.

Acked-by: Qi Zhang <[email protected]>

Applied to dpdk-next-net-intel.

Reply via email to