virtio-net: Introduce a new bottom half packet TX
authorAlex Williamson <alex.williamson@redhat.com>
Thu, 2 Sep 2010 15:01:10 +0000 (09:01 -0600)
committerMichael S. Tsirkin <mst@redhat.com>
Tue, 7 Sep 2010 17:29:29 +0000 (20:29 +0300)
commit702024b158779a662179c59a0d70becbaa1fa7f3
treef22a82803a21178bb158a63fe9647a7155d25693
parent3b18bc5b9f59ebdd5b08f2701e85da52f8866e2e
virtio-net: Introduce a new bottom half packet TX

Based on a patch from Mark McLoughlin, this patch introduces a new
bottom half packet transmitter that avoids the latency imposed by
the tx_timer approach.  Rather than scheduling a timer when a TX
packet comes in, schedule a bottom half to be run from the iothread.
The bottom half handler first attempts to flush the queue with
notification disabled (this is where we could race with a guest
without txburst).  If we flush a full burst, reschedule immediately.
If we send short of a full burst, try to re-enable notification.
To avoid a race with TXs that may have occurred, we must then
flush again.  If we find some packets to send, the guest it probably
active, so we can reschedule again.

tx_timer and tx_bh are mutually exclusive, so we can re-use the
tx_waiting flag to indicate one or the other needs to be setup.
This allows us to seamlessly migrate between timer and bh TX
handling.

The bottom half handler becomes the new default and we add a new
tx= option to virtio-net-pci.  Usage:

-device virtio-net-pci,tx=timer # select timer mitigation vs "bh"

Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
hw/s390-virtio-bus.c
hw/syborg_virtio.c
hw/virtio-net.c
hw/virtio-net.h
hw/virtio-pci.c