Skip to content

Commit b3f7525

Browse files
torvaldsgregkh
authored andcommitted
Revert "mm: mmap: allow for the maximum number of bits for randomizing mmap_base by default"
commit 14d7c92 upstream. This reverts commit 3afb76a. This was a wrongheaded workaround for an issue that had already been fixed much better by commit 4ef9ad1 ("mm: huge_memory: don't force huge page alignment on 32 bit"). Asking users questions at kernel compile time that they can't make sense of is not a viable strategy. And the fact that even the kernel VM maintainers apparently didn't catch that this "fix" is not a fix any more pretty much proves the point that people can't be expected to understand the implications of the question. It may well be the case that we could improve things further, and that __thp_get_unmapped_area() should take the mapping randomization into account even for 64-bit kernels. Maybe we should not be so eager to use THP mappings. But in no case should this be a kernel config option. Cc: Rafael Aquini <[email protected]> Cc: Andrew Morton <[email protected]> Cc: Jiri Slaby <[email protected]> Cc: Suren Baghdasaryan <[email protected]> Cc: Matthew Wilcox (Oracle) <[email protected]> Signed-off-by: Linus Torvalds <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
1 parent 9de62e8 commit b3f7525

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

arch/Kconfig

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1037,21 +1037,10 @@ config ARCH_MMAP_RND_BITS_MAX
10371037
config ARCH_MMAP_RND_BITS_DEFAULT
10381038
int
10391039

1040-
config FORCE_MAX_MMAP_RND_BITS
1041-
bool "Force maximum number of bits to use for ASLR of mmap base address"
1042-
default y if !64BIT
1043-
help
1044-
ARCH_MMAP_RND_BITS and ARCH_MMAP_RND_COMPAT_BITS represent the number
1045-
of bits to use for ASLR and if no custom value is assigned (EXPERT)
1046-
then the architecture's lower bound (minimum) value is assumed.
1047-
This toggle changes that default assumption to assume the arch upper
1048-
bound (maximum) value instead.
1049-
10501040
config ARCH_MMAP_RND_BITS
10511041
int "Number of bits to use for ASLR of mmap base address" if EXPERT
10521042
range ARCH_MMAP_RND_BITS_MIN ARCH_MMAP_RND_BITS_MAX
10531043
default ARCH_MMAP_RND_BITS_DEFAULT if ARCH_MMAP_RND_BITS_DEFAULT
1054-
default ARCH_MMAP_RND_BITS_MAX if FORCE_MAX_MMAP_RND_BITS
10551044
default ARCH_MMAP_RND_BITS_MIN
10561045
depends on HAVE_ARCH_MMAP_RND_BITS
10571046
help
@@ -1086,7 +1075,6 @@ config ARCH_MMAP_RND_COMPAT_BITS
10861075
int "Number of bits to use for ASLR of mmap base address for compatible applications" if EXPERT
10871076
range ARCH_MMAP_RND_COMPAT_BITS_MIN ARCH_MMAP_RND_COMPAT_BITS_MAX
10881077
default ARCH_MMAP_RND_COMPAT_BITS_DEFAULT if ARCH_MMAP_RND_COMPAT_BITS_DEFAULT
1089-
default ARCH_MMAP_RND_COMPAT_BITS_MAX if FORCE_MAX_MMAP_RND_BITS
10901078
default ARCH_MMAP_RND_COMPAT_BITS_MIN
10911079
depends on HAVE_ARCH_MMAP_RND_COMPAT_BITS
10921080
help

0 commit comments

Comments
 (0)