| Message ID | 20240720181025.work.002-kees@kernel.org (mailing list archive) |
|---|---|
| State | Handled Elsewhere |
| Headers | show |
| Series | lib: Move KUnit tests into tests/ subdirectory | expand |
On Sat, 20 Jul 2024 11:10:30 -0700 Kees Cook wrote: > Following from the recent KUnit file naming discussion[1], move all > KUnit tests in lib/ into lib/tests/. > > Link: https://lore.kernel.org/lkml/20240720165441.it.320-kees@kernel.org/ [1] > Signed-off-by: Kees Cook <kees@kernel.org> Makes sense: Acked-by: Jakub Kicinski <kuba@kernel.org> BTW love the suggested guidelines hidden in the Link.
On Sat, 20 Jul 2024 11:10:30 -0700 Kees Cook <kees@kernel.org> wrote: > Following from the recent KUnit file naming discussion[1], move all > KUnit tests in lib/ into lib/tests/. > > Link: https://lore.kernel.org/lkml/20240720165441.it.320-kees@kernel.org/ [1] > Signed-off-by: Kees Cook <kees@kernel.org> This looks good to me. Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org> For kprobes and fprobe tests. BTW, should we also rename the file from tests/test_foo.c to tests/foo_kunit.c ? Thank you, > --- > I can carry this in the hardening tree. To disrupt people as little as > possible, I'm hoping to send this either at the end of -rc1 or early > in -rc2. > Cc: Andrew Morton <akpm@linux-foundation.org> > Cc: Yury Norov <yury.norov@gmail.com> > Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk> > Cc: David Gow <davidgow@google.com> > Cc: "Jason A. Donenfeld" <Jason@zx2c4.com> > Cc: Andy Shevchenko <andy@kernel.org> > Cc: "Naveen N. Rao" <naveen.n.rao@linux.ibm.com> > Cc: Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Masami Hiramatsu <mhiramat@kernel.org> > Cc: Mark Brown <broonie@kernel.org> > Cc: Matti Vaittinen <mazziesaccount@gmail.com> > Cc: linux-hardening@vger.kernel.org > Cc: linux-kselftest@vger.kernel.org > Cc: kunit-dev@googlegroups.com > Cc: linux-trace-kernel@vger.kernel.org > --- > MAINTAINERS | 18 ++++++------- > lib/Makefile | 35 +----------------------- > lib/tests/Makefile | 37 ++++++++++++++++++++++++++ > lib/{ => tests}/bitfield_kunit.c | 0 > lib/{ => tests}/checksum_kunit.c | 0 > lib/{ => tests}/cmdline_kunit.c | 0 > lib/{ => tests}/cpumask_kunit.c | 0 > lib/{ => tests}/fortify_kunit.c | 0 > lib/{ => tests}/hashtable_test.c | 0 > lib/{ => tests}/is_signed_type_kunit.c | 0 > lib/{ => tests}/kunit_iov_iter.c | 0 > lib/{ => tests}/list-test.c | 0 > lib/{ => tests}/memcpy_kunit.c | 0 > lib/{ => tests}/overflow_kunit.c | 0 > lib/{ => tests}/siphash_kunit.c | 0 > lib/{ => tests}/slub_kunit.c | 0 > lib/{ => tests}/stackinit_kunit.c | 0 > lib/{ => tests}/string_helpers_kunit.c | 0 > lib/{ => tests}/string_kunit.c | 0 > lib/{ => tests}/test_bits.c | 0 > lib/{ => tests}/test_fprobe.c | 0 > lib/{ => tests}/test_hash.c | 0 > lib/{ => tests}/test_kprobes.c | 0 > lib/{ => tests}/test_linear_ranges.c | 0 > lib/{ => tests}/test_list_sort.c | 0 > lib/{ => tests}/test_sort.c | 0 > 26 files changed, 47 insertions(+), 43 deletions(-) > create mode 100644 lib/tests/Makefile > rename lib/{ => tests}/bitfield_kunit.c (100%) > rename lib/{ => tests}/checksum_kunit.c (100%) > rename lib/{ => tests}/cmdline_kunit.c (100%) > rename lib/{ => tests}/cpumask_kunit.c (100%) > rename lib/{ => tests}/fortify_kunit.c (100%) > rename lib/{ => tests}/hashtable_test.c (100%) > rename lib/{ => tests}/is_signed_type_kunit.c (100%) > rename lib/{ => tests}/kunit_iov_iter.c (100%) > rename lib/{ => tests}/list-test.c (100%) > rename lib/{ => tests}/memcpy_kunit.c (100%) > rename lib/{ => tests}/overflow_kunit.c (100%) > rename lib/{ => tests}/siphash_kunit.c (100%) > rename lib/{ => tests}/slub_kunit.c (100%) > rename lib/{ => tests}/stackinit_kunit.c (100%) > rename lib/{ => tests}/string_helpers_kunit.c (100%) > rename lib/{ => tests}/string_kunit.c (100%) > rename lib/{ => tests}/test_bits.c (100%) > rename lib/{ => tests}/test_fprobe.c (100%) > rename lib/{ => tests}/test_hash.c (100%) > rename lib/{ => tests}/test_kprobes.c (100%) > rename lib/{ => tests}/test_linear_ranges.c (100%) > rename lib/{ => tests}/test_list_sort.c (100%) > rename lib/{ => tests}/test_sort.c (100%) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 8754ac2c259d..3f4b9d007cbb 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -3737,10 +3737,10 @@ F: include/vdso/bits.h > F: lib/bitmap-str.c > F: lib/bitmap.c > F: lib/cpumask.c > -F: lib/cpumask_kunit.c > F: lib/find_bit.c > F: lib/find_bit_benchmark.c > F: lib/test_bitmap.c > +F: lib/tests/cpumask_kunit.c > F: tools/include/linux/bitfield.h > F: tools/include/linux/bitmap.h > F: tools/include/linux/bits.h > @@ -8618,9 +8618,9 @@ L: linux-hardening@vger.kernel.org > S: Supported > T: git git://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git for-next/hardening > F: include/linux/fortify-string.h > -F: lib/fortify_kunit.c > -F: lib/memcpy_kunit.c > F: lib/test_fortify/* > +F: lib/tests/fortify_kunit.c > +F: lib/tests/memcpy_kunit.c > F: scripts/test_fortify.sh > K: \b__NO_FORTIFY\b > > @@ -9246,9 +9246,9 @@ F: include/linux/string.h > F: include/linux/string_choices.h > F: include/linux/string_helpers.h > F: lib/string.c > -F: lib/string_kunit.c > F: lib/string_helpers.c > -F: lib/string_helpers_kunit.c > +F: lib/tests/string_helpers_kunit.c > +F: lib/tests/string_kunit.c > F: scripts/coccinelle/api/string_choices.cocci > > GENERIC UIO DRIVER FOR PCI DEVICES > @@ -12347,7 +12347,7 @@ F: Documentation/trace/kprobes.rst > F: include/asm-generic/kprobes.h > F: include/linux/kprobes.h > F: kernel/kprobes.c > -F: lib/test_kprobes.c > +F: lib/tests/test_kprobes.c > F: samples/kprobes > > KS0108 LCD CONTROLLER DRIVER > @@ -12697,7 +12697,7 @@ M: Mark Brown <broonie@kernel.org> > R: Matti Vaittinen <mazziesaccount@gmail.com> > F: include/linux/linear_range.h > F: lib/linear_ranges.c > -F: lib/test_linear_ranges.c > +F: lib/tests/test_linear_ranges.c > > LINUX FOR POWER MACINTOSH > L: linuxppc-dev@lists.ozlabs.org > @@ -12824,7 +12824,7 @@ M: David Gow <davidgow@google.com> > L: linux-kselftest@vger.kernel.org > L: kunit-dev@googlegroups.com > S: Maintained > -F: lib/list-test.c > +F: lib/tests/list-test.c > > LITEX PLATFORM > M: Karol Gugala <kgugala@antmicro.com> > @@ -20498,7 +20498,7 @@ M: Jason A. Donenfeld <Jason@zx2c4.com> > S: Maintained > F: include/linux/siphash.h > F: lib/siphash.c > -F: lib/siphash_kunit.c > +F: lib/tests/siphash_kunit.c > > SIS 190 ETHERNET DRIVER > M: Francois Romieu <romieu@fr.zoreil.com> > diff --git a/lib/Makefile b/lib/Makefile > index 3b1769045651..f00fe120ee9e 100644 > --- a/lib/Makefile > +++ b/lib/Makefile > @@ -49,9 +49,7 @@ obj-y += bcd.o sort.o parser.o debug_locks.o random32.o \ > percpu-refcount.o rhashtable.o base64.o \ > once.o refcount.o rcuref.o usercopy.o errseq.o bucket_locks.o \ > generic-radix-tree.o bitmap-str.o > -obj-$(CONFIG_STRING_KUNIT_TEST) += string_kunit.o > obj-y += string_helpers.o > -obj-$(CONFIG_STRING_HELPERS_KUNIT_TEST) += string_helpers_kunit.o > obj-y += hexdump.o > obj-$(CONFIG_TEST_HEXDUMP) += test_hexdump.o > obj-y += kstrtox.o > @@ -62,22 +60,17 @@ obj-$(CONFIG_TEST_DHRY) += test_dhry.o > obj-$(CONFIG_TEST_FIRMWARE) += test_firmware.o > obj-$(CONFIG_TEST_BITOPS) += test_bitops.o > CFLAGS_test_bitops.o += -Werror > -obj-$(CONFIG_CPUMASK_KUNIT_TEST) += cpumask_kunit.o > obj-$(CONFIG_TEST_SYSCTL) += test_sysctl.o > -obj-$(CONFIG_TEST_IOV_ITER) += kunit_iov_iter.o > -obj-$(CONFIG_HASH_KUNIT_TEST) += test_hash.o > obj-$(CONFIG_TEST_IDA) += test_ida.o > obj-$(CONFIG_TEST_UBSAN) += test_ubsan.o > CFLAGS_test_ubsan.o += $(call cc-disable-warning, vla) > CFLAGS_test_ubsan.o += $(call cc-disable-warning, unused-but-set-variable) > UBSAN_SANITIZE_test_ubsan.o := y > obj-$(CONFIG_TEST_KSTRTOX) += test-kstrtox.o > -obj-$(CONFIG_TEST_LIST_SORT) += test_list_sort.o > obj-$(CONFIG_TEST_MIN_HEAP) += test_min_heap.o > obj-$(CONFIG_TEST_LKM) += test_module.o > obj-$(CONFIG_TEST_VMALLOC) += test_vmalloc.o > obj-$(CONFIG_TEST_RHASHTABLE) += test_rhashtable.o > -obj-$(CONFIG_TEST_SORT) += test_sort.o > obj-$(CONFIG_TEST_USER_COPY) += test_user_copy.o > obj-$(CONFIG_TEST_STATIC_KEYS) += test_static_keys.o > obj-$(CONFIG_TEST_STATIC_KEYS) += test_static_key_base.o > @@ -104,10 +97,7 @@ obj-$(CONFIG_TEST_MEMINIT) += test_meminit.o > obj-$(CONFIG_TEST_LOCKUP) += test_lockup.o > obj-$(CONFIG_TEST_HMM) += test_hmm.o > obj-$(CONFIG_TEST_FREE_PAGES) += test_free_pages.o > -obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o > obj-$(CONFIG_TEST_REF_TRACKER) += test_ref_tracker.o > -CFLAGS_test_fprobe.o += $(CC_FLAGS_FTRACE) > -obj-$(CONFIG_FPROBE_SANITY_TEST) += test_fprobe.o > obj-$(CONFIG_TEST_OBJPOOL) += test_objpool.o > > obj-$(CONFIG_TEST_FPU) += test_fpu.o > @@ -129,7 +119,7 @@ endif > obj-$(CONFIG_DEBUG_INFO_REDUCED) += debug_info.o > CFLAGS_debug_info.o += $(call cc-option, -femit-struct-debug-detailed=any) > > -obj-y += math/ crypto/ > +obj-y += math/ crypto/ tests/ > > obj-$(CONFIG_GENERIC_IOMAP) += iomap.o > obj-$(CONFIG_HAS_IOMEM) += iomap_copy.o devres.o > @@ -366,29 +356,6 @@ obj-$(CONFIG_OBJAGG) += objagg.o > # pldmfw library > obj-$(CONFIG_PLDMFW) += pldmfw/ > > -# KUnit tests > -CFLAGS_bitfield_kunit.o := $(DISABLE_STRUCTLEAK_PLUGIN) > -obj-$(CONFIG_BITFIELD_KUNIT) += bitfield_kunit.o > -obj-$(CONFIG_CHECKSUM_KUNIT) += checksum_kunit.o > -obj-$(CONFIG_LIST_KUNIT_TEST) += list-test.o > -obj-$(CONFIG_HASHTABLE_KUNIT_TEST) += hashtable_test.o > -obj-$(CONFIG_LINEAR_RANGES_TEST) += test_linear_ranges.o > -obj-$(CONFIG_BITS_TEST) += test_bits.o > -obj-$(CONFIG_CMDLINE_KUNIT_TEST) += cmdline_kunit.o > -obj-$(CONFIG_SLUB_KUNIT_TEST) += slub_kunit.o > -obj-$(CONFIG_MEMCPY_KUNIT_TEST) += memcpy_kunit.o > -obj-$(CONFIG_IS_SIGNED_TYPE_KUNIT_TEST) += is_signed_type_kunit.o > -CFLAGS_overflow_kunit.o = $(call cc-disable-warning, tautological-constant-out-of-range-compare) > -obj-$(CONFIG_OVERFLOW_KUNIT_TEST) += overflow_kunit.o > -CFLAGS_stackinit_kunit.o += $(call cc-disable-warning, switch-unreachable) > -obj-$(CONFIG_STACKINIT_KUNIT_TEST) += stackinit_kunit.o > -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, unsequenced) > -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-overread) > -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-truncation) > -CFLAGS_fortify_kunit.o += $(DISABLE_STRUCTLEAK_PLUGIN) > -obj-$(CONFIG_FORTIFY_KUNIT_TEST) += fortify_kunit.o > -obj-$(CONFIG_SIPHASH_KUNIT_TEST) += siphash_kunit.o > - > obj-$(CONFIG_GENERIC_LIB_DEVMEM_IS_ALLOWED) += devmem_is_allowed.o > > obj-$(CONFIG_FIRMWARE_TABLE) += fw_table.o > diff --git a/lib/tests/Makefile b/lib/tests/Makefile > new file mode 100644 > index 000000000000..c6a14cc8663e > --- /dev/null > +++ b/lib/tests/Makefile > @@ -0,0 +1,37 @@ > +# SPDX-License-Identifier: GPL-2.0 > +# > +# Makefile for tests of kernel library functions. > + > +# KUnit tests > +CFLAGS_bitfield_kunit.o := $(DISABLE_STRUCTLEAK_PLUGIN) > +obj-$(CONFIG_BITFIELD_KUNIT) += bitfield_kunit.o > +obj-$(CONFIG_BITS_TEST) += test_bits.o > +obj-$(CONFIG_CHECKSUM_KUNIT) += checksum_kunit.o > +obj-$(CONFIG_CMDLINE_KUNIT_TEST) += cmdline_kunit.o > +obj-$(CONFIG_CPUMASK_KUNIT_TEST) += cpumask_kunit.o > +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, unsequenced) > +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-overread) > +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-truncation) > +CFLAGS_fortify_kunit.o += $(DISABLE_STRUCTLEAK_PLUGIN) > +obj-$(CONFIG_FORTIFY_KUNIT_TEST) += fortify_kunit.o > +CFLAGS_test_fprobe.o += $(CC_FLAGS_FTRACE) > +obj-$(CONFIG_FPROBE_SANITY_TEST) += test_fprobe.o > +obj-$(CONFIG_HASHTABLE_KUNIT_TEST) += hashtable_test.o > +obj-$(CONFIG_HASH_KUNIT_TEST) += test_hash.o > +obj-$(CONFIG_TEST_IOV_ITER) += kunit_iov_iter.o > +obj-$(CONFIG_IS_SIGNED_TYPE_KUNIT_TEST) += is_signed_type_kunit.o > +obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o > +obj-$(CONFIG_LIST_KUNIT_TEST) += list-test.o > +obj-$(CONFIG_TEST_LIST_SORT) += test_list_sort.o > +obj-$(CONFIG_LINEAR_RANGES_TEST) += test_linear_ranges.o > +obj-$(CONFIG_MEMCPY_KUNIT_TEST) += memcpy_kunit.o > +CFLAGS_overflow_kunit.o = $(call cc-disable-warning, tautological-constant-out-of-range-compare) > +obj-$(CONFIG_OVERFLOW_KUNIT_TEST) += overflow_kunit.o > +obj-$(CONFIG_SIPHASH_KUNIT_TEST) += siphash_kunit.o > +obj-$(CONFIG_SLUB_KUNIT_TEST) += slub_kunit.o > +obj-$(CONFIG_TEST_SORT) += test_sort.o > +CFLAGS_stackinit_kunit.o += $(call cc-disable-warning, switch-unreachable) > +obj-$(CONFIG_STACKINIT_KUNIT_TEST) += stackinit_kunit.o > +obj-$(CONFIG_STRING_KUNIT_TEST) += string_kunit.o > +obj-$(CONFIG_STRING_HELPERS_KUNIT_TEST) += string_helpers_kunit.o > + > diff --git a/lib/bitfield_kunit.c b/lib/tests/bitfield_kunit.c > similarity index 100% > rename from lib/bitfield_kunit.c > rename to lib/tests/bitfield_kunit.c > diff --git a/lib/checksum_kunit.c b/lib/tests/checksum_kunit.c > similarity index 100% > rename from lib/checksum_kunit.c > rename to lib/tests/checksum_kunit.c > diff --git a/lib/cmdline_kunit.c b/lib/tests/cmdline_kunit.c > similarity index 100% > rename from lib/cmdline_kunit.c > rename to lib/tests/cmdline_kunit.c > diff --git a/lib/cpumask_kunit.c b/lib/tests/cpumask_kunit.c > similarity index 100% > rename from lib/cpumask_kunit.c > rename to lib/tests/cpumask_kunit.c > diff --git a/lib/fortify_kunit.c b/lib/tests/fortify_kunit.c > similarity index 100% > rename from lib/fortify_kunit.c > rename to lib/tests/fortify_kunit.c > diff --git a/lib/hashtable_test.c b/lib/tests/hashtable_test.c > similarity index 100% > rename from lib/hashtable_test.c > rename to lib/tests/hashtable_test.c > diff --git a/lib/is_signed_type_kunit.c b/lib/tests/is_signed_type_kunit.c > similarity index 100% > rename from lib/is_signed_type_kunit.c > rename to lib/tests/is_signed_type_kunit.c > diff --git a/lib/kunit_iov_iter.c b/lib/tests/kunit_iov_iter.c > similarity index 100% > rename from lib/kunit_iov_iter.c > rename to lib/tests/kunit_iov_iter.c > diff --git a/lib/list-test.c b/lib/tests/list-test.c > similarity index 100% > rename from lib/list-test.c > rename to lib/tests/list-test.c > diff --git a/lib/memcpy_kunit.c b/lib/tests/memcpy_kunit.c > similarity index 100% > rename from lib/memcpy_kunit.c > rename to lib/tests/memcpy_kunit.c > diff --git a/lib/overflow_kunit.c b/lib/tests/overflow_kunit.c > similarity index 100% > rename from lib/overflow_kunit.c > rename to lib/tests/overflow_kunit.c > diff --git a/lib/siphash_kunit.c b/lib/tests/siphash_kunit.c > similarity index 100% > rename from lib/siphash_kunit.c > rename to lib/tests/siphash_kunit.c > diff --git a/lib/slub_kunit.c b/lib/tests/slub_kunit.c > similarity index 100% > rename from lib/slub_kunit.c > rename to lib/tests/slub_kunit.c > diff --git a/lib/stackinit_kunit.c b/lib/tests/stackinit_kunit.c > similarity index 100% > rename from lib/stackinit_kunit.c > rename to lib/tests/stackinit_kunit.c > diff --git a/lib/string_helpers_kunit.c b/lib/tests/string_helpers_kunit.c > similarity index 100% > rename from lib/string_helpers_kunit.c > rename to lib/tests/string_helpers_kunit.c > diff --git a/lib/string_kunit.c b/lib/tests/string_kunit.c > similarity index 100% > rename from lib/string_kunit.c > rename to lib/tests/string_kunit.c > diff --git a/lib/test_bits.c b/lib/tests/test_bits.c > similarity index 100% > rename from lib/test_bits.c > rename to lib/tests/test_bits.c > diff --git a/lib/test_fprobe.c b/lib/tests/test_fprobe.c > similarity index 100% > rename from lib/test_fprobe.c > rename to lib/tests/test_fprobe.c > diff --git a/lib/test_hash.c b/lib/tests/test_hash.c > similarity index 100% > rename from lib/test_hash.c > rename to lib/tests/test_hash.c > diff --git a/lib/test_kprobes.c b/lib/tests/test_kprobes.c > similarity index 100% > rename from lib/test_kprobes.c > rename to lib/tests/test_kprobes.c > diff --git a/lib/test_linear_ranges.c b/lib/tests/test_linear_ranges.c > similarity index 100% > rename from lib/test_linear_ranges.c > rename to lib/tests/test_linear_ranges.c > diff --git a/lib/test_list_sort.c b/lib/tests/test_list_sort.c > similarity index 100% > rename from lib/test_list_sort.c > rename to lib/tests/test_list_sort.c > diff --git a/lib/test_sort.c b/lib/tests/test_sort.c > similarity index 100% > rename from lib/test_sort.c > rename to lib/tests/test_sort.c > -- > 2.34.1 >
On Sun, 21 Jul 2024 at 02:10, Kees Cook <kees@kernel.org> wrote: > > Following from the recent KUnit file naming discussion[1], move all > KUnit tests in lib/ into lib/tests/. > > Link: https://lore.kernel.org/lkml/20240720165441.it.320-kees@kernel.org/ [1] > Signed-off-by: Kees Cook <kees@kernel.org> > --- > I can carry this in the hardening tree. To disrupt people as little as > possible, I'm hoping to send this either at the end of -rc1 or early > in -rc2. Thanks. I'm happy if you want to rename list-test.c -> list_kunit.c, too. Cheers -- David > Cc: Andrew Morton <akpm@linux-foundation.org> > Cc: Yury Norov <yury.norov@gmail.com> > Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk> > Cc: David Gow <davidgow@google.com> > Cc: "Jason A. Donenfeld" <Jason@zx2c4.com> > Cc: Andy Shevchenko <andy@kernel.org> > Cc: "Naveen N. Rao" <naveen.n.rao@linux.ibm.com> > Cc: Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Masami Hiramatsu <mhiramat@kernel.org> > Cc: Mark Brown <broonie@kernel.org> > Cc: Matti Vaittinen <mazziesaccount@gmail.com> > Cc: linux-hardening@vger.kernel.org > Cc: linux-kselftest@vger.kernel.org > Cc: kunit-dev@googlegroups.com > Cc: linux-trace-kernel@vger.kernel.org > --- > MAINTAINERS | 18 ++++++------- > lib/Makefile | 35 +----------------------- > lib/tests/Makefile | 37 ++++++++++++++++++++++++++ > lib/{ => tests}/bitfield_kunit.c | 0 > lib/{ => tests}/checksum_kunit.c | 0 > lib/{ => tests}/cmdline_kunit.c | 0 > lib/{ => tests}/cpumask_kunit.c | 0 > lib/{ => tests}/fortify_kunit.c | 0 > lib/{ => tests}/hashtable_test.c | 0 > lib/{ => tests}/is_signed_type_kunit.c | 0 > lib/{ => tests}/kunit_iov_iter.c | 0 > lib/{ => tests}/list-test.c | 0 > lib/{ => tests}/memcpy_kunit.c | 0 > lib/{ => tests}/overflow_kunit.c | 0 > lib/{ => tests}/siphash_kunit.c | 0 > lib/{ => tests}/slub_kunit.c | 0 > lib/{ => tests}/stackinit_kunit.c | 0 > lib/{ => tests}/string_helpers_kunit.c | 0 > lib/{ => tests}/string_kunit.c | 0 > lib/{ => tests}/test_bits.c | 0 > lib/{ => tests}/test_fprobe.c | 0 > lib/{ => tests}/test_hash.c | 0 > lib/{ => tests}/test_kprobes.c | 0 > lib/{ => tests}/test_linear_ranges.c | 0 > lib/{ => tests}/test_list_sort.c | 0 > lib/{ => tests}/test_sort.c | 0 > 26 files changed, 47 insertions(+), 43 deletions(-) > create mode 100644 lib/tests/Makefile > rename lib/{ => tests}/bitfield_kunit.c (100%) > rename lib/{ => tests}/checksum_kunit.c (100%) > rename lib/{ => tests}/cmdline_kunit.c (100%) > rename lib/{ => tests}/cpumask_kunit.c (100%) > rename lib/{ => tests}/fortify_kunit.c (100%) > rename lib/{ => tests}/hashtable_test.c (100%) > rename lib/{ => tests}/is_signed_type_kunit.c (100%) > rename lib/{ => tests}/kunit_iov_iter.c (100%) > rename lib/{ => tests}/list-test.c (100%) > rename lib/{ => tests}/memcpy_kunit.c (100%) > rename lib/{ => tests}/overflow_kunit.c (100%) > rename lib/{ => tests}/siphash_kunit.c (100%) > rename lib/{ => tests}/slub_kunit.c (100%) > rename lib/{ => tests}/stackinit_kunit.c (100%) > rename lib/{ => tests}/string_helpers_kunit.c (100%) > rename lib/{ => tests}/string_kunit.c (100%) > rename lib/{ => tests}/test_bits.c (100%) > rename lib/{ => tests}/test_fprobe.c (100%) > rename lib/{ => tests}/test_hash.c (100%) > rename lib/{ => tests}/test_kprobes.c (100%) > rename lib/{ => tests}/test_linear_ranges.c (100%) > rename lib/{ => tests}/test_list_sort.c (100%) > rename lib/{ => tests}/test_sort.c (100%) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 8754ac2c259d..3f4b9d007cbb 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -3737,10 +3737,10 @@ F: include/vdso/bits.h > F: lib/bitmap-str.c > F: lib/bitmap.c > F: lib/cpumask.c > -F: lib/cpumask_kunit.c > F: lib/find_bit.c > F: lib/find_bit_benchmark.c > F: lib/test_bitmap.c > +F: lib/tests/cpumask_kunit.c > F: tools/include/linux/bitfield.h > F: tools/include/linux/bitmap.h > F: tools/include/linux/bits.h > @@ -8618,9 +8618,9 @@ L: linux-hardening@vger.kernel.org > S: Supported > T: git git://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git for-next/hardening > F: include/linux/fortify-string.h > -F: lib/fortify_kunit.c > -F: lib/memcpy_kunit.c > F: lib/test_fortify/* > +F: lib/tests/fortify_kunit.c > +F: lib/tests/memcpy_kunit.c > F: scripts/test_fortify.sh > K: \b__NO_FORTIFY\b > > @@ -9246,9 +9246,9 @@ F: include/linux/string.h > F: include/linux/string_choices.h > F: include/linux/string_helpers.h > F: lib/string.c > -F: lib/string_kunit.c > F: lib/string_helpers.c > -F: lib/string_helpers_kunit.c > +F: lib/tests/string_helpers_kunit.c > +F: lib/tests/string_kunit.c > F: scripts/coccinelle/api/string_choices.cocci > > GENERIC UIO DRIVER FOR PCI DEVICES > @@ -12347,7 +12347,7 @@ F: Documentation/trace/kprobes.rst > F: include/asm-generic/kprobes.h > F: include/linux/kprobes.h > F: kernel/kprobes.c > -F: lib/test_kprobes.c > +F: lib/tests/test_kprobes.c > F: samples/kprobes > > KS0108 LCD CONTROLLER DRIVER > @@ -12697,7 +12697,7 @@ M: Mark Brown <broonie@kernel.org> > R: Matti Vaittinen <mazziesaccount@gmail.com> > F: include/linux/linear_range.h > F: lib/linear_ranges.c > -F: lib/test_linear_ranges.c > +F: lib/tests/test_linear_ranges.c > > LINUX FOR POWER MACINTOSH > L: linuxppc-dev@lists.ozlabs.org > @@ -12824,7 +12824,7 @@ M: David Gow <davidgow@google.com> > L: linux-kselftest@vger.kernel.org > L: kunit-dev@googlegroups.com > S: Maintained > -F: lib/list-test.c > +F: lib/tests/list-test.c > > LITEX PLATFORM > M: Karol Gugala <kgugala@antmicro.com> > @@ -20498,7 +20498,7 @@ M: Jason A. Donenfeld <Jason@zx2c4.com> > S: Maintained > F: include/linux/siphash.h > F: lib/siphash.c > -F: lib/siphash_kunit.c > +F: lib/tests/siphash_kunit.c > > SIS 190 ETHERNET DRIVER > M: Francois Romieu <romieu@fr.zoreil.com> > diff --git a/lib/Makefile b/lib/Makefile > index 3b1769045651..f00fe120ee9e 100644 > --- a/lib/Makefile > +++ b/lib/Makefile > @@ -49,9 +49,7 @@ obj-y += bcd.o sort.o parser.o debug_locks.o random32.o \ > percpu-refcount.o rhashtable.o base64.o \ > once.o refcount.o rcuref.o usercopy.o errseq.o bucket_locks.o \ > generic-radix-tree.o bitmap-str.o > -obj-$(CONFIG_STRING_KUNIT_TEST) += string_kunit.o > obj-y += string_helpers.o > -obj-$(CONFIG_STRING_HELPERS_KUNIT_TEST) += string_helpers_kunit.o > obj-y += hexdump.o > obj-$(CONFIG_TEST_HEXDUMP) += test_hexdump.o > obj-y += kstrtox.o > @@ -62,22 +60,17 @@ obj-$(CONFIG_TEST_DHRY) += test_dhry.o > obj-$(CONFIG_TEST_FIRMWARE) += test_firmware.o > obj-$(CONFIG_TEST_BITOPS) += test_bitops.o > CFLAGS_test_bitops.o += -Werror > -obj-$(CONFIG_CPUMASK_KUNIT_TEST) += cpumask_kunit.o > obj-$(CONFIG_TEST_SYSCTL) += test_sysctl.o > -obj-$(CONFIG_TEST_IOV_ITER) += kunit_iov_iter.o > -obj-$(CONFIG_HASH_KUNIT_TEST) += test_hash.o > obj-$(CONFIG_TEST_IDA) += test_ida.o > obj-$(CONFIG_TEST_UBSAN) += test_ubsan.o > CFLAGS_test_ubsan.o += $(call cc-disable-warning, vla) > CFLAGS_test_ubsan.o += $(call cc-disable-warning, unused-but-set-variable) > UBSAN_SANITIZE_test_ubsan.o := y > obj-$(CONFIG_TEST_KSTRTOX) += test-kstrtox.o > -obj-$(CONFIG_TEST_LIST_SORT) += test_list_sort.o > obj-$(CONFIG_TEST_MIN_HEAP) += test_min_heap.o > obj-$(CONFIG_TEST_LKM) += test_module.o > obj-$(CONFIG_TEST_VMALLOC) += test_vmalloc.o > obj-$(CONFIG_TEST_RHASHTABLE) += test_rhashtable.o > -obj-$(CONFIG_TEST_SORT) += test_sort.o > obj-$(CONFIG_TEST_USER_COPY) += test_user_copy.o > obj-$(CONFIG_TEST_STATIC_KEYS) += test_static_keys.o > obj-$(CONFIG_TEST_STATIC_KEYS) += test_static_key_base.o > @@ -104,10 +97,7 @@ obj-$(CONFIG_TEST_MEMINIT) += test_meminit.o > obj-$(CONFIG_TEST_LOCKUP) += test_lockup.o > obj-$(CONFIG_TEST_HMM) += test_hmm.o > obj-$(CONFIG_TEST_FREE_PAGES) += test_free_pages.o > -obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o > obj-$(CONFIG_TEST_REF_TRACKER) += test_ref_tracker.o > -CFLAGS_test_fprobe.o += $(CC_FLAGS_FTRACE) > -obj-$(CONFIG_FPROBE_SANITY_TEST) += test_fprobe.o > obj-$(CONFIG_TEST_OBJPOOL) += test_objpool.o > > obj-$(CONFIG_TEST_FPU) += test_fpu.o > @@ -129,7 +119,7 @@ endif > obj-$(CONFIG_DEBUG_INFO_REDUCED) += debug_info.o > CFLAGS_debug_info.o += $(call cc-option, -femit-struct-debug-detailed=any) > > -obj-y += math/ crypto/ > +obj-y += math/ crypto/ tests/ > > obj-$(CONFIG_GENERIC_IOMAP) += iomap.o > obj-$(CONFIG_HAS_IOMEM) += iomap_copy.o devres.o > @@ -366,29 +356,6 @@ obj-$(CONFIG_OBJAGG) += objagg.o > # pldmfw library > obj-$(CONFIG_PLDMFW) += pldmfw/ > > -# KUnit tests > -CFLAGS_bitfield_kunit.o := $(DISABLE_STRUCTLEAK_PLUGIN) > -obj-$(CONFIG_BITFIELD_KUNIT) += bitfield_kunit.o > -obj-$(CONFIG_CHECKSUM_KUNIT) += checksum_kunit.o > -obj-$(CONFIG_LIST_KUNIT_TEST) += list-test.o > -obj-$(CONFIG_HASHTABLE_KUNIT_TEST) += hashtable_test.o > -obj-$(CONFIG_LINEAR_RANGES_TEST) += test_linear_ranges.o > -obj-$(CONFIG_BITS_TEST) += test_bits.o > -obj-$(CONFIG_CMDLINE_KUNIT_TEST) += cmdline_kunit.o > -obj-$(CONFIG_SLUB_KUNIT_TEST) += slub_kunit.o > -obj-$(CONFIG_MEMCPY_KUNIT_TEST) += memcpy_kunit.o > -obj-$(CONFIG_IS_SIGNED_TYPE_KUNIT_TEST) += is_signed_type_kunit.o > -CFLAGS_overflow_kunit.o = $(call cc-disable-warning, tautological-constant-out-of-range-compare) > -obj-$(CONFIG_OVERFLOW_KUNIT_TEST) += overflow_kunit.o > -CFLAGS_stackinit_kunit.o += $(call cc-disable-warning, switch-unreachable) > -obj-$(CONFIG_STACKINIT_KUNIT_TEST) += stackinit_kunit.o > -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, unsequenced) > -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-overread) > -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-truncation) > -CFLAGS_fortify_kunit.o += $(DISABLE_STRUCTLEAK_PLUGIN) > -obj-$(CONFIG_FORTIFY_KUNIT_TEST) += fortify_kunit.o > -obj-$(CONFIG_SIPHASH_KUNIT_TEST) += siphash_kunit.o > - > obj-$(CONFIG_GENERIC_LIB_DEVMEM_IS_ALLOWED) += devmem_is_allowed.o > > obj-$(CONFIG_FIRMWARE_TABLE) += fw_table.o > diff --git a/lib/tests/Makefile b/lib/tests/Makefile > new file mode 100644 > index 000000000000..c6a14cc8663e > --- /dev/null > +++ b/lib/tests/Makefile > @@ -0,0 +1,37 @@ > +# SPDX-License-Identifier: GPL-2.0 > +# > +# Makefile for tests of kernel library functions. > + > +# KUnit tests > +CFLAGS_bitfield_kunit.o := $(DISABLE_STRUCTLEAK_PLUGIN) > +obj-$(CONFIG_BITFIELD_KUNIT) += bitfield_kunit.o > +obj-$(CONFIG_BITS_TEST) += test_bits.o > +obj-$(CONFIG_CHECKSUM_KUNIT) += checksum_kunit.o > +obj-$(CONFIG_CMDLINE_KUNIT_TEST) += cmdline_kunit.o > +obj-$(CONFIG_CPUMASK_KUNIT_TEST) += cpumask_kunit.o > +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, unsequenced) > +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-overread) > +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-truncation) > +CFLAGS_fortify_kunit.o += $(DISABLE_STRUCTLEAK_PLUGIN) > +obj-$(CONFIG_FORTIFY_KUNIT_TEST) += fortify_kunit.o > +CFLAGS_test_fprobe.o += $(CC_FLAGS_FTRACE) > +obj-$(CONFIG_FPROBE_SANITY_TEST) += test_fprobe.o > +obj-$(CONFIG_HASHTABLE_KUNIT_TEST) += hashtable_test.o > +obj-$(CONFIG_HASH_KUNIT_TEST) += test_hash.o > +obj-$(CONFIG_TEST_IOV_ITER) += kunit_iov_iter.o > +obj-$(CONFIG_IS_SIGNED_TYPE_KUNIT_TEST) += is_signed_type_kunit.o > +obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o > +obj-$(CONFIG_LIST_KUNIT_TEST) += list-test.o > +obj-$(CONFIG_TEST_LIST_SORT) += test_list_sort.o > +obj-$(CONFIG_LINEAR_RANGES_TEST) += test_linear_ranges.o > +obj-$(CONFIG_MEMCPY_KUNIT_TEST) += memcpy_kunit.o > +CFLAGS_overflow_kunit.o = $(call cc-disable-warning, tautological-constant-out-of-range-compare) > +obj-$(CONFIG_OVERFLOW_KUNIT_TEST) += overflow_kunit.o > +obj-$(CONFIG_SIPHASH_KUNIT_TEST) += siphash_kunit.o > +obj-$(CONFIG_SLUB_KUNIT_TEST) += slub_kunit.o > +obj-$(CONFIG_TEST_SORT) += test_sort.o > +CFLAGS_stackinit_kunit.o += $(call cc-disable-warning, switch-unreachable) > +obj-$(CONFIG_STACKINIT_KUNIT_TEST) += stackinit_kunit.o > +obj-$(CONFIG_STRING_KUNIT_TEST) += string_kunit.o > +obj-$(CONFIG_STRING_HELPERS_KUNIT_TEST) += string_helpers_kunit.o > + > diff --git a/lib/bitfield_kunit.c b/lib/tests/bitfield_kunit.c > similarity index 100% > rename from lib/bitfield_kunit.c > rename to lib/tests/bitfield_kunit.c > diff --git a/lib/checksum_kunit.c b/lib/tests/checksum_kunit.c > similarity index 100% > rename from lib/checksum_kunit.c > rename to lib/tests/checksum_kunit.c > diff --git a/lib/cmdline_kunit.c b/lib/tests/cmdline_kunit.c > similarity index 100% > rename from lib/cmdline_kunit.c > rename to lib/tests/cmdline_kunit.c > diff --git a/lib/cpumask_kunit.c b/lib/tests/cpumask_kunit.c > similarity index 100% > rename from lib/cpumask_kunit.c > rename to lib/tests/cpumask_kunit.c > diff --git a/lib/fortify_kunit.c b/lib/tests/fortify_kunit.c > similarity index 100% > rename from lib/fortify_kunit.c > rename to lib/tests/fortify_kunit.c > diff --git a/lib/hashtable_test.c b/lib/tests/hashtable_test.c > similarity index 100% > rename from lib/hashtable_test.c > rename to lib/tests/hashtable_test.c > diff --git a/lib/is_signed_type_kunit.c b/lib/tests/is_signed_type_kunit.c > similarity index 100% > rename from lib/is_signed_type_kunit.c > rename to lib/tests/is_signed_type_kunit.c > diff --git a/lib/kunit_iov_iter.c b/lib/tests/kunit_iov_iter.c > similarity index 100% > rename from lib/kunit_iov_iter.c > rename to lib/tests/kunit_iov_iter.c > diff --git a/lib/list-test.c b/lib/tests/list-test.c > similarity index 100% > rename from lib/list-test.c > rename to lib/tests/list-test.c > diff --git a/lib/memcpy_kunit.c b/lib/tests/memcpy_kunit.c > similarity index 100% > rename from lib/memcpy_kunit.c > rename to lib/tests/memcpy_kunit.c > diff --git a/lib/overflow_kunit.c b/lib/tests/overflow_kunit.c > similarity index 100% > rename from lib/overflow_kunit.c > rename to lib/tests/overflow_kunit.c > diff --git a/lib/siphash_kunit.c b/lib/tests/siphash_kunit.c > similarity index 100% > rename from lib/siphash_kunit.c > rename to lib/tests/siphash_kunit.c > diff --git a/lib/slub_kunit.c b/lib/tests/slub_kunit.c > similarity index 100% > rename from lib/slub_kunit.c > rename to lib/tests/slub_kunit.c > diff --git a/lib/stackinit_kunit.c b/lib/tests/stackinit_kunit.c > similarity index 100% > rename from lib/stackinit_kunit.c > rename to lib/tests/stackinit_kunit.c > diff --git a/lib/string_helpers_kunit.c b/lib/tests/string_helpers_kunit.c > similarity index 100% > rename from lib/string_helpers_kunit.c > rename to lib/tests/string_helpers_kunit.c > diff --git a/lib/string_kunit.c b/lib/tests/string_kunit.c > similarity index 100% > rename from lib/string_kunit.c > rename to lib/tests/string_kunit.c > diff --git a/lib/test_bits.c b/lib/tests/test_bits.c > similarity index 100% > rename from lib/test_bits.c > rename to lib/tests/test_bits.c > diff --git a/lib/test_fprobe.c b/lib/tests/test_fprobe.c > similarity index 100% > rename from lib/test_fprobe.c > rename to lib/tests/test_fprobe.c > diff --git a/lib/test_hash.c b/lib/tests/test_hash.c > similarity index 100% > rename from lib/test_hash.c > rename to lib/tests/test_hash.c > diff --git a/lib/test_kprobes.c b/lib/tests/test_kprobes.c > similarity index 100% > rename from lib/test_kprobes.c > rename to lib/tests/test_kprobes.c > diff --git a/lib/test_linear_ranges.c b/lib/tests/test_linear_ranges.c > similarity index 100% > rename from lib/test_linear_ranges.c > rename to lib/tests/test_linear_ranges.c > diff --git a/lib/test_list_sort.c b/lib/tests/test_list_sort.c > similarity index 100% > rename from lib/test_list_sort.c > rename to lib/tests/test_list_sort.c > diff --git a/lib/test_sort.c b/lib/tests/test_sort.c > similarity index 100% > rename from lib/test_sort.c > rename to lib/tests/test_sort.c > -- > 2.34.1 > > -- > You received this message because you are subscribed to the Google Groups "KUnit Development" group. > To unsubscribe from this group and stop receiving emails from it, send an email to kunit-dev+unsubscribe@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/kunit-dev/20240720181025.work.002-kees%40kernel.org.
On Wed, Jul 24, 2024 at 02:43:15PM +0800, David Gow wrote: > On Sun, 21 Jul 2024 at 02:10, Kees Cook <kees@kernel.org> wrote: > > > > Following from the recent KUnit file naming discussion[1], move all > > KUnit tests in lib/ into lib/tests/. > > > > Link: https://lore.kernel.org/lkml/20240720165441.it.320-kees@kernel.org/ [1] > > Signed-off-by: Kees Cook <kees@kernel.org> > > --- > > I can carry this in the hardening tree. To disrupt people as little as > > possible, I'm hoping to send this either at the end of -rc1 or early > > in -rc2. > > Thanks. > > I'm happy if you want to rename list-test.c -> list_kunit.c, too. Sure! I was thinking about doing some separate follow-up changes for individual tests. -Kees
On Sat, 20 Jul 2024 11:10:30 -0700 Kees Cook <kees@kernel.org> wrote: > Following from the recent KUnit file naming discussion[1], move all > KUnit tests in lib/ into lib/tests/. > > Link: https://lore.kernel.org/lkml/20240720165441.it.320-kees@kernel.org/ [1] > Signed-off-by: Kees Cook <kees@kernel.org> > --- Strong-Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org> I was talking with Masami and asking him where the kunit test code lives and after pointing to lib/ I was disgusted with what I saw. I told him, "why isn't this in its own "tests/" directory"? And he mentioned this patch (which I forgot about). So yes, please move it. Thanks, -- Steve > I can carry this in the hardening tree. To disrupt people as little as > possible, I'm hoping to send this either at the end of -rc1 or early > in -rc2. > Cc: Andrew Morton <akpm@linux-foundation.org> > Cc: Yury Norov <yury.norov@gmail.com> > Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk> > Cc: David Gow <davidgow@google.com> > Cc: "Jason A. Donenfeld" <Jason@zx2c4.com> > Cc: Andy Shevchenko <andy@kernel.org> > Cc: "Naveen N. Rao" <naveen.n.rao@linux.ibm.com> > Cc: Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Masami Hiramatsu <mhiramat@kernel.org> > Cc: Mark Brown <broonie@kernel.org> > Cc: Matti Vaittinen <mazziesaccount@gmail.com> > Cc: linux-hardening@vger.kernel.org > Cc: linux-kselftest@vger.kernel.org > Cc: kunit-dev@googlegroups.com > Cc: linux-trace-kernel@vger.kernel.org >
On Sun, 21 Jul 2024 at 02:10, Kees Cook <kees@kernel.org> wrote: > > Following from the recent KUnit file naming discussion[1], move all > KUnit tests in lib/ into lib/tests/. > > Link: https://lore.kernel.org/lkml/20240720165441.it.320-kees@kernel.org/ [1] > Signed-off-by: Kees Cook <kees@kernel.org> > --- > I can carry this in the hardening tree. To disrupt people as little as > possible, I'm hoping to send this either at the end of -rc1 or early > in -rc2. > Cc: Andrew Morton <akpm@linux-foundation.org> > Cc: Yury Norov <yury.norov@gmail.com> > Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk> > Cc: David Gow <davidgow@google.com> > Cc: "Jason A. Donenfeld" <Jason@zx2c4.com> > Cc: Andy Shevchenko <andy@kernel.org> > Cc: "Naveen N. Rao" <naveen.n.rao@linux.ibm.com> > Cc: Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Masami Hiramatsu <mhiramat@kernel.org> > Cc: Mark Brown <broonie@kernel.org> > Cc: Matti Vaittinen <mazziesaccount@gmail.com> > Cc: linux-hardening@vger.kernel.org > Cc: linux-kselftest@vger.kernel.org > Cc: kunit-dev@googlegroups.com > Cc: linux-trace-kernel@vger.kernel.org > --- I've just rebased this and sent it out as part of a series of renames to Andrew: https://lore.kernel.org/linux-kselftest/20241011072509.3068328-5-davidgow@google.com/T/#u Hopefully this will be a workable way of getting all of the different renames (and other things entangled with renames) in, but let me know if it causes problems / if I broke it. Cheers, -- David > MAINTAINERS | 18 ++++++------- > lib/Makefile | 35 +----------------------- > lib/tests/Makefile | 37 ++++++++++++++++++++++++++ > lib/{ => tests}/bitfield_kunit.c | 0 > lib/{ => tests}/checksum_kunit.c | 0 > lib/{ => tests}/cmdline_kunit.c | 0 > lib/{ => tests}/cpumask_kunit.c | 0 > lib/{ => tests}/fortify_kunit.c | 0 > lib/{ => tests}/hashtable_test.c | 0 > lib/{ => tests}/is_signed_type_kunit.c | 0 > lib/{ => tests}/kunit_iov_iter.c | 0 > lib/{ => tests}/list-test.c | 0 > lib/{ => tests}/memcpy_kunit.c | 0 > lib/{ => tests}/overflow_kunit.c | 0 > lib/{ => tests}/siphash_kunit.c | 0 > lib/{ => tests}/slub_kunit.c | 0 > lib/{ => tests}/stackinit_kunit.c | 0 > lib/{ => tests}/string_helpers_kunit.c | 0 > lib/{ => tests}/string_kunit.c | 0 > lib/{ => tests}/test_bits.c | 0 > lib/{ => tests}/test_fprobe.c | 0 > lib/{ => tests}/test_hash.c | 0 > lib/{ => tests}/test_kprobes.c | 0 > lib/{ => tests}/test_linear_ranges.c | 0 > lib/{ => tests}/test_list_sort.c | 0 > lib/{ => tests}/test_sort.c | 0 > 26 files changed, 47 insertions(+), 43 deletions(-) > create mode 100644 lib/tests/Makefile > rename lib/{ => tests}/bitfield_kunit.c (100%) > rename lib/{ => tests}/checksum_kunit.c (100%) > rename lib/{ => tests}/cmdline_kunit.c (100%) > rename lib/{ => tests}/cpumask_kunit.c (100%) > rename lib/{ => tests}/fortify_kunit.c (100%) > rename lib/{ => tests}/hashtable_test.c (100%) > rename lib/{ => tests}/is_signed_type_kunit.c (100%) > rename lib/{ => tests}/kunit_iov_iter.c (100%) > rename lib/{ => tests}/list-test.c (100%) > rename lib/{ => tests}/memcpy_kunit.c (100%) > rename lib/{ => tests}/overflow_kunit.c (100%) > rename lib/{ => tests}/siphash_kunit.c (100%) > rename lib/{ => tests}/slub_kunit.c (100%) > rename lib/{ => tests}/stackinit_kunit.c (100%) > rename lib/{ => tests}/string_helpers_kunit.c (100%) > rename lib/{ => tests}/string_kunit.c (100%) > rename lib/{ => tests}/test_bits.c (100%) > rename lib/{ => tests}/test_fprobe.c (100%) > rename lib/{ => tests}/test_hash.c (100%) > rename lib/{ => tests}/test_kprobes.c (100%) > rename lib/{ => tests}/test_linear_ranges.c (100%) > rename lib/{ => tests}/test_list_sort.c (100%) > rename lib/{ => tests}/test_sort.c (100%) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 8754ac2c259d..3f4b9d007cbb 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -3737,10 +3737,10 @@ F: include/vdso/bits.h > F: lib/bitmap-str.c > F: lib/bitmap.c > F: lib/cpumask.c > -F: lib/cpumask_kunit.c > F: lib/find_bit.c > F: lib/find_bit_benchmark.c > F: lib/test_bitmap.c > +F: lib/tests/cpumask_kunit.c > F: tools/include/linux/bitfield.h > F: tools/include/linux/bitmap.h > F: tools/include/linux/bits.h > @@ -8618,9 +8618,9 @@ L: linux-hardening@vger.kernel.org > S: Supported > T: git git://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git for-next/hardening > F: include/linux/fortify-string.h > -F: lib/fortify_kunit.c > -F: lib/memcpy_kunit.c > F: lib/test_fortify/* > +F: lib/tests/fortify_kunit.c > +F: lib/tests/memcpy_kunit.c > F: scripts/test_fortify.sh > K: \b__NO_FORTIFY\b > > @@ -9246,9 +9246,9 @@ F: include/linux/string.h > F: include/linux/string_choices.h > F: include/linux/string_helpers.h > F: lib/string.c > -F: lib/string_kunit.c > F: lib/string_helpers.c > -F: lib/string_helpers_kunit.c > +F: lib/tests/string_helpers_kunit.c > +F: lib/tests/string_kunit.c > F: scripts/coccinelle/api/string_choices.cocci > > GENERIC UIO DRIVER FOR PCI DEVICES > @@ -12347,7 +12347,7 @@ F: Documentation/trace/kprobes.rst > F: include/asm-generic/kprobes.h > F: include/linux/kprobes.h > F: kernel/kprobes.c > -F: lib/test_kprobes.c > +F: lib/tests/test_kprobes.c > F: samples/kprobes > > KS0108 LCD CONTROLLER DRIVER > @@ -12697,7 +12697,7 @@ M: Mark Brown <broonie@kernel.org> > R: Matti Vaittinen <mazziesaccount@gmail.com> > F: include/linux/linear_range.h > F: lib/linear_ranges.c > -F: lib/test_linear_ranges.c > +F: lib/tests/test_linear_ranges.c > > LINUX FOR POWER MACINTOSH > L: linuxppc-dev@lists.ozlabs.org > @@ -12824,7 +12824,7 @@ M: David Gow <davidgow@google.com> > L: linux-kselftest@vger.kernel.org > L: kunit-dev@googlegroups.com > S: Maintained > -F: lib/list-test.c > +F: lib/tests/list-test.c > > LITEX PLATFORM > M: Karol Gugala <kgugala@antmicro.com> > @@ -20498,7 +20498,7 @@ M: Jason A. Donenfeld <Jason@zx2c4.com> > S: Maintained > F: include/linux/siphash.h > F: lib/siphash.c > -F: lib/siphash_kunit.c > +F: lib/tests/siphash_kunit.c > > SIS 190 ETHERNET DRIVER > M: Francois Romieu <romieu@fr.zoreil.com> > diff --git a/lib/Makefile b/lib/Makefile > index 3b1769045651..f00fe120ee9e 100644 > --- a/lib/Makefile > +++ b/lib/Makefile > @@ -49,9 +49,7 @@ obj-y += bcd.o sort.o parser.o debug_locks.o random32.o \ > percpu-refcount.o rhashtable.o base64.o \ > once.o refcount.o rcuref.o usercopy.o errseq.o bucket_locks.o \ > generic-radix-tree.o bitmap-str.o > -obj-$(CONFIG_STRING_KUNIT_TEST) += string_kunit.o > obj-y += string_helpers.o > -obj-$(CONFIG_STRING_HELPERS_KUNIT_TEST) += string_helpers_kunit.o > obj-y += hexdump.o > obj-$(CONFIG_TEST_HEXDUMP) += test_hexdump.o > obj-y += kstrtox.o > @@ -62,22 +60,17 @@ obj-$(CONFIG_TEST_DHRY) += test_dhry.o > obj-$(CONFIG_TEST_FIRMWARE) += test_firmware.o > obj-$(CONFIG_TEST_BITOPS) += test_bitops.o > CFLAGS_test_bitops.o += -Werror > -obj-$(CONFIG_CPUMASK_KUNIT_TEST) += cpumask_kunit.o > obj-$(CONFIG_TEST_SYSCTL) += test_sysctl.o > -obj-$(CONFIG_TEST_IOV_ITER) += kunit_iov_iter.o > -obj-$(CONFIG_HASH_KUNIT_TEST) += test_hash.o > obj-$(CONFIG_TEST_IDA) += test_ida.o > obj-$(CONFIG_TEST_UBSAN) += test_ubsan.o > CFLAGS_test_ubsan.o += $(call cc-disable-warning, vla) > CFLAGS_test_ubsan.o += $(call cc-disable-warning, unused-but-set-variable) > UBSAN_SANITIZE_test_ubsan.o := y > obj-$(CONFIG_TEST_KSTRTOX) += test-kstrtox.o > -obj-$(CONFIG_TEST_LIST_SORT) += test_list_sort.o > obj-$(CONFIG_TEST_MIN_HEAP) += test_min_heap.o > obj-$(CONFIG_TEST_LKM) += test_module.o > obj-$(CONFIG_TEST_VMALLOC) += test_vmalloc.o > obj-$(CONFIG_TEST_RHASHTABLE) += test_rhashtable.o > -obj-$(CONFIG_TEST_SORT) += test_sort.o > obj-$(CONFIG_TEST_USER_COPY) += test_user_copy.o > obj-$(CONFIG_TEST_STATIC_KEYS) += test_static_keys.o > obj-$(CONFIG_TEST_STATIC_KEYS) += test_static_key_base.o > @@ -104,10 +97,7 @@ obj-$(CONFIG_TEST_MEMINIT) += test_meminit.o > obj-$(CONFIG_TEST_LOCKUP) += test_lockup.o > obj-$(CONFIG_TEST_HMM) += test_hmm.o > obj-$(CONFIG_TEST_FREE_PAGES) += test_free_pages.o > -obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o > obj-$(CONFIG_TEST_REF_TRACKER) += test_ref_tracker.o > -CFLAGS_test_fprobe.o += $(CC_FLAGS_FTRACE) > -obj-$(CONFIG_FPROBE_SANITY_TEST) += test_fprobe.o > obj-$(CONFIG_TEST_OBJPOOL) += test_objpool.o > > obj-$(CONFIG_TEST_FPU) += test_fpu.o > @@ -129,7 +119,7 @@ endif > obj-$(CONFIG_DEBUG_INFO_REDUCED) += debug_info.o > CFLAGS_debug_info.o += $(call cc-option, -femit-struct-debug-detailed=any) > > -obj-y += math/ crypto/ > +obj-y += math/ crypto/ tests/ > > obj-$(CONFIG_GENERIC_IOMAP) += iomap.o > obj-$(CONFIG_HAS_IOMEM) += iomap_copy.o devres.o > @@ -366,29 +356,6 @@ obj-$(CONFIG_OBJAGG) += objagg.o > # pldmfw library > obj-$(CONFIG_PLDMFW) += pldmfw/ > > -# KUnit tests > -CFLAGS_bitfield_kunit.o := $(DISABLE_STRUCTLEAK_PLUGIN) > -obj-$(CONFIG_BITFIELD_KUNIT) += bitfield_kunit.o > -obj-$(CONFIG_CHECKSUM_KUNIT) += checksum_kunit.o > -obj-$(CONFIG_LIST_KUNIT_TEST) += list-test.o > -obj-$(CONFIG_HASHTABLE_KUNIT_TEST) += hashtable_test.o > -obj-$(CONFIG_LINEAR_RANGES_TEST) += test_linear_ranges.o > -obj-$(CONFIG_BITS_TEST) += test_bits.o > -obj-$(CONFIG_CMDLINE_KUNIT_TEST) += cmdline_kunit.o > -obj-$(CONFIG_SLUB_KUNIT_TEST) += slub_kunit.o > -obj-$(CONFIG_MEMCPY_KUNIT_TEST) += memcpy_kunit.o > -obj-$(CONFIG_IS_SIGNED_TYPE_KUNIT_TEST) += is_signed_type_kunit.o > -CFLAGS_overflow_kunit.o = $(call cc-disable-warning, tautological-constant-out-of-range-compare) > -obj-$(CONFIG_OVERFLOW_KUNIT_TEST) += overflow_kunit.o > -CFLAGS_stackinit_kunit.o += $(call cc-disable-warning, switch-unreachable) > -obj-$(CONFIG_STACKINIT_KUNIT_TEST) += stackinit_kunit.o > -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, unsequenced) > -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-overread) > -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-truncation) > -CFLAGS_fortify_kunit.o += $(DISABLE_STRUCTLEAK_PLUGIN) > -obj-$(CONFIG_FORTIFY_KUNIT_TEST) += fortify_kunit.o > -obj-$(CONFIG_SIPHASH_KUNIT_TEST) += siphash_kunit.o > - > obj-$(CONFIG_GENERIC_LIB_DEVMEM_IS_ALLOWED) += devmem_is_allowed.o > > obj-$(CONFIG_FIRMWARE_TABLE) += fw_table.o > diff --git a/lib/tests/Makefile b/lib/tests/Makefile > new file mode 100644 > index 000000000000..c6a14cc8663e > --- /dev/null > +++ b/lib/tests/Makefile > @@ -0,0 +1,37 @@ > +# SPDX-License-Identifier: GPL-2.0 > +# > +# Makefile for tests of kernel library functions. > + > +# KUnit tests > +CFLAGS_bitfield_kunit.o := $(DISABLE_STRUCTLEAK_PLUGIN) > +obj-$(CONFIG_BITFIELD_KUNIT) += bitfield_kunit.o > +obj-$(CONFIG_BITS_TEST) += test_bits.o > +obj-$(CONFIG_CHECKSUM_KUNIT) += checksum_kunit.o > +obj-$(CONFIG_CMDLINE_KUNIT_TEST) += cmdline_kunit.o > +obj-$(CONFIG_CPUMASK_KUNIT_TEST) += cpumask_kunit.o > +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, unsequenced) > +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-overread) > +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-truncation) > +CFLAGS_fortify_kunit.o += $(DISABLE_STRUCTLEAK_PLUGIN) > +obj-$(CONFIG_FORTIFY_KUNIT_TEST) += fortify_kunit.o > +CFLAGS_test_fprobe.o += $(CC_FLAGS_FTRACE) > +obj-$(CONFIG_FPROBE_SANITY_TEST) += test_fprobe.o > +obj-$(CONFIG_HASHTABLE_KUNIT_TEST) += hashtable_test.o > +obj-$(CONFIG_HASH_KUNIT_TEST) += test_hash.o > +obj-$(CONFIG_TEST_IOV_ITER) += kunit_iov_iter.o > +obj-$(CONFIG_IS_SIGNED_TYPE_KUNIT_TEST) += is_signed_type_kunit.o > +obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o > +obj-$(CONFIG_LIST_KUNIT_TEST) += list-test.o > +obj-$(CONFIG_TEST_LIST_SORT) += test_list_sort.o > +obj-$(CONFIG_LINEAR_RANGES_TEST) += test_linear_ranges.o > +obj-$(CONFIG_MEMCPY_KUNIT_TEST) += memcpy_kunit.o > +CFLAGS_overflow_kunit.o = $(call cc-disable-warning, tautological-constant-out-of-range-compare) > +obj-$(CONFIG_OVERFLOW_KUNIT_TEST) += overflow_kunit.o > +obj-$(CONFIG_SIPHASH_KUNIT_TEST) += siphash_kunit.o > +obj-$(CONFIG_SLUB_KUNIT_TEST) += slub_kunit.o > +obj-$(CONFIG_TEST_SORT) += test_sort.o > +CFLAGS_stackinit_kunit.o += $(call cc-disable-warning, switch-unreachable) > +obj-$(CONFIG_STACKINIT_KUNIT_TEST) += stackinit_kunit.o > +obj-$(CONFIG_STRING_KUNIT_TEST) += string_kunit.o > +obj-$(CONFIG_STRING_HELPERS_KUNIT_TEST) += string_helpers_kunit.o > + > diff --git a/lib/bitfield_kunit.c b/lib/tests/bitfield_kunit.c > similarity index 100% > rename from lib/bitfield_kunit.c > rename to lib/tests/bitfield_kunit.c > diff --git a/lib/checksum_kunit.c b/lib/tests/checksum_kunit.c > similarity index 100% > rename from lib/checksum_kunit.c > rename to lib/tests/checksum_kunit.c > diff --git a/lib/cmdline_kunit.c b/lib/tests/cmdline_kunit.c > similarity index 100% > rename from lib/cmdline_kunit.c > rename to lib/tests/cmdline_kunit.c > diff --git a/lib/cpumask_kunit.c b/lib/tests/cpumask_kunit.c > similarity index 100% > rename from lib/cpumask_kunit.c > rename to lib/tests/cpumask_kunit.c > diff --git a/lib/fortify_kunit.c b/lib/tests/fortify_kunit.c > similarity index 100% > rename from lib/fortify_kunit.c > rename to lib/tests/fortify_kunit.c > diff --git a/lib/hashtable_test.c b/lib/tests/hashtable_test.c > similarity index 100% > rename from lib/hashtable_test.c > rename to lib/tests/hashtable_test.c > diff --git a/lib/is_signed_type_kunit.c b/lib/tests/is_signed_type_kunit.c > similarity index 100% > rename from lib/is_signed_type_kunit.c > rename to lib/tests/is_signed_type_kunit.c > diff --git a/lib/kunit_iov_iter.c b/lib/tests/kunit_iov_iter.c > similarity index 100% > rename from lib/kunit_iov_iter.c > rename to lib/tests/kunit_iov_iter.c > diff --git a/lib/list-test.c b/lib/tests/list-test.c > similarity index 100% > rename from lib/list-test.c > rename to lib/tests/list-test.c > diff --git a/lib/memcpy_kunit.c b/lib/tests/memcpy_kunit.c > similarity index 100% > rename from lib/memcpy_kunit.c > rename to lib/tests/memcpy_kunit.c > diff --git a/lib/overflow_kunit.c b/lib/tests/overflow_kunit.c > similarity index 100% > rename from lib/overflow_kunit.c > rename to lib/tests/overflow_kunit.c > diff --git a/lib/siphash_kunit.c b/lib/tests/siphash_kunit.c > similarity index 100% > rename from lib/siphash_kunit.c > rename to lib/tests/siphash_kunit.c > diff --git a/lib/slub_kunit.c b/lib/tests/slub_kunit.c > similarity index 100% > rename from lib/slub_kunit.c > rename to lib/tests/slub_kunit.c > diff --git a/lib/stackinit_kunit.c b/lib/tests/stackinit_kunit.c > similarity index 100% > rename from lib/stackinit_kunit.c > rename to lib/tests/stackinit_kunit.c > diff --git a/lib/string_helpers_kunit.c b/lib/tests/string_helpers_kunit.c > similarity index 100% > rename from lib/string_helpers_kunit.c > rename to lib/tests/string_helpers_kunit.c > diff --git a/lib/string_kunit.c b/lib/tests/string_kunit.c > similarity index 100% > rename from lib/string_kunit.c > rename to lib/tests/string_kunit.c > diff --git a/lib/test_bits.c b/lib/tests/test_bits.c > similarity index 100% > rename from lib/test_bits.c > rename to lib/tests/test_bits.c > diff --git a/lib/test_fprobe.c b/lib/tests/test_fprobe.c > similarity index 100% > rename from lib/test_fprobe.c > rename to lib/tests/test_fprobe.c > diff --git a/lib/test_hash.c b/lib/tests/test_hash.c > similarity index 100% > rename from lib/test_hash.c > rename to lib/tests/test_hash.c > diff --git a/lib/test_kprobes.c b/lib/tests/test_kprobes.c > similarity index 100% > rename from lib/test_kprobes.c > rename to lib/tests/test_kprobes.c > diff --git a/lib/test_linear_ranges.c b/lib/tests/test_linear_ranges.c > similarity index 100% > rename from lib/test_linear_ranges.c > rename to lib/tests/test_linear_ranges.c > diff --git a/lib/test_list_sort.c b/lib/tests/test_list_sort.c > similarity index 100% > rename from lib/test_list_sort.c > rename to lib/tests/test_list_sort.c > diff --git a/lib/test_sort.c b/lib/tests/test_sort.c > similarity index 100% > rename from lib/test_sort.c > rename to lib/tests/test_sort.c > -- > 2.34.1 > > -- > You received this message because you are subscribed to the Google Groups "KUnit Development" group. > To unsubscribe from this group and stop receiving emails from it, send an email to kunit-dev+unsubscribe@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/kunit-dev/20240720181025.work.002-kees%40kernel.org.
diff --git a/MAINTAINERS b/MAINTAINERS index 8754ac2c259d..3f4b9d007cbb 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3737,10 +3737,10 @@ F: include/vdso/bits.h F: lib/bitmap-str.c F: lib/bitmap.c F: lib/cpumask.c -F: lib/cpumask_kunit.c F: lib/find_bit.c F: lib/find_bit_benchmark.c F: lib/test_bitmap.c +F: lib/tests/cpumask_kunit.c F: tools/include/linux/bitfield.h F: tools/include/linux/bitmap.h F: tools/include/linux/bits.h @@ -8618,9 +8618,9 @@ L: linux-hardening@vger.kernel.org S: Supported T: git git://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git for-next/hardening F: include/linux/fortify-string.h -F: lib/fortify_kunit.c -F: lib/memcpy_kunit.c F: lib/test_fortify/* +F: lib/tests/fortify_kunit.c +F: lib/tests/memcpy_kunit.c F: scripts/test_fortify.sh K: \b__NO_FORTIFY\b @@ -9246,9 +9246,9 @@ F: include/linux/string.h F: include/linux/string_choices.h F: include/linux/string_helpers.h F: lib/string.c -F: lib/string_kunit.c F: lib/string_helpers.c -F: lib/string_helpers_kunit.c +F: lib/tests/string_helpers_kunit.c +F: lib/tests/string_kunit.c F: scripts/coccinelle/api/string_choices.cocci GENERIC UIO DRIVER FOR PCI DEVICES @@ -12347,7 +12347,7 @@ F: Documentation/trace/kprobes.rst F: include/asm-generic/kprobes.h F: include/linux/kprobes.h F: kernel/kprobes.c -F: lib/test_kprobes.c +F: lib/tests/test_kprobes.c F: samples/kprobes KS0108 LCD CONTROLLER DRIVER @@ -12697,7 +12697,7 @@ M: Mark Brown <broonie@kernel.org> R: Matti Vaittinen <mazziesaccount@gmail.com> F: include/linux/linear_range.h F: lib/linear_ranges.c -F: lib/test_linear_ranges.c +F: lib/tests/test_linear_ranges.c LINUX FOR POWER MACINTOSH L: linuxppc-dev@lists.ozlabs.org @@ -12824,7 +12824,7 @@ M: David Gow <davidgow@google.com> L: linux-kselftest@vger.kernel.org L: kunit-dev@googlegroups.com S: Maintained -F: lib/list-test.c +F: lib/tests/list-test.c LITEX PLATFORM M: Karol Gugala <kgugala@antmicro.com> @@ -20498,7 +20498,7 @@ M: Jason A. Donenfeld <Jason@zx2c4.com> S: Maintained F: include/linux/siphash.h F: lib/siphash.c -F: lib/siphash_kunit.c +F: lib/tests/siphash_kunit.c SIS 190 ETHERNET DRIVER M: Francois Romieu <romieu@fr.zoreil.com> diff --git a/lib/Makefile b/lib/Makefile index 3b1769045651..f00fe120ee9e 100644 --- a/lib/Makefile +++ b/lib/Makefile @@ -49,9 +49,7 @@ obj-y += bcd.o sort.o parser.o debug_locks.o random32.o \ percpu-refcount.o rhashtable.o base64.o \ once.o refcount.o rcuref.o usercopy.o errseq.o bucket_locks.o \ generic-radix-tree.o bitmap-str.o -obj-$(CONFIG_STRING_KUNIT_TEST) += string_kunit.o obj-y += string_helpers.o -obj-$(CONFIG_STRING_HELPERS_KUNIT_TEST) += string_helpers_kunit.o obj-y += hexdump.o obj-$(CONFIG_TEST_HEXDUMP) += test_hexdump.o obj-y += kstrtox.o @@ -62,22 +60,17 @@ obj-$(CONFIG_TEST_DHRY) += test_dhry.o obj-$(CONFIG_TEST_FIRMWARE) += test_firmware.o obj-$(CONFIG_TEST_BITOPS) += test_bitops.o CFLAGS_test_bitops.o += -Werror -obj-$(CONFIG_CPUMASK_KUNIT_TEST) += cpumask_kunit.o obj-$(CONFIG_TEST_SYSCTL) += test_sysctl.o -obj-$(CONFIG_TEST_IOV_ITER) += kunit_iov_iter.o -obj-$(CONFIG_HASH_KUNIT_TEST) += test_hash.o obj-$(CONFIG_TEST_IDA) += test_ida.o obj-$(CONFIG_TEST_UBSAN) += test_ubsan.o CFLAGS_test_ubsan.o += $(call cc-disable-warning, vla) CFLAGS_test_ubsan.o += $(call cc-disable-warning, unused-but-set-variable) UBSAN_SANITIZE_test_ubsan.o := y obj-$(CONFIG_TEST_KSTRTOX) += test-kstrtox.o -obj-$(CONFIG_TEST_LIST_SORT) += test_list_sort.o obj-$(CONFIG_TEST_MIN_HEAP) += test_min_heap.o obj-$(CONFIG_TEST_LKM) += test_module.o obj-$(CONFIG_TEST_VMALLOC) += test_vmalloc.o obj-$(CONFIG_TEST_RHASHTABLE) += test_rhashtable.o -obj-$(CONFIG_TEST_SORT) += test_sort.o obj-$(CONFIG_TEST_USER_COPY) += test_user_copy.o obj-$(CONFIG_TEST_STATIC_KEYS) += test_static_keys.o obj-$(CONFIG_TEST_STATIC_KEYS) += test_static_key_base.o @@ -104,10 +97,7 @@ obj-$(CONFIG_TEST_MEMINIT) += test_meminit.o obj-$(CONFIG_TEST_LOCKUP) += test_lockup.o obj-$(CONFIG_TEST_HMM) += test_hmm.o obj-$(CONFIG_TEST_FREE_PAGES) += test_free_pages.o -obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o obj-$(CONFIG_TEST_REF_TRACKER) += test_ref_tracker.o -CFLAGS_test_fprobe.o += $(CC_FLAGS_FTRACE) -obj-$(CONFIG_FPROBE_SANITY_TEST) += test_fprobe.o obj-$(CONFIG_TEST_OBJPOOL) += test_objpool.o obj-$(CONFIG_TEST_FPU) += test_fpu.o @@ -129,7 +119,7 @@ endif obj-$(CONFIG_DEBUG_INFO_REDUCED) += debug_info.o CFLAGS_debug_info.o += $(call cc-option, -femit-struct-debug-detailed=any) -obj-y += math/ crypto/ +obj-y += math/ crypto/ tests/ obj-$(CONFIG_GENERIC_IOMAP) += iomap.o obj-$(CONFIG_HAS_IOMEM) += iomap_copy.o devres.o @@ -366,29 +356,6 @@ obj-$(CONFIG_OBJAGG) += objagg.o # pldmfw library obj-$(CONFIG_PLDMFW) += pldmfw/ -# KUnit tests -CFLAGS_bitfield_kunit.o := $(DISABLE_STRUCTLEAK_PLUGIN) -obj-$(CONFIG_BITFIELD_KUNIT) += bitfield_kunit.o -obj-$(CONFIG_CHECKSUM_KUNIT) += checksum_kunit.o -obj-$(CONFIG_LIST_KUNIT_TEST) += list-test.o -obj-$(CONFIG_HASHTABLE_KUNIT_TEST) += hashtable_test.o -obj-$(CONFIG_LINEAR_RANGES_TEST) += test_linear_ranges.o -obj-$(CONFIG_BITS_TEST) += test_bits.o -obj-$(CONFIG_CMDLINE_KUNIT_TEST) += cmdline_kunit.o -obj-$(CONFIG_SLUB_KUNIT_TEST) += slub_kunit.o -obj-$(CONFIG_MEMCPY_KUNIT_TEST) += memcpy_kunit.o -obj-$(CONFIG_IS_SIGNED_TYPE_KUNIT_TEST) += is_signed_type_kunit.o -CFLAGS_overflow_kunit.o = $(call cc-disable-warning, tautological-constant-out-of-range-compare) -obj-$(CONFIG_OVERFLOW_KUNIT_TEST) += overflow_kunit.o -CFLAGS_stackinit_kunit.o += $(call cc-disable-warning, switch-unreachable) -obj-$(CONFIG_STACKINIT_KUNIT_TEST) += stackinit_kunit.o -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, unsequenced) -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-overread) -CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-truncation) -CFLAGS_fortify_kunit.o += $(DISABLE_STRUCTLEAK_PLUGIN) -obj-$(CONFIG_FORTIFY_KUNIT_TEST) += fortify_kunit.o -obj-$(CONFIG_SIPHASH_KUNIT_TEST) += siphash_kunit.o - obj-$(CONFIG_GENERIC_LIB_DEVMEM_IS_ALLOWED) += devmem_is_allowed.o obj-$(CONFIG_FIRMWARE_TABLE) += fw_table.o diff --git a/lib/tests/Makefile b/lib/tests/Makefile new file mode 100644 index 000000000000..c6a14cc8663e --- /dev/null +++ b/lib/tests/Makefile @@ -0,0 +1,37 @@ +# SPDX-License-Identifier: GPL-2.0 +# +# Makefile for tests of kernel library functions. + +# KUnit tests +CFLAGS_bitfield_kunit.o := $(DISABLE_STRUCTLEAK_PLUGIN) +obj-$(CONFIG_BITFIELD_KUNIT) += bitfield_kunit.o +obj-$(CONFIG_BITS_TEST) += test_bits.o +obj-$(CONFIG_CHECKSUM_KUNIT) += checksum_kunit.o +obj-$(CONFIG_CMDLINE_KUNIT_TEST) += cmdline_kunit.o +obj-$(CONFIG_CPUMASK_KUNIT_TEST) += cpumask_kunit.o +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, unsequenced) +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-overread) +CFLAGS_fortify_kunit.o += $(call cc-disable-warning, stringop-truncation) +CFLAGS_fortify_kunit.o += $(DISABLE_STRUCTLEAK_PLUGIN) +obj-$(CONFIG_FORTIFY_KUNIT_TEST) += fortify_kunit.o +CFLAGS_test_fprobe.o += $(CC_FLAGS_FTRACE) +obj-$(CONFIG_FPROBE_SANITY_TEST) += test_fprobe.o +obj-$(CONFIG_HASHTABLE_KUNIT_TEST) += hashtable_test.o +obj-$(CONFIG_HASH_KUNIT_TEST) += test_hash.o +obj-$(CONFIG_TEST_IOV_ITER) += kunit_iov_iter.o +obj-$(CONFIG_IS_SIGNED_TYPE_KUNIT_TEST) += is_signed_type_kunit.o +obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o +obj-$(CONFIG_LIST_KUNIT_TEST) += list-test.o +obj-$(CONFIG_TEST_LIST_SORT) += test_list_sort.o +obj-$(CONFIG_LINEAR_RANGES_TEST) += test_linear_ranges.o +obj-$(CONFIG_MEMCPY_KUNIT_TEST) += memcpy_kunit.o +CFLAGS_overflow_kunit.o = $(call cc-disable-warning, tautological-constant-out-of-range-compare) +obj-$(CONFIG_OVERFLOW_KUNIT_TEST) += overflow_kunit.o +obj-$(CONFIG_SIPHASH_KUNIT_TEST) += siphash_kunit.o +obj-$(CONFIG_SLUB_KUNIT_TEST) += slub_kunit.o +obj-$(CONFIG_TEST_SORT) += test_sort.o +CFLAGS_stackinit_kunit.o += $(call cc-disable-warning, switch-unreachable) +obj-$(CONFIG_STACKINIT_KUNIT_TEST) += stackinit_kunit.o +obj-$(CONFIG_STRING_KUNIT_TEST) += string_kunit.o +obj-$(CONFIG_STRING_HELPERS_KUNIT_TEST) += string_helpers_kunit.o + diff --git a/lib/bitfield_kunit.c b/lib/tests/bitfield_kunit.c similarity index 100% rename from lib/bitfield_kunit.c rename to lib/tests/bitfield_kunit.c diff --git a/lib/checksum_kunit.c b/lib/tests/checksum_kunit.c similarity index 100% rename from lib/checksum_kunit.c rename to lib/tests/checksum_kunit.c diff --git a/lib/cmdline_kunit.c b/lib/tests/cmdline_kunit.c similarity index 100% rename from lib/cmdline_kunit.c rename to lib/tests/cmdline_kunit.c diff --git a/lib/cpumask_kunit.c b/lib/tests/cpumask_kunit.c similarity index 100% rename from lib/cpumask_kunit.c rename to lib/tests/cpumask_kunit.c diff --git a/lib/fortify_kunit.c b/lib/tests/fortify_kunit.c similarity index 100% rename from lib/fortify_kunit.c rename to lib/tests/fortify_kunit.c diff --git a/lib/hashtable_test.c b/lib/tests/hashtable_test.c similarity index 100% rename from lib/hashtable_test.c rename to lib/tests/hashtable_test.c diff --git a/lib/is_signed_type_kunit.c b/lib/tests/is_signed_type_kunit.c similarity index 100% rename from lib/is_signed_type_kunit.c rename to lib/tests/is_signed_type_kunit.c diff --git a/lib/kunit_iov_iter.c b/lib/tests/kunit_iov_iter.c similarity index 100% rename from lib/kunit_iov_iter.c rename to lib/tests/kunit_iov_iter.c diff --git a/lib/list-test.c b/lib/tests/list-test.c similarity index 100% rename from lib/list-test.c rename to lib/tests/list-test.c diff --git a/lib/memcpy_kunit.c b/lib/tests/memcpy_kunit.c similarity index 100% rename from lib/memcpy_kunit.c rename to lib/tests/memcpy_kunit.c diff --git a/lib/overflow_kunit.c b/lib/tests/overflow_kunit.c similarity index 100% rename from lib/overflow_kunit.c rename to lib/tests/overflow_kunit.c diff --git a/lib/siphash_kunit.c b/lib/tests/siphash_kunit.c similarity index 100% rename from lib/siphash_kunit.c rename to lib/tests/siphash_kunit.c diff --git a/lib/slub_kunit.c b/lib/tests/slub_kunit.c similarity index 100% rename from lib/slub_kunit.c rename to lib/tests/slub_kunit.c diff --git a/lib/stackinit_kunit.c b/lib/tests/stackinit_kunit.c similarity index 100% rename from lib/stackinit_kunit.c rename to lib/tests/stackinit_kunit.c diff --git a/lib/string_helpers_kunit.c b/lib/tests/string_helpers_kunit.c similarity index 100% rename from lib/string_helpers_kunit.c rename to lib/tests/string_helpers_kunit.c diff --git a/lib/string_kunit.c b/lib/tests/string_kunit.c similarity index 100% rename from lib/string_kunit.c rename to lib/tests/string_kunit.c diff --git a/lib/test_bits.c b/lib/tests/test_bits.c similarity index 100% rename from lib/test_bits.c rename to lib/tests/test_bits.c diff --git a/lib/test_fprobe.c b/lib/tests/test_fprobe.c similarity index 100% rename from lib/test_fprobe.c rename to lib/tests/test_fprobe.c diff --git a/lib/test_hash.c b/lib/tests/test_hash.c similarity index 100% rename from lib/test_hash.c rename to lib/tests/test_hash.c diff --git a/lib/test_kprobes.c b/lib/tests/test_kprobes.c similarity index 100% rename from lib/test_kprobes.c rename to lib/tests/test_kprobes.c diff --git a/lib/test_linear_ranges.c b/lib/tests/test_linear_ranges.c similarity index 100% rename from lib/test_linear_ranges.c rename to lib/tests/test_linear_ranges.c diff --git a/lib/test_list_sort.c b/lib/tests/test_list_sort.c similarity index 100% rename from lib/test_list_sort.c rename to lib/tests/test_list_sort.c diff --git a/lib/test_sort.c b/lib/tests/test_sort.c similarity index 100% rename from lib/test_sort.c rename to lib/tests/test_sort.c
Following from the recent KUnit file naming discussion[1], move all KUnit tests in lib/ into lib/tests/. Link: https://lore.kernel.org/lkml/20240720165441.it.320-kees@kernel.org/ [1] Signed-off-by: Kees Cook <kees@kernel.org> --- I can carry this in the hardening tree. To disrupt people as little as possible, I'm hoping to send this either at the end of -rc1 or early in -rc2. Cc: Andrew Morton <akpm@linux-foundation.org> Cc: Yury Norov <yury.norov@gmail.com> Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk> Cc: David Gow <davidgow@google.com> Cc: "Jason A. Donenfeld" <Jason@zx2c4.com> Cc: Andy Shevchenko <andy@kernel.org> Cc: "Naveen N. Rao" <naveen.n.rao@linux.ibm.com> Cc: Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com> Cc: "David S. Miller" <davem@davemloft.net> Cc: Masami Hiramatsu <mhiramat@kernel.org> Cc: Mark Brown <broonie@kernel.org> Cc: Matti Vaittinen <mazziesaccount@gmail.com> Cc: linux-hardening@vger.kernel.org Cc: linux-kselftest@vger.kernel.org Cc: kunit-dev@googlegroups.com Cc: linux-trace-kernel@vger.kernel.org --- MAINTAINERS | 18 ++++++------- lib/Makefile | 35 +----------------------- lib/tests/Makefile | 37 ++++++++++++++++++++++++++ lib/{ => tests}/bitfield_kunit.c | 0 lib/{ => tests}/checksum_kunit.c | 0 lib/{ => tests}/cmdline_kunit.c | 0 lib/{ => tests}/cpumask_kunit.c | 0 lib/{ => tests}/fortify_kunit.c | 0 lib/{ => tests}/hashtable_test.c | 0 lib/{ => tests}/is_signed_type_kunit.c | 0 lib/{ => tests}/kunit_iov_iter.c | 0 lib/{ => tests}/list-test.c | 0 lib/{ => tests}/memcpy_kunit.c | 0 lib/{ => tests}/overflow_kunit.c | 0 lib/{ => tests}/siphash_kunit.c | 0 lib/{ => tests}/slub_kunit.c | 0 lib/{ => tests}/stackinit_kunit.c | 0 lib/{ => tests}/string_helpers_kunit.c | 0 lib/{ => tests}/string_kunit.c | 0 lib/{ => tests}/test_bits.c | 0 lib/{ => tests}/test_fprobe.c | 0 lib/{ => tests}/test_hash.c | 0 lib/{ => tests}/test_kprobes.c | 0 lib/{ => tests}/test_linear_ranges.c | 0 lib/{ => tests}/test_list_sort.c | 0 lib/{ => tests}/test_sort.c | 0 26 files changed, 47 insertions(+), 43 deletions(-) create mode 100644 lib/tests/Makefile rename lib/{ => tests}/bitfield_kunit.c (100%) rename lib/{ => tests}/checksum_kunit.c (100%) rename lib/{ => tests}/cmdline_kunit.c (100%) rename lib/{ => tests}/cpumask_kunit.c (100%) rename lib/{ => tests}/fortify_kunit.c (100%) rename lib/{ => tests}/hashtable_test.c (100%) rename lib/{ => tests}/is_signed_type_kunit.c (100%) rename lib/{ => tests}/kunit_iov_iter.c (100%) rename lib/{ => tests}/list-test.c (100%) rename lib/{ => tests}/memcpy_kunit.c (100%) rename lib/{ => tests}/overflow_kunit.c (100%) rename lib/{ => tests}/siphash_kunit.c (100%) rename lib/{ => tests}/slub_kunit.c (100%) rename lib/{ => tests}/stackinit_kunit.c (100%) rename lib/{ => tests}/string_helpers_kunit.c (100%) rename lib/{ => tests}/string_kunit.c (100%) rename lib/{ => tests}/test_bits.c (100%) rename lib/{ => tests}/test_fprobe.c (100%) rename lib/{ => tests}/test_hash.c (100%) rename lib/{ => tests}/test_kprobes.c (100%) rename lib/{ => tests}/test_linear_ranges.c (100%) rename lib/{ => tests}/test_list_sort.c (100%) rename lib/{ => tests}/test_sort.c (100%)