Re: [PATCH] kbuild: devtest - new make target for build all and run tests

[Shuah Khan <shuah.kh@samsung.com>]

On 08/06/2014 07:51 AM, Michal Marek wrote:
> On 2014-07-18 22:44, Shuah Khan wrote:
>> Add a new devtest make target to enable developer testing. This
>> new target does full build (make all) and then runs selftests.
>
> I agree with Sam that the name is rather ambiguous, but I can't suggest
> a better alternative right now.

Does kselftest or kern_selftest sound better?

>
>
>> Signed-off-by: Shuah Khan <shuah.kh@samsung.com>
>> ---
>>   Makefile | 11 +++++++++++
>>   1 file changed, 11 insertions(+)
>>
>> diff --git a/Makefile b/Makefile
>> index f3c543d..1ef3128 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -1034,6 +1034,14 @@ headers_check: headers_install
>>   	$(Q)$(MAKE) $(hdr-inst)=arch/$(hdr-arch)/include/uapi/asm $(hdr-dst) HDRCHECK=1
>>
>>   # ---------------------------------------------------------------------------
>> +# Kernel devtest
>> +
>> +PHONY += devtest
>> +devtest:
>> +	make all
>
> devtest: all
>
>
>> +	make -C tools/testing/selftests run_tests
>
> Please use $(Q)$(MAKE), to keep the build silent by default and to
> consistently use the same make program with the same options throughout
> the build.
>

Working on the change - send v2 patch.

-- Shuah


-- 
Shuah Khan
Senior Linux Kernel Developer - Open Source Group
Samsung Research America(Silicon Valley)
shuah.kh@samsung.com | (970) 672-0658