All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Monjalon <thomas_ml@monjalon.net>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v2 4/5] target-ppc: fix RFI by clearing upper bytes of MSR
Date: Tue, 18 May 2010 17:17:02 +0200	[thread overview]
Message-ID: <201005181717.03044.thomas_ml@monjalon.net> (raw)
In-Reply-To: <4BF2A5CE.8020709@suse.de>

Alexander Graf wrote:
> Thomas Monjalon wrote:
> > I'm running Linux for SBC834x in Qemu. The interrupt controller and board
> > definition are not yet published.
>
> Wow, I didn't know there were still new products based on e300.

Sorry, I was not clear. By "not yet published", I mean that I've written Qemu 
code to emulate e300 but I haven't yet send it to the ML. I would prefer to 
fix this RFI issue first.
SBC834x is not a new product.


> > From the e300 reference manual (e300CORERM):
> > "The TGPR bit is cleared by an rfi instruction."
> >
> > My first try was to clear only TGPR. But it doesn't work properly if POW
> > and ILE are not cleared.
>
> According to the 2.06 ISA again, rfi does the following:
>
> The contents of SRR1 are placed into the MSR. If the new MSR value does
> not enable any pending exceptions, then the next instruction is fetched,
> under control of the new MSR value, from the address SRR0 0:64 || 0b00.
>
> If rfi would clear ILE, how would it be enabled then?

You should be right. I have to fix a bug elsewhere.

-- 
Thomas

  reply	other threads:[~2010-05-18 15:17 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-27 15:31 [Qemu-devel] [PATCH v2 0/5] fix & clean PPC e300 Thomas Monjalon
2010-04-27 15:31 ` [Qemu-devel] [PATCH v2 1/5] target-ppc: fix processor versions (PVR) for e300 Thomas Monjalon
2010-04-27 15:31 ` [Qemu-devel] [PATCH v2 2/5] target-ppc: fix interrupt vectors for MPC603 and e300 Thomas Monjalon
2010-04-27 15:31 ` [Qemu-devel] [PATCH v2 3/5] target-ppc: exception model of 603e inherits from 603 Thomas Monjalon
2010-04-27 15:31 ` [Qemu-devel] [PATCH v2 4/5] target-ppc: fix RFI by clearing upper bytes of MSR Thomas Monjalon
2010-05-02  8:12   ` Alexander Graf
2010-05-18 14:00     ` Thomas Monjalon
2010-05-18 14:35       ` Alexander Graf
2010-05-18 15:17         ` Thomas Monjalon [this message]
2010-04-27 15:31 ` [Qemu-devel] [PATCH v2 5/5] ppc: remove dead code Thomas Monjalon
2010-04-27 18:47 ` [Qemu-devel] Re: [PATCH v2 0/5] fix & clean PPC e300 Blue Swirl
2010-04-27 21:18   ` Thomas Monjalon

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=201005181717.03044.thomas_ml@monjalon.net \
    --to=thomas_ml@monjalon.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.