| Message ID | 20220405190619.540177-1-dlatypov@google.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | dcbb2ee24601fabbb547fb0c747ca98b876fe5df |
| Headers | show |
| Series | lib/Kconfig.debug: change KUnit tests to default to KUNIT_ALL_TESTS | expand |
On Wed, Apr 6, 2022 at 3:06 AM Daniel Latypov <dlatypov@google.com> wrote: > > This is in line with Documentation/dev-tools/kunit/style.rst. > Some of these tests predate that so they don't follow this convention. > > With this and commit b0841b51cac9 ("kunit: arch/um/configs: Enable > KUNIT_ALL_TESTS by default"), kunit.py will now run these tests by > default. This hopefully makes it easier to run and maintain the tests. > If any of these were to start failing, people would notice much quicker. > > Note: this commit doesn't update LINEAR_RANGES_TEST since that would > select its dependency (LINEAR_RANGES). We don't want KUNIT_ALL_TESTS > to enable anything other than test kconfigs. > > Signed-off-by: Daniel Latypov <dlatypov@google.com> > --- I'd forgotten these were missing KUNIT_ALL_TESTS support, thanks! Reviewed-by: David Gow <davidgow@google.com> Cheers, -- David
On 4/5/22 15:06, Daniel Latypov wrote: > This is in line with Documentation/dev-tools/kunit/style.rst. > Some of these tests predate that so they don't follow this convention. > > With this and commit b0841b51cac9 ("kunit: arch/um/configs: Enable > KUNIT_ALL_TESTS by default"), kunit.py will now run these tests by > default. This hopefully makes it easier to run and maintain the tests. > If any of these were to start failing, people would notice much quicker. > > Note: this commit doesn't update LINEAR_RANGES_TEST since that would > select its dependency (LINEAR_RANGES). We don't want KUNIT_ALL_TESTS > to enable anything other than test kconfigs. > > Signed-off-by: Daniel Latypov <dlatypov@google.com> > --- I have tried posting something similar in the past! https://lore.kernel.org/lkml/f427ed5cbc08da83086c504fbb3ad1bab50340cd.1618388989.git.npache@redhat.com/ Reviewed-by: Nico Pache <npache@redhat.com> Acked-by: Nico Pache <npache@redhat.com>
On Mon, Apr 18, 2022 at 1:43 PM Nico Pache <npache@redhat.com> wrote: > <snip> > I have tried posting something similar in the past! > https://lore.kernel.org/lkml/f427ed5cbc08da83086c504fbb3ad1bab50340cd.1618388989.git.npache@redhat.com/ Aha, so that's why I vaguely thought someone had already done this. > > Reviewed-by: Nico Pache <npache@redhat.com> > Acked-by: Nico Pache <npache@redhat.com> Thanks!
On Tue, Apr 5, 2022 at 3:06 PM Daniel Latypov <dlatypov@google.com> wrote: > > This is in line with Documentation/dev-tools/kunit/style.rst. > Some of these tests predate that so they don't follow this convention. > > With this and commit b0841b51cac9 ("kunit: arch/um/configs: Enable > KUNIT_ALL_TESTS by default"), kunit.py will now run these tests by > default. This hopefully makes it easier to run and maintain the tests. > If any of these were to start failing, people would notice much quicker. > > Note: this commit doesn't update LINEAR_RANGES_TEST since that would > select its dependency (LINEAR_RANGES). We don't want KUNIT_ALL_TESTS > to enable anything other than test kconfigs. > > Signed-off-by: Daniel Latypov <dlatypov@google.com> Reviewed-by: Brendan Higgins <brendanhiggins@google.com>
On Mon, Apr 18, 2022 at 2:54 PM Daniel Latypov <dlatypov@google.com> wrote: > > On Mon, Apr 18, 2022 at 1:43 PM Nico Pache <npache@redhat.com> wrote: > > > > <snip> > > > I have tried posting something similar in the past! > > https://lore.kernel.org/lkml/f427ed5cbc08da83086c504fbb3ad1bab50340cd.1618388989.git.npache@redhat.com/ Ooops, sorry, we should probably go back and pull those in as well. Thanks for your fixes, and sorry for not picking them up. > Aha, so that's why I vaguely thought someone had already done this. > > > > > Reviewed-by: Nico Pache <npache@redhat.com> > > Acked-by: Nico Pache <npache@redhat.com> > > Thanks!
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug index 075cd25363ac..36865b37b33b 100644 --- a/lib/Kconfig.debug +++ b/lib/Kconfig.debug @@ -2140,10 +2140,11 @@ config TEST_DIV64 If unsure, say N. config KPROBES_SANITY_TEST - tristate "Kprobes sanity tests" + tristate "Kprobes sanity tests" if !KUNIT_ALL_TESTS depends on DEBUG_KERNEL depends on KPROBES depends on KUNIT + default KUNIT_ALL_TESTS help This option provides for testing basic kprobes functionality on boot. Samples of kprobe and kretprobe are inserted and @@ -2417,8 +2418,9 @@ config TEST_SYSCTL If unsure, say N. config BITFIELD_KUNIT - tristate "KUnit test bitfield functions at runtime" + tristate "KUnit test bitfield functions at runtime" if !KUNIT_ALL_TESTS depends on KUNIT + default KUNIT_ALL_TESTS help Enable this option to test the bitfield functions at boot. @@ -2452,8 +2454,9 @@ config HASH_KUNIT_TEST optimized versions. If unsure, say N. config RESOURCE_KUNIT_TEST - tristate "KUnit test for resource API" + tristate "KUnit test for resource API" if !KUNIT_ALL_TESTS depends on KUNIT + default KUNIT_ALL_TESTS help This builds the resource API unit test. Tests the logic of API provided by resource.c and ioport.h. @@ -2506,8 +2509,9 @@ config LINEAR_RANGES_TEST If unsure, say N. config CMDLINE_KUNIT_TEST - tristate "KUnit test for cmdline API" + tristate "KUnit test for cmdline API" if !KUNIT_ALL_TESTS depends on KUNIT + default KUNIT_ALL_TESTS help This builds the cmdline API unit test. Tests the logic of API provided by cmdline.c. @@ -2517,8 +2521,9 @@ config CMDLINE_KUNIT_TEST If unsure, say N. config BITS_TEST - tristate "KUnit test for bits.h" + tristate "KUnit test for bits.h" if !KUNIT_ALL_TESTS depends on KUNIT + default KUNIT_ALL_TESTS help This builds the bits unit test. Tests the logic of macros defined in bits.h.
This is in line with Documentation/dev-tools/kunit/style.rst. Some of these tests predate that so they don't follow this convention. With this and commit b0841b51cac9 ("kunit: arch/um/configs: Enable KUNIT_ALL_TESTS by default"), kunit.py will now run these tests by default. This hopefully makes it easier to run and maintain the tests. If any of these were to start failing, people would notice much quicker. Note: this commit doesn't update LINEAR_RANGES_TEST since that would select its dependency (LINEAR_RANGES). We don't want KUNIT_ALL_TESTS to enable anything other than test kconfigs. Signed-off-by: Daniel Latypov <dlatypov@google.com> --- lib/Kconfig.debug | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) base-commit: 3123109284176b1532874591f7c81f3837bbdc17