Re: [Xen-devel] [PATCH 03/24] xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64

2017-06-16 Thread Stefano Stabellini
On Fri, 16 Jun 2017, Julien Grall wrote: > Hi Stefano, > > On 06/16/2017 06:33 PM, Stefano Stabellini wrote: > > On Fri, 16 Jun 2017, Julien Grall wrote: > > > Hi Stefano, > > > > > > On 15/06/2017 23:28, Stefano Stabellini wrote: > > > > On Tue, 13 Jun 2017, Julien Grall wrote: > > > > >

Re: [Xen-devel] [PATCH 03/24] xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64

2017-06-16 Thread Julien Grall
Hi Stefano, On 06/16/2017 06:33 PM, Stefano Stabellini wrote: On Fri, 16 Jun 2017, Julien Grall wrote: Hi Stefano, On 15/06/2017 23:28, Stefano Stabellini wrote: On Tue, 13 Jun 2017, Julien Grall wrote: xenheap_mfn_end is storing an MFN and not a physical address. Thankfully xenheap_mfn_end

Re: [Xen-devel] [PATCH 03/24] xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64

2017-06-16 Thread Stefano Stabellini
On Fri, 16 Jun 2017, Julien Grall wrote: > Hi Stefano, > > On 15/06/2017 23:28, Stefano Stabellini wrote: > > On Tue, 13 Jun 2017, Julien Grall wrote: > > > xenheap_mfn_end is storing an MFN and not a physical address. Thankfully > > > xenheap_mfn_end is not used in the arm64 code. So drop it. >

Re: [Xen-devel] [PATCH 03/24] xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64

2017-06-16 Thread Julien Grall
Hi Stefano, On 15/06/2017 23:28, Stefano Stabellini wrote: On Tue, 13 Jun 2017, Julien Grall wrote: xenheap_mfn_end is storing an MFN and not a physical address. Thankfully xenheap_mfn_end is not used in the arm64 code. So drop it. That's fine, but in that case I would prefer to move the

Re: [Xen-devel] [PATCH 03/24] xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64

2017-06-15 Thread Stefano Stabellini
On Tue, 13 Jun 2017, Julien Grall wrote: > xenheap_mfn_end is storing an MFN and not a physical address. Thankfully > xenheap_mfn_end is not used in the arm64 code. So drop it. That's fine, but in that case I would prefer to move the definition of xenheap_mfn_end under #ifdef CONFIG_ARM_32. In

[Xen-devel] [PATCH 03/24] xen/arm: setup: Remove bogus xenheap_mfn_end in setup_mm for arm64

2017-06-13 Thread Julien Grall
xenheap_mfn_end is storing an MFN and not a physical address. Thankfully xenheap_mfn_end is not used in the arm64 code. So drop it. Signed-off-by: Julien Grall --- xen/arch/arm/setup.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/xen/arch/arm/setup.c