From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=54311 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OsICR-0000yR-Pb for qemu-devel@nongnu.org; Sun, 05 Sep 2010 12:36:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OsICQ-0005gR-LF for qemu-devel@nongnu.org; Sun, 05 Sep 2010 12:36:11 -0400 Received: from mx1.redhat.com ([209.132.183.28]:25693) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OsICQ-0005gL-Cc for qemu-devel@nongnu.org; Sun, 05 Sep 2010 12:36:10 -0400 Date: Sun, 5 Sep 2010 19:30:14 +0300 From: "Michael S. Tsirkin" Message-ID: <20100905163014.GA23307@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Subject: [Qemu-devel] [PATCH] qemu: e1000 fix TOR math List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, Anthony Liguori Patch b0b900070c7cb29bbefb732ec00397abe5de6d73 made TOR valuer incorrect: the spec says it should always include the CRC field, while size does not include CRC now. No one seems to use TOR field (which is likely why current code works fine), but better to stick to spec. Lightly tested with a linux guest. Signed-off-by: Michael S. Tsirkin --- hw/e1000.c | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/hw/e1000.c b/hw/e1000.c index 80b78bc..eb9faf2 100644 --- a/hw/e1000.c +++ b/hw/e1000.c @@ -345,7 +345,7 @@ is_vlan_txd(uint32_t txd_lower) /* FCS aka Ethernet CRC-32. We don't get it from backends and can't * fill it in, just pad descriptor length by 4 bytes unless guest - * told us to trip it off the packet. */ + * told us to strip it off the packet. */ static inline int fcs_len(E1000State *s) { @@ -690,8 +690,12 @@ e1000_receive(VLANClientState *nc, const uint8_t *buf, size_t size) s->mac_reg[GPRC]++; s->mac_reg[TPR]++; + /* TOR - Total Octets Received: + * This register includes bytes received in a packet from the field through the field, inclusively. + */ n = s->mac_reg[TORL]; - if ((s->mac_reg[TORL] += size) < n) + if ((s->mac_reg[TORL] += size + 4 /* Always include FCS length. */) < n) s->mac_reg[TORH]++; n = E1000_ICS_RXT0; -- 1.7.2.rc0.14.g41c1c