diff options
| author | Pavel Begunkov <asml.silence@gmail.com> | 2025-11-30 23:35:16 +0000 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2025-12-02 11:08:39 -0800 |
| commit | 854858848bc7ac6ea3bf896ef1b541cb9d4bca20 (patch) | |
| tree | 5d6a2802bf0c27ef117bad9689ce70c5ec7c8a21 | |
| parent | 3c4159b3019cc3444495f54c18083cda579cba84 (diff) | |
| download | net-854858848bc7ac6ea3bf896ef1b541cb9d4bca20.tar.gz | |
net: page pool: xa init with destroy on pp init
The free_ptr_ring label path initialises ->dma_mapped xarray but doesn't
destroy it in case of an error. That's not a real problem since init
itself doesn't do anything requiring destruction, but still match it
with xa_destroy() to silence warnings.
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://patch.msgid.link/02904c6d83dbe5cc1c671106a5c97bd93ab31006.1764542851.git.asml.silence@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
| -rw-r--r-- | net/core/page_pool.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/core/page_pool.c b/net/core/page_pool.c index 1a5edec485f141..a085fd199ff012 100644 --- a/net/core/page_pool.c +++ b/net/core/page_pool.c @@ -307,6 +307,7 @@ static int page_pool_init(struct page_pool *pool, free_ptr_ring: ptr_ring_cleanup(&pool->ring, NULL); + xa_destroy(&pool->dma_mapped); #ifdef CONFIG_PAGE_POOL_STATS if (!pool->system) free_percpu(pool->recycle_stats); |