]> git.ipfire.org Git - thirdparty/linux.git/commit
net: sched: generalize check for no-queue qdisc on TX queue
authorJesper Dangaard Brouer <hawk@kernel.org>
Fri, 25 Apr 2025 14:55:31 +0000 (16:55 +0200)
committerJakub Kicinski <kuba@kernel.org>
Mon, 28 Apr 2025 21:06:58 +0000 (14:06 -0700)
commit34dd0fecaa02d654c447d43a7e4c72f9b18b7033
treebf14e87a1e0afe98fa011cb961ab4ec5c1ec8437
parentccc25158c22bf9aaf1242182b852bb6c73e88ac3
net: sched: generalize check for no-queue qdisc on TX queue

The "noqueue" qdisc can either be directly attached, or get default
attached if net_device priv_flags has IFF_NO_QUEUE. In both cases, the
allocated Qdisc structure gets it's enqueue function pointer reset to
NULL by noqueue_init() via noqueue_qdisc_ops.

This is a common case for software virtual net_devices. For these devices
with no-queue, the transmission path in __dev_queue_xmit() will bypass
the qdisc layer. Directly invoking device drivers ndo_start_xmit (via
dev_hard_start_xmit).  In this mode the device driver is not allowed to
ask for packets to be queued (either via returning NETDEV_TX_BUSY or
stopping the TXQ).

The simplest and most reliable way to identify this no-queue case is by
checking if enqueue == NULL.

The vrf driver currently open-codes this check (!qdisc->enqueue). While
functionally correct, this low-level detail is better encapsulated in a
dedicated helper for clarity and long-term maintainability.

To make this behavior more explicit and reusable, this patch introduce a
new helper: qdisc_txq_has_no_queue(). Helper will also be used by the
veth driver in the next patch, which introduces optional qdisc-based
backpressure.

This is a non-functional change.

Reviewed-by: David Ahern <dsahern@kernel.org>
Reviewed-by: Toke Høiland-Jørgensen <toke@redhat.com>
Signed-off-by: Jesper Dangaard Brouer <hawk@kernel.org>
Link: https://patch.msgid.link/174559293172.827981.7583862632045264175.stgit@firesoul
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/vrf.c
include/net/sch_generic.h