Re: common KUnit Kconfig and file naming (was: Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions)

2020-06-19 Thread Brendan Higgins
On Thu, Jun 18, 2020 at 11:39 PM David Gow wrote: [...] > On Fri, Jun 19, 2020 at 4:28 AM Brendan Higgins > wrote: > > > > On Tue, Jun 16, 2020 at 9:21 PM David Gow wrote: > > > > > > On Tue, Jun 16, 2020 at 5:40 PM Alan Maguire > > > wrote: > > > > > > > > On Tue, 16 Jun 2020, David Gow wrote

Re: common KUnit Kconfig and file naming (was: Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions)

2020-06-18 Thread David Gow
I'm in the process of writing up some documentation for this now. I hope to post a draft soon, but here's the overview of what's going in it: - Test filenames should be _kunit.c - (If the suite name is too long/namespaced, the source filename may have prefixes removed, so long as the module name

Re: common KUnit Kconfig and file naming (was: Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions)

2020-06-18 Thread Kees Cook
On Thu, Jun 18, 2020 at 01:27:55PM -0700, Brendan Higgins wrote: > I am cool with changing *-test.c to *-kunit.c. The *-test.c was a hold I am fine with basically any decision as long as there's a single naming convention, *except* for this part. Dashes in source files creates confusion for module

Re: common KUnit Kconfig and file naming (was: Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions)

2020-06-18 Thread Brendan Higgins
On Tue, Jun 16, 2020 at 9:21 PM David Gow wrote: > > On Tue, Jun 16, 2020 at 5:40 PM Alan Maguire wrote: > > > > On Tue, 16 Jun 2020, David Gow wrote: > > > > > CONFIG_PM_QOS_KUNIT_TESTOn Mon, Jun 15, 2020 at 1:48 AM Kees Cook > > > wrote: > > > > > > > > On Sat, Jun 13, 2020 at 02:51:17PM +0800

Re: common KUnit Kconfig and file naming (was: Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions)

2020-06-16 Thread David Gow
On Tue, Jun 16, 2020 at 5:40 PM Alan Maguire wrote: > > On Tue, 16 Jun 2020, David Gow wrote: > > > CONFIG_PM_QOS_KUNIT_TESTOn Mon, Jun 15, 2020 at 1:48 AM Kees Cook > > wrote: > > > > > > On Sat, Jun 13, 2020 at 02:51:17PM +0800, David Gow wrote: > > > > Yeah, _KUNIT_TEST was what we've sort-of

Re: common KUnit Kconfig and file naming (was: Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions)

2020-06-16 Thread Alan Maguire
On Tue, 16 Jun 2020, David Gow wrote: > CONFIG_PM_QOS_KUNIT_TESTOn Mon, Jun 15, 2020 at 1:48 AM Kees Cook > wrote: > > > > On Sat, Jun 13, 2020 at 02:51:17PM +0800, David Gow wrote: > > > Yeah, _KUNIT_TEST was what we've sort-of implicitly decided on for > > > config names, but the documentation

Re: common KUnit Kconfig and file naming (was: Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions)

2020-06-16 Thread David Gow
CONFIG_PM_QOS_KUNIT_TESTOn Mon, Jun 15, 2020 at 1:48 AM Kees Cook wrote: > > On Sat, Jun 13, 2020 at 02:51:17PM +0800, David Gow wrote: > > Yeah, _KUNIT_TEST was what we've sort-of implicitly decided on for > > config names, but the documentation does need to happen. > > That works for me. It stil

Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions

2020-06-15 Thread Kees Cook
On Mon, Jun 15, 2020 at 01:30:42PM -0300, Vitor Massaru Iha wrote: > On Fri, 2020-06-12 at 15:36 -0700, Kees Cook wrote: > > Why drop the __initconst? > > I removed __initconst because of these warnings below, as it is used > for the kernel during the module initialization, and I do not use the >

Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions

2020-06-15 Thread Vitor Massaru Iha
On Sat, 2020-06-13 at 14:56 +0800, David Gow wrote: > On Fri, Jun 12, 2020 at 5:55 AM Vitor Massaru Iha > wrote: > > This adds the convertion of the runtime tests of check_*_overflow > > fuctions, > > from `lib/test_overflow.c`to KUnit tests. > > Nit: couple of minor typos here: convertion -> con

Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions

2020-06-15 Thread Vitor Massaru Iha
On Fri, 2020-06-12 at 15:36 -0700, Kees Cook wrote: > On Thu, Jun 11, 2020 at 06:55:01PM -0300, Vitor Massaru Iha wrote: > > This adds the convertion of the runtime tests of check_*_overflow > > fuctions, > > from `lib/test_overflow.c`to KUnit tests. > > > > The log similar to the one seen in dmes

common KUnit Kconfig and file naming (was: Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions)

2020-06-14 Thread Kees Cook
On Sat, Jun 13, 2020 at 02:51:17PM +0800, David Gow wrote: > Yeah, _KUNIT_TEST was what we've sort-of implicitly decided on for > config names, but the documentation does need to happen. That works for me. It still feels redundant, but all I really want is a standard name. :) > We haven't put as

Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions

2020-06-12 Thread David Gow
On Fri, Jun 12, 2020 at 5:55 AM Vitor Massaru Iha wrote: > > This adds the convertion of the runtime tests of check_*_overflow fuctions, > from `lib/test_overflow.c`to KUnit tests. Nit: couple of minor typos here: convertion -> conversion, and functions -> functions > The log similar to the one

Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions

2020-06-12 Thread David Gow
On Sat, Jun 13, 2020 at 6:36 AM Kees Cook wrote: > > On Thu, Jun 11, 2020 at 06:55:01PM -0300, Vitor Massaru Iha wrote: > > This adds the convertion of the runtime tests of check_*_overflow fuctions, > > from `lib/test_overflow.c`to KUnit tests. > > > > The log similar to the one seen in dmesg run

Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions

2020-06-12 Thread Kees Cook
On Thu, Jun 11, 2020 at 06:55:01PM -0300, Vitor Massaru Iha wrote: > This adds the convertion of the runtime tests of check_*_overflow fuctions, > from `lib/test_overflow.c`to KUnit tests. > > The log similar to the one seen in dmesg running test_overflow can be > seen in `test.log`. > > Signed-off

Re: [PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions

2020-06-12 Thread Brendan Higgins
On Thu, Jun 11, 2020 at 2:55 PM Vitor Massaru Iha wrote: > > This adds the convertion of the runtime tests of check_*_overflow fuctions, > from `lib/test_overflow.c`to KUnit tests. > > The log similar to the one seen in dmesg running test_overflow can be > seen in `test.log`. > > Signed-off-by: Vi

[PATCH] lib: kunit_test_overflow: add KUnit test of check_*_overflow functions

2020-06-11 Thread Vitor Massaru Iha
This adds the convertion of the runtime tests of check_*_overflow fuctions, from `lib/test_overflow.c`to KUnit tests. The log similar to the one seen in dmesg running test_overflow can be seen in `test.log`. Signed-off-by: Vitor Massaru Iha --- lib/Kconfig.debug | 17 ++ lib/Makefile