net: dev: Always serialize on Qdisc::busylock in __dev_xmit_skb() on PREEMPT_RT.
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>
Mon, 13 Dec 2021 10:53:29 +0000 (11:53 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 13 Dec 2021 14:57:03 +0000 (14:57 +0000)
commit64445dda9d8384975eca54e3f01886fca61e1db6
tree16dda030b630cf2f363e0ab71e08322f6a994393
parent93d576f54e0f697c7b54f4ddd718e68ccc52c4f2
net: dev: Always serialize on Qdisc::busylock in __dev_xmit_skb() on PREEMPT_RT.

The root-lock is dropped before dev_hard_start_xmit() is invoked and after
setting the __QDISC___STATE_RUNNING bit. If the Qdisc owner is preempted
by another sender/task with a higher priority then this new sender won't
be able to submit packets to the NIC directly instead they will be
enqueued into the Qdisc. The NIC will remain idle until the Qdisc owner
is scheduled again and finishes the job.

By serializing every task on the ->busylock then the task will be
preempted by a sender only after the Qdisc has no owner.

Always serialize on the busylock on PREEMPT_RT.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c