| Message ID | 20220503231655.211346-1-broonie@kernel.org |
|---|---|
| State | Accepted |
| Commit | ae60e0763e97e977b03af1ac6ba782a4a86c3a5a |
| Headers | show |
| Series | kselftest/arm64: Fix ABI header directory location | expand |
On Wed, 4 May 2022 00:16:55 +0100, Mark Brown wrote: > Currently the arm64 kselftests attempt to locate the ABI headers using > custom logic which doesn't work correctly in the case of out of tree builds > if KBUILD_OUTPUT is not specified. Since lib.mk defines KHDR_INCLUDES with > the appropriate flags we can simply remove the custom logic and use that > instead. > > > [...] Applied to arm64 (for-next/kselftest), thanks! [1/1] kselftest/arm64: Fix ABI header directory location https://git.kernel.org/arm64/c/ae60e0763e97
diff --git a/tools/testing/selftests/arm64/Makefile b/tools/testing/selftests/arm64/Makefile index 1e8d9a8f59df..9460cbe81bcc 100644 --- a/tools/testing/selftests/arm64/Makefile +++ b/tools/testing/selftests/arm64/Makefile @@ -17,16 +17,7 @@ top_srcdir = $(realpath ../../../../) # Additional include paths needed by kselftest.h and local headers CFLAGS += -I$(top_srcdir)/tools/testing/selftests/ -# Guessing where the Kernel headers could have been installed -# depending on ENV config -ifeq ($(KBUILD_OUTPUT),) -khdr_dir = $(top_srcdir)/usr/include -else -# the KSFT preferred location when KBUILD_OUTPUT is set -khdr_dir = $(KBUILD_OUTPUT)/kselftest/usr/include -endif - -CFLAGS += -I$(khdr_dir) +CFLAGS += $(KHDR_INCLUDES) export CFLAGS export top_srcdir
Currently the arm64 kselftests attempt to locate the ABI headers using custom logic which doesn't work correctly in the case of out of tree builds if KBUILD_OUTPUT is not specified. Since lib.mk defines KHDR_INCLUDES with the appropriate flags we can simply remove the custom logic and use that instead. Signed-off-by: Mark Brown <broonie@kernel.org> --- This fix is required to get us able to run the arm64 kselftests in KernelCI, it does out of tree kselftest builds triggering the issue especially in conjunction with the addition of the new definitions for SME. tools/testing/selftests/arm64/Makefile | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-)