Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751066Ab2HLEFG (ORCPT ); Sun, 12 Aug 2012 00:05:06 -0400 Received: from mail-qa0-f46.google.com ([209.85.216.46]:53461 "EHLO mail-qa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750770Ab2HLEFC (ORCPT ); Sun, 12 Aug 2012 00:05:02 -0400 Date: Sun, 12 Aug 2012 00:04:59 -0400 (EDT) From: Nicolas Pitre To: Cyril Chemparathy cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, arnd@arndb.de, catalin.marinas@arm.com, grant.likely@secretlab.ca, linux@arm.linux.org.uk, will.deacon@arm.com, Vitaly Andrianov Subject: Re: [PATCH v2 10/22] ARM: LPAE: use phys_addr_t in switch_mm() In-Reply-To: <1344648306-15619-11-git-send-email-cyril@ti.com> Message-ID: References: <1344648306-15619-1-git-send-email-cyril@ti.com> <1344648306-15619-11-git-send-email-cyril@ti.com> User-Agent: Alpine 2.02 (LFD 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3001 Lines: 90 On Fri, 10 Aug 2012, Cyril Chemparathy wrote: > This patch modifies the switch_mm() processor functions to use phys_addr_t. > On LPAE systems, we now honor the upper 32-bits of the physical address that > is being passed in, and program these into TTBR as expected. > > Signed-off-by: Cyril Chemparathy > Signed-off-by: Vitaly Andrianov > --- > arch/arm/include/asm/proc-fns.h | 4 ++-- > arch/arm/mm/proc-v7-3level.S | 26 ++++++++++++++++++++++---- > 2 files changed, 24 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/include/asm/proc-fns.h b/arch/arm/include/asm/proc-fns.h > index f3628fb..75b5f14 100644 > --- a/arch/arm/include/asm/proc-fns.h > +++ b/arch/arm/include/asm/proc-fns.h > @@ -60,7 +60,7 @@ extern struct processor { > /* > * Set the page table > */ > - void (*switch_mm)(unsigned long pgd_phys, struct mm_struct *mm); > + void (*switch_mm)(phys_addr_t pgd_phys, struct mm_struct *mm); > /* > * Set a possibly extended PTE. Non-extended PTEs should > * ignore 'ext'. > @@ -82,7 +82,7 @@ extern void cpu_proc_init(void); > extern void cpu_proc_fin(void); > extern int cpu_do_idle(void); > extern void cpu_dcache_clean_area(void *, int); > -extern void cpu_do_switch_mm(unsigned long pgd_phys, struct mm_struct *mm); > +extern void cpu_do_switch_mm(phys_addr_t pgd_phys, struct mm_struct *mm); > #ifdef CONFIG_ARM_LPAE > extern void cpu_set_pte_ext(pte_t *ptep, pte_t pte); > #else > diff --git a/arch/arm/mm/proc-v7-3level.S b/arch/arm/mm/proc-v7-3level.S > index 8de0f1d..78bd88c 100644 > --- a/arch/arm/mm/proc-v7-3level.S > +++ b/arch/arm/mm/proc-v7-3level.S > @@ -39,6 +39,22 @@ > #define TTB_FLAGS_SMP (TTB_IRGN_WBWA|TTB_S|TTB_RGN_OC_WBWA) > #define PMD_FLAGS_SMP (PMD_SECT_WBWA|PMD_SECT_S) > > +#define rzero r3 > +#ifndef CONFIG_ARM_LPAE > +# define rpgdl r0 > +# define rpgdh rzero > +# define rmm r1 > +#else > +# define rmm r2 > +#ifndef __ARMEB__ > +# define rpgdl r0 > +# define rpgdh r1 > +#else > +# define rpgdl r1 > +# define rpgdh r0 > +#endif > +#endif Given proc-v7-3level.S is used only when CONFIG_ARM_LPAE is defined, you shouldn't need all the above. > /* > * cpu_v7_switch_mm(pgd_phys, tsk) > * > @@ -47,10 +63,12 @@ > */ > ENTRY(cpu_v7_switch_mm) > #ifdef CONFIG_MMU > - ldr r1, [r1, #MM_CONTEXT_ID] @ get mm->context.id > - and r3, r1, #0xff > - mov r3, r3, lsl #(48 - 32) @ ASID > - mcrr p15, 0, r0, r3, c2 @ set TTB 0 > + mov rzero, #0 > + ldr rmm, [rmm, #MM_CONTEXT_ID] @ get mm->context.id > + and rmm, rmm, #0xff > + mov rmm, rmm, lsl #(48 - 32) @ ASID > + orr rpgdh, rpgdh, rmm @ upper 32-bits of pgd phys > + mcrr p15, 0, rpgdl, rpgdh, c2 @ set TTB 0 > isb > #endif > mov pc, lr > -- > 1.7.9.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/