[Qemu-devel] Re: openbsd image on freeoszoo

["Jim C. Brown" <jbrown106@phreaker.net>]

[-- Attachment #1: Type: text/plain, Size: 706 bytes --]

On Fri, Jul 09, 2004 at 07:18:08PM -0400, Jim C. Brown wrote:
> 
> I also get lots of "remote transmit DMA failed to complete" errors, for
> both the ISA and the PCI nic. I haven't solved this one yet but it seems
> harmless. (For the ISA nic, it seems that it MUST be called ne2 btw or network
> won't work.) For the PCI nic, I also get "send_packet: Input/output error",
> not solved yet either.

Fixed in qemu. Attached is the patch that fixes it. This looks like a bug
in qemu, as it transfers an odd number of bytes in words, leaving a count of
-1 when done. But if so, it has never been caught before...

-- 
Infinite complexity begets infinite beauty.
Infinite precision begets infinite perfection.

[-- Attachment #2: nox.patch --]
[-- Type: text/plain, Size: 1701 bytes --]

--- qemu/hw/ne2000.c	Mon Jun 21 15:42:46 2004
+++ qemu/hw/ne2000.c	Fri Jul  9 20:57:32 2004
@@ -447,7 +447,7 @@
 #ifdef DEBUG_NE2000
     printf("NE2000: asic write val=0x%04x\n", val);
 #endif
-    if (s->rcnt == 0)
+    if ((s->rcnt == 0) || (s->rcnt == (uint16_t)-1))
 	    return;
     if (s->dcfg & 0x01) {
         /* 16 bit access */
@@ -463,7 +463,7 @@
     /* wrap */
     if (s->rsar == s->stop)
         s->rsar = s->start;
-    if (s->rcnt == 0) {
+    if ((s->rcnt == 0) || (s->rcnt == (uint16_t)-1)) {
         /* signal end of transfert */
         s->isr |= ENISR_RDC;
         ne2000_update_irq(s);
@@ -489,7 +489,7 @@
     /* wrap */
     if (s->rsar == s->stop)
         s->rsar = s->start;
-    if (s->rcnt == 0) {
+    if ((s->rcnt == 0) || (s->rcnt == (uint16_t)-1)) {
         /* signal end of transfert */
         s->isr |= ENISR_RDC;
         ne2000_update_irq(s);
@@ -507,7 +507,7 @@
 #ifdef DEBUG_NE2000
     printf("NE2000: asic writel val=0x%04x\n", val);
 #endif
-    if (s->rcnt == 0)
+    if ((s->rcnt == 0) || (s->rcnt == (uint16_t)-1))
 	    return;
     /* 32 bit access */
     ne2000_mem_writel(s, s->rsar, val);
@@ -516,7 +516,7 @@
     /* wrap */
     if (s->rsar == s->stop)
         s->rsar = s->start;
-    if (s->rcnt == 0) {
+    if ((s->rcnt == 0) || (s->rcnt == (uint16_t)-1)) {
         /* signal end of transfert */
         s->isr |= ENISR_RDC;
         ne2000_update_irq(s);
@@ -536,7 +536,7 @@
     /* wrap */
     if (s->rsar == s->stop)
         s->rsar = s->start;
-    if (s->rcnt == 0) {
+    if ((s->rcnt == 0) || (s->rcnt == (uint16_t)-1)) {
         /* signal end of transfert */
         s->isr |= ENISR_RDC;
         ne2000_update_irq(s);