Commit 7fb9b66d authored by Yunsheng Lin's avatar Yunsheng Lin Committed by David S. Miller
Browse files

page_pool: use relaxed atomic for release side accounting 



There is no need to synchronize the account updating, so
use the relaxed atomic to avoid some memory barrier in the
data path.

Acked-by: default avatarJesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: default avatarYunsheng Lin <linyunsheng@huawei.com>
Acked-by: default avatarIlias Apalodimas <ilias.apalodimas@linaro.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 669f047e

net/core/page_pool.c

+1 −1
Original line number Diff line number Diff line
@@ -370,7 +370,7 @@ void page_pool_release_page(struct page_pool *pool, struct page *page) 
	/* This may be the last page returned, releasing the pool, so

	 * it is not safe to reference pool afterwards.

	 */

	count = atomic_inc_return(&pool->pages_state_release_cnt);

	count = atomic_inc_return_relaxed(&pool->pages_state_release_cnt);

	trace_page_pool_state_release(pool, page, count);

}

EXPORT_SYMBOL(page_pool_release_page);