Re: [RFC v1 4/6] init: main: add KUnit to kernel init

From: Stephen Boyd
Date: Tue Dec 17 2019 - 02:58:39 EST


Quoting Brendan Higgins (2019-12-16 14:05:53)
> Remove KUnit from init calls entirely, instead call directly from
> kernel_init().

Yes, but why? Is it desired to run the unit tests earlier than opening
the console or something?

>
> diff --git a/lib/kunit/executor.c b/lib/kunit/executor.c
> index 978086cfd257d..ca880224c0bab 100644
> --- a/lib/kunit/executor.c
> +++ b/lib/kunit/executor.c
> @@ -32,12 +32,10 @@ static bool kunit_run_all_tests(void)
> return !has_test_failed;
> }
>
> -static int kunit_executor_init(void)
> +int kunit_executor_init(void)

Should be marked __init? Even before this patch presumably.

> {
> if (kunit_run_all_tests())
> return 0;
> else
> return -EFAULT;
> }
> -
> -late_initcall(kunit_executor_init);