net: devmem: don't call queue stop / start when the interface is down
authorJakub Kicinski <kuba@kernel.org>
Thu, 6 Feb 2025 22:56:36 +0000 (14:56 -0800)
committerJakub Kicinski <kuba@kernel.org>
Sat, 8 Feb 2025 01:21:02 +0000 (17:21 -0800)
commit3e7efc3f4f03bca0ea630c302e7c79cf807476bb
tree2edc81d1789e4d721d4caedfc09e09c748747b52
parent1eb824d69f8d88405e4e80c568e8f07080309fb0
net: devmem: don't call queue stop / start when the interface is down

We seem to be missing a netif_running() check from the devmem
installation path. Starting a queue on a stopped device makes
no sense. We still want to be able to allocate the memory, just
to test that the device is indeed setting up the page pools
in a memory provider compatible way.

This is not a bug fix, because existing drivers check if
the interface is down as part of the ops. But new drivers
shouldn't have to do this, as long as they can correctly
alloc/free while down.

Reviewed-by: Mina Almasry <almasrymina@google.com>
Link: https://patch.msgid.link/20250206225638.1387810-3-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
include/net/netdev_queues.h
net/core/netdev_rx_queue.c