]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
Merge branch 'net-make-sure-we-retain-napi-ordering-on-netdev-napi_list'
authorPaolo Abeni <pabeni@redhat.com>
Thu, 9 Jan 2025 14:33:10 +0000 (15:33 +0100)
committerPaolo Abeni <pabeni@redhat.com>
Thu, 9 Jan 2025 14:33:11 +0000 (15:33 +0100)
commit11c668db098507207d55c5d6e04cc34a55288636
tree9eb6e497555e24d5ae38f2a4b0c4871e9384b900
parenta3b3d2dc389568a77d0e25da17203e3616218e93
parenteb721f117e7d43b561e81dd878c4acfa2de13ee2
Merge branch 'net-make-sure-we-retain-napi-ordering-on-netdev-napi_list'

Jakub Kicinski says:

====================
net: make sure we retain NAPI ordering on netdev->napi_list

I promised Eric to remove the rtnl protection of the NAPI list,
when I sat down to implement it over the break I realized that
the recently added NAPI ID retention will break the list ordering
assumption we have in netlink dump. The ordering used to happen
"naturally", because we'd always add NAPIs that the head of the
list, and assign a new monotonically increasing ID.

Before the first patch of this series we'd still only add at
the head of the list but now the newly added NAPI may inherit
from its config an ID lower than something else already on the list.

The fix is in the first patch, the rest is netdevsim churn to test it.
I'm posting this for net-next, because AFAICT the problem can't
be triggered in net, given the very limited queue API adoption.

v2:
 - [patch 2] allocate the array with kcalloc() instead of kvcalloc()
 - [patch 2] set GFP_KERNEL_ACCOUNT when allocating queues
 - [patch 6] don't null-check page pool before page_pool_destroy()
 - [patch 6] controled -> controlled
 - [patch 7] change mode to 0200
 - [patch 7] reorder removal to be inverse of add
 - [patch 7] fix the spaces vs tabs
v1: https://lore.kernel.org/20250103185954.1236510-1-kuba@kernel.org
====================

Link: https://patch.msgid.link/20250107160846.2223263-1-kuba@kernel.org
Signed-off-by: Paolo Abeni <pabeni@redhat.com>