Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

selftests: pp-bench: remove page_pool_put_page wrapper

Minor cleanup: remove the pointless looking _ wrapper around
page_pool_put_page, and just do the call directly.

Signed-off-by: Mina Almasry <almasrymina@google.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Reviewed-by: Toke Høiland-Jørgensen <toke@redhat.com>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Acked-by: Jesper Dangaard Brouer <hawk@kernel.org>
Link: https://patch.msgid.link/20250627200501.1712389-2-almasrymina@google.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

authored by

Mina Almasry and committed by
Jakub Kicinski
be75d319 8d3e0982

+3 -9
+3 -9
tools/testing/selftests/net/bench/page_pool/bench_page_pool_simple.c
··· 16 16 static int verbose = 1; 17 17 #define MY_POOL_SIZE 1024 18 18 19 - static void _page_pool_put_page(struct page_pool *pool, struct page *page, 20 - bool allow_direct) 21 - { 22 - page_pool_put_page(pool, page, -1, allow_direct); 23 - } 24 - 25 19 /* Makes tests selectable. Useful for perf-record to analyze a single test. 26 20 * Hint: Bash shells support writing binary number like: $((2#101010) 27 21 * ··· 115 121 for (i = 0; i < elems; i++) 116 122 array[i] = page_pool_alloc_pages(pp, gfp_mask); 117 123 for (i = 0; i < elems; i++) 118 - _page_pool_put_page(pp, array[i], false); 124 + page_pool_put_page(pp, array[i], -1, false); 119 125 120 126 kfree(array); 121 127 } ··· 174 180 175 181 } else if (type == type_ptr_ring) { 176 182 /* Normal return path */ 177 - _page_pool_put_page(pp, page, false); 183 + page_pool_put_page(pp, page, -1, false); 178 184 179 185 } else if (type == type_page_allocator) { 180 186 /* Test if not pages are recycled, but instead 181 187 * returned back into systems page allocator 182 188 */ 183 189 get_page(page); /* cause no-recycling */ 184 - _page_pool_put_page(pp, page, false); 190 + page_pool_put_page(pp, page, -1, false); 185 191 put_page(page); 186 192 } else { 187 193 BUILD_BUG();