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>

diff --git a/net/core/page_pool.c b/net/core/page_pool.c
index 1a5edec485f141b2c13599dc84a60af20b809f78..a085fd199ff012ae6574df7d62186bb12e7205d4 100644 (file)
--- 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);