From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oa1-x2b.google.com (mail-oa1-x2b.google.com [IPv6:2001:4860:4864:20::2b]) by mail.toke.dk (Postfix) with ESMTPS id 6D5BA9C4A4D for ; Tue, 22 Nov 2022 19:08:44 +0100 (CET) Authentication-Results: mail.toke.dk; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20210112 header.b=DvYBNWd1 Received: by mail-oa1-x2b.google.com with SMTP id 586e51a60fabf-14279410bf4so16348232fac.8 for ; Tue, 22 Nov 2022 10:08:44 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=G/+CkOXYfB8ZlUe0lQxCv/js581DawFiKZd9/L8amPg=; b=DvYBNWd1xYf7d67xFw1/x61RfV/5IW+zP9pZIJ2PiG+227mHRlD1q5hoSn68yaA+zn mbSK/wCfR24MhIyezTDIvLNVx2YTk9wHO3PQxir2C6pogUq61dtkBiFhSIBmRTA2eQTy 6euybBgmtWqw+KtVQWW1exti8Id2WMQXBLOTw7l7IV2j8P711KtDtR8pQ3FHg96DowZu Cikk88vSkkrilH7k5wbqMJf5DOzvNAh88cQdzZ++ssNLid0rH5s2EZO1Jh/xoWRHnMzy MZIEWSHIdzfjQ3zecTKcKG2oCgXRZ/odIz5Xj4VXt8nEIwgeecZKAywmwiS83FouxZqw OZYg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=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=G/+CkOXYfB8ZlUe0lQxCv/js581DawFiKZd9/L8amPg=; b=RsNHmLy0nyH+HjA3ngSMJL4JrpSCWbmvhLRdaCcnqx/2dNK3FkO//TPmV7I4ktmdpF 1zH/pq603WaTJ6pKH9SVDMmRCjavFSxPY9cuWmjqaXwdr0S+IIDrf0qdapo4gUtJ73ly 7AympM8tNQQDwL0hxffwU5Cs6dGh2kCjlFWtNM9TAYTuIjsq1ZnrQgJOyTCPBGgr1Zay 08l7MvogC2TNwhx7z2+IgUW8qXtLt6LwhWretTuOTwGV9pwXN+NgekdFo99n3+nMZs6C ImpDFmBtQR8aGCido1ZSxn9pMSTD2BqOJ6+tRKttH7Sp/QjoWuRkCJI6mcDLr+c6PTci YwEw== X-Gm-Message-State: ANoB5pk7PFCbSDJmJj9fYbKKuS5HlwUDm4TNyCL8WMNF1alKkAabaEmh C1cXwMZoscHOxKQ4AUOufgoTEkYwx/A4AKAcrudDwA== X-Google-Smtp-Source: AA0mqf7FGjCj2A1KXbRxjU0cU5r57Sbn9O5FkntMogUbeHHmFLH2gAcufRD++azWzORZiybSCrvmO8x2KYql1YfDs4o= X-Received: by 2002:a05:6870:9d95:b0:13b:a163:ca6 with SMTP id pv21-20020a0568709d9500b0013ba1630ca6mr17548541oab.125.1669140520654; Tue, 22 Nov 2022 10:08:40 -0800 (PST) MIME-Version: 1.0 References: <20221121182552.2152891-1-sdf@google.com> <20221121182552.2152891-7-sdf@google.com> In-Reply-To: From: Stanislav Fomichev Date: Tue, 22 Nov 2022 10:08:29 -0800 Message-ID: To: Tariq Toukan Content-Type: text/plain; charset="UTF-8" Message-ID-Hash: BNFR6EQJCUUDHGU2PM3GUZ26TSY4237M X-Message-ID-Hash: BNFR6EQJCUUDHGU2PM3GUZ26TSY4237M X-MailFrom: sdf@google.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header 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, Tariq Toukan , 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 X-Mailman-Version: 3.3.7 Precedence: list Subject: [xdp-hints] Re: [PATCH bpf-next v2 6/8] mlx4: Introduce mlx4_xdp_buff wrapper for xdp_buff List-Id: XDP hardware hints design discussion Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: On Tue, Nov 22, 2022 at 5:49 AM Tariq Toukan wrote: > > > > On 11/21/2022 8:25 PM, Stanislav Fomichev wrote: > > No functional changes. Boilerplate to allow stuffing more data after xdp_buff. > > > > Cc: Tariq Toukan > > 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: Stanislav Fomichev > > --- > > drivers/net/ethernet/mellanox/mlx4/en_rx.c | 26 +++++++++++++--------- > > 1 file changed, 15 insertions(+), 11 deletions(-) > > > > diff --git a/drivers/net/ethernet/mellanox/mlx4/en_rx.c b/drivers/net/ethernet/mellanox/mlx4/en_rx.c > > index 8f762fc170b3..467356633172 100644 > > --- a/drivers/net/ethernet/mellanox/mlx4/en_rx.c > > +++ b/drivers/net/ethernet/mellanox/mlx4/en_rx.c > > @@ -661,17 +661,21 @@ static int check_csum(struct mlx4_cqe *cqe, struct sk_buff *skb, void *va, > > #define MLX4_CQE_STATUS_IP_ANY (MLX4_CQE_STATUS_IPV4) > > #endif > > > > +struct mlx4_xdp_buff { > > + struct xdp_buff xdp; > > +}; > > + > > int mlx4_en_process_rx_cq(struct net_device *dev, struct mlx4_en_cq *cq, int budget) > > { > > struct mlx4_en_priv *priv = netdev_priv(dev); > > int factor = priv->cqe_factor; > > struct mlx4_en_rx_ring *ring; > > + struct mlx4_xdp_buff this would helpmxbuf; > > as it doesn't go through an init function (only mxbuf.xdp does), better > init to zero. SG, will do, thanks! > > struct bpf_prog *xdp_prog; > > int cq_ring = cq->ring; > > bool doorbell_pending; > > bool xdp_redir_flush; > > struct mlx4_cqe *cqe; > > - struct xdp_buff xdp; > > int polled = 0; > > int index; > > > > @@ -681,7 +685,7 @@ int mlx4_en_process_rx_cq(struct net_device *dev, struct mlx4_en_cq *cq, int bud > > ring = priv->rx_ring[cq_ring]; > > > > xdp_prog = rcu_dereference_bh(ring->xdp_prog); > > - xdp_init_buff(&xdp, priv->frag_info[0].frag_stride, &ring->xdp_rxq); > > + xdp_init_buff(&mxbuf.xdp, priv->frag_info[0].frag_stride, &ring->xdp_rxq); > > doorbell_pending = false; > > xdp_redir_flush = false; > > > > @@ -776,24 +780,24 @@ int mlx4_en_process_rx_cq(struct net_device *dev, struct mlx4_en_cq *cq, int bud > > priv->frag_info[0].frag_size, > > DMA_FROM_DEVICE); > > > > - xdp_prepare_buff(&xdp, va - frags[0].page_offset, > > + xdp_prepare_buff(&mxbuf.xdp, va - frags[0].page_offset, > > frags[0].page_offset, length, false); > > - orig_data = xdp.data; > > + orig_data = mxbuf.xdp.data; > > > > - act = bpf_prog_run_xdp(xdp_prog, &xdp); > > + act = bpf_prog_run_xdp(xdp_prog, &mxbuf.xdp); > > > > - length = xdp.data_end - xdp.data; > > - if (xdp.data != orig_data) { > > - frags[0].page_offset = xdp.data - > > - xdp.data_hard_start; > > - va = xdp.data; > > + length = mxbuf.xdp.data_end - mxbuf.xdp.data; > > + if (mxbuf.xdp.data != orig_data) { > > + frags[0].page_offset = mxbuf.xdp.data - > > + mxbuf.xdp.data_hard_start; > > + va = mxbuf.xdp.data; > > } > > > > switch (act) { > > case XDP_PASS: > > break; > > case XDP_REDIRECT: > > - if (likely(!xdp_do_redirect(dev, &xdp, xdp_prog))) { > > + if (likely(!xdp_do_redirect(dev, &mxbuf.xdp, xdp_prog))) { > > ring->xdp_redirect++; > > xdp_redir_flush = true; > > frags[0].page = NULL;