Re: [PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-18 Thread Ard Biesheuvel
On 16 November 2016 at 17:32, Catalin Marinas wrote: > On Tue, Nov 15, 2016 at 04:09:07PM -0800, Laura Abbott wrote: >> On 11/15/2016 10:35 AM, Catalin Marinas wrote: >> > I'm fine with __pa_symbol use entirely from under arch/arm64. But if you >> > want to use __pa_symbol, I tried to change most

Re: [PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-16 Thread Catalin Marinas
On Tue, Nov 15, 2016 at 04:09:07PM -0800, Laura Abbott wrote: > On 11/15/2016 10:35 AM, Catalin Marinas wrote: > > I'm fine with __pa_symbol use entirely from under arch/arm64. But if you > > want to use __pa_symbol, I tried to change most (all?) places where > > necessary, together with making vir

Re: [PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-15 Thread Laura Abbott
On 11/15/2016 10:35 AM, Catalin Marinas wrote: > On Mon, Nov 14, 2016 at 10:41:29AM -0800, Laura Abbott wrote: >> On 11/14/2016 10:19 AM, Catalin Marinas wrote: >>> On Thu, Nov 03, 2016 at 03:51:07PM +, Mark Rutland wrote: On Wed, Nov 02, 2016 at 05:56:42PM -0600, Laura Abbott wrote: >

Re: [PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-15 Thread Catalin Marinas
On Mon, Nov 14, 2016 at 10:41:29AM -0800, Laura Abbott wrote: > On 11/14/2016 10:19 AM, Catalin Marinas wrote: > > On Thu, Nov 03, 2016 at 03:51:07PM +, Mark Rutland wrote: > >> On Wed, Nov 02, 2016 at 05:56:42PM -0600, Laura Abbott wrote: > >>> On 11/02/2016 04:52 PM, Mark Rutland wrote: > >>>

Re: [PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-14 Thread Laura Abbott
On 11/14/2016 10:19 AM, Catalin Marinas wrote: > On Thu, Nov 03, 2016 at 03:51:07PM +, Mark Rutland wrote: >> On Wed, Nov 02, 2016 at 05:56:42PM -0600, Laura Abbott wrote: >>> On 11/02/2016 04:52 PM, Mark Rutland wrote: On Wed, Nov 02, 2016 at 03:00:53PM -0600, Laura Abbott wrote: > >>

Re: [PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-14 Thread Catalin Marinas
On Thu, Nov 03, 2016 at 03:51:07PM +, Mark Rutland wrote: > On Wed, Nov 02, 2016 at 05:56:42PM -0600, Laura Abbott wrote: > > On 11/02/2016 04:52 PM, Mark Rutland wrote: > > >On Wed, Nov 02, 2016 at 03:00:53PM -0600, Laura Abbott wrote: > > >> > > >>__pa_symbol is technically the marco that sho

Re: [PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-03 Thread Mark Rutland
On Wed, Nov 02, 2016 at 05:56:42PM -0600, Laura Abbott wrote: > On 11/02/2016 04:52 PM, Mark Rutland wrote: > >On Wed, Nov 02, 2016 at 03:00:53PM -0600, Laura Abbott wrote: > >> > >>__pa_symbol is technically the marco that should be used for kernel > >>symbols. Switch to this as a pre-requisite fo

Re: [PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-02 Thread Laura Abbott
On 11/02/2016 04:52 PM, Mark Rutland wrote: On Wed, Nov 02, 2016 at 03:00:53PM -0600, Laura Abbott wrote: __pa_symbol is technically the marco that should be used for kernel symbols. Switch to this as a pre-requisite for DEBUG_VIRTUAL. Nit: s/marco/macro/ I see there are some other uses of _

Re: [PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-02 Thread Mark Rutland
On Wed, Nov 02, 2016 at 03:00:53PM -0600, Laura Abbott wrote: > > __pa_symbol is technically the marco that should be used for kernel > symbols. Switch to this as a pre-requisite for DEBUG_VIRTUAL. Nit: s/marco/macro/ I see there are some other uses of __pa() that look like they could/should be

[PATCHv2 5/6] arm64: Use __pa_symbol for _end

2016-11-02 Thread Laura Abbott
__pa_symbol is technically the marco that should be used for kernel symbols. Switch to this as a pre-requisite for DEBUG_VIRTUAL. Signed-off-by: Laura Abbott --- arch/arm64/mm/init.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/mm/init.c b/arch/arm64/mm/ini