sched: Fix detection of empty queues in child qdiscs
authorToke Høiland-Jørgensen <toke@redhat.com>
Wed, 9 Jan 2019 16:09:43 +0000 (17:09 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 16 Jan 2019 04:12:00 +0000 (20:12 -0800)
commit37d9cf1a3ce35de3df6f7d209bfb1f50cf188cea
tree29e8d6fccf9da0a9c19129f477a0ef0ae244464d
parentf6bab199315b70fd83fe3ee0947bc84c7a35f3d4
sched: Fix detection of empty queues in child qdiscs

Several qdiscs check on enqueue whether the packet was enqueued to a class
with an empty queue, in which case the class is activated. This is done by
checking if the qlen is exactly 1 after enqueue. However, if GSO splitting
is enabled in the child qdisc, a single packet can result in a qlen longer
than 1. This means the activation check fails, leading to a stalled queue.

Fix this by checking if the queue is empty *before* enqueue, and running
the activation logic if this was the case.

Reported-by: Pete Heist <pete@heistp.net>
Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/sch_drr.c
net/sched/sch_hfsc.c
net/sched/sch_qfq.c