Re: [RFC PATCH v2] Documentation: dev-tools: Add KTAP specification

From: Shuah Khan
Date: Fri Dec 10 2021 - 16:11:44 EST


On 12/10/21 2:05 PM, Jonathan Corbet wrote:
Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx> writes:

On 12/7/21 3:02 PM, Kees Cook wrote:
On Tue, Dec 07, 2021 at 11:02:51AM -0800, David Gow wrote:
From: Rae Moar <rmoar@xxxxxxxxxx>

It does not make any significant additions or changes other than those
already in use in the kernel: additional features can be added as they
become necessary and used.

[1]: https://testanything.org/tap-version-13-specification.html

Signed-off-by: Rae Moar <rmoar@xxxxxxxxxx>
Co-developed-by: David Gow <davidgow@xxxxxxxxxx>
Signed-off-by: David Gow <davidgow@xxxxxxxxxx>

I like it! Thank you so much for suffering through my earlier reviews.
:)

The only concern I have is wonder what'll be needed to kselftest to
deal with indentation changes. As long as this can be implemented
without a subtest knowing it is a subtest, we're good.


A lot of this TAP output is in the wrappers - hopefully it will be okay.
Fingers crossed. :)

Reviewed-by: Kees Cook <keescook@xxxxxxxxxxxx>


Looks good to me as well. Thanks for doing this work.

Reviewed-by: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>

Would you like me to take this through the docs tree, or do you have
other plans for merging?


Please take this through docs.

thanks,
-- Shuah