diff options
| author | Shuah Khan <skhan@linuxfoundation.org> | 2019-05-10 21:38:39 -0400 |
|---|---|---|
| committer | Shuah Khan <skhan@linuxfoundation.org> | 2019-05-13 12:19:43 -0400 |
| commit | 27d79a2b2bf0dbec1cc3d8ea269db4d5a0dac2f3 (patch) | |
| tree | cb0a4b547ceadb81dc10c093fa706997b3803f22 | |
| parent | c3c599281f2b9915200fe0390afe1ae7aca69feb (diff) | |
selftests: fix bpf build/test workflow regression when KBUILD_OUTPUT is set
commit 8ce72dc32578 ("selftests: fix headers_install circular dependency")
broke bpf build/test workflow. When KBUILD_OUTPUT is set, bpf objects end
up in KBUILD_OUTPUT build directory instead of in ../selftests/bpf.
The following bpf workflow breaks when it can't find the test_verifier:
cd tools/testing/selftests/bpf; make; ./test_verifier;
Fix it to set OUTPUT only when it is undefined in lib.mk. It didn't need
to be set in the first place.
Fixes: 8ce72dc32578 ("selftests: fix headers_install circular dependency")
Reported-by: Alexei Starovoitov <alexei.starovoitov@gmail.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
| -rw-r--r-- | tools/testing/selftests/lib.mk | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/tools/testing/selftests/lib.mk b/tools/testing/selftests/lib.mk index 098dd0065fb1..077337195783 100644 --- a/tools/testing/selftests/lib.mk +++ b/tools/testing/selftests/lib.mk | |||
| @@ -3,15 +3,9 @@ | |||
| 3 | CC := $(CROSS_COMPILE)gcc | 3 | CC := $(CROSS_COMPILE)gcc |
| 4 | 4 | ||
| 5 | ifeq (0,$(MAKELEVEL)) | 5 | ifeq (0,$(MAKELEVEL)) |
| 6 | ifneq ($(O),) | 6 | ifeq ($(OUTPUT),) |
| 7 | OUTPUT := $(O) | 7 | OUTPUT := $(shell pwd) |
| 8 | else | 8 | DEFAULT_INSTALL_HDR_PATH := 1 |
| 9 | ifneq ($(KBUILD_OUTPUT),) | ||
| 10 | OUTPUT := $(KBUILD_OUTPUT) | ||
| 11 | else | ||
| 12 | OUTPUT := $(shell pwd) | ||
| 13 | DEFAULT_INSTALL_HDR_PATH := 1 | ||
| 14 | endif | ||
| 15 | endif | 9 | endif |
| 16 | endif | 10 | endif |
| 17 | selfdir = $(realpath $(dir $(filter %/lib.mk,$(MAKEFILE_LIST)))) | 11 | selfdir = $(realpath $(dir $(filter %/lib.mk,$(MAKEFILE_LIST)))) |
