From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f170.google.com (mail-pf1-f170.google.com [209.85.210.170]) (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 9A0F039479B for ; Tue, 3 Mar 2026 18:41:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.170 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772563277; cv=none; b=V/cuiHkrsZsGoN42FHNS/wVqI4co0xPe9RvXBafpkxZMVuXxfPt/o53svDawogp14ikt77U5EGTXJRgKcmwFDBAIZ4tXoqmqGoTQTGZvrfxx5ix0wOr//61x9k67fXAXKyu7BOZaVnxUZ0q9zRBUGPzeKjkpbyj2BzVLvXqoCM8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772563277; 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=t6z8zVNWbwavkzB8OKW31cvaaHoehPBV4U0x2hfa6jisjki/zAUqt9PGUPuqSSZqaQrwDc203AMs4gK6pmb2D0ksukX3Bzn6B1Z5XEEL5//9H+xeJ+iWPe7YAzBke1W8tO3nCbNIOQVU92JwQsn8jjoJe6DJIT7mXjVR/cS6AIo= 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.170 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-f170.google.com with SMTP id d2e1a72fcca58-824a829f9bbso2776574b3a.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=UlaBphtnpsyBbOTBGG75dOm+bHtkHNXQB6gKsLkuCSWEUe/+jP1KEKWN6y5sg7HFwl ksBypNgsnNba3Trnpnr8QCK+Yf3bgQUiYl8Yf77gSBU3ZL1jlmPas6/pBQRVX+uqpYR5 nXsktuojJZA/0Nd1DGdQfs5XzSIhQgFh3ualbbHgCH/kqmcp+usLBYDoyjpAZ1BLBvKp tNfxxmrCrBfCNeS1er5WCy0MD0PWjOUA10cNkttmwrued1yOD8kKqlLApvL2XFud8C2L vSiOZUpBRo7JDFBaudbrSXir6eVH261rzuUmFmuv6yLETuEbjxQtlHlJ1Qr4hRh9ZpXM E8EQ== X-Forwarded-Encrypted: i=1; AJvYcCWlLIb1Os5ra/3H5SkUS+nHlHCEysoOhDMjaJqT6DN/DXruL1/OmITtuWBy7pWIHH7ZeUkXUf5RJucMsqU=@vger.kernel.org X-Gm-Message-State: AOJu0YxUijxA5VTKTzoTw41KBy44NAZA5y/DgAYj4n4tHdQbESP+9FlC sN6wJX7gnHawI/P43bLwTD08cZTu5i73sbjn6LKPUq65iWCsGrNnf2zLYUThAyQj++8= X-Gm-Gg: ATEYQzxZIp6iEEHd7ycdZFXQTNMs82GTJrzj6g2Uxf4x1xlgN6rPjOjyY7OpL3jtlHk IXig2p3fJSC8N29vCwb6qJtvuthtIaf/tYay86N4/DDT5nlg+81ydWQvGwnd0b6AwWUVWGIouPo YPhJVXET7Exvc616AddAueCajXx0MStxvXvK+erUMMOgqKN8TPtSa6zRr9oSdmG18yPwjo/mvy0 5NwshYYF0nlLpBIVvCWWiF4pz8/2mEQLAxBm1pwB5A+SmNKMBJwdnyWRB2ej6CEayidfxjnt3KD XQtsQlu5aYjiACwG823N5SXzez2sUc4Y9TpXT97ragsJT2ltQpSDb85qJj+GUSSfVJACtPEuRN/ Y+Zsa4gqW/bJR5nLhGBn/Hv4WhjCWi30f0i3MeS03e++WG7RHc9SBaSiwdldA4Vyc7bNPQmJw18 WrByM= 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: linux-kernel@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