| Message ID | 20170619165753.25049-3-julien.grall@arm.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | xen/arm: Clean-up memory subsystems | expand |
On Mon, 19 Jun 2017, Julien Grall wrote: > xenheap_mfn_end is storing an MFN and not a physical address. Xen is not > currently using xenheap_mfn_end and the value will be reset after the > loop. So drop this bogus xenheap_mfn_end. > > Signed-off-by: Julien Grall <julien.grall@arm.com> I would drop the mention of the fact that Xen is not currently using xenheap_mfn_end entirely. I might do that as I commit. Reviewed-by: Stefano Stabellini <sstabellini@kernel.org> > --- > Changes in v2: > - Update commit message > --- > xen/arch/arm/setup.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/xen/arch/arm/setup.c b/xen/arch/arm/setup.c > index f00f29a45b..ab4d8e4218 100644 > --- a/xen/arch/arm/setup.c > +++ b/xen/arch/arm/setup.c > @@ -654,8 +654,6 @@ static void __init setup_mm(unsigned long dtb_paddr, size_t dtb_size) > if ( e > bank_end ) > e = bank_end; > > - xenheap_mfn_end = e; > - > dt_unreserved_regions(s, e, init_boot_pages, 0); > s = n; > } > -- > 2.11.0 >
diff --git a/xen/arch/arm/setup.c b/xen/arch/arm/setup.c index f00f29a45b..ab4d8e4218 100644 --- a/xen/arch/arm/setup.c +++ b/xen/arch/arm/setup.c @@ -654,8 +654,6 @@ static void __init setup_mm(unsigned long dtb_paddr, size_t dtb_size) if ( e > bank_end ) e = bank_end; - xenheap_mfn_end = e; - dt_unreserved_regions(s, e, init_boot_pages, 0); s = n; }
xenheap_mfn_end is storing an MFN and not a physical address. Xen is not currently using xenheap_mfn_end and the value will be reset after the loop. So drop this bogus xenheap_mfn_end. Signed-off-by: Julien Grall <julien.grall@arm.com> --- Changes in v2: - Update commit message --- xen/arch/arm/setup.c | 2 -- 1 file changed, 2 deletions(-)