[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20200821233848.3904680-1-ckennelly@google.com>
Date: Fri, 21 Aug 2020 19:38:46 -0400
From: Chris Kennelly <ckennelly@...gle.com>
To: Alexander Viro <viro@...iv.linux.org.uk>,
Alexey Dobriyan <adobriyan@...il.com>,
Song Liu <songliubraving@...com>
Cc: David Rientjes <rientjes@...gle.com>,
Ian Rogers <irogers@...gle.com>,
Hugh Dickens <hughd@...gle.com>,
Andrew Morton <akpm@...ux-foundation.org>,
Suren Baghdasaryan <surenb@...gle.com>,
Sandeep Patil <sspatil@...gle.com>,
Fangrui Song <maskray@...gle.com>,
Nick Desaulniers <ndesaulniers@...gle.com>,
clang-built-linux@...glegroups.com, linux-fsdevel@...r.kernel.org,
linux-kernel@...r.kernel.org, Chris Kennelly <ckennelly@...gle.com>
Subject: [PATCH v4 0/2] Selecting Load Addresses According to p_align
The current ELF loading mechancism provides page-aligned mappings. This
can lead to the program being loaded in a way unsuitable for
file-backed, transparent huge pages when handling PIE executables.
While specifying -z,max-page-size=0x200000 to the linker will generate
suitably aligned segments for huge pages on x86_64, the executable needs
to be loaded at a suitably aligned address as well. This alignment
requires the binary's cooperation, as distinct segments need to be
appropriately paddded to be eligible for THP.
For binaries built with increased alignment, this limits the number of
bits usable for ASLR, but provides some randomization over using fixed
load addresses/non-PIE binaries.
Changes V3 -> V4:
* Code tweaks based on on-thread feedback
* Addressed compiler warning
Changes V2 -> V3:
* Minor code tweaks based on off-thread feedback
Changes V1 -> V2:
* Added test
Chris Kennelly (2):
fs/binfmt_elf: Use PT_LOAD p_align values for suitable start address.
Add self-test for verifying load alignment.
fs/binfmt_elf.c | 25 ++++++++
tools/testing/selftests/exec/.gitignore | 1 +
tools/testing/selftests/exec/Makefile | 9 ++-
tools/testing/selftests/exec/load_address.c | 68 +++++++++++++++++++++
4 files changed, 101 insertions(+), 2 deletions(-)
create mode 100644 tools/testing/selftests/exec/load_address.c
--
2.28.0.297.g1956fa8f8d-goog
Powered by blists - more mailing lists