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 <ag...@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. -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
