From: "Cédric Le Goater" <clg@fr.ibm.com>
To: David Gibson <david@gibson.dropbear.id.au>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: peter.maydell@linaro.org, qemu-devel@nongnu.org, aik@ozlabs.ru,
mdroth@linux.vnet.ibm.com, agraf@suse.de, qemu-ppc@nongnu.org,
pbonzini@redhat.com
Subject: Re: [Qemu-devel] [PULL 1/3] ppc: Rework POWER7 & POWER8 exception model
Date: Tue, 5 Apr 2016 09:03:25 +0200 [thread overview]
Message-ID: <5703633D.8080101@fr.ibm.com> (raw)
In-Reply-To: <20160405032533.GS16485@voom.fritz.box>
On 04/05/2016 05:25 AM, David Gibson wrote:
> On Tue, Apr 05, 2016 at 12:19:50PM +1000, Benjamin Herrenschmidt wrote:
>> On Tue, 2016-04-05 at 12:17 +1000, David Gibson wrote:
>>> From: Cédric Le Goater <clg@fr.ibm.com>
>>>
>>> From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
>>>
>>> This patch fixes the current AIL implementation for POWER8. The
>>> interrupt vector address can be calculated directly from LPCR when
>>> the
>>> exception is handled. The excp_prefix update becomes useless and we
>>> can cleanup the H_SET_MODE hcall.
>>
>> Beware, iirc, this depends on the new cpu_set_papr() stuff I did so we
>> get the right LPCR values in PAPR mode.
>
> Right, Cédric already submitted that before the 2.6 freeze, and it was
> merged as 26a7f129, AFAICT.
Well, yes, but cpu_ppc_set_papr() only handles the AMOR setting, the LPCR
settings were kept for later as they were not bug fixes.
As for now, powerpc_excp() checks the ILE bit and uses the AIL bits to
calculate the vector address, which was done before in the H_SET_MODE
hcall. This allows some simplification, getting rid of excp_prefix,
and fixes a migration bug in TCG.
C.
next prev parent reply other threads:[~2016-04-05 7:03 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-05 2:17 [Qemu-devel] [PULL 0/3] ppc-for-2.6 queue 20160405 David Gibson
2016-04-05 2:17 ` [Qemu-devel] [PULL 1/3] ppc: Rework POWER7 & POWER8 exception model David Gibson
2016-04-05 2:19 ` Benjamin Herrenschmidt
2016-04-05 3:25 ` David Gibson
2016-04-05 7:03 ` Cédric Le Goater [this message]
2016-04-05 20:42 ` Benjamin Herrenschmidt
2016-04-07 9:13 ` [Qemu-devel] [Qemu-ppc] " Laurent Vivier
2016-04-07 10:27 ` Cédric Le Goater
2016-04-07 10:45 ` Laurent Vivier
2016-04-07 15:01 ` Cédric Le Goater
2016-04-08 1:22 ` David Gibson
2016-04-08 1:20 ` David Gibson
2016-04-05 2:17 ` [Qemu-devel] [PULL 2/3] spapr_drc: enable immediate detach for unsignalled devices David Gibson
2016-04-05 2:17 ` [Qemu-devel] [PULL 3/3] vl: Move cpu_synchronize_all_states() into qemu_system_reset() David Gibson
2016-04-05 10:02 ` [Qemu-devel] [PULL 0/3] ppc-for-2.6 queue 20160405 Peter Maydell
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=5703633D.8080101@fr.ibm.com \
--to=clg@fr.ibm.com \
--cc=agraf@suse.de \
--cc=aik@ozlabs.ru \
--cc=benh@kernel.crashing.org \
--cc=david@gibson.dropbear.id.au \
--cc=mdroth@linux.vnet.ibm.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@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.