diff options
| author | Jakub Kicinski <kuba@kernel.org> | 2024-04-09 18:20:32 -0700 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2024-04-09 18:20:32 -0700 |
| commit | 811b836285779067cefa2285607f2fc508f3f2ee (patch) | |
| tree | 7137308fbcdba00cb1b087c1efd78be02fc5b1b8 /include/linux | |
| parent | 445e60303883950161f67e18b9f048b18d7fb706 (diff) | |
| parent | f58f3c9563409e618e591d0d540316286cb0665f (diff) | |
Merge branch 'minor-cleanups-to-skb-frag-ref-unref'
Mina Almasry says:
====================
Minor cleanups to skb frag ref/unref (part)
This series is largely motivated by a recent discussion where there was
some confusion on how to properly ref/unref pp pages vs non pp pages:
https://lore.kernel.org/netdev/CAHS8izOoO-EovwMwAm9tLYetwikNPxC0FKyVGu1TPJWSz4bGoA@mail.gmail.com/T/#t
There is some subtely there because pp uses page->pp_ref_count for
refcounting, while non-pp uses get_page()/put_page() for ref counting.
Getting the refcounting pairs wrong can lead to kernel crash.
[...]
https://lore.kernel.org/lkml/CAHS8izN436pn3SndrzsCyhmqvJHLyxgCeDpWXA4r1ANt3RCDLQ@mail.gmail.com/T/
====================
Link: https://lore.kernel.org/r/20240408153000.2152844-1-almasrymina@google.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/skbuff.h | 16 |
1 files changed, 2 insertions, 14 deletions
diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index 7dfb906d92f7..7135a3e94afd 100644 --- a/include/linux/skbuff.h +++ b/include/linux/skbuff.h @@ -3522,21 +3522,9 @@ int skb_cow_data_for_xdp(struct page_pool *pool, struct sk_buff **pskb, bool napi_pp_put_page(struct page *page); static inline void -skb_page_unref(const struct sk_buff *skb, struct page *page) +skb_page_unref(struct page *page, bool recycle) { #ifdef CONFIG_PAGE_POOL - if (skb->pp_recycle && napi_pp_put_page(page)) - return; -#endif - put_page(page); -} - -static inline void -napi_frag_unref(skb_frag_t *frag, bool recycle) -{ - struct page *page = skb_frag_page(frag); - -#ifdef CONFIG_PAGE_POOL if (recycle && napi_pp_put_page(page)) return; #endif @@ -3553,7 +3541,7 @@ napi_frag_unref(skb_frag_t *frag, bool recycle) */ static inline void __skb_frag_unref(skb_frag_t *frag, bool recycle) { - napi_frag_unref(frag, recycle); + skb_page_unref(skb_frag_page(frag), recycle); } /** |