[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <1440703299-21243-1-git-send-email-phil@nwl.cc>
Date: Thu, 27 Aug 2015 21:21:35 +0200
From: Phil Sutter <phil@....cc>
To: netdev@...r.kernel.org
Cc: davem@...emloft.net, eric.dumazet@...il.com, brouer@...hat.com,
geert@...ux-m68k.org
Subject: [net-next PATCH 0/4] fixup IFF_NO_QUEUE conversion
This series serves two purposes:
On one hand it fixes a quite embarrassing bug around the warning I added for
drivers still setting tx_queue_len = 0 to achieve noqueue operation. It turned
out to be quite useless as due to using alloc_netdev(), many in-kernel drivers
fell into the trap by accident, as well. Instead this place serves pretty well
as a sanitizing point to set IFF_NO_QUEUE for drivers not initializing
tx_queue_len, which in turn allows to drop all special treatment of the latter
being zero since that can not happen anymore without IFF_NO_QUEUE being set.
On the other hand, it provides a better solution for Eric Dumazet's concern
regarding how to assign noqueue to an interface which does not default to it
already. In order to make this possible, noqueue is being registered so users
can 'tc qd add dev eth0 root noqueue'. In addition, it resolves the ugly
situation of 'tc qd show' not showing noqueue. Finally, the former changes
allow for some code cleanup.
Phil Sutter (4):
net: fix IFF_NO_QUEUE for drivers using alloc_netdev
net: sched: ignore tx_queue_len when assigning default qdisc
net: sched: register noqueue qdisc
net: sched: simplify attach_one_default_qdisc()
include/net/sch_generic.h | 1 +
net/core/dev.c | 2 +-
net/sched/sch_api.c | 1 +
net/sched/sch_generic.c | 54 ++++++++++++++++++++---------------------------
4 files changed, 26 insertions(+), 32 deletions(-)
--
2.1.2
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists