Re: port-sparc/46530 (Install failure of NetBSD/sparc 5.1.2 and 6.0_BETA2 on qemu-1.0.1 and 1.1.0)

[Ryo ONODERA <ryo_on%yk.rim.or.jp@localhost>]

The following reply was made to PR port-sparc/46530; it has been noted by GNATS.

From: Ryo ONODERA <ryo_on%yk.rim.or.jp@localhost>
To: gnats-bugs%NetBSD.org@localhost, tsutsui%ceres.dti.ne.jp@localhost
Cc: martin%NetBSD.org@localhost
Subject: Re: port-sparc/46530 (Install failure of NetBSD/sparc 5.1.2 and
 6.0_BETA2 on qemu-1.0.1 and 1.1.0)
Date: Tue, 25 Sep 2012 02:38:33 +0900 (JST)

 Hi,
 
 From: Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>, Date: Sun, 23 Sep 
2012 20:10:07 +0000 (UTC)
 
 > The following reply was made to PR port-sparc/46530; it has been noted by 
 > GNATS.
 > 
 > From: Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc: martin%NetBSD.org@localhost, ryoon%NetBSD.org@localhost, 
 > tsutsui%ceres.dti.ne.jp@localhost
 > Subject: Re: port-sparc/46530 (Install failure of NetBSD/sparc 5.1.2 and 
 > 6.0_BETA2
 >       on qemu-1.0.1 and 1.1.0)
 > Date: Mon, 24 Sep 2012 05:07:54 +0900
 > 
 >  > I think this is a qemu bug - I'll look at it.
 >  
 >  It looks qemu's esp.c has some inconsistent ops around
 >  DMA command xfer termination.
 >  
 >  The following patch for qemu-1.2.0 would suppress most warning messages.
 >  (still not sure what it actually tries though)
 >  
 >  --- qemu-1.2.0/hw/esp.c.orig        2012-09-05 23:03:06.000000000 +0900
 >  +++ qemu-1.2.0/hw/esp.c     2012-09-24 04:57:37.000000000 +0900
 >  @@ -91,6 +91,10 @@
 >           dmalen |= s->rregs[ESP_TCMID] << 8;
 >           dmalen |= s->rregs[ESP_TCHI] << 16;
 >           s->dma_memory_read(s->dma_opaque, buf, dmalen);
 >  +        s->rregs[ESP_RSTAT] |= STAT_TC;
 >  +        s->rregs[ESP_TCLO] = 0;
 >  +        s->rregs[ESP_TCMID] = 0;
 >  +        s->rregs[ESP_TCHI] = 0;
 >       } else {
 >           dmalen = s->ti_size;
 >           memcpy(buf, s->ti_buf, dmalen);
 >  @@ -242,6 +246,10 @@
 >       if (s->do_cmd) {
 >           trace_esp_do_dma(s->cmdlen, len);
 >           s->dma_memory_read(s->dma_opaque, &s->cmdbuf[s->cmdlen], len);
 >  +        s->rregs[ESP_RSTAT] |= STAT_TC;
 >  +        s->rregs[ESP_TCLO] = 0;
 >  +        s->rregs[ESP_TCMID] = 0;
 >  +        s->rregs[ESP_TCHI] = 0;
 >           s->ti_size = 0;
 >           s->cmdlen = 0;
 >           s->do_cmd = 0;
 
 Your patch suppress the output of errors.
 I will pkgsrc bulk build on the VM and check the error message.
 
 If bulk build does not make errors, I will integrate the patch to
 emulators/qemu.
 
 Thank you.
 
 --
 Ryo ONODERA // ryo_on%yk.rim.or.jp@localhost
 PGP fingerprint = 82A2 DC91 76E0 A10A 8ABB  FD1B F404 27FA C7D1 15F3