Commit 8d522566 authored by Pavel Begunkov's avatar Pavel Begunkov Committed by Jakub Kicinski
Browse files

net: page_pool: don't cast mp param to devmem 



page_pool_check_memory_provider() is a generic path and shouldn't assume
anything about the actual type of the memory provider argument. It's
fine while devmem is the only provider, but cast away the devmem
specific binding types to avoid confusion.

Reviewed-by: default avatarJakub Kicinski <kuba@kernel.org>
Reviewed-by: default avatarMina Almasry <almasrymina@google.com>
Signed-off-by: default avatarPavel Begunkov <asml.silence@gmail.com>
Signed-off-by: default avatarDavid Wei <dw@davidwei.uk>
Link: https://patch.msgid.link/20250204215622.695511-2-dw@davidwei.uk


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 2014c95a

net/core/page_pool_user.c

+1 −1
Original line number Diff line number Diff line
@@ -356,7 +356,7 @@ void page_pool_unlist(struct page_pool *pool) 
int page_pool_check_memory_provider(struct net_device *dev,

				    struct netdev_rx_queue *rxq)

{

	struct net_devmem_dmabuf_binding *binding = rxq->mp_params.mp_priv;

	void *binding = rxq->mp_params.mp_priv;

	struct page_pool *pool;

	struct hlist_node *n;