On 5/30/25 10:59, Shuah Khan wrote:
On 5/30/25 07:58, Jakub Kicinski wrote:
This reverts commit a571a9a1b120264e24b41eddf1ac5140131bfa84.
The commit in question breaks kunit for older compilers:
> $ gcc --version
gcc (GCC) 11.5.0 20240719 (Red Hat 11.5.0-5)
$ ./tools/testing/kunit/kunit.py run --alltests --json --arch=x86_64
Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=x86_64 O=.kunit olddefconfig
ERROR:root:Not all Kconfig options selected in kunitconfig were in the
generated .config.
This is probably due to unsatisfied dependencies.
Missing: CONFIG_INIT_STACK_ALL_PATTERN=y
Does adding config option work for you?
./tools/testing/kunit/kunit.py run --kconfig_add CONFIG_INIT_STACK_ALL_PATTERN
Link: https://lore.kernel.org/20250529083811.778bc...@kernel.org
Fixes: a571a9a1b120 ("kunit: configs: Enable CONFIG_INIT_STACK_ALL_PATTERN in
all_tests")
Signed-off-by: Jakub Kicinski <k...@kernel.org>
---
I'd like to take this in via netdev since it fixes our CI.
We'll send it to Linus next week.
I am good with reverting it for now.
Meant to add Ack.
Acked-by: Shuah Khan <sk...@linuxfoundation.org>
David, Brendan,
We will have to enable this at a later time. Also we saw this problem
before with other configs. Anyway way to fix this for alltests case?
thanks,
-- Shuah