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 <[email protected]>
---
 net/core/page_pool.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/core/page_pool.c b/net/core/page_pool.c
index 1a5edec485f1..a085fd199ff0 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);
-- 
2.52.0


Reply via email to