From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f169.google.com (mail-pf1-f169.google.com [209.85.210.169]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9EBE43CB2FB for ; Tue, 3 Mar 2026 18:41:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772563279; cv=none; b=j2XkNaPThTbPCr7LGq2fyDxKu/+QOBjeG3aTeYsYCWau6iXSBLihKE4ZTsMl9UvqNN1V9DJ4jGL6vxaK48PzGg9dvtFSRWlV/Si7RP4WCNO7NcZFsm0g7DGrm6WLXYUTR6aWxPRH5AS0Gj+mf5fAcZskepnbJsHB/B4MUDBVNAo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772563279; c=relaxed/simple; bh=COoepY3IHCGDvrOdJwMdZwFCBacWNUZWI+Ma3V+69Gk=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=DCF+2EzjD6gAgQiowV7JlsOOHFXo/7+BCnEu2Hgj4KqEl2/sU1qZYSzgN762RPtTqdmA5Xvq5zS6gze5IdksBRqi27UKnC/pmUjJAEWsZn2oHDz71nq02KcGV5mdrp+fzLqA+3IYYVszQ1rdx9y8Xv2YCI/Gh6fDoZlRvYm5UMc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=dama.to; spf=none smtp.mailfrom=dama.to; dkim=pass (2048-bit key) header.d=dama-to.20230601.gappssmtp.com header.i=@dama-to.20230601.gappssmtp.com header.b=VTsC5HTE; arc=none smtp.client-ip=209.85.210.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=dama.to Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=dama.to Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=dama-to.20230601.gappssmtp.com header.i=@dama-to.20230601.gappssmtp.com header.b="VTsC5HTE" Received: by mail-pf1-f169.google.com with SMTP id d2e1a72fcca58-824a829f9bbso2776573b3a.0 for ; Tue, 03 Mar 2026 10:41:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=dama-to.20230601.gappssmtp.com; s=20230601; t=1772563276; x=1773168076; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to; bh=hxvBm/svbAndJJLk+ZSBdX8462dd6PpqDbcqOUbb0IQ=; b=VTsC5HTEcDSYUJk0tgjA/ur95nQVMnG83zCrJgsznxBvPYMkyYNvcDnudDwHWIt8v5 vTNnMeAKSt+LPHJzPmSvMq0N5Vjt2aL1qGxlDuwtWIDdG7OZF3t+/O+aXq3rFpWJmVdn kFv89fihdQFM2MQAHRxAcl1BeNO2lWEM9ciqGVc/TdKnNqKDmQ07p6aQjVG00ULwMIVy T/ywOeA8uLH88j8OOGwQ1gnI6heSFyE+/AtY5GRDcdnedcPJ/d4D3h/xtGp9tv6IbVOa Y902vgXH99MqUzmnyFHkCjZRcaEJCsYKmnnLFNxuiB/2jptjGIEnjYOcAvuZpbFLOA79 rm8g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772563276; x=1773168076; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=hxvBm/svbAndJJLk+ZSBdX8462dd6PpqDbcqOUbb0IQ=; b=ncwKFoG+IGwEZX4ii4cSCPPlJXA7HY+qGaa7UNj4Hs61rLv1IME9haUZe7ghCboixJ /8lCfBZCeT+kKtCeaHf1ZyZFfgkABQdGNrcqgZizG5QxYV4/Pco6bz/pATR/99Yx0vYz bpDjJyTZi9tC4lSSQv0l0nGHkhgkmLrpIXzHZhe6bwZaTiqbbN3oq/06/FcJSbZgHv9T DmEFcox54Mgs8wxs7pPLEVjSQCLbV2tpHVQJiPbdzWXtcnMKeYjnWS8T+USzjMPI2U+t WR+OCBUkulY923CJhVgvnCOBGJej1AZQywGBAuf5m+GyIaNqqU1C84wt2cLjHL29FByn fdug== X-Forwarded-Encrypted: i=1; AJvYcCUhm9dIOH0s7vN7oRoYbDxHyZXKuAgifjaVpEDOQjy0f2wPQYv7bZb5s/CVpZGojywyMjU=@vger.kernel.org X-Gm-Message-State: AOJu0Yz7TDej/qfp4QS+mj5HP8Vypv0VeiFE9D2pA0GSw4aLxMH9VT/m c6EWH1Rwf1nR/5nFUlbrBEUP7/5JoNlAzoKR3DhN4c0bMlsKjMtoP0fdBE5fRnPHFg8= X-Gm-Gg: ATEYQzy8WD45u/hfOZ85MzlwxZoREe24gdIqWS+j9J2ciEyJ+8AIwZbwTQipaDj2V4j DQJFxDqEJPjY4KaD6B5kgvQw+5ZO4dvkAj64S3vB0tkNO++MKEo3D/tl5GW7ljePvJL3wyeufA1 VYYJuIjNZKyuMo+OpObrPvfiIur+D+SukpgrpMVDxR/a07DIgDjH/ikN2+OAc9jsxUnZ6ZGQqpB htx9rJ1Za7Hf9841qwAiH4zVleXbpdK/TcAl4tdidPTzupAqMwnmqed1Zo4BF6ExM6j1zXR55ni xXws9Y9RGMC8YfAlGbSccQMiCBhJnmirQW0wBIPcmzoGnHfdw9r9bei1M8A6asIJXic3cXLgz3j FiK88238tRwBIwWT9PqCcg040V881lmRj531Qs2LPSUe7X5ySy0zsG+8J2yo2DiRb+phrOm+jJI Bpgko= X-Received: by 2002:a05:6a00:189f:b0:827:2d0f:1ed5 with SMTP id d2e1a72fcca58-8274da07d17mr16120083b3a.45.1772563275952; Tue, 03 Mar 2026 10:41:15 -0800 (PST) Received: from localhost ([2a03:2880:2ff:9::]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82739d87e78sm19760699b3a.17.2026.03.03.10.41.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 03 Mar 2026 10:41:15 -0800 (PST) Date: Tue, 3 Mar 2026 10:41:14 -0800 From: Joe Damato To: Chris J Arges Cc: michael.chan@broadcom.com, pavan.chebbi@broadcom.com, kuba@kernel.org, Andrew Lunn , "David S. Miller" , Eric Dumazet , Paolo Abeni , Alexei Starovoitov , Daniel Borkmann , Jesper Dangaard Brouer , John Fastabend , Stanislav Fomichev , kernel-team@cloudflare.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org Subject: Re: [PATCH net-next v2 1/4] bnxt_en: use bnxt_xdp_buff for xdp context Message-ID: Mail-Followup-To: Joe Damato , Chris J Arges , michael.chan@broadcom.com, pavan.chebbi@broadcom.com, kuba@kernel.org, Andrew Lunn , "David S. Miller" , Eric Dumazet , Paolo Abeni , Alexei Starovoitov , Daniel Borkmann , Jesper Dangaard Brouer , John Fastabend , Stanislav Fomichev , kernel-team@cloudflare.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org References: <20260213192449.1294830-1-carges@cloudflare.com> <20260303024510.644962-1-carges@cloudflare.com> <20260303024510.644962-2-carges@cloudflare.com> Precedence: bulk X-Mailing-List: bpf@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260303024510.644962-2-carges@cloudflare.com> On Mon, Mar 02, 2026 at 08:43:49PM -0600, Chris J Arges wrote: > This adds bnxt_xdp_buff which embeds the xdp_buff struct and stores > pointers to hardware RX completion descriptors (rx_cmp and rx_cmp_ext) > along with the completion type. > > Signed-off-by: Chris J Arges > --- > drivers/net/ethernet/broadcom/bnxt/bnxt.c | 25 +++++++++++-------- > drivers/net/ethernet/broadcom/bnxt/bnxt_xdp.h | 7 ++++++ > 2 files changed, 22 insertions(+), 10 deletions(-) > > diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c b/drivers/net/ethernet/broadcom/bnxt/bnxt.c > index c982aac714d1..ab73aad40593 100644 > --- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c > +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c > @@ -2118,7 +2118,7 @@ static int bnxt_rx_pkt(struct bnxt *bp, struct bnxt_cp_ring_info *cpr, > bool xdp_active = false; > dma_addr_t dma_addr; > struct sk_buff *skb; > - struct xdp_buff xdp; > + struct bnxt_xdp_buff bnxt_xdp; Not sure if it matters but similar RCT issue here. Otherwise the rest of the implementation looks OK to me, you can retain my Reviewed-by if you are respinning to fix a cosmetic thing: Reviewed-by: Joe Damato