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

[Shuah Khan]

On 07/18/2014 05:29 PM, Sam Ravnborg wrote:
> On Fri, Jul 18, 2014 at 02:44:34PM -0600, 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.
> >
> > Signed-off-by: Shuah Khan <shuah.kh@xxxxxxxxxxx>
> > ---
> >   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
> > +	make -C tools/testing/selftests run_tests
>
> The name "devtest" does not give any hints that we are actually
> running the suite of selftest programs.
>
> Is it so because the plan is to extend devtest to cover more than just selftest?
> If not please fix it so it is logical for the user what happens - in other
> words name is selftest or something like that.

Yes the intent is to cover more than just selftests in the future.
That is why the target is called devtest.

-- Shuah


--
Shuah Khan
Senior Linux Kernel Developer - Open Source Group
Samsung Research America(Silicon Valley)
shuah.kh@xxxxxxxxxxx | (970) 672-0658
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/