[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <202006151142.8B78AAE7D2@keescook>
Date: Mon, 15 Jun 2020 11:44:06 -0700
From: Kees Cook <keescook@...omium.org>
To: "Bird, Tim" <Tim.Bird@...y.com>
Cc: David Gow <davidgow@...gle.com>,
Vitor Massaru Iha <vitor@...saru.org>,
KUnit Development <kunit-dev@...glegroups.com>,
Shuah Khan <skhan@...uxfoundation.org>,
"open list:KERNEL SELFTEST FRAMEWORK"
<linux-kselftest@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Brendan Higgins <brendanhiggins@...gle.com>,
"linux-kernel-mentees@...ts.linuxfoundation.org"
<linux-kernel-mentees@...ts.linuxfoundation.org>,
"linux@...musvillemoes.dk" <linux@...musvillemoes.dk>
Subject: Re: RFC - kernel selftest result documentation (KTAP)
On Mon, Jun 15, 2020 at 05:45:28PM +0000, Bird, Tim wrote:
> Personally, as a human I find the space prefix a bit easier to read.
> However, I think that in "normal" kernel log output it is unusual for
> a line to be prefixed with a hash (#), so this might be easier to
> both visually distinguish and for automated parsing.
> So I'm torn. I don't have a strong opinion on space vs. hash prefix
> for indicating sub-test. I think the KUnit convention of matching
> whatever was the prefix of the "TAP version 14" line is clever, but
> it would be nice to just pick a prefix and be done with it.
Are there plans in kernelci for doing any parsing of subtests? (As in,
what do we break if we move from "# " to " "?)
I'm really thinking " " makes sense now. :)
--
Kees Cook
Powered by blists - more mailing lists