summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaolo Bonzini <pbonzini@redhat.com>2012-08-09 16:45:56 +0200
committerStefan Hajnoczi <stefanha@gmail.com>2012-09-14 08:40:31 +0100
commite8b4c680b41bd960ecccd9ff076b7b058e0afcd4 (patch)
tree6ab1aca5201c3b0bab967c0d40829bd5a4bae386
parent987a9b4800003567b1a47a379255e886a77d57ea (diff)
downloadqemu-e8b4c680b41bd960ecccd9ff076b7b058e0afcd4.tar.gz
qemu-e8b4c680b41bd960ecccd9ff076b7b058e0afcd4.tar.bz2
qemu-e8b4c680b41bd960ecccd9ff076b7b058e0afcd4.zip
e1000: flush queue whenever can_receive can go from false to true
When the guests replenish the receive ring buffer, the network device should flush its queue of pending packets. This is done with qemu_flush_queued_packets. e1000's can_receive can go from false to true when RCTL or RDT are modified. Reported-by: Luigi Rizzo <rizzo@iet.unipi.it> Cc: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> Cc: Jan Kiszka <jan.kiszka@siemens.de> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Amos Kong <akong@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
-rw-r--r--hw/e1000.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/hw/e1000.c b/hw/e1000.c
index ae8a6c5523..ec3a7c4ecc 100644
--- a/hw/e1000.c
+++ b/hw/e1000.c
@@ -295,6 +295,7 @@ set_rx_control(E1000State *s, int index, uint32_t val)
s->rxbuf_min_shift = ((val / E1000_RCTL_RDMTS_QUAT) & 3) + 1;
DBGOUT(RX, "RCTL: %d, mac_reg[RCTL] = 0x%x\n", s->mac_reg[RDT],
s->mac_reg[RCTL]);
+ qemu_flush_queued_packets(&s->nic->nc);
}
static void
@@ -926,6 +927,9 @@ set_rdt(E1000State *s, int index, uint32_t val)
{
s->check_rxov = 0;
s->mac_reg[index] = val & 0xffff;
+ if (e1000_has_rxbufs(s, 1)) {
+ qemu_flush_queued_packets(&s->nic->nc);
+ }
}
static void