From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756208AbbLANGV (ORCPT ); Tue, 1 Dec 2015 08:06:21 -0500 Received: from mail-db3on0066.outbound.protection.outlook.com ([157.55.234.66]:44160 "EHLO emea01-db3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754256AbbLANGP (ORCPT ); Tue, 1 Dec 2015 08:06:15 -0500 Authentication-Results: spf=fail (sender IP is 212.179.42.66) smtp.mailfrom=ezchip.com; vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=none action=none header.from=ezchip.com; From: Noam Camus To: CC: , , Noam Camus Subject: [PATCH v3 05/18] ARC: Set vmalloc size from configuration Date: Tue, 1 Dec 2015 15:02:52 +0200 Message-ID: <1448974985-11487-6-git-send-email-noamc@ezchip.com> X-Mailer: git-send-email 1.7.1 In-Reply-To: <1448974985-11487-1-git-send-email-noamc@ezchip.com> References: <1448974985-11487-1-git-send-email-noamc@ezchip.com> MIME-Version: 1.0 Content-Type: text/plain X-TM-AS-Product-Ver: SMEX-11.0.0.1191-8.000.1202-21974.007 X-TM-AS-Result: No--9.768900-8.000000-31 X-TM-AS-User-Approved-Sender: No X-TM-AS-User-Blocked-Sender: No X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1;AM1FFO11FD031;1:gcHJtt2jOXUlSOOmrQBbj1t3JuZ+KqLrg7Uw5Q7hS1IRvsb2h5yX0fwZns6T9ZLEzRuE6pqW7MRmoKJi3+sM3BZ9qBTUuN9eRi/1EeGbRq0V+PrpJCGEv4T7aGysjFJM21ONmmgPp6jCS0f+x7xqVOw/QSX7E1IKuEMEhkHjm6IX0ODG3eZ77qFyW4vaRBt/wY1pzuROCD1G1vzofpoumAayQDMPypSQLFqE+x02p8A9T7/qXZDi62kshQayFJ174gnhpNA77vRF9vd7GUEV3Ff8IBhlG0KpJpk8ae2c0UT3AK/bexl3yvFn5Puov1znUUNdQWVb5MU6l2yui9gHZEMmk+DEiKoCJWoviMvnPtuTpgoeRjXmf1P1zI+GAfQxPIGaFBOrQlgoO+bHuH3Uhrzg0uUPee7dOPd3/pT2dh0= X-Forefront-Antispam-Report: CIP:212.179.42.66;CTRY:IL;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(2980300002)(1109001)(1110001)(339900001)(189002)(199003)(92566002)(76176999)(110136002)(189998001)(77096005)(86362001)(6806005)(50986999)(19580405001)(107886002)(2351001)(33646002)(1220700001)(5008740100001)(229853001)(50466002)(85426001)(19580395003)(104016004)(1096002)(47776003)(87936001)(11100500001)(5001970100001)(5003940100001)(49486002)(4001430100002)(105606002)(106466001)(50226001)(36756003)(48376002)(2950100001)(586003);DIR:OUT;SFP:1101;SCL:1;SRVR:DB4PR02MB192;H:ezex10.ezchip.com;FPR:;SPF:Fail;PTR:ezmail.ezchip.com;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;DB4PR02MB192;2:MsLIDHmd0zyBsQeodS2l5kBszdODZcSIYpB/+X8hD3HLQU/V4CDDImlCmz46+iDo2s0kkFb9tqlME4QJ4zUWewenU1fpGp6QiPiO4X+KYtiF3OxE2yS8o2tZ6tnmokZ4QsOTE5MO8EY3emLj9OQD2w==;3:DGVoG4G20FPfLaJviYcf7Atd3f+ijj/GDp7b1kDBeRlG4OE/OCBKYOQfoMylLfP4XYqG7WpZcuJteLuWWr6TtUKdPcpemvjX2o0GN5pPx2yxxfovog+Nw5AD1R0rUKzE9gZtRvjKpVePElTadld/fh0pWsOakEoHS1xxzKdf1gl7tbfDRiqKKF3mVhms3WN6/7JjRYKZc3Y0hqS5wccUGzzUYD5KXPUs9j6FxyEvRME=;25:J7QuM8+CDAI9vPXDtKO9DQPmbU7VhdzsNP+pMQxf1vW40R5Zs1c891orV1Ci+Xxvr4npa/h2QJda4Mk1tw2s62wb6K781mRakcQ7NjvUAeish3kGKh3PW70jDkjWxDWqTyxoGKrdpUt78Tobp+qeANY5MViY+8r/oxlF84afdI0HkYNzOV2gYPF+0YjiCojiE8egZVvp6SNhcIw/F+Gso9ML2I7QKc19nuQhvLDUD6TfQ7gfslblDYQh/oDoSW2Et69Wkc8/y2CFKScGW0B1Hg==;20:kqSa+yZWWggtq6NMtI6p0JQ6pZb2GEIssJ3Sn03VVxB7QASSbifaliyo//YHq9cxripwccmjF1B6iD4JLV8uUeBiZH9aiNQ6HfzgelCFvwXzTPZPWpCEBxe0atoyMuJnmaG2a/jk9ArM3RgPXZux1UXyIp1o6itoGI3DxnQoMlI= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DB4PR02MB192; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(121898900299872); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(8121501046)(520078)(5005006)(10201501046)(3002001);SRVR:DB4PR02MB192;BCL:0;PCL:0;RULEID:;SRVR:DB4PR02MB192; X-Microsoft-Exchange-Diagnostics: 1;DB4PR02MB192;4:hHX/Sposo3RCV3xHHofA7xyCd3Wf9qbUzMLYYNvD3F2KGeQ3+BBjBm+LW9yB2WRo8rH3iZzfHQZQZRflLp1EWxe7bHid7ITiwSY8xvjQFFYyjvZkbw2lGT5y+EVYaCMu5S1zsmxSRZS0O9tPQAbT8i3nf35lngOtKuCjHGcHiAeOw9TOwC6jmFa6dkSAU7RE6TtwGYDT/rIpNlxCyZXczhzwyOEmHNYVCTSWk82A3WdDz6AZVgBrBG51y6UDpV/RSwY9GQxM6tFwfjBX+ksxBM5jr62xsbtQtfHxXF0XjWIRAZIZwqIgDwH8daFGHv72GXISWv9+1YHbUZouO89hIl7MTgSSkk7wbbbhSN7/tDtNvEn3WFwXIfDwEf8NjNdwSXGFeFST41HBiPVuzTILJq00hlZ0HbsRvvdrpE43rmXQ7yxpacU2WhFlW2wWqcjP X-Forefront-PRVS: 07778E4001 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;DB4PR02MB192;23:ULLcVE8j34v2FH8XnYXnsLrspyw4AoY2VVyA9NN5Jj?= =?us-ascii?Q?sQyn/huOQqYy+4Yxc0hOr7sE0mzaORmAH2EosiR7VC7FgNmT5iLGB9GVfdoI?= =?us-ascii?Q?ZPor5CtBTpEo8AQft9Er7QDVOF0Ia24qfuIfuxvKm+xpgnjslOQq6yPARG/H?= =?us-ascii?Q?OtKVJIzGtHzmkN77i/08o5XzqQxiDY93a7xW8ihShoUi+gE4pAmofoNwNPjh?= =?us-ascii?Q?D45D5Lz6/EPWkLD/lJTWVFj7gwFLphWDY8B8Iek8lTZiBJ0XOwmJUs6h736Y?= =?us-ascii?Q?ogRhDcLbfybZMlrAUKbHcgrnz+olHBtM3xb79rt9+0TM85KGRxvAtu6cCbHd?= =?us-ascii?Q?agBPExS6xt7x/MRT+G/3FWX29Z97HeY2AeuxQ8q1CqaSFY9s4sRDTJAKJV0l?= =?us-ascii?Q?ZxTsVo17w8O7UJbXwSqC65CaCfifIzWh0//w+xvRaWWlfH1lt6/eKr4eu2/u?= =?us-ascii?Q?UCsD6edPy717gnxY3I9rSOeFwhf8XspqrBPntDE9EBH9GnqdANaE5BT+upTD?= =?us-ascii?Q?qWkBuaABJtu+ZDwoQLOOvccwOy/w7/rTe83f8o6Ct9niqzkY2uFQK+ddhfUO?= =?us-ascii?Q?65UZ/YXob5yqEwJlkc2VNXvwEbfvIgKk09Y8t83iNZo1gW5kUaFv3C57+dto?= =?us-ascii?Q?xK5mzEEXrivPHhCOdFN+jodyOOSECp48cfnH12RKswO4WZ3m3aZS/YMRzcKK?= =?us-ascii?Q?Sk7j2wNuQnSVQd+blXn4ypxgUkAt6aFGOOsNGmxnqC42qsopfeI/2lOS2Aui?= =?us-ascii?Q?MZzG6PsnMjJ149rIcAOc3Jc4ztvEQbRLK12YWftzZBcNVt3s7AzaT+8iqDJk?= =?us-ascii?Q?Yx/opS9M2EObZRBc8hg+pMaz2+r1Rj8G8aHCxX3bhZgL7W+rxFiySIzKxMbq?= =?us-ascii?Q?9wJ4vkuJt9jWHJpvWEYPqDm9o4u0b0we/qEYs4rm/x063j8Bn/Aktj29QDxd?= =?us-ascii?Q?JYuixPnSmey2LLAFlBwiGyb6kK4P3KW7qnBmn1LEl5rkWGYkRsXv1wQRSnMl?= =?us-ascii?Q?v0e38bx7u3FFA0kqLGvV8fJytyQQ53UlGDqCxK+nGf38PSfgB5Pq69iIaYFA?= =?us-ascii?Q?GNHn5GJu5SdO13jR92HIV0gF9ygHy+dH9YvakQmWLPJzK+Pw=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1;DB4PR02MB192;5:2f3qs0p+dAMsSRf1kC4yKwWGmEXIhoGFaJ9OhB/E1/cc/jHedWVNZK3l4bgNpjFiMzbs8J9xBnEWtE+i4wVAhqdCqoEvV/lrcn2VqCnLsuvtx9AyGkrXDotYDCgUb+iEhBM1R0nSzeA/4UwaODeoyg==;24:l3kOdEfycycNQgLHEuFK4/iMMzFKSxM4NF6r1bpAmhYfs0ODaQEZ2rIthRXIAwIWZESsAJQPayNX3JMRaHSMwitfj+wN2+PPyc0e3MSvHgQ= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: ezchip.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Dec 2015 13:06:12.5426 (UTC) X-MS-Exchange-CrossTenant-Id: 0fc16e0a-3cd3-4092-8b2f-0a42cff122c3 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=0fc16e0a-3cd3-4092-8b2f-0a42cff122c3;Ip=[212.179.42.66];Helo=[ezex10.ezchip.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB4PR02MB192 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Noam Camus User space use lower 2G of the virtual address space. However kernel steals upper 512M of this space. This stolen space is used partially for vmalloc and the rest serves as gutter between kernel and user space. The vmalloc size is depend on NR_CPUS since "per cpu" mechanism use vmalloc to allocate chunks for its use. Historically vmalloc size was 256M however it is not enough in case of many CPUs e.g. 4K CPUs. For such case an extra 192M is allocated. We are setting default vmalloc size to be 256M, any addtional space will be taken from kernel/user gutter. This introduce build error since PGDIR_SIZE() cannot use "1UL" ibecause it is called from mm/tlbex.S by VMALLOC_START. Using _BITUL() solves this build error. Signed-off-by: Noam Camus Acked-by: Vineet Gupta --- arch/arc/Kconfig | 8 ++++++++ arch/arc/include/asm/pgtable.h | 2 +- arch/arc/include/asm/processor.h | 10 +++++----- arch/arc/mm/tlb.c | 6 ++++++ 4 files changed, 20 insertions(+), 6 deletions(-) diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig index 2c2ac3f..689ccb3 100644 --- a/arch/arc/Kconfig +++ b/arch/arc/Kconfig @@ -468,6 +468,14 @@ config ARCH_PHYS_ADDR_T_64BIT config ARCH_DMA_ADDR_T_64BIT bool +config ARC_VMALLOC_SIZE + hex "Vmalloc size (MB)" + range 0 512 + default "256" + help + By default equals to 256MB and the rest of 512MB is + left for gutter between kernel and user space. + config ARC_CURR_IN_REG bool "Dedicate Register r25 for current_task pointer" default y diff --git a/arch/arc/include/asm/pgtable.h b/arch/arc/include/asm/pgtable.h index 57af2f0..372a282 100644 --- a/arch/arc/include/asm/pgtable.h +++ b/arch/arc/include/asm/pgtable.h @@ -210,7 +210,7 @@ #define BITS_FOR_PGD (32 - BITS_FOR_PTE - BITS_IN_PAGE) #define PGDIR_SHIFT (32 - BITS_FOR_PGD) -#define PGDIR_SIZE (1UL << PGDIR_SHIFT) /* vaddr span, not PDG sz */ +#define PGDIR_SIZE _BITUL(PGDIR_SHIFT) /* vaddr span, not PDG sz */ #define PGDIR_MASK (~(PGDIR_SIZE-1)) #define PTRS_PER_PTE _BITUL(BITS_FOR_PTE) diff --git a/arch/arc/include/asm/processor.h b/arch/arc/include/asm/processor.h index 1d694c1..8a77ccd 100644 --- a/arch/arc/include/asm/processor.h +++ b/arch/arc/include/asm/processor.h @@ -109,18 +109,18 @@ extern unsigned int get_wchan(struct task_struct *p); * 0xC000_0000 0xFFFF_FFFF (peripheral uncached space) * ----------------------------------------------------------------------------- */ -#define VMALLOC_START 0x70000000 /* * 1 PGDIR_SIZE each for fixmap/pkmap, 2 PGDIR_SIZE gutter * See asm/highmem.h for details */ -#define VMALLOC_SIZE (PAGE_OFFSET - VMALLOC_START - PGDIR_SIZE * 4) -#define VMALLOC_END (VMALLOC_START + VMALLOC_SIZE) +#define VMALLOC_END (PAGE_OFFSET - PGDIR_SIZE * 4) +#define VMALLOC_SIZE (CONFIG_ARC_VMALLOC_SIZE << 20) +#define VMALLOC_START (VMALLOC_END - VMALLOC_SIZE) -#define USER_KERNEL_GUTTER 0x10000000 +#define TASK_SIZE 0x60000000 -#define TASK_SIZE (VMALLOC_START - USER_KERNEL_GUTTER) +#define USER_KERNEL_GUTTER (VMALLOC_START - TASK_SIZE) #define STACK_TOP TASK_SIZE #define STACK_TOP_MAX STACK_TOP diff --git a/arch/arc/mm/tlb.c b/arch/arc/mm/tlb.c index daf2bf5..843e1a5 100644 --- a/arch/arc/mm/tlb.c +++ b/arch/arc/mm/tlb.c @@ -814,6 +814,12 @@ void arc_mmu_init(void) printk(arc_mmu_mumbojumbo(0, str, sizeof(str))); + /* + * vmalloc size (in MB) sanity check, + * Can't be done in processor.h due to header include depenedencies + */ + BUILD_BUG_ON(!IS_ALIGNED((CONFIG_ARC_VMALLOC_SIZE << 20), PMD_SIZE)); + /* For efficiency sake, kernel is compile time built for a MMU ver * This must match the hardware it is running on. * Linux built for MMU V2, if run on MMU V1 will break down because V1 -- 1.7.1 From mboxrd@z Thu Jan 1 00:00:00 1970 From: noamc@ezchip.com (Noam Camus) Date: Tue, 1 Dec 2015 15:02:52 +0200 Subject: [PATCH v3 05/18] ARC: Set vmalloc size from configuration In-Reply-To: <1448974985-11487-1-git-send-email-noamc@ezchip.com> References: <1448974985-11487-1-git-send-email-noamc@ezchip.com> List-ID: Message-ID: <1448974985-11487-6-git-send-email-noamc@ezchip.com> To: linux-snps-arc@lists.infradead.org From: Noam Camus User space use lower 2G of the virtual address space. However kernel steals upper 512M of this space. This stolen space is used partially for vmalloc and the rest serves as gutter between kernel and user space. The vmalloc size is depend on NR_CPUS since "per cpu" mechanism use vmalloc to allocate chunks for its use. Historically vmalloc size was 256M however it is not enough in case of many CPUs e.g. 4K CPUs. For such case an extra 192M is allocated. We are setting default vmalloc size to be 256M, any addtional space will be taken from kernel/user gutter. This introduce build error since PGDIR_SIZE() cannot use "1UL" ibecause it is called from mm/tlbex.S by VMALLOC_START. Using _BITUL() solves this build error. Signed-off-by: Noam Camus Acked-by: Vineet Gupta --- arch/arc/Kconfig | 8 ++++++++ arch/arc/include/asm/pgtable.h | 2 +- arch/arc/include/asm/processor.h | 10 +++++----- arch/arc/mm/tlb.c | 6 ++++++ 4 files changed, 20 insertions(+), 6 deletions(-) diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig index 2c2ac3f..689ccb3 100644 --- a/arch/arc/Kconfig +++ b/arch/arc/Kconfig @@ -468,6 +468,14 @@ config ARCH_PHYS_ADDR_T_64BIT config ARCH_DMA_ADDR_T_64BIT bool +config ARC_VMALLOC_SIZE + hex "Vmalloc size (MB)" + range 0 512 + default "256" + help + By default equals to 256MB and the rest of 512MB is + left for gutter between kernel and user space. + config ARC_CURR_IN_REG bool "Dedicate Register r25 for current_task pointer" default y diff --git a/arch/arc/include/asm/pgtable.h b/arch/arc/include/asm/pgtable.h index 57af2f0..372a282 100644 --- a/arch/arc/include/asm/pgtable.h +++ b/arch/arc/include/asm/pgtable.h @@ -210,7 +210,7 @@ #define BITS_FOR_PGD (32 - BITS_FOR_PTE - BITS_IN_PAGE) #define PGDIR_SHIFT (32 - BITS_FOR_PGD) -#define PGDIR_SIZE (1UL << PGDIR_SHIFT) /* vaddr span, not PDG sz */ +#define PGDIR_SIZE _BITUL(PGDIR_SHIFT) /* vaddr span, not PDG sz */ #define PGDIR_MASK (~(PGDIR_SIZE-1)) #define PTRS_PER_PTE _BITUL(BITS_FOR_PTE) diff --git a/arch/arc/include/asm/processor.h b/arch/arc/include/asm/processor.h index 1d694c1..8a77ccd 100644 --- a/arch/arc/include/asm/processor.h +++ b/arch/arc/include/asm/processor.h @@ -109,18 +109,18 @@ extern unsigned int get_wchan(struct task_struct *p); * 0xC000_0000 0xFFFF_FFFF (peripheral uncached space) * ----------------------------------------------------------------------------- */ -#define VMALLOC_START 0x70000000 /* * 1 PGDIR_SIZE each for fixmap/pkmap, 2 PGDIR_SIZE gutter * See asm/highmem.h for details */ -#define VMALLOC_SIZE (PAGE_OFFSET - VMALLOC_START - PGDIR_SIZE * 4) -#define VMALLOC_END (VMALLOC_START + VMALLOC_SIZE) +#define VMALLOC_END (PAGE_OFFSET - PGDIR_SIZE * 4) +#define VMALLOC_SIZE (CONFIG_ARC_VMALLOC_SIZE << 20) +#define VMALLOC_START (VMALLOC_END - VMALLOC_SIZE) -#define USER_KERNEL_GUTTER 0x10000000 +#define TASK_SIZE 0x60000000 -#define TASK_SIZE (VMALLOC_START - USER_KERNEL_GUTTER) +#define USER_KERNEL_GUTTER (VMALLOC_START - TASK_SIZE) #define STACK_TOP TASK_SIZE #define STACK_TOP_MAX STACK_TOP diff --git a/arch/arc/mm/tlb.c b/arch/arc/mm/tlb.c index daf2bf5..843e1a5 100644 --- a/arch/arc/mm/tlb.c +++ b/arch/arc/mm/tlb.c @@ -814,6 +814,12 @@ void arc_mmu_init(void) printk(arc_mmu_mumbojumbo(0, str, sizeof(str))); + /* + * vmalloc size (in MB) sanity check, + * Can't be done in processor.h due to header include depenedencies + */ + BUILD_BUG_ON(!IS_ALIGNED((CONFIG_ARC_VMALLOC_SIZE << 20), PMD_SIZE)); + /* For efficiency sake, kernel is compile time built for a MMU ver * This must match the hardware it is running on. * Linux built for MMU V2, if run on MMU V1 will break down because V1 -- 1.7.1