Message ID | 20190925125236.4043-6-david@redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | s390x/mmu: DAT translation rewrite | expand |
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 <david@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 <thuth@redhat.com>
On 9/25/19 5:52 AM, David Hildenbrand wrote: > While ASCE_ORIGIN is not wrong, it is certainly confusing. We want a > page frame address. > > Signed-off-by: David Hildenbrand <david@redhat.com> > --- > target/s390x/mmu_helper.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Richard Henderson <richard.henderson@linaro.org> r~
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; }
While ASCE_ORIGIN is not wrong, it is certainly confusing. We want a page frame address. Signed-off-by: David Hildenbrand <david@redhat.com> --- target/s390x/mmu_helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)