Re: [PATCH v2] net/ice: write rx timestamp to the first mbuf segment in scattered rx

[Martin Weiser]

Hi,

sorry for the first patch. I did not manage to get git send-mail to work 
with office365 OAuth2 and Thunderbird messed up the format even in text 
mode.

By the way, are there any plans to support hardware timestamping in the 
vectorized codepath?

Best regards,
Martin


Am 08.08.23 um 13:39 schrieb Martin Weiser:
Previously, the rx timestamp was written to the last segment of the mbuf
chain, which was unexpected.

Signed-off-by: Martin Weiser <martin.wei...@allegro-packets.com>
---
  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;