All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thiemo Seufer <ths@networkno.de>
To: Aurelien Jarno <aurelien@aurel32.net>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] MIPS: Fix dmtc0 instruction
Date: Mon, 25 Aug 2008 13:28:41 +0200	[thread overview]
Message-ID: <20080825112841.GB994@networkno.de> (raw)
In-Reply-To: <20080823172404.GK32516@volta.aurel32.net>

Aurelien Jarno wrote:
> On Sat, Aug 23, 2008 at 06:42:34PM +0200, Hervé Poussineau wrote:
> > Hi,
> > 
> > In gen_dmtc0 function, TCG temporary variable t0 is freed at the end of
> > the function. Variable is freed again in the gen_dmtc0 caller.
> > I removed the free in gen_dmtc0, to do like in gen_dmfc0, gen_mfc0,
> > gen_mtc0.
> > 
> > Incidentally, this unregresses NetBSD on Pica 61.
>  
> This was also affecting Linux MIPS64 and I have noticed the problem this
> morning (though it seems to be a few weeks old).

My old 64-bit testcase worked for some reason despite that bug. The current
debian/testing kernel image for malta/5kc failed, though.

> I have applied the patch, and also removed another tcg_temp_free(t0)
> which has been left. Thanks!

I can confirm it works here as well, thank you. However, I still see
QEMU hang with the debian kernel at IDE detection:

[...]
[    2.040127] pcnet32.c:v1.34-NAPI 14.Aug.2007 tsbogend@alpha.franken.de
[    2.040127] pcnet32: PCnet/PCI II 79C970A at 0x1020, 52:54:00:12:34:56 assigned IRQ 10.
[    2.040127] eth0: registered as PCnet/PCI II 79C970A
[    2.040127] pcnet32: 1 cards_found.
[    2.040127] Uniform Multi-Platform E-IDE driver
[    2.040127] ide: Assuming 33MHz system bus speed for PIO modes; override with idebus=xx
[    2.040127] PIIX4: IDE controller (0x8086:0x7111 rev 0x00) at  PCI slot 0000:00:0a.1
[    2.040127] PCI: Enabling device 0000:00:0a.1 (0000 -> 0001)
[    2.040127] PIIX4: not 100% native mode: will probe irqs later
[    2.040127]     ide0: BM-DMA at 0x1040-0x1047, BIOS settings: hda:PIO, hdb:PIO
[    2.040127]     ide1: BM-DMA at 0x1048-0x104f, BIOS settings: hdc:PIO, hdd:PIO

I figure CONFIG_NO_HZ has too high demands on the count/compare emulation.


Thiemo

  reply	other threads:[~2008-08-25 11:29 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-23 16:42 [Qemu-devel] [PATCH] MIPS: Fix dmtc0 instruction Hervé Poussineau
2008-08-23 17:24 ` Aurelien Jarno
2008-08-25 11:28   ` Thiemo Seufer [this message]
2008-08-25 12:55     ` Aurelien Jarno
2008-08-25 14:12       ` Thiemo Seufer
2008-08-25 18:51     ` Stefan Weil
2008-08-25 21:10       ` Thiemo Seufer
2008-08-25 22:26         ` Aurelien Jarno
  -- strict thread matches above, loose matches on Subject: below --
2008-08-23 16:40 Hervé Poussineau

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080825112841.GB994@networkno.de \
    --to=ths@networkno.de \
    --cc=aurelien@aurel32.net \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.