Re: [PATCH v3 0/6] Add kselftest_harness.h

From: Shuah Khan
Date: Thu May 04 2017 - 09:58:47 EST


On 05/03/2017 04:26 PM, MickaÃl SalaÃn wrote:
> Hi,
>
> This third patch series make the seccomp/test_harness.h more generally
> available [1] and update the kselftest documentation with the Sphinx format. It
> also improve the Makefile of seccomp tests to take into account any
> kselftest_harness.h update.

Thanks for the test harness as well as updating kselftest documentation
to rst format. Awesome. I will merge these into linux-kselftest after
the 4.12-rc1 comes out.

I will have to defer to Jon Corbet for Documentation related changes
and patches. Jon! Could you please review and give me an Ack.

>
> [1] https://lkml.kernel.org/r/CAGXu5j+8CVz8vL51DRYXqOY=xc3zuKFf=PTENe88XYHzFYidUQ@xxxxxxxxxxxxxx
>
> Regards,
>
> MickaÃl SalaÃn (6):
> selftests: Make test_harness.h more generally available
> selftests: Cosmetic renames in kselftest_harness.h
> selftests/seccomp: Force rebuild according to dependencies
> Documentation/dev-tools: Add kselftest
> Documentation/dev-tools: Use reStructuredText markups for kselftest
> Documentation/dev-tools: Add kselftest_harness documentation
>
> Documentation/00-INDEX | 2 -
> Documentation/dev-tools/index.rst | 1 +
> .../{kselftest.txt => dev-tools/kselftest.rst} | 125 +++++++---
> MAINTAINERS | 1 +
> .../test_harness.h => kselftest_harness.h} | 270 +++++++++++++++------
> tools/testing/selftests/seccomp/Makefile | 2 +
> tools/testing/selftests/seccomp/seccomp_bpf.c | 2 +-
> 7 files changed, 296 insertions(+), 107 deletions(-)
> rename Documentation/{kselftest.txt => dev-tools/kselftest.rst} (50%)
> rename tools/testing/selftests/{seccomp/test_harness.h => kselftest_harness.h} (80%)
>

thanks,
-- Shuah