diff options
author | David Gow <davidgow@google.com> | 2022-11-25 09:43:05 +0100 |
---|---|---|
committer | Shuah Khan <skhan@linuxfoundation.org> | 2022-12-12 22:13:47 +0100 |
commit | 91e93592219f74c4d5cd4f27006d726ac86ae15d (patch) | |
tree | 085447ad7679587245dfed13d28fd37da9b16d84 /block/blk-settings.c | |
parent | kunit: Provide a static key to check if KUnit is actively running tests (diff) | |
download | linux-91e93592219f74c4d5cd4f27006d726ac86ae15d.tar.xz linux-91e93592219f74c4d5cd4f27006d726ac86ae15d.zip |
kunit: Use the static key when retrieving the current test
In order to detect if a KUnit test is running, and to access its
context, the 'kunit_test' member of the current task_struct is used.
Usually, this is accessed directly or via the kunit_fail_current_task()
function.
In order to speed up the case where no test is running, add a wrapper,
kunit_get_current_test(), which uses the static key to fail early.
Equally, Speed up kunit_fail_current_test() by using the static key.
This should make it convenient for code to call this
unconditionally in fakes or error paths, without worrying that this will
slow the code down significantly.
If CONFIG_KUNIT=n (or m), this compiles away to nothing. If
CONFIG_KUNIT=y, it will compile down to a NOP (on most architectures) if
no KUnit test is currently running.
Note that kunit_get_current_test() does not work if KUnit is built as a
module. This mirrors the existing restriction on kunit_fail_current_test().
Note that the definition of kunit_fail_current_test() still wraps an
empty, inline function if KUnit is not built-in. This is to ensure that
the printf format string __attribute__ will still work.
Also update the documentation to suggest users use the new
kunit_get_current_test() function, update the example, and to describe
the behaviour when KUnit is disabled better.
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Sadiya Kazi <sadiyakazi@google.com>
Signed-off-by: David Gow <davidgow@google.com>
Reviewed-by: Daniel Latypov <dlatypov@google.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'block/blk-settings.c')
0 files changed, 0 insertions, 0 deletions