2020-08-04 20:48:52

by Brendan Higgins

[permalink] [raw]
Subject: [PATCH v6 5/5] Documentation: kunit: add a brief blurb about kunit_test_suite

Add a brief blurb saying how and when the kunit_test_suite() macro
works to the usage documentation.

Signed-off-by: Brendan Higgins <[email protected]>
---
Documentation/dev-tools/kunit/usage.rst | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/dev-tools/kunit/usage.rst b/Documentation/dev-tools/kunit/usage.rst
index 3c3fe8b5feccf..961d3ea3ca19a 100644
--- a/Documentation/dev-tools/kunit/usage.rst
+++ b/Documentation/dev-tools/kunit/usage.rst
@@ -211,6 +211,11 @@ KUnit test framework.
.. note::
A test case will only be run if it is associated with a test suite.

+``kunit_test_suite(...)`` is a macro which tells the linker to put the specified
+test suite in a special linker section so that it can be run by KUnit either
+after late_init, or when the test module is loaded (depending on whether the
+test was built in or not).
+
For more information on these types of things see the :doc:`api/test`.

Isolating Behavior
--
2.28.0.163.g6104cc2f0b6-goog


2020-08-05 06:02:40

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH v6 5/5] Documentation: kunit: add a brief blurb about kunit_test_suite

Quoting Brendan Higgins (2020-08-04 13:47:45)
> Add a brief blurb saying how and when the kunit_test_suite() macro
> works to the usage documentation.
>
> Signed-off-by: Brendan Higgins <[email protected]>
> ---
> Documentation/dev-tools/kunit/usage.rst | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/dev-tools/kunit/usage.rst b/Documentation/dev-tools/kunit/usage.rst
> index 3c3fe8b5feccf..961d3ea3ca19a 100644
> --- a/Documentation/dev-tools/kunit/usage.rst
> +++ b/Documentation/dev-tools/kunit/usage.rst
> @@ -211,6 +211,11 @@ KUnit test framework.
> .. note::
> A test case will only be run if it is associated with a test suite.
>
> +``kunit_test_suite(...)`` is a macro which tells the linker to put the specified
> +test suite in a special linker section so that it can be run by KUnit either
> +after late_init, or when the test module is loaded (depending on whether the

"late_init" probably needs some italics or something, but otherwise.

Reviewed-by: Stephen Boyd <[email protected]>