]> git.baikalelectronics.ru Git - kernel.git/commit
net: sch_generic: aviod concurrent reset and enqueue op for lockless qdisc
authorYunsheng Lin <linyunsheng@huawei.com>
Tue, 8 Sep 2020 11:02:34 +0000 (19:02 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 10 Sep 2020 19:38:26 +0000 (12:38 -0700)
commit148ab6c673cc0da5462cf2b4eafe4521e09a7a1b
tree68e49da296f724a5cc66e17a6f561e26472c8109
parent6e7573eb48a2c38e5f32259d9666c3588d2a2e7d
net: sch_generic: aviod concurrent reset and enqueue op for lockless qdisc

Currently there is concurrent reset and enqueue operation for the
same lockless qdisc when there is no lock to synchronize the
q->enqueue() in __dev_xmit_skb() with the qdisc reset operation in
qdisc_deactivate() called by dev_deactivate_queue(), which may cause
out-of-bounds access for priv->ring[] in hns3 driver if user has
requested a smaller queue num when __dev_xmit_skb() still enqueue a
skb with a larger queue_mapping after the corresponding qdisc is
reset, and call hns3_nic_net_xmit() with that skb later.

Reused the existing synchronize_net() in dev_deactivate_many() to
make sure skb with larger queue_mapping enqueued to old qdisc(which
is saved in dev_queue->qdisc_sleeping) will always be reset when
dev_reset_queue() is called.

Fixes: 168006f6b275 ("net: sched: allow qdiscs to handle locking")
Signed-off-by: Yunsheng Lin <linyunsheng@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/sch_generic.c