]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
kunit: test: Drop CONFIG_MODULE ifdeffery
authorThomas Weißschuh <thomas.weissschuh@linutronix.de>
Fri, 11 Jul 2025 13:31:38 +0000 (15:31 +0200)
committerDaniel Gomez <da.gomez@samsung.com>
Thu, 31 Jul 2025 11:51:34 +0000 (13:51 +0200)
The function stubs exposed by module.h allow the code to compile properly
without the ifdeffery. The generated object code stays the same, as the
compiler can optimize away all the dead code.
As the code is still typechecked developer errors can be detected faster.

Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
Acked-by: David Gow <davidgow@google.com>
Reviewed-by: Daniel Gomez <da.gomez@samsung.com>
Link: https://lore.kernel.org/r/20250711-kunit-ifdef-modules-v2-3-39443decb1f8@linutronix.de
Signed-off-by: Daniel Gomez <da.gomez@samsung.com>
lib/kunit/test.c

index f3c6b11f12b81d1c81ea581305b685f115d2bf7f..d2bfa331a2b1eb4a793b99ba7822f1a5a9831219 100644 (file)
@@ -802,7 +802,6 @@ void __kunit_test_suites_exit(struct kunit_suite **suites, int num_suites)
 }
 EXPORT_SYMBOL_GPL(__kunit_test_suites_exit);
 
-#ifdef CONFIG_MODULES
 static void kunit_module_init(struct module *mod)
 {
        struct kunit_suite_set suite_set, filtered_set;
@@ -890,7 +889,6 @@ static struct notifier_block kunit_mod_nb = {
        .notifier_call = kunit_module_notify,
        .priority = 0,
 };
-#endif
 
 KUNIT_DEFINE_ACTION_WRAPPER(kfree_action_wrapper, kfree, const void *)
 
@@ -981,20 +979,14 @@ static int __init kunit_init(void)
        kunit_debugfs_init();
 
        kunit_bus_init();
-#ifdef CONFIG_MODULES
        return register_module_notifier(&kunit_mod_nb);
-#else
-       return 0;
-#endif
 }
 late_initcall(kunit_init);
 
 static void __exit kunit_exit(void)
 {
        memset(&kunit_hooks, 0, sizeof(kunit_hooks));
-#ifdef CONFIG_MODULES
        unregister_module_notifier(&kunit_mod_nb);
-#endif
 
        kunit_bus_shutdown();