From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932263AbbAESVz (ORCPT ); Mon, 5 Jan 2015 13:21:55 -0500 Received: from gw1.transmode.se ([195.58.98.146]:59463 "EHLO gw1.transmode.se" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753303AbbAESVx convert rfc822-to-8bit (ORCPT ); Mon, 5 Jan 2015 13:21:53 -0500 From: Joakim Tjernlund To: "christophe.leroy@c-s.fr" CC: "scottwood@freescale.com" , "paulus@samba.org" , "mpe@ellerman.id.au" , "benh@kernel.crashing.org" , "linux-kernel@vger.kernel.org" , "linuxppc-dev@lists.ozlabs.org" Subject: Re: [PATCH 11/11] powerpc/8xx: Add support for TASK_SIZE greater than 0x80000000 Thread-Topic: [PATCH 11/11] powerpc/8xx: Add support for TASK_SIZE greater than 0x80000000 Thread-Index: AQHQGUGA+jZMVazuK0CDbf6TNIDjdpyx5ayA Date: Mon, 5 Jan 2015 18:21:51 +0000 Message-ID: <1420482111.25047.22.camel@transmode.se> References: <20141216150340.55A031A5E07@localhost.localdomain> In-Reply-To: <20141216150340.55A031A5E07@localhost.localdomain> Accept-Language: en-US, sv-SE Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [192.168.200.4] Content-Type: text/plain; charset=US-ASCII Content-ID: Content-Transfer-Encoding: 7BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2014-12-16 at 16:03 +0100, Christophe Leroy wrote: > By default, TASK_SIZE is set to 0x80000000 for PPC_8xx, which is most likely > sufficient for most cases. However, kernel configuration allows to set TASK_SIZE > to another value, so the 8xx shall handle it. On 8xx I would be just as happy with not supporting different TASK_SIZEs, but why not .. Acked-by: Joakim Tjernlund > > Signed-off-by: Christophe Leroy > > --- > arch/powerpc/kernel/head_8xx.S | 29 +++++++++++++++++++++-------- > 1 file changed, 21 insertions(+), 8 deletions(-) > > diff --git a/arch/powerpc/kernel/head_8xx.S b/arch/powerpc/kernel/head_8xx.S > index dbe110e..d380658 100644 > --- a/arch/powerpc/kernel/head_8xx.S > +++ b/arch/powerpc/kernel/head_8xx.S > @@ -48,6 +48,19 @@ > mtspr spr, reg > #endif > > +/* Macro to test if an address is a kernel address */ > +#if CONFIG_TASK_SIZE <= 0x80000000 > +#define IS_KERNEL(tmp, addr) \ > + andis. tmp, addr, 0x8000/* Address >= 0x80000000 */ > +#define BRANCH_UNLESS_KERNEL(label) beq label > +#else > +#define IS_KERNEL(tmp, addr) \ > + rlwinm tmp, addr, 16, 16, 31;\ > + cmpli cr0, tmp, PAGE_OFFSET >> 16 > +#define BRANCH_UNLESS_KERNEL(label) blt label > +#endif > + > + > /* > * Value for the bits that have fixed value in RPN entries. > * Also used for tagging DAR for DTLBerror. > @@ -323,15 +336,15 @@ InstructionTLBMiss: > mfspr r10, SPRN_SRR0/* Get effective address of fault */ > DO_8xx_CPU15(r11, r10) > mfcr r3 > - andis. r11, r10, 0x8000/* Address >= 0x80000000 */ > + IS_KERNEL(r11, r10) > #else > mfspr r11, SPRN_SRR0/* Get effective address of fault */ > DO_8xx_CPU15(r10, r11) > mfcr r10 > - andis. r11, r11, 0x8000/* Address >= 0x80000000 */ > + IS_KERNEL(r11, r11) > #endif > mfspr r11, SPRN_M_TW/* Get level 1 table base address */ > - beq 3f > + BRANCH_UNLESS_KERNEL(3f) > lis r11, (swapper_pg_dir-PAGE_OFFSET)@ha > 3: > #ifdef CONFIG_8xx_CPU6 > @@ -390,14 +403,14 @@ DataStoreTLBMiss: > mtspr SPRN_SPRG_SCRATCH2, r3 > mfcr r3 > mfspr r10, SPRN_MD_EPN > - andis. r11, r10, 0x8000 > + IS_KERNEL(r11, r10) > #else > mfcr r10 > mfspr r11, SPRN_MD_EPN > - andis. r11, r11, 0x8000 > + IS_KERNEL(r11, r11) > #endif > mfspr r11, SPRN_M_TW/* Get level 1 table base address */ > - beq 3f > + BRANCH_UNLESS_KERNEL(3f) > lis r11, (swapper_pg_dir-PAGE_OFFSET)@ha > 3: > #ifdef CONFIG_8xx_CPU6 > @@ -536,9 +549,9 @@ FixupDAR:/* Entry point for dcbx workaround. */ > mtspr SPRN_SPRG_SCRATCH2, r10 > /* fetch instruction from memory. */ > mfspr r10, SPRN_SRR0 > - andis. r11, r10, 0x8000/* Address >= 0x80000000 */ > + IS_KERNEL(r11, r10) > mfspr r11, SPRN_M_TW/* Get level 1 table base address */ > - beq 3f > + BRANCH_UNLESS_KERNEL(3f) > lis r11, (swapper_pg_dir-PAGE_OFFSET)@ha > /* Insert level 1 index */ > 3: rlwimi r11, r10, 32 - ((PAGE_SHIFT - 2) << 1), (PAGE_SHIFT - 2) << 1, 29