linux-kernel - Re: [PATCH v1 14/17] Documentation: kunit: add documentation for KUnit

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAFd5g45CD7=dMvvRgmLKomirXDWpO59hFSed9A=o0kdBxp7v-w@mail.gmail.com>
Date:   Thu, 25 Apr 2019 14:37:11 -0700
From:   Brendan Higgins <brendanhiggins@...gle.com>
To:     Masayoshi Mizuma <msys.mizuma@...il.com>
Cc:     Jonathan Corbet <corbet@....net>,
        Frank Rowand <frowand.list@...il.com>,
        Kees Cook <keescook@...gle.com>,
        Kieran Bingham <kieran.bingham@...asonboard.com>,
        Luis Chamberlain <mcgrof@...nel.org>,
        Rob Herring <robh@...nel.org>, shuah@...nel.org,
        yamada.masahiro@...ionext.com, Petr Mladek <pmladek@...e.com>,
        linux-doc@...r.kernel.org, Amir Goldstein <amir73il@...il.com>,
        dri-devel <dri-devel@...ts.freedesktop.org>,
        Sasha Levin <Alexander.Levin@...rosoft.com>,
        linux-kselftest@...r.kernel.org,
        linux-nvdimm <linux-nvdimm@...ts.01.org>,
        Kevin Hilman <khilman@...libre.com>,
        Knut Omang <knut.omang@...cle.com>,
        Felix Guo <felixguoxiuping@...il.com>, wfg@...ux.intel.com,
        Joel Stanley <joel@....id.au>, Jeff Dike <jdike@...toit.com>,
        Dan Carpenter <dan.carpenter@...cle.com>,
        devicetree <devicetree@...r.kernel.org>,
        linux-kbuild@...r.kernel.org, "Bird, Timothy" <Tim.Bird@...y.com>,
        linux-um@...ts.infradead.org, Steven Rostedt <rostedt@...dmis.org>,
        Julia Lawall <julia.lawall@...6.fr>,
        kunit-dev@...glegroups.com, Richard Weinberger <richard@....at>,
        Greg KH <gregkh@...uxfoundation.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Daniel Vetter <daniel@...ll.ch>,
        Michael Ellerman <mpe@...erman.id.au>,
        linux-fsdevel@...r.kernel.org
Subject: Re: [PATCH v1 14/17] Documentation: kunit: add documentation for KUnit

On Wed, Apr 24, 2019 at 11:58 AM Masayoshi Mizuma <msys.mizuma@...il.com> wrote:
>
> Hi Brendan,
>
> KUNIT_ASSERT_NOT_ERR_OR_NULL() should be replaced to
> KUNIT_EXPECT_NOT_ERR_OR_NULL(), right?

Generally speaking, no.

There may be places that I have used it improperly, but I think it is
generally okay to use.

If you are refering to the discussion with Frank and Stephen in RFC
v4, my take was that the resolution was that there were some things I
needed to fix in its implementation, but it is conceptually okay, and
that Frank doesn't want me to use it in drivers/of/unittest.c.
Consequently, I fixed the implementation issues that Frank and Stephen
pointed out, and I am holding off on drivers/of/unittest.c for this
patchset. The usage in this file should be fine.

>
> On Thu, Apr 04, 2019 at 03:06:49PM -0700, Brendan Higgins wrote:
> > Add documentation for KUnit, the Linux kernel unit testing framework.
> > - Add intro and usage guide for KUnit
> > - Add API reference
> >
> > Signed-off-by: Felix Guo <felixguoxiuping@...il.com>
> > Signed-off-by: Brendan Higgins <brendanhiggins@...gle.com>
> > ---
>
> <snip>

<snip>

Cheers