* [PATCH] Makefile: replace headers_install with headers for kselftest
@ 2022-07-13 6:33 Guillaume Tucker
2022-07-14 20:45 ` Shuah Khan
0 siblings, 1 reply; 6+ messages in thread
From: Guillaume Tucker @ 2022-07-13 6:33 UTC (permalink / raw)
To: Masahiro Yamada, Michal Marek, Nick Desaulniers, Shuah Khan, Kees Cook
Cc: Anders Roxell, Tim.Bird, kernel, linux-kbuild, linux-kernel,
linux-kselftest
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@collabora.com>
Reported-by: Masahiro Yamada <masahiroy@kernel.org>
---
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
--
2.30.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] Makefile: replace headers_install with headers for kselftest
2022-07-13 6:33 [PATCH] Makefile: replace headers_install with headers for kselftest Guillaume Tucker
@ 2022-07-14 20:45 ` Shuah Khan
2022-07-19 12:37 ` Guillaume Tucker
0 siblings, 1 reply; 6+ messages in thread
From: Shuah Khan @ 2022-07-14 20:45 UTC (permalink / raw)
To: Guillaume Tucker, Masahiro Yamada, Michal Marek,
Nick Desaulniers, Shuah Khan, Kees Cook
Cc: Anders Roxell, Tim.Bird, kernel, linux-kbuild, linux-kernel,
linux-kselftest, Shuah Khan
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@collabora.com>
> Reported-by: Masahiro Yamada <masahiroy@kernel.org>
> ---
> 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?
Acked-by: Shuah Khan <skhan@linuxfoundation.org>
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Makefile: replace headers_install with headers for kselftest
2022-07-14 20:45 ` Shuah Khan
@ 2022-07-19 12:37 ` Guillaume Tucker
2022-07-19 15:06 ` Masahiro Yamada
0 siblings, 1 reply; 6+ messages in thread
From: Guillaume Tucker @ 2022-07-19 12:37 UTC (permalink / raw)
To: Shuah Khan, Masahiro Yamada, Michal Marek, Nick Desaulniers,
Shuah Khan, Kees Cook
Cc: Anders Roxell, Tim.Bird, kernel, linux-kbuild, linux-kernel,
linux-kselftest
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@collabora.com>
>> Reported-by: Masahiro Yamada <masahiroy@kernel.org>
>> ---
>> 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@linuxfoundation.org>
>
> thanks,
> -- Shuah
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Makefile: replace headers_install with headers for kselftest
2022-07-19 12:37 ` Guillaume Tucker
@ 2022-07-19 15:06 ` Masahiro Yamada
2022-07-19 17:31 ` Guillaume Tucker
0 siblings, 1 reply; 6+ messages in thread
From: Masahiro Yamada @ 2022-07-19 15:06 UTC (permalink / raw)
To: Guillaume Tucker
Cc: Shuah Khan, Michal Marek, Nick Desaulniers, Shuah Khan,
Kees Cook, Anders Roxell, Tim Bird, kernel,
Linux Kbuild mailing list, Linux Kernel Mailing List,
open list:KERNEL SELFTEST FRAMEWORK
On Tue, Jul 19, 2022 at 9:37 PM Guillaume Tucker
<guillaume.tucker@collabora.com> wrote:
>
> 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@collabora.com>
> >> Reported-by: Masahiro Yamada <masahiroy@kernel.org>
> >> ---
> >> 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?
No.
This is a fix-up patch on top of the previous one [1],
which was applied to the kselftest tree.
This cannot apply to the kbuild tree.
[1]: https://lore.kernel.org/linux-kbuild/168ede35-12e0-c535-9d94-23b65a1beb28@collabora.com/T/#mb8ec7004f7c9b89fbb50c19022760e19701af7f6
> Best wishes,
> Guillaume
>
> > Acked-by: Shuah Khan <skhan@linuxfoundation.org>
> >
> > thanks,
> > -- Shuah
> >
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Makefile: replace headers_install with headers for kselftest
2022-07-19 15:06 ` Masahiro Yamada
@ 2022-07-19 17:31 ` Guillaume Tucker
2022-07-27 0:13 ` Shuah Khan
0 siblings, 1 reply; 6+ messages in thread
From: Guillaume Tucker @ 2022-07-19 17:31 UTC (permalink / raw)
To: Masahiro Yamada, Shuah Khan
Cc: Michal Marek, Nick Desaulniers, Shuah Khan, Kees Cook,
Anders Roxell, Tim Bird, kernel, Linux Kbuild mailing list,
Linux Kernel Mailing List, open list:KERNEL SELFTEST FRAMEWORK
On 19/07/2022 17:06, Masahiro Yamada wrote:
> On Tue, Jul 19, 2022 at 9:37 PM Guillaume Tucker
> <guillaume.tucker@collabora.com> wrote:
>>
>> 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@collabora.com>
>>>> Reported-by: Masahiro Yamada <masahiroy@kernel.org>
>>>> ---
>>>> 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?
>
>
> No.
>
> This is a fix-up patch on top of the previous one [1],
> which was applied to the kselftest tree.
>
> This cannot apply to the kbuild tree.
OK thank you for confirming.
Shuah, I guess you're happy to apply it to the kselftest tree
instead then?
Guillaume
> [1]: https://lore.kernel.org/linux-kbuild/168ede35-12e0-c535-9d94-23b65a1beb28@collabora.com/T/#mb8ec7004f7c9b89fbb50c19022760e19701af7f6
>
>
>
>
>
>
>> Best wishes,
>> Guillaume
>>
>>> Acked-by: Shuah Khan <skhan@linuxfoundation.org>
>>>
>>> thanks,
>>> -- Shuah
>>>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Makefile: replace headers_install with headers for kselftest
2022-07-19 17:31 ` Guillaume Tucker
@ 2022-07-27 0:13 ` Shuah Khan
0 siblings, 0 replies; 6+ messages in thread
From: Shuah Khan @ 2022-07-27 0:13 UTC (permalink / raw)
To: Guillaume Tucker, Masahiro Yamada
Cc: Michal Marek, Nick Desaulniers, Shuah Khan, Kees Cook,
Anders Roxell, Tim Bird, kernel, Linux Kbuild mailing list,
Linux Kernel Mailing List, open list:KERNEL SELFTEST FRAMEWORK,
Shuah Khan
On 7/19/22 11:31 AM, Guillaume Tucker wrote:
> On 19/07/2022 17:06, Masahiro Yamada wrote:
>> On Tue, Jul 19, 2022 at 9:37 PM Guillaume Tucker
>> <guillaume.tucker@collabora.com> wrote:
>>>
>>>
>>> Masahiro-san,
>>>
>>> A you OK with applying this in the kbuild tree ahead of the
>>> upcoming merge window?
>>
>>
>> No.
>>
>> This is a fix-up patch on top of the previous one [1],
>> which was applied to the kselftest tree.
>>
>> This cannot apply to the kbuild tree.
>
> OK thank you for confirming.
>
> Shuah, I guess you're happy to apply it to the kselftest tree
> instead then?
>
Sorry for a late response due to time off. Working through my Inbox.
Applied now to linux-kselftest next for 5.20-rc1.
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-07-27 0:13 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-13 6:33 [PATCH] Makefile: replace headers_install with headers for kselftest Guillaume Tucker
2022-07-14 20:45 ` Shuah Khan
2022-07-19 12:37 ` Guillaume Tucker
2022-07-19 15:06 ` Masahiro Yamada
2022-07-19 17:31 ` Guillaume Tucker
2022-07-27 0:13 ` Shuah Khan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).