From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: "Hervé Poussineau" <hpoussin@reactos.org>,
qemu-devel <qemu-devel@nongnu.org>,
"qemu-ppc@nongnu.org" <qemu-ppc@nongnu.org>
Subject: Re: [Qemu-devel] ppc: regression in 2.7.0-rc3 on rfi/rfid/hrfi instructions
Date: Thu, 25 Aug 2016 08:22:57 +1000 [thread overview]
Message-ID: <1472077377.2246.41.camel@kernel.crashing.org> (raw)
In-Reply-To: <4a78a3a5-91ba-b3bd-a59c-d1ebf9f1470b@reactos.org>
On Wed, 2016-08-24 at 20:39 +0200, Hervé Poussineau wrote:
> Hi,
>
> Following patch regresses Linux boot on PReP machine:
Where do I find the source for that kernel ?
Cheers,
Ben.
> commit a2e71b28e832346409efc795ecd1f0a2bcb705a3
> Author: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> Date: Tue Jun 21 23:48:46 2016 +0200
>
> ppc: Fix rfi/rfid/hrfi/... emulation
>
> This reworks emulation of the various "rfi" variants. I removed
> some masking bits that I couldn't make sense of, the only bit
> that
> I am aware we should mask here is POW, the CPU's MSR mask should
> take care of the rest.
>
> This also fixes some problems when running 32-bit userspace
> under
> a 64-bit kernel.
>
> This patch broke 32bit OpenBIOS when run under a 970 cpu. A fix
> was
> proposed here :
>
> https://www.coreboot.org/pipermail/openbios/2016-June/009452
> .html
>
> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
> [clg: updated the commit log with the reference of the openbios
> fix ]
> Signed-off-by: Cédric Le Goater <clg@kaod.org>
> [dwg: Remove hunk which disabled rfi on 64-bit CPUS. The change
> was
> correct, but we need to fix OpenBIOS before applying it]
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
>
> Test case:
> - Download: http://www.juneau-lug.org/zImage.initrd.sandalfoot
> - Run: qemu-system-ppc -M prep -kernel zImage.initrd.sandalfoot
>
> Firmware starts, loads kernel, but seems to error out just after PS/2
> detection.
>
> Reverting the commit and fixing the conflict makes it work again, up
> to shell.
>
> Hervé
next prev parent reply other threads:[~2016-08-24 22:23 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-24 18:39 [Qemu-devel] ppc: regression in 2.7.0-rc3 on rfi/rfid/hrfi instructions Hervé Poussineau
2016-08-24 21:25 ` Mark Cave-Ayland
2016-08-24 22:22 ` Benjamin Herrenschmidt [this message]
2016-08-25 1:31 ` Benjamin Herrenschmidt
2016-08-25 4:37 ` Benjamin Herrenschmidt
2016-08-25 5:33 ` Benjamin Herrenschmidt
2016-08-25 20:14 ` Hervé Poussineau
2016-08-25 21:27 ` Benjamin Herrenschmidt
2016-08-26 6:43 ` 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=1472077377.2246.41.camel@kernel.crashing.org \
--to=benh@kernel.crashing.org \
--cc=hpoussin@reactos.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.