diff options
| author | Masahiro Yamada <masahiroy@kernel.org> | 2020-12-18 11:45:40 +0900 |
|---|---|---|
| committer | Will Deacon <will@kernel.org> | 2021-01-20 12:18:46 +0000 |
| commit | a5b8ca97fbf8300a5e21c393df25ce6f521e7939 (patch) | |
| tree | e408b44b8200420fb5e05c14edacbf566dbb503e /arch/arm64/kernel/vdso/vdso.S | |
| parent | 19c329f6808995b142b3966301f217c831e7cf31 (diff) | |
| download | linux-a5b8ca97fbf8300a5e21c393df25ce6f521e7939.tar.bz2 | |
arm64: do not descend to vdso directories twice
arm64 descends into each vdso directory twice; first in vdso_prepare,
second during the ordinary build process.
PPC mimicked it and uncovered a problem [1]. In the first descend,
Kbuild directly visits the vdso directories, therefore it does not
inherit subdir-ccflags-y from upper directories.
This means the command line parameters may differ between the two.
If it happens, the offset values in the generated headers might be
different from real offsets of vdso.so in the kernel.
This potential danger should be avoided. The vdso directories are
built in the vdso_prepare stage, so the second descend is unneeded.
[1]: https://lore.kernel.org/linux-kbuild/CAK7LNARAkJ3_-4gX0VA2UkapbOftuzfSTVMBbgbw=HD8n7N+7w@mail.gmail.com/T/#ma10dcb961fda13f36d42d58fa6cb2da988b7e73a
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Link: https://lore.kernel.org/r/20201218024540.1102650-1-masahiroy@kernel.org
Signed-off-by: Will Deacon <will@kernel.org>
Diffstat (limited to 'arch/arm64/kernel/vdso/vdso.S')
| -rw-r--r-- | arch/arm64/kernel/vdso/vdso.S | 24 |
1 files changed, 0 insertions, 24 deletions
diff --git a/arch/arm64/kernel/vdso/vdso.S b/arch/arm64/kernel/vdso/vdso.S deleted file mode 100644 index c4b1990bf2be..000000000000 --- a/arch/arm64/kernel/vdso/vdso.S +++ /dev/null @@ -1,24 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ -/* - * Copyright (C) 2012 ARM Limited - * - * Author: Will Deacon <will.deacon@arm.com> - */ - -#include <linux/init.h> -#include <linux/linkage.h> -#include <linux/const.h> -#include <asm/assembler.h> -#include <asm/page.h> - - .globl vdso_start, vdso_end - .section .rodata - .balign PAGE_SIZE -vdso_start: - .incbin "arch/arm64/kernel/vdso/vdso.so" - .balign PAGE_SIZE -vdso_end: - - .previous - -emit_aarch64_feature_1_and |