On Fri, May 03, 2019 at 03:53:09PM +1000, Suraj Jitindar Singh wrote: > 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 <sjitindarsi...@gmail.com>
Reviewed-by: David Gibson <da...@gibson.dropbear.id.au> > --- > target/ppc/mmu-radix64.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/target/ppc/mmu-radix64.c b/target/ppc/mmu-radix64.c > index a6ab290323..afa5ba506a 100644 > --- a/target/ppc/mmu-radix64.c > +++ b/target/ppc/mmu-radix64.c > @@ -249,6 +249,8 @@ 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; > } -- 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