diff options
author | Hugh Dickins <hugh@veritas.com> | 2005-04-19 16:29:22 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@ppc970.osdl.org.(none)> | 2005-04-19 16:29:22 -0400 |
commit | cdfb82fff33cf3b1a367a427e5d89a012dc568b1 (patch) | |
tree | e06fe6c70972feac1c0ba1f6451c9b3d825c84c9 | |
parent | 6119be0bba10e7e7b1e5650a87812fa2fd8d3a60 (diff) |
[PATCH] freepgt: arm26 FIRST_USER_ADDRESS PAGE_SIZE
ARM26 define FIRST_USER_ADDRESS as PAGE_SIZE (beyond the machine vectors when
they are mapped low), and use that definition in place of locally defined
MIN_MAP_ADDR. Previously, ARM26 permitted user mappings at 0 if the machine
vectors were mapped high; but that's inconsistent with ARM, and
FIRST_USER_ADDRESS would then have to be determined at runtime. Let's fix it
at PAGE_SIZE throughout the architecture.
Signed-off-by: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | arch/arm26/kernel/sys_arm.c | 9 | ||||
-rw-r--r-- | include/asm-arm26/pgtable.h | 7 |
2 files changed, 11 insertions, 5 deletions
diff --git a/arch/arm26/kernel/sys_arm.c b/arch/arm26/kernel/sys_arm.c index e7edd201579a..85457897b8a9 100644 --- a/arch/arm26/kernel/sys_arm.c +++ b/arch/arm26/kernel/sys_arm.c | |||
@@ -64,10 +64,10 @@ inline long do_mmap2( | |||
64 | flags &= ~(MAP_EXECUTABLE | MAP_DENYWRITE); | 64 | flags &= ~(MAP_EXECUTABLE | MAP_DENYWRITE); |
65 | 65 | ||
66 | /* | 66 | /* |
67 | * If we are doing a fixed mapping, and address < PAGE_SIZE, | 67 | * If we are doing a fixed mapping, and address < FIRST_USER_ADDRESS, |
68 | * then deny it. | 68 | * then deny it. |
69 | */ | 69 | */ |
70 | if (flags & MAP_FIXED && addr < PAGE_SIZE && vectors_base() == 0) | 70 | if (flags & MAP_FIXED && addr < FIRST_USER_ADDRESS) |
71 | goto out; | 71 | goto out; |
72 | 72 | ||
73 | error = -EBADF; | 73 | error = -EBADF; |
@@ -121,11 +121,10 @@ sys_arm_mremap(unsigned long addr, unsigned long old_len, | |||
121 | unsigned long ret = -EINVAL; | 121 | unsigned long ret = -EINVAL; |
122 | 122 | ||
123 | /* | 123 | /* |
124 | * If we are doing a fixed mapping, and address < PAGE_SIZE, | 124 | * If we are doing a fixed mapping, and address < FIRST_USER_ADDRESS, |
125 | * then deny it. | 125 | * then deny it. |
126 | */ | 126 | */ |
127 | if (flags & MREMAP_FIXED && new_addr < PAGE_SIZE && | 127 | if (flags & MREMAP_FIXED && new_addr < FIRST_USER_ADDRESS) |
128 | vectors_base() == 0) | ||
129 | goto out; | 128 | goto out; |
130 | 129 | ||
131 | down_write(¤t->mm->mmap_sem); | 130 | down_write(¤t->mm->mmap_sem); |
diff --git a/include/asm-arm26/pgtable.h b/include/asm-arm26/pgtable.h index af0b8907dc14..4a0a00da425f 100644 --- a/include/asm-arm26/pgtable.h +++ b/include/asm-arm26/pgtable.h | |||
@@ -62,6 +62,13 @@ | |||
62 | #define PTRS_PER_PMD 1 | 62 | #define PTRS_PER_PMD 1 |
63 | #define PTRS_PER_PTE 32 | 63 | #define PTRS_PER_PTE 32 |
64 | 64 | ||
65 | /* | ||
66 | * This is the lowest virtual address we can permit any user space | ||
67 | * mapping to be mapped at. This is particularly important for | ||
68 | * non-high vector CPUs. | ||
69 | */ | ||
70 | #define FIRST_USER_ADDRESS PAGE_SIZE | ||
71 | |||
65 | #define FIRST_USER_PGD_NR 1 | 72 | #define FIRST_USER_PGD_NR 1 |
66 | #define USER_PTRS_PER_PGD ((TASK_SIZE/PGD_SIZE) - FIRST_USER_PGD_NR) | 73 | #define USER_PTRS_PER_PGD ((TASK_SIZE/PGD_SIZE) - FIRST_USER_PGD_NR) |
67 | 74 | ||