Re: [PATCH 1/6] KVM: PPC: Book3S PR: Ignore PMU SPRs

[Paul Mackerras <paulus@samba.org>]

On Fri, May 02, 2014 at 10:35:09AM +0200, Alexander Graf wrote:
> On 05/01/2014 12:12 AM, Paul Mackerras wrote:
> >On Tue, Apr 29, 2014 at 06:17:37PM +0200, Alexander Graf wrote:
> >>When we expose a POWER8 CPU into the guest, it will start accessing PMU SPRs
> >>that we don't emulate. Just ignore accesses to them.
> >>
> >>Signed-off-by: Alexander Graf <agraf@suse.de>
> >This patch is OK as it stands, but in fact the architecture says that
> >kernel accesses to unimplemented SPRs are mostly supposed to be no-ops
> >rather than causing a trap (mostly = excluding mtspr to 0 or mfspr
> >from 0, 4, 5 or 6).  I have a patch to implement that, which I'll
> >post.
> 
> I think what we want is a flag similar to x86 where we can force
> ignore unknown SPRs, but leave it at triggering an interrupt as
> default. We usually have to be at least aware of unknown SPRs and
> check that not implementing them is ok for the guest.
> 
> Debugging a program interrupt because of an unknown SPR is usually a
> lot easier than debugging a breaking guest because it was using the
> SPR as storage and we didn't back it by anything.

That has not been my experience, for accesses by the Linux kernel
early in the boot process; usually we end up in a loop of ISI
interrupts because the HPT isn't set up yet, with the original
interrupt cause (and PC) lost long ago.

The Power ISA was changed in version 2.05 (POWER6) to specify that
accesses to unimplemented SPRs by privileged code must be no-ops on
server processors.  Before that the architecture allowed either an
illegal instruction interrupt or "boundedly undefined" behaviour
(which would include a no-op).

So, if we're emulating POWERx for x >= 6, to be correct we need to do
the no-op behaviour, even if we retain the option of making them trap
for debugging purposes.  Of course at the moment we basically never
look at what specific CPU we're emulating, but maybe now we have to.

Regards,
Paul.