diff options
author | Andrii Nakryiko <andrii@kernel.org> | 2020-12-10 17:59:46 -0800 |
---|---|---|
committer | Alexei Starovoitov <ast@kernel.org> | 2020-12-10 21:16:29 -0800 |
commit | a67079b03165a17f9aceab3dd26b1638af68e0fc (patch) | |
tree | 50eb1603f3a50f0849bff7cec78e12fb9c38f8bb /lib/crypto | |
parent | 092fde0f863b72b67c4d6dc03844f5658fc00a35 (diff) | |
download | linux-a67079b03165a17f9aceab3dd26b1638af68e0fc.tar.bz2 |
selftests/bpf: fix bpf_testmod.ko recompilation logic
bpf_testmod.ko build rule declared dependency on VMLINUX_BTF, but the variable
itself was initialized after the rule was declared, which often caused
bpf_testmod.ko to not be re-compiled. Fix by moving VMLINUX_BTF determination
sooner.
Also enforce bpf_testmod.ko recompilation when we detect that vmlinux image
changed by removing bpf_testmod/bpf_testmod.ko. This is necessary to generate
correct module's split BTF. Without it, Kbuild's module build logic might
determine that nothing changed on the kernel side and thus bpf_testmod.ko
shouldn't be rebuilt, so won't re-generate module BTF, which often leads to
module's BTF with wrong string offsets against vmlinux BTF. Removing .ko file
forces Kbuild to re-build the module.
Reported-by: Alexei Starovoitov <ast@kernel.org>
Fixes: 9f7fa225894c ("selftests/bpf: Add bpf_testmod kernel module for testing")
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/r/20201211015946.4062098-1-andrii@kernel.org
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'lib/crypto')
0 files changed, 0 insertions, 0 deletions