mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-31 03:35:32 -04:00
Documentation: kunit: Test Kconfig entries shouldn't select other configs
Add a note to the Kconfig section of style.rst to use 'depends on' rather than 'selects' for dependencies, as this can cause users of CONFIG_KUNIT_ALL_TESTS to suddenly grow unexpected dependencies. Link: https://lore.kernel.org/r/20260627082921.1709181-1-david@davidgow.net Signed-off-by: David Gow <david@davidgow.net> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
This commit is contained in:
@@ -164,9 +164,10 @@ This Kconfig entry must:
|
||||
* be visible only if ``CONFIG_KUNIT_ALL_TESTS`` is not enabled.
|
||||
* have a default value of ``CONFIG_KUNIT_ALL_TESTS``.
|
||||
* have a brief description of KUnit in the help text.
|
||||
|
||||
If we are not able to meet above conditions (for example, the test is unable to
|
||||
be built as a module), Kconfig entries for tests should be tristate.
|
||||
* depend on the feature being tested, rather than selecting it (so that
|
||||
enabling ``CONFIG_KUNIT_ALL_TESTS`` does not enable unrelated functionality).
|
||||
* be ``tristate``, unless there is a specific reason that the test cannot be
|
||||
built as a module.
|
||||
|
||||
For example, a Kconfig entry might look like:
|
||||
|
||||
|
||||
Reference in New Issue
Block a user