Message ID | 20200330094946.24678-2-clg@kaod.org |
---|---|
State | New |
Headers | show |
Series | target/ppc: Add support for Radix partition-scoped translation | expand |
On Mon, 30 Mar 2020 11:49:40 +0200 Cédric Le Goater <clg@kaod.org> wrote: > From: Suraj Jitindar Singh <sjitindarsingh@gmail.com> > > According to the ISA the root page directory size of a radix tree for > either process- or partition-scoped translation must be >= 5. > > Thus add this to the list of conditions checked when validating the > partition table entry in validate_pate(); > > Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com> > Reviewed-by: David Gibson <david@gibson.dropbear.id.au> > Signed-off-by: Cédric Le Goater <clg@kaod.org> > --- Reviewed-by: Greg Kurz <groug@kaod.org> > target/ppc/mmu-radix64.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/target/ppc/mmu-radix64.c b/target/ppc/mmu-radix64.c > index 224e646c5094..99678570581b 100644 > --- a/target/ppc/mmu-radix64.c > +++ b/target/ppc/mmu-radix64.c > @@ -212,6 +212,9 @@ static bool validate_pate(PowerPCCPU *cpu, uint64_t lpid, ppc_v3_pate_t *pate) > if (lpid == 0 && !msr_hv) { > return false; > } > + if ((pate->dw0 & PATE1_R_PRTS) < 5) { > + return false; > + } > /* More checks ... */ > return true; > }
On Mon, Mar 30, 2020 at 11:49:40AM +0200, Cédric Le Goater wrote: > From: Suraj Jitindar Singh <sjitindarsingh@gmail.com> > > According to the ISA the root page directory size of a radix tree for > either process- or partition-scoped translation must be >= 5. > > Thus add this to the list of conditions checked when validating the > partition table entry in validate_pate(); > > Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com> > Reviewed-by: David Gibson <david@gibson.dropbear.id.au> > Signed-off-by: Cédric Le Goater <clg@kaod.org> Applied to ppc-for-5.1. > --- > target/ppc/mmu-radix64.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/target/ppc/mmu-radix64.c b/target/ppc/mmu-radix64.c > index 224e646c5094..99678570581b 100644 > --- a/target/ppc/mmu-radix64.c > +++ b/target/ppc/mmu-radix64.c > @@ -212,6 +212,9 @@ static bool validate_pate(PowerPCCPU *cpu, uint64_t lpid, ppc_v3_pate_t *pate) > if (lpid == 0 && !msr_hv) { > return false; > } > + if ((pate->dw0 & PATE1_R_PRTS) < 5) { > + return false; > + } > /* More checks ... */ > return true; > }
diff --git a/target/ppc/mmu-radix64.c b/target/ppc/mmu-radix64.c index 224e646c5094..99678570581b 100644 --- a/target/ppc/mmu-radix64.c +++ b/target/ppc/mmu-radix64.c @@ -212,6 +212,9 @@ static bool validate_pate(PowerPCCPU *cpu, uint64_t lpid, ppc_v3_pate_t *pate) if (lpid == 0 && !msr_hv) { return false; } + if ((pate->dw0 & PATE1_R_PRTS) < 5) { + return false; + } /* More checks ... */ return true; }