[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 13/17] target/ppc: Remove env->immu_idx and env->dmmu_idx
From: |
David Gibson |
Subject: |
Re: [PATCH v4 13/17] target/ppc: Remove env->immu_idx and env->dmmu_idx |
Date: |
Tue, 23 Mar 2021 11:01:03 +1100 |
On Mon, Mar 22, 2021 at 11:27:49AM -0600, Richard Henderson wrote:
> On 3/21/21 10:26 PM, David Gibson wrote:
> > On Mon, Mar 15, 2021 at 12:46:11PM -0600, Richard Henderson wrote:
> > > We weren't recording MSR_GS in hflags, which means that BookE
> > > memory accesses were essentially random vs Guest State.
> > >
> > > Instead of adding this bit directly, record the completed mmu
> > > indexes instead. This makes it obvious that we are recording
> > > exactly the information that we need.
> > >
> > > This also means that we can stop directly recording MSR_IR.
> >
> > What still uses MSR_DR, that you can't also drop it?
>
> #define CHK_HVRM \
> do { \
> if (unlikely(ctx->pr || !ctx->hv || ctx->dr)) { \
>
> I have this notion that this (and CHK_HV and CHK_SV) could be a test against
> mmu_idx instead, but was reluctant to make that change.
Yeah, that's checking for hypervisor real mode (hence "HVRM") for
ldcix and friends, so it should be equivalent to (mmu_idx != 7).
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature