Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1634663

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

Show key headers only | View raw

On Tue, May 2, 2017 at 3:26 PM, Mickaël Salaün <mic@digikod.net> wrote:
> Hi,
>
> This second 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.
>
> [1] https://lkml.kernel.org/r/CAGXu5j+8CVz8vL51DRYXqOY=xc3zuKFf=PTENe88XYHzFYidUQ@mail.gmail.com
>
> 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

For these four:

Acked-by: Kees Cook <keescook@chromium.org>

>   Documentation/dev-tools: Use reStructuredText markups for kselftest
>   Documentation/dev-tools: Add kselftest_harness documentation

These two have some minor nits I emailed about, but if those are
fixed, consider them Acked-by me too.

Thanks for this!

-Kees

>
>  Documentation/00-INDEX                             |   2 -
>  Documentation/dev-tools/index.rst                  |   1 +
>  .../{kselftest.txt => dev-tools/kselftest.rst}     | 134 +++++++++--
>  MAINTAINERS                                        |   1 +
>  .../test_harness.h => kselftest_harness.h}         | 268 +++++++++++++++------
>  tools/testing/selftests/seccomp/Makefile           |   2 +
>  tools/testing/selftests/seccomp/seccomp_bpf.c      |   2 +-
>  7 files changed, 307 insertions(+), 103 deletions(-)
>  rename Documentation/{kselftest.txt => dev-tools/kselftest.rst} (52%)
>  rename tools/testing/selftests/{seccomp/test_harness.h => kselftest_harness.h} (81%)
>
> --
> 2.11.0
>



-- 
Kees Cook
Pixel Security

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/6] Add kselftest_harness.h Mickaël Salaün <mic@digikod.net> - 2017-05-03 00:30 +0200
  [PATCH v2 5/6] Documentation/dev-tools: Use reStructuredText markups for kselftest Mickaël Salaün <mic@digikod.net> - 2017-05-03 00:30 +0200
    Re: [PATCH v2 5/6] Documentation/dev-tools: Use reStructuredText  markups for kselftest Kees Cook <keescook@chromium.org> - 2017-05-03 01:20 +0200
  [PATCH v2 4/6] Documentation/dev-tools: Add kselftest Mickaël Salaün <mic@digikod.net> - 2017-05-03 00:30 +0200
  [PATCH v2 6/6] Documentation/dev-tools: Add kselftest_harness documentation Mickaël Salaün <mic@digikod.net> - 2017-05-03 00:30 +0200
    Re: [PATCH v2 6/6] Documentation/dev-tools: Add kselftest_harness documentation Kees Cook <keescook@chromium.org> - 2017-05-03 01:20 +0200
  [PATCH v2 3/6] selftests/seccomp: Force rebuild according to dependencies Mickaël Salaün <mic@digikod.net> - 2017-05-03 00:30 +0200
  [PATCH v2 1/6] selftests: Make test_harness.h more generally available Mickaël Salaün <mic@digikod.net> - 2017-05-03 00:30 +0200
  [PATCH v2 2/6] selftests: Cosmetic renames in kselftest_harness.h Mickaël Salaün <mic@digikod.net> - 2017-05-03 00:30 +0200
  Re: [PATCH v2 0/6] Add kselftest_harness.h Kees Cook <keescook@chromium.org> - 2017-05-03 01:20 +0200

csiph-web