On 25/09/2019 14.52, David Hildenbrand wrote: > While ASCE_ORIGIN is not wrong, it is certainly confusing. We want a > page frame address. > > Signed-off-by: David Hildenbrand <da...@redhat.com> > --- > target/s390x/mmu_helper.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target/s390x/mmu_helper.c b/target/s390x/mmu_helper.c > index c9fde78614..20e9c13202 100644 > --- a/target/s390x/mmu_helper.c > +++ b/target/s390x/mmu_helper.c > @@ -126,7 +126,7 @@ static int mmu_translate_pte(CPUS390XState *env, > target_ulong vaddr, > *flags &= ~PAGE_WRITE; > } > > - *raddr = pt_entry & ASCE_ORIGIN; > + *raddr = pt_entry & TARGET_PAGE_MASK; > return 0; > }
Reviewed-by: Thomas Huth <th...@redhat.com>
