On 14/07/2022 22:45, Shuah Khan wrote: > On 7/13/22 12:33 AM, Guillaume Tucker wrote: >> Replace headers_install with headers as kselftest uses the header >> files from within the kernel tree rather than from a system-wide >> installation. >> >> We can still run this directly: >> >> $ make O=build kselftest-all >> >> and when building from the selftests directory: >> >> $ make O=build headers >> $ make O=build -C tools/testing/selftests all >> >> Signed-off-by: Guillaume Tucker <guillaume.tucker@xxxxxxxxxxxxx> >> Reported-by: Masahiro Yamada <masahiroy@xxxxxxxxxx> >> --- >> Makefile | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/Makefile b/Makefile >> index fb2f3bb53a6b..5c934d16664c 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -1347,10 +1347,10 @@ tools/%: FORCE >> # Kernel selftest >> PHONY += kselftest >> -kselftest: headers_install >> +kselftest: headers >> $(Q)$(MAKE) -C $(srctree)/tools/testing/selftests run_tests >> -kselftest-%: headers_install FORCE >> +kselftest-%: headers FORCE >> $(Q)$(MAKE) -C $(srctree)/tools/testing/selftests $* >> PHONY += kselftest-merge >> > > Thank you for taking care of this. This will go through kbuild? You're welcome. Masahiro-san, A you OK with applying this in the kbuild tree ahead of the upcoming merge window? Best wishes, Guillaume > Acked-by: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx> > > thanks, > -- Shuah >
