net: sch_prio: When ungrafting, replace with FIFO
authorPetr Machata <petrm@mellanox.com>
Mon, 6 Jan 2020 18:01:56 +0000 (18:01 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 8 Jan 2020 20:45:53 +0000 (12:45 -0800)
commit240ce7f6428ff5188b9eedc066e1e4d645b8635f
tree38df958b10b05e5d459aeec5c5bd18e6785d7a61
parent3971a535b839489e4ea31796cc086e6ce616318c
net: sch_prio: When ungrafting, replace with FIFO

When a child Qdisc is removed from one of the PRIO Qdisc's bands, it is
replaced unconditionally by a NOOP qdisc. As a result, any traffic hitting
that band gets dropped. That is incorrect--no Qdisc was explicitly added
when PRIO was created, and after removal, none should have to be added
either.

Fix PRIO by first attempting to create a default Qdisc and only falling
back to noop when that fails. This pattern of attempting to create an
invisible FIFO, using NOOP only as a fallback, is also seen in other
Qdiscs.

Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Signed-off-by: Petr Machata <petrm@mellanox.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/sch_prio.c