Re: [Intel-wired-lan] [PATCH net-next v1] idpf: export RX hardware timestamping information to XDP

[Loktionov, Aleksandr]

> -----Original Message-----
> From: YiFei Zhu <zhuyi...@google.com>
> Sent: Monday, November 24, 2025 11:52 AM
> To: Loktionov, Aleksandr <aleksandr.loktio...@intel.com>
> Cc: Mina Almasry <almasrym...@google.com>; net...@vger.kernel.org;
> b...@vger.kernel.org; linux-ker...@vger.kernel.org; Alexei Starovoitov
> <a...@kernel.org>; Daniel Borkmann <dan...@iogearbox.net>; David S.
> Miller <da...@davemloft.net>; Jakub Kicinski <k...@kernel.org>; Jesper
> Dangaard Brouer <h...@kernel.org>; John Fastabend
> <john.fastab...@gmail.com>; Stanislav Fomichev <s...@fomichev.me>;
> Nguyen, Anthony L <anthony.l.ngu...@intel.com>; Kitszel, Przemyslaw
> <przemyslaw.kits...@intel.com>; Andrew Lunn <andrew+net...@lunn.ch>;
> Eric Dumazet <eduma...@google.com>; Paolo Abeni <pab...@redhat.com>;
> Lobakin, Aleksander <aleksander.loba...@intel.com>; Richard Cochran
> <richardcoch...@gmail.com>; intel-wired-...@lists.osuosl.org
> Subject: Re: [Intel-wired-lan] [PATCH net-next v1] idpf: export RX
> hardware timestamping information to XDP
> 
> On Mon, Nov 24, 2025 at 12:33 AM Loktionov, Aleksandr
> <aleksandr.loktio...@intel.com> wrote:
> >
> >
> >
> > > -----Original Message-----
> > > From: Intel-wired-lan <intel-wired-lan-boun...@osuosl.org> On
> Behalf
> > > Of Mina Almasry
> > > Sent: Saturday, November 22, 2025 3:09 PM
> > > To: net...@vger.kernel.org; b...@vger.kernel.org; linux-
> > > ker...@vger.kernel.org
> > > Cc: YiFei Zhu <zhuyi...@google.com>; Alexei Starovoitov
> > > <a...@kernel.org>; Daniel Borkmann <dan...@iogearbox.net>; David S.
> > > Miller <da...@davemloft.net>; Jakub Kicinski <k...@kernel.org>;
> > > Jesper Dangaard Brouer <h...@kernel.org>; John Fastabend
> > > <john.fastab...@gmail.com>; Stanislav Fomichev <s...@fomichev.me>;
> > > Nguyen, Anthony L <anthony.l.ngu...@intel.com>; Kitszel,
> Przemyslaw
> > > <przemyslaw.kits...@intel.com>; Andrew Lunn
> <andrew+net...@lunn.ch>;
> > > Eric Dumazet <eduma...@google.com>; Paolo Abeni
> <pab...@redhat.com>;
> > > Lobakin, Aleksander <aleksander.loba...@intel.com>; Richard
> Cochran
> > > <richardcoch...@gmail.com>; intel-wired-...@lists.osuosl.org; Mina
> > > Almasry <almasrym...@google.com>
> > > Subject: [Intel-wired-lan] [PATCH net-next v1] idpf: export RX
> > > hardware timestamping information to XDP
> > >
> > > From: YiFei Zhu <zhuyi...@google.com>
> > >
> > > The logic is similar to idpf_rx_hwtstamp, but the data is exported
> > > as a BPF kfunc instead of appended to an skb.
> > >
> > > A idpf_queue_has(PTP, rxq) condition is added to check the queue
> > > supports PTP similar to idpf_rx_process_skb_fields.
> > >
> > > Cc: intel-wired-...@lists.osuosl.org
> > >
> > > Signed-off-by: YiFei Zhu <zhuyi...@google.com>
> > > Signed-off-by: Mina Almasry <almasrym...@google.com>
> > > ---
> > >  drivers/net/ethernet/intel/idpf/xdp.c | 27
> > > +++++++++++++++++++++++++++
> > >  1 file changed, 27 insertions(+)
> > >
> > > diff --git a/drivers/net/ethernet/intel/idpf/xdp.c
> > > b/drivers/net/ethernet/intel/idpf/xdp.c
> > > index 21ce25b0567f..850389ca66b6 100644
> > > --- a/drivers/net/ethernet/intel/idpf/xdp.c
> > > +++ b/drivers/net/ethernet/intel/idpf/xdp.c
> > > @@ -2,6 +2,7 @@
> > >  /* Copyright (C) 2025 Intel Corporation */
> > >
> > >  #include "idpf.h"
> > > +#include "idpf_ptp.h"
> > >  #include "idpf_virtchnl.h"
> > >  #include "xdp.h"
> > >  #include "xsk.h"
> > > @@ -369,6 +370,31 @@ int idpf_xdp_xmit(struct net_device *dev, int
> > > n, struct xdp_frame **frames,
> > >                                      idpf_xdp_tx_finalize);  }
> > >
> > > +static int idpf_xdpmo_rx_timestamp(const struct xdp_md *ctx, u64
> > > +*timestamp) {
> > > +     const struct virtchnl2_rx_flex_desc_adv_nic_3 *rx_desc;
> > > +     const struct libeth_xdp_buff *xdp = (typeof(xdp))ctx;
> > > +     const struct idpf_rx_queue *rxq;
> > > +     u64 cached_time, ts_ns;
> > > +     u32 ts_high;
> > > +
> > > +     rx_desc = xdp->desc;
> > > +     rxq = libeth_xdp_buff_to_rq(xdp, typeof(*rxq), xdp_rxq);
> > > +
> > > +     if (!idpf_queue_has(PTP, rxq))
> > > +             return -ENODATA;
> > > +     if (!(rx_desc->ts_low & VIRTCHNL2_RX_FLEX_TSTAMP_VALID))
> > > +             return -ENODATA;
> > RX flex desc fields are little‑endian.
> > You already convert ts_high with le32_to_cpu(), but test ts_low
> directly against the mask.
> > On big‑endian this can misdetect the bit and spuriously return -
> ENODATA.
> > Please convert ts_low to host order before the bit test.
> > See existing IDPF/ICE patterns where descriptor words are
> leXX_to_cpu()‑converted prior to FIELD_GET() / bit checks.
> > Also, per the XDP RX metadata kfunc docs, -ENODATA must reflect true
> absence of per‑packet metadata; endianness‑correct testing is required
> to uphold the semantic.
> 
> The logic is copied as verbatim from idpf_rx_hwtstamp:
> 
> static void
> idpf_rx_hwtstamp(const struct idpf_rx_queue *rxq,
>                  const struct virtchnl2_rx_flex_desc_adv_nic_3
> *rx_desc,
>                  struct sk_buff *skb)
> {
>         u64 cached_time, ts_ns;
>         u32 ts_high;
> 
>         if (!(rx_desc->ts_low & VIRTCHNL2_RX_FLEX_TSTAMP_VALID))
>                 return;
> 
>         cached_time = READ_ONCE(rxq->cached_phc_time);
> 
>         ts_high = le32_to_cpu(rx_desc->ts_high);
>         ts_ns = idpf_ptp_tstamp_extend_32b_to_64b(cached_time,
> ts_high); [...]
> 
> I assume that is incorrect and would need to be fixed too?
> 
I think must be fixed.

With the best regards
Alex

> YiFei Zhu


...