Re: [PATCH] docs/kselftest: add more guidelines for adding new tests
From: Muhammad Usama Anjum
Date: Sat May 21 2022 - 03:02:13 EST
On 3/18/22 9:50 AM, Bagas Sanjaya wrote:
>> + * Use lib.mk instead of writing Makefile from sratch. Specify flags and
>> + binaries generation flags on need basis before including lib.mk. ::
>> +
>> + CFLAGS = $(KHDR_INCLUDES)
>> + TEST_GEN_PROGS := close_range_test
>> + include ../lib.mk
>> +
>
> I think what you mean is "In your Makefile, use facilities from lib.mk by
> including it instead of reinventing the wheel.", right?
Yes, right.
>
>> + * Add new test name in TARGETS in selftests/Makefile::
>> +
>> + TARGETS += android
>> +
>> + * All of the following build commands should be successful
>> +
>> + - Same directory build of kselftests::
>> +
>> + make kselftest-all
>> + make kselftest-install
>> + make kselftest-clean
>> + make kselftest-gen_tar
>> +
>> + - Build with absolute output directory path::
>> +
>> + make kselftest-all O=/abs_build_path
>> + make kselftest-install O=/abs_build_path
>> + make kselftest-clean O=/abs_build_path
>> + make kselftest-gen_tar O=/abs_build_path
>> +
>> + - Build with relative output directory path::
>> +
>> + make kselftest-all O=relative_path
>> + make kselftest-install O=relative_path
>> + make kselftest-clean O=relative_path
>> + make kselftest-gen_tar O=relative_path
>> +
>> + - Build from Makefile of selftests directly::
>> +
>> + make -C tools/testing/selftests
>> + make -C tools/testing/selftests O=/abs_build_path
>> + make -C tools/testing/selftests O=relative_path
>> +
>
> For simplicity, we can say "All changes should pass
> kselftest-{all,install,clean,gen_tar} builds."
We (me and maintainer) want a list of commands to run before patch
submission. I'll use the {...} short hand method and update.
> You don't need to spell out full command-line in the guideline unless
> absolutely necessary, in general.
>
--
Muhammad Usama Anjum