From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3213AC63797 for ; Thu, 12 Jan 2023 19:22:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240342AbjALTWl (ORCPT ); Thu, 12 Jan 2023 14:22:41 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44832 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240500AbjALTWT (ORCPT ); Thu, 12 Jan 2023 14:22:19 -0500 Received: from mail-pg1-x529.google.com (mail-pg1-x529.google.com [IPv6:2607:f8b0:4864:20::529]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2412167BF8 for ; Thu, 12 Jan 2023 11:10:08 -0800 (PST) Received: by mail-pg1-x529.google.com with SMTP id b12so13437763pgj.6 for ; Thu, 12 Jan 2023 11:10:08 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=CG7CR6w0M5I2GSnGZToCTG9bTDe48L+eIrr5AXRYBSc=; b=mE5EtTFdPxcz3Kf5nVv/TGCF/np5rchgTFw+oPZHdMnvGkXQtKOlwqZb4AFwZotSV9 kP1TRTWgzzXFFdDrQsiNz6Sk0tohm3+DfYWTbUXEdFaYW0w/0KYpTwC7XSp/1pAEKCwW 30wuDT6ywMrGD0ISIkk4DvcsTl6N6AOTDR/MhmWjUOYgr+iJ32ss3unrvksKDeA14XcS DWal5cx0AcCRqKb9oSahjzIGEQzIR8hYWH3VPC5R6pDFzl7k2WReBPHWr5EQqbmjzr16 uC2/z5IjX4tUb6OPi9XGCdBKR9mf5BEubPNsz52rcxHjdsH1bTyGlFNjMwc52ggCDGip GaGw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=CG7CR6w0M5I2GSnGZToCTG9bTDe48L+eIrr5AXRYBSc=; b=WIFGGXVDcQLT7bnPZiTVTKU7QoWDoiKKZB2FuAf/epyLIS1tHS2RWWf7AUPdfv7a35 PzOqJxKlulNRpjlM+loH7zwV4eajLuxjBvO1Bwfrd31jmwb/z189xYI1tTIIWhomXT89 MRcNj6OmpGWZ34WzHqb3i5AiGsybYrs9XkXKcKnxh+pxc+sjuHP/KQ5LaWeJYbVQl7Mc 0O+6muKN74ID0+h6ymp4X72dDJtbyrliMeavw/0gOW0eBRnI/mxkzRs/kwNO/YMRfrf2 lEb6oeuuSb8ddSlbT/0x3gpANd1rn4tSBdQzV48isghqbio5P9U+2AMSiWBP22EfC+ss U6Ig== X-Gm-Message-State: AFqh2koSSG5gLJeQW1vJS7A7jdZGXvZh5xRDNmV9OOBHtC1h1eGjJ5Zj c3GkedZfV7uhR+15kfw3tjpP2y58/zNH5LyqnVhlqQ== X-Google-Smtp-Source: AMrXdXuWVTymlmYkLzjqrrjtbEBmwThK5kGQsVxmk415qiG8R9VaBSA7wqMhJoRkboqnhoulUgwIpZn4TIn9zs2gy5M= X-Received: by 2002:a05:6a00:1d95:b0:583:2058:e4 with SMTP id z21-20020a056a001d9500b00583205800e4mr2232166pfw.42.1673550607354; Thu, 12 Jan 2023 11:10:07 -0800 (PST) MIME-Version: 1.0 References: <20230112003230.3779451-1-sdf@google.com> <20230112003230.3779451-17-sdf@google.com> <9545d379-9894-c7b6-d000-4f8eb6705bb1@gmail.com> In-Reply-To: <9545d379-9894-c7b6-d000-4f8eb6705bb1@gmail.com> From: Stanislav Fomichev Date: Thu, 12 Jan 2023 11:09:55 -0800 Message-ID: Subject: Re: [PATCH bpf-next v7 16/17] net/mlx5e: Support RX XDP metadata To: Tariq Toukan Cc: bpf@vger.kernel.org, ast@kernel.org, daniel@iogearbox.net, andrii@kernel.org, martin.lau@linux.dev, song@kernel.org, yhs@fb.com, john.fastabend@gmail.com, kpsingh@kernel.org, haoluo@google.com, jolsa@kernel.org, =?UTF-8?B?VG9rZSBIw7hpbGFuZC1Kw7hyZ2Vuc2Vu?= , Tariq Toukan , Saeed Mahameed , David Ahern , Jakub Kicinski , Willem de Bruijn , Jesper Dangaard Brouer , Anatoly Burakov , Alexander Lobakin , Magnus Karlsson , Maryam Tahhan , xdp-hints@xdp-project.net, netdev@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Thu, Jan 12, 2023 at 12:13 AM Tariq Toukan wro= te: > > > > On 12/01/2023 2:32, Stanislav Fomichev wrote: > > From: Toke H=C3=B8iland-J=C3=B8rgensen > > > > Support RX hash and timestamp metadata kfuncs. We need to pass in the c= qe > > pointer to the mlx5e_skb_from* functions so it can be retrieved from th= e > > XDP ctx to do this. > > > > Cc: Tariq Toukan > > Cc: Saeed Mahameed > > Cc: John Fastabend > > Cc: David Ahern > > Cc: Martin KaFai Lau > > Cc: Jakub Kicinski > > Cc: Willem de Bruijn > > Cc: Jesper Dangaard Brouer > > Cc: Anatoly Burakov > > Cc: Alexander Lobakin > > Cc: Magnus Karlsson > > Cc: Maryam Tahhan > > Cc: xdp-hints@xdp-project.net > > Cc: netdev@vger.kernel.org > > Signed-off-by: Toke H=C3=B8iland-J=C3=B8rgensen > > Signed-off-by: Stanislav Fomichev > > --- > > drivers/net/ethernet/mellanox/mlx5/core/en.h | 5 +- > > .../net/ethernet/mellanox/mlx5/core/en/txrx.h | 5 ++ > > .../net/ethernet/mellanox/mlx5/core/en/xdp.c | 23 +++++++++ > > .../net/ethernet/mellanox/mlx5/core/en/xdp.h | 5 ++ > > .../ethernet/mellanox/mlx5/core/en/xsk/rx.c | 10 ++++ > > .../ethernet/mellanox/mlx5/core/en/xsk/rx.h | 2 + > > .../net/ethernet/mellanox/mlx5/core/en_main.c | 6 +++ > > .../net/ethernet/mellanox/mlx5/core/en_rx.c | 49 ++++++++++--------= - > > 8 files changed, 80 insertions(+), 25 deletions(-) > > > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en.h b/drivers/net= /ethernet/mellanox/mlx5/core/en.h > > index af663978d1b4..6de02d8aeab8 100644 > > --- a/drivers/net/ethernet/mellanox/mlx5/core/en.h > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en.h > > @@ -627,10 +627,11 @@ struct mlx5e_rq; > > typedef void (*mlx5e_fp_handle_rx_cqe)(struct mlx5e_rq*, struct mlx5_= cqe64*); > > typedef struct sk_buff * > > (*mlx5e_fp_skb_from_cqe_mpwrq)(struct mlx5e_rq *rq, struct mlx5e_mpw_= info *wi, > > - u16 cqe_bcnt, u32 head_offset, u32 page_id= x); > > + struct mlx5_cqe64 *cqe, u16 cqe_bcnt, > > + u32 head_offset, u32 page_idx); > > typedef struct sk_buff * > > (*mlx5e_fp_skb_from_cqe)(struct mlx5e_rq *rq, struct mlx5e_wqe_frag_i= nfo *wi, > > - u32 cqe_bcnt); > > + struct mlx5_cqe64 *cqe, u32 cqe_bcnt); > > typedef bool (*mlx5e_fp_post_rx_wqes)(struct mlx5e_rq *rq); > > typedef void (*mlx5e_fp_dealloc_wqe)(struct mlx5e_rq*, u16); > > typedef void (*mlx5e_fp_shampo_dealloc_hd)(struct mlx5e_rq*, u16, u16= , bool); > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en/txrx.h b/driver= s/net/ethernet/mellanox/mlx5/core/en/txrx.h > > index 853f312cd757..757c012ece27 100644 > > --- a/drivers/net/ethernet/mellanox/mlx5/core/en/txrx.h > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en/txrx.h > > @@ -73,6 +73,11 @@ int mlx5e_poll_rx_cq(struct mlx5e_cq *cq, int budget= ); > > void mlx5e_free_rx_descs(struct mlx5e_rq *rq); > > void mlx5e_free_rx_in_progress_descs(struct mlx5e_rq *rq); > > > > +static inline bool mlx5e_rx_hw_stamp(struct hwtstamp_config *config) > > +{ > > + return config->rx_filter =3D=3D HWTSTAMP_FILTER_ALL; > > +} > > + > > /* TX */ > > netdev_tx_t mlx5e_xmit(struct sk_buff *skb, struct net_device *dev); > > bool mlx5e_poll_tx_cq(struct mlx5e_cq *cq, int napi_budget); > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en/xdp.c b/drivers= /net/ethernet/mellanox/mlx5/core/en/xdp.c > > index 31bb6806bf5d..d10d31e12ba2 100644 > > --- a/drivers/net/ethernet/mellanox/mlx5/core/en/xdp.c > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en/xdp.c > > @@ -156,6 +156,29 @@ mlx5e_xmit_xdp_buff(struct mlx5e_xdpsq *sq, struct= mlx5e_rq *rq, > > return true; > > } > > > > +int mlx5e_xdp_rx_timestamp(const struct xdp_md *ctx, u64 *timestamp) > > +{ > > + const struct mlx5e_xdp_buff *_ctx =3D (void *)ctx; > > + > > + if (unlikely(!mlx5e_rx_hw_stamp(_ctx->rq->tstamp))) > > + return -EOPNOTSUPP; > > + > > + *timestamp =3D mlx5e_cqe_ts_to_ns(_ctx->rq->ptp_cyc2time, > > + _ctx->rq->clock, get_cqe_ts(_ctx= ->cqe)); > > + return 0; > > +} > > + > > +int mlx5e_xdp_rx_hash(const struct xdp_md *ctx, u32 *hash) > > +{ > > + const struct mlx5e_xdp_buff *_ctx =3D (void *)ctx; > > + > > + if (unlikely(!(_ctx->xdp.rxq->dev->features & NETIF_F_RXHASH))) > > + return -EOPNOTSUPP; > > + > > + *hash =3D be32_to_cpu(_ctx->cqe->rss_hash_result); > > + return 0; > > +} > > + > > /* returns true if packet was consumed by xdp */ > > bool mlx5e_xdp_handle(struct mlx5e_rq *rq, struct page *page, > > struct bpf_prog *prog, struct mlx5e_xdp_buff *mxbuf= ) > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en/xdp.h b/drivers= /net/ethernet/mellanox/mlx5/core/en/xdp.h > > index 389818bf6833..cb568c62aba0 100644 > > --- a/drivers/net/ethernet/mellanox/mlx5/core/en/xdp.h > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en/xdp.h > > @@ -46,6 +46,8 @@ > > > > struct mlx5e_xdp_buff { > > struct xdp_buff xdp; > > + struct mlx5_cqe64 *cqe; > > + struct mlx5e_rq *rq; > > }; > > > > struct mlx5e_xsk_param; > > @@ -60,6 +62,9 @@ void mlx5e_xdp_rx_poll_complete(struct mlx5e_rq *rq); > > int mlx5e_xdp_xmit(struct net_device *dev, int n, struct xdp_frame **= frames, > > u32 flags); > > > > +int mlx5e_xdp_rx_timestamp(const struct xdp_md *ctx, u64 *timestamp); > > +int mlx5e_xdp_rx_hash(const struct xdp_md *ctx, u32 *hash); > > + > > INDIRECT_CALLABLE_DECLARE(bool mlx5e_xmit_xdp_frame_mpwqe(struct mlx5= e_xdpsq *sq, > > struct mlx5e_xm= it_data *xdptxd, > > struct skb_shar= ed_info *sinfo, > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en/xsk/rx.c b/driv= ers/net/ethernet/mellanox/mlx5/core/en/xsk/rx.c > > index 9cff82d764e3..8bf3029abd3c 100644 > > --- a/drivers/net/ethernet/mellanox/mlx5/core/en/xsk/rx.c > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en/xsk/rx.c > > @@ -49,6 +49,7 @@ int mlx5e_xsk_alloc_rx_mpwqe(struct mlx5e_rq *rq, u16= ix) > > umr_wqe->inline_mtts[i] =3D (struct mlx5_mtt) { > > .ptag =3D cpu_to_be64(addr | MLX5_EN_WR), > > }; > > + wi->alloc_units[i].mxbuf->rq =3D rq; > > } > > } else if (unlikely(rq->mpwqe.umr_mode =3D=3D MLX5E_MPWRQ_UMR_MOD= E_UNALIGNED)) { > > for (i =3D 0; i < batch; i++) { > > @@ -58,6 +59,7 @@ int mlx5e_xsk_alloc_rx_mpwqe(struct mlx5e_rq *rq, u16= ix) > > .key =3D rq->mkey_be, > > .va =3D cpu_to_be64(addr), > > }; > > + wi->alloc_units[i].mxbuf->rq =3D rq; > > } > > } else if (likely(rq->mpwqe.umr_mode =3D=3D MLX5E_MPWRQ_UMR_MODE_= TRIPLE)) { > > u32 mapping_size =3D 1 << (rq->mpwqe.page_shift - 2); > > @@ -81,6 +83,7 @@ int mlx5e_xsk_alloc_rx_mpwqe(struct mlx5e_rq *rq, u16= ix) > > .key =3D rq->mkey_be, > > .va =3D cpu_to_be64(rq->wqe_overflow.addr= ), > > }; > > + wi->alloc_units[i].mxbuf->rq =3D rq; > > } > > } else { > > __be32 pad_size =3D cpu_to_be32((1 << rq->mpwqe.page_shif= t) - > > @@ -100,6 +103,7 @@ int mlx5e_xsk_alloc_rx_mpwqe(struct mlx5e_rq *rq, u= 16 ix) > > .va =3D cpu_to_be64(rq->wqe_overflow.addr= ), > > .bcount =3D pad_size, > > }; > > + wi->alloc_units[i].mxbuf->rq =3D rq; > > } > > } > > > > @@ -230,6 +234,7 @@ static struct sk_buff *mlx5e_xsk_construct_skb(stru= ct mlx5e_rq *rq, struct xdp_b > > > > struct sk_buff *mlx5e_xsk_skb_from_cqe_mpwrq_linear(struct mlx5e_rq *= rq, > > struct mlx5e_mpw_info= *wi, > > + struct mlx5_cqe64 *cq= e, > > u16 cqe_bcnt, > > u32 head_offset, > > u32 page_idx) > > @@ -250,6 +255,8 @@ struct sk_buff *mlx5e_xsk_skb_from_cqe_mpwrq_linear= (struct mlx5e_rq *rq, > > */ > > WARN_ON_ONCE(head_offset); > > > > + /* mxbuf->rq is set on allocation, but cqe is per-packet so set i= t here */ > > + mxbuf->cqe =3D cqe; > > xsk_buff_set_size(&mxbuf->xdp, cqe_bcnt); > > xsk_buff_dma_sync_for_cpu(&mxbuf->xdp, rq->xsk_pool); > > net_prefetch(mxbuf->xdp.data); > > @@ -284,6 +291,7 @@ struct sk_buff *mlx5e_xsk_skb_from_cqe_mpwrq_linear= (struct mlx5e_rq *rq, > > > > struct sk_buff *mlx5e_xsk_skb_from_cqe_linear(struct mlx5e_rq *rq, > > struct mlx5e_wqe_frag_info = *wi, > > + struct mlx5_cqe64 *cqe, > > u32 cqe_bcnt) > > { > > struct mlx5e_xdp_buff *mxbuf =3D wi->au->mxbuf; > > @@ -296,6 +304,8 @@ struct sk_buff *mlx5e_xsk_skb_from_cqe_linear(struc= t mlx5e_rq *rq, > > */ > > WARN_ON_ONCE(wi->offset); > > > > + /* mxbuf->rq is set on allocation, but cqe is per-packet so set i= t here */ > > + mxbuf->cqe =3D cqe; > > xsk_buff_set_size(&mxbuf->xdp, cqe_bcnt); > > xsk_buff_dma_sync_for_cpu(&mxbuf->xdp, rq->xsk_pool); > > net_prefetch(mxbuf->xdp.data); > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en/xsk/rx.h b/driv= ers/net/ethernet/mellanox/mlx5/core/en/xsk/rx.h > > index 087c943bd8e9..cefc0ef6105d 100644 > > --- a/drivers/net/ethernet/mellanox/mlx5/core/en/xsk/rx.h > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en/xsk/rx.h > > @@ -13,11 +13,13 @@ int mlx5e_xsk_alloc_rx_wqes_batched(struct mlx5e_rq= *rq, u16 ix, int wqe_bulk); > > int mlx5e_xsk_alloc_rx_wqes(struct mlx5e_rq *rq, u16 ix, int wqe_bulk= ); > > struct sk_buff *mlx5e_xsk_skb_from_cqe_mpwrq_linear(struct mlx5e_rq *= rq, > > struct mlx5e_mpw_info= *wi, > > + struct mlx5_cqe64 *cq= e, > > u16 cqe_bcnt, > > u32 head_offset, > > u32 page_idx); > > struct sk_buff *mlx5e_xsk_skb_from_cqe_linear(struct mlx5e_rq *rq, > > struct mlx5e_wqe_frag_info = *wi, > > + struct mlx5_cqe64 *cqe, > > u32 cqe_bcnt); > > > > #endif /* __MLX5_EN_XSK_RX_H__ */ > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_main.c b/driver= s/net/ethernet/mellanox/mlx5/core/en_main.c > > index cff5f2e29e1e..be942c060774 100644 > > --- a/drivers/net/ethernet/mellanox/mlx5/core/en_main.c > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_main.c > > @@ -4913,6 +4913,11 @@ const struct net_device_ops mlx5e_netdev_ops =3D= { > > #endif > > }; > > > > +static const struct xdp_metadata_ops mlx5_xdp_metadata_ops =3D { > > + .xmo_rx_timestamp =3D mlx5e_xdp_rx_timestamp, > > + .xmo_rx_hash =3D mlx5e_xdp_rx_hash, > > +}; > > + > > Instead of exposing every single xdp function in the xdp header, move > this struct into the en/xdp.c, and expose it via en/xdp.h. > > See example struct and its usage: > netdev->ethtool_ops =3D &mlx5e_ethtool_ops; SG, will put "extern const struct xdp_metadata_ops mlx5_xdp_metadata_ops;" in en/xdp.h. LMK if you prefer that to go into en.h next to mlx5e_ethtool_ops extern. > > static u32 mlx5e_choose_lro_timeout(struct mlx5_core_dev *mdev, u32 w= anted_timeout) > > { > > int i; > > @@ -5053,6 +5058,7 @@ static void mlx5e_build_nic_netdev(struct net_dev= ice *netdev) > > SET_NETDEV_DEV(netdev, mdev->device); > > > > netdev->netdev_ops =3D &mlx5e_netdev_ops; > > + netdev->xdp_metadata_ops =3D &mlx5_xdp_metadata_ops; > > > > mlx5e_dcbnl_build_netdev(netdev); > > > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_rx.c b/drivers/= net/ethernet/mellanox/mlx5/core/en_rx.c > > index 6affdddf5bcf..7b08653be000 100644 > > --- a/drivers/net/ethernet/mellanox/mlx5/core/en_rx.c > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_rx.c > > @@ -62,10 +62,12 @@ > > > > static struct sk_buff * > > mlx5e_skb_from_cqe_mpwrq_linear(struct mlx5e_rq *rq, struct mlx5e_mpw= _info *wi, > > - u16 cqe_bcnt, u32 head_offset, u32 page_i= dx); > > + struct mlx5_cqe64 *cqe, u16 cqe_bcnt, u32= head_offset, > > + u32 page_idx); > > static struct sk_buff * > > mlx5e_skb_from_cqe_mpwrq_nonlinear(struct mlx5e_rq *rq, struct mlx5e_= mpw_info *wi, > > - u16 cqe_bcnt, u32 head_offset, u32 pag= e_idx); > > + struct mlx5_cqe64 *cqe, u16 cqe_bcnt, = u32 head_offset, > > + u32 page_idx); > > static void mlx5e_handle_rx_cqe(struct mlx5e_rq *rq, struct mlx5_cqe6= 4 *cqe); > > static void mlx5e_handle_rx_cqe_mpwrq(struct mlx5e_rq *rq, struct mlx= 5_cqe64 *cqe); > > static void mlx5e_handle_rx_cqe_mpwrq_shampo(struct mlx5e_rq *rq, str= uct mlx5_cqe64 *cqe); > > @@ -76,11 +78,6 @@ const struct mlx5e_rx_handlers mlx5e_rx_handlers_nic= =3D { > > .handle_rx_cqe_mpwqe_shampo =3D mlx5e_handle_rx_cqe_mpwrq_shampo, > > }; > > > > -static inline bool mlx5e_rx_hw_stamp(struct hwtstamp_config *config) > > -{ > > - return config->rx_filter =3D=3D HWTSTAMP_FILTER_ALL; > > -} > > - > > static inline void mlx5e_read_cqe_slot(struct mlx5_cqwq *wq, > > u32 cqcc, void *data) > > { > > @@ -1575,16 +1572,19 @@ struct sk_buff *mlx5e_build_linear_skb(struct m= lx5e_rq *rq, void *va, > > return skb; > > } > > > > -static void mlx5e_fill_mxbuf(struct mlx5e_rq *rq, void *va, u16 headro= om, > > - u32 len, struct mlx5e_xdp_buff *mxbuf) > > +static void mlx5e_fill_mxbuf(struct mlx5e_rq *rq, struct mlx5_cqe64 *c= qe, > > + void *va, u16 headroom, u32 len, > > + struct mlx5e_xdp_buff *mxbuf) > > { > > xdp_init_buff(&mxbuf->xdp, rq->buff.frame0_sz, &rq->xdp_rxq); > > xdp_prepare_buff(&mxbuf->xdp, va, headroom, len, true); > > + mxbuf->cqe =3D cqe; > > + mxbuf->rq =3D rq; > > } > > > > static struct sk_buff * > > mlx5e_skb_from_cqe_linear(struct mlx5e_rq *rq, struct mlx5e_wqe_frag_= info *wi, > > - u32 cqe_bcnt) > > + struct mlx5_cqe64 *cqe, u32 cqe_bcnt) > > { > > union mlx5e_alloc_unit *au =3D wi->au; > > u16 rx_headroom =3D rq->buff.headroom; > > @@ -1630,7 +1630,7 @@ mlx5e_skb_from_cqe_linear(struct mlx5e_rq *rq, st= ruct mlx5e_wqe_frag_info *wi, > > > > static struct sk_buff * > > mlx5e_skb_from_cqe_nonlinear(struct mlx5e_rq *rq, struct mlx5e_wqe_fr= ag_info *wi, > > - u32 cqe_bcnt) > > + struct mlx5_cqe64 *cqe, u32 cqe_bcnt) > > { > > struct mlx5e_rq_frag_info *frag_info =3D &rq->wqe.info.arr[0]; > > struct mlx5e_wqe_frag_info *head_wi =3D wi; > > @@ -1654,7 +1654,7 @@ mlx5e_skb_from_cqe_nonlinear(struct mlx5e_rq *rq,= struct mlx5e_wqe_frag_info *wi > > net_prefetchw(va); /* xdp_frame data area */ > > net_prefetch(va + rx_headroom); > > > > - mlx5e_fill_mxbuf(rq, va, rx_headroom, frag_consumed_bytes, &mxbuf= ); > > + mlx5e_fill_mxbuf(rq, cqe, va, rx_headroom, frag_consumed_bytes, &= mxbuf); > > sinfo =3D xdp_get_shared_info_from_buff(&mxbuf.xdp); > > truesize =3D 0; > > > > @@ -1777,7 +1777,7 @@ static void mlx5e_handle_rx_cqe(struct mlx5e_rq *= rq, struct mlx5_cqe64 *cqe) > > mlx5e_skb_from_cqe_linear, > > mlx5e_skb_from_cqe_nonlinear, > > mlx5e_xsk_skb_from_cqe_linear, > > - rq, wi, cqe_bcnt); > > + rq, wi, cqe, cqe_bcnt); > > if (!skb) { > > /* probably for XDP */ > > if (__test_and_clear_bit(MLX5E_RQ_FLAG_XDP_XMIT, rq->flag= s)) { > > @@ -1830,7 +1830,7 @@ static void mlx5e_handle_rx_cqe_rep(struct mlx5e_= rq *rq, struct mlx5_cqe64 *cqe) > > skb =3D INDIRECT_CALL_2(rq->wqe.skb_from_cqe, > > mlx5e_skb_from_cqe_linear, > > mlx5e_skb_from_cqe_nonlinear, > > - rq, wi, cqe_bcnt); > > + rq, wi, cqe, cqe_bcnt); > > if (!skb) { > > /* probably for XDP */ > > if (__test_and_clear_bit(MLX5E_RQ_FLAG_XDP_XMIT, rq->flag= s)) { > > @@ -1889,7 +1889,7 @@ static void mlx5e_handle_rx_cqe_mpwrq_rep(struct = mlx5e_rq *rq, struct mlx5_cqe64 > > skb =3D INDIRECT_CALL_2(rq->mpwqe.skb_from_cqe_mpwrq, > > mlx5e_skb_from_cqe_mpwrq_linear, > > mlx5e_skb_from_cqe_mpwrq_nonlinear, > > - rq, wi, cqe_bcnt, head_offset, page_idx); > > + rq, wi, cqe, cqe_bcnt, head_offset, page_id= x); > > if (!skb) > > goto mpwrq_cqe_out; > > > > @@ -1940,7 +1940,8 @@ mlx5e_fill_skb_data(struct sk_buff *skb, struct m= lx5e_rq *rq, > > > > static struct sk_buff * > > mlx5e_skb_from_cqe_mpwrq_nonlinear(struct mlx5e_rq *rq, struct mlx5e_= mpw_info *wi, > > - u16 cqe_bcnt, u32 head_offset, u32 pag= e_idx) > > + struct mlx5_cqe64 *cqe, u16 cqe_bcnt, = u32 head_offset, > > + u32 page_idx) > > { > > union mlx5e_alloc_unit *au =3D &wi->alloc_units[page_idx]; > > u16 headlen =3D min_t(u16, MLX5E_RX_MAX_HEAD, cqe_bcnt); > > @@ -1979,7 +1980,8 @@ mlx5e_skb_from_cqe_mpwrq_nonlinear(struct mlx5e_r= q *rq, struct mlx5e_mpw_info *w > > > > static struct sk_buff * > > mlx5e_skb_from_cqe_mpwrq_linear(struct mlx5e_rq *rq, struct mlx5e_mpw= _info *wi, > > - u16 cqe_bcnt, u32 head_offset, u32 page_i= dx) > > + struct mlx5_cqe64 *cqe, u16 cqe_bcnt, u32= head_offset, > > + u32 page_idx) > > { > > union mlx5e_alloc_unit *au =3D &wi->alloc_units[page_idx]; > > u16 rx_headroom =3D rq->buff.headroom; > > @@ -2010,7 +2012,7 @@ mlx5e_skb_from_cqe_mpwrq_linear(struct mlx5e_rq *= rq, struct mlx5e_mpw_info *wi, > > struct mlx5e_xdp_buff mxbuf; > > > > net_prefetchw(va); /* xdp_frame data area */ > > - mlx5e_fill_mxbuf(rq, va, rx_headroom, cqe_bcnt, &mxbuf); > > + mlx5e_fill_mxbuf(rq, cqe, va, rx_headroom, cqe_bcnt, &mxb= uf); > > if (mlx5e_xdp_handle(rq, au->page, prog, &mxbuf)) { > > if (__test_and_clear_bit(MLX5E_RQ_FLAG_XDP_XMIT, = rq->flags)) > > __set_bit(page_idx, wi->xdp_xmit_bitmap);= /* non-atomic */ > > @@ -2174,8 +2176,8 @@ static void mlx5e_handle_rx_cqe_mpwrq_shampo(stru= ct mlx5e_rq *rq, struct mlx5_cq > > if (likely(head_size)) > > *skb =3D mlx5e_skb_from_cqe_shampo(rq, wi, cqe, h= eader_index); > > else > > - *skb =3D mlx5e_skb_from_cqe_mpwrq_nonlinear(rq, w= i, cqe_bcnt, data_offset, > > - page_id= x); > > + *skb =3D mlx5e_skb_from_cqe_mpwrq_nonlinear(rq, w= i, cqe, cqe_bcnt, > > + data_of= fset, page_idx); > > if (unlikely(!*skb)) > > goto free_hd_entry; > > > > @@ -2249,7 +2251,8 @@ static void mlx5e_handle_rx_cqe_mpwrq(struct mlx5= e_rq *rq, struct mlx5_cqe64 *cq > > mlx5e_skb_from_cqe_mpwrq_linear, > > mlx5e_skb_from_cqe_mpwrq_nonlinear, > > mlx5e_xsk_skb_from_cqe_mpwrq_linear, > > - rq, wi, cqe_bcnt, head_offset, page_idx); > > + rq, wi, cqe, cqe_bcnt, head_offset, > > + page_idx); > > if (!skb) > > goto mpwrq_cqe_out; > > > > @@ -2494,7 +2497,7 @@ static void mlx5i_handle_rx_cqe(struct mlx5e_rq *= rq, struct mlx5_cqe64 *cqe) > > skb =3D INDIRECT_CALL_2(rq->wqe.skb_from_cqe, > > mlx5e_skb_from_cqe_linear, > > mlx5e_skb_from_cqe_nonlinear, > > - rq, wi, cqe_bcnt); > > + rq, wi, cqe, cqe_bcnt); > > if (!skb) > > goto wq_free_wqe; > > > > @@ -2586,7 +2589,7 @@ static void mlx5e_trap_handle_rx_cqe(struct mlx5e= _rq *rq, struct mlx5_cqe64 *cqe > > goto free_wqe; > > } > > > > - skb =3D mlx5e_skb_from_cqe_nonlinear(rq, wi, cqe_bcnt); > > + skb =3D mlx5e_skb_from_cqe_nonlinear(rq, wi, cqe, cqe_bcnt); > > if (!skb) > > goto free_wqe; > >