[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20251119011146.27493-1-byungchul@sk.com>
Date: Wed, 19 Nov 2025 10:11:46 +0900
From: Byungchul Park <byungchul@...com>
To: kuba@...nel.org,
netdev@...r.kernel.org
Cc: linux-kernel@...r.kernel.org,
linux-mm@...ck.org,
kernel_team@...ynix.com,
harry.yoo@...cle.com,
hawk@...nel.org,
andrew+netdev@...n.ch,
david@...hat.com,
lorenzo.stoakes@...cle.com,
Liam.Howlett@...cle.com,
vbabka@...e.cz,
ziy@...dia.com,
willy@...radead.org,
toke@...hat.com,
asml.silence@...il.com,
alexanderduyck@...com,
kernel-team@...a.com,
davem@...emloft.net,
edumazet@...gle.com,
pabeni@...hat.com,
mohsin.bashr@...il.com,
almasrymina@...gle.com,
jdamato@...tly.com
Subject: [RFC net-next] eth: fbnic: use ring->page_pool instead of page->pp in fbnic_clean_twq1()
With the planned removal of @pp from struct page, we should access the
page pool pointer through other means. Use @page_pool in struct
fbnic_ring instead.
Signed-off-by: Byungchul Park <byungchul@...com>
---
I should admit I'm not used to the following code. So I'd like to ask
how to alter page->pp to avoid accessing @pp through struct page
directly. Does the following change work? Or can you suggest other
ways to achieve it?
Byungchul
---
drivers/net/ethernet/meta/fbnic/fbnic_txrx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/meta/fbnic/fbnic_txrx.c b/drivers/net/ethernet/meta/fbnic/fbnic_txrx.c
index b1e8ce89870f..95f158ba6fa2 100644
--- a/drivers/net/ethernet/meta/fbnic/fbnic_txrx.c
+++ b/drivers/net/ethernet/meta/fbnic/fbnic_txrx.c
@@ -653,7 +653,7 @@ static void fbnic_clean_twq1(struct fbnic_napi_vector *nv, bool pp_allow_direct,
FBNIC_TWD_TYPE_AL;
total_bytes += FIELD_GET(FBNIC_TWD_LEN_MASK, twd);
- page_pool_put_page(page->pp, page, -1, pp_allow_direct);
+ page_pool_put_page(ring->page_pool, page, -1, pp_allow_direct);
next_desc:
head++;
head &= ring->size_mask;
--
2.17.1
Powered by blists - more mailing lists