Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1342779 > unrolled thread

[PATCH] arm64/mm: remove unnecessary boundary check

Started by<miles.chen@mediatek.com>
First post2016-02-25 04:50 +0100
Last post2016-02-26 14:50 +0100
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] arm64/mm: remove unnecessary boundary check <miles.chen@mediatek.com> - 2016-02-25 04:50 +0100
    Re: [PATCH] arm64/mm: remove unnecessary boundary check Will Deacon <will.deacon@arm.com> - 2016-02-26 10:50 +0100
    Re: [PATCH] arm64/mm: remove unnecessary boundary check Catalin Marinas <catalin.marinas@arm.com> - 2016-02-26 14:50 +0100

#1342779 — [PATCH] arm64/mm: remove unnecessary boundary check

From<miles.chen@mediatek.com>
Date2016-02-25 04:50 +0100
Subject[PATCH] arm64/mm: remove unnecessary boundary check
Message-ID<r5V69-6mZ-1@gated-at.bofh.it>
From: Miles Chen <miles.chen@mediatek.com>

Remove the unnecessary boundary check since there is a huge
gap between user and kernel address that they would never overlap.
(arm64 does not have enough levels of page tables to cover 64-bit
virtual address)

See Documentation/arm64/memory.txt

Signed-off-by: Miles Chen <miles.chen@mediatek.com>
---
 arch/arm64/mm/init.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm64/mm/init.c b/arch/arm64/mm/init.c
index f3b061e..4579604 100644
--- a/arch/arm64/mm/init.c
+++ b/arch/arm64/mm/init.c
@@ -343,8 +343,6 @@ void __init mem_init(void)
 #ifdef CONFIG_COMPAT
 	BUILD_BUG_ON(TASK_SIZE_32			> TASK_SIZE_64);
 #endif
-	BUILD_BUG_ON(TASK_SIZE_64			> MODULES_VADDR);
-	BUG_ON(TASK_SIZE_64				> MODULES_VADDR);
 
 	if (PAGE_SIZE >= 16384 && get_num_physpages() <= 128) {
 		extern int sysctl_overcommit_memory;
-- 
1.9.1

[toc] | [next] | [standalone]


#1343965

FromWill Deacon <will.deacon@arm.com>
Date2016-02-26 10:50 +0100
Message-ID<r6nc6-17o-5@gated-at.bofh.it>
In reply to#1342779
On Thu, Feb 25, 2016 at 11:44:34AM +0800, miles.chen@mediatek.com wrote:
> From: Miles Chen <miles.chen@mediatek.com>
> 
> Remove the unnecessary boundary check since there is a huge
> gap between user and kernel address that they would never overlap.
> (arm64 does not have enough levels of page tables to cover 64-bit
> virtual address)
> 
> See Documentation/arm64/memory.txt
> 
> Signed-off-by: Miles Chen <miles.chen@mediatek.com>
> ---
>  arch/arm64/mm/init.c | 2 --
>  1 file changed, 2 deletions(-)

Acked-by: Will Deacon <will.deacon@arm.com>

Will

[toc] | [prev] | [next] | [standalone]


#1344257

FromCatalin Marinas <catalin.marinas@arm.com>
Date2016-02-26 14:50 +0100
Message-ID<r6qWm-3Zw-15@gated-at.bofh.it>
In reply to#1342779
On Thu, Feb 25, 2016 at 11:44:34AM +0800, miles.chen@mediatek.com wrote:
> From: Miles Chen <miles.chen@mediatek.com>
> 
> Remove the unnecessary boundary check since there is a huge
> gap between user and kernel address that they would never overlap.
> (arm64 does not have enough levels of page tables to cover 64-bit
> virtual address)
> 
> See Documentation/arm64/memory.txt
> 
> Signed-off-by: Miles Chen <miles.chen@mediatek.com>

Applied. Thanks.

-- 
Catalin

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web