[PATCH] powerpc: Fix definition of SIAR register
Paul Mackerras
paulus at ozlabs.org
Thu May 12 20:40:17 AEST 2016
On Thu, May 12, 2016 at 09:27:40AM +0200, Thomas Huth wrote:
> On 12.05.2016 06:57, Paul Mackerras wrote:
> > On Mon, Apr 25, 2016 at 10:15:47AM +0200, Alexander Graf wrote:
> >>
> >>>> +#define SPRN_SIAR 796
> >>
> >> I'm sure there's a reason (iSeries?) we used the r/o version before. Better introduce a new constant that gives us rw access and use that in the kvm entry/exit code.
> >
> > I don't think we ever did any performance monitoring on iSeries, and
> > in any case, we have removed the iSeries code. I think we should just
> > use the privileged, read/write number (i.e. 796).
>
> Ok, then I'll simply add the fix for SDAR to my patch as well, test it
> and submit it again.
Great! Could you switch MMCR2 to the privileged number as well while
you're at it?
Thanks,
Paul.
More information about the Linuxppc-dev
mailing list