diff options
| author | Guenter Roeck <linux@roeck-us.net> | 2025-07-21 16:19:17 -0700 |
|---|---|---|
| committer | Eric Biggers <ebiggers@kernel.org> | 2025-07-21 20:10:36 -0700 |
| commit | 8cd876e7833758f8a924db2e39530d2a9afd2703 (patch) | |
| tree | c4239f5e7964d639a654f3451a71d154805e4460 /lib | |
| parent | lib/crypto: tests: Add KUnit tests for SHA-1 and HMAC-SHA1 (diff) | |
| download | linux-8cd876e7833758f8a924db2e39530d2a9afd2703.tar.gz linux-8cd876e7833758f8a924db2e39530d2a9afd2703.zip | |
lib/crypto: tests: Annotate worker to be on stack
The following warning traceback is seen if object debugging is enabled
with the new crypto test code.
ODEBUG: object 9000000106237c50 is on stack 9000000106234000, but NOT annotated.
------------[ cut here ]------------
WARNING: lib/debugobjects.c:655 at lookup_object_or_alloc.part.0+0x19c/0x1f4, CPU#0: kunit_try_catch/468
...
This also results in a boot stall when running the code in qemu:loongarch.
Initializing the worker with INIT_WORK_ONSTACK() fixes the problem.
Fixes: 950a81224e8b ("lib/crypto: tests: Add hash-test-template.h and gen-hash-testvecs.py")
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Link: https://lore.kernel.org/r/20250721231917.3182029-1-linux@roeck-us.net
Signed-off-by: Eric Biggers <ebiggers@kernel.org>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/crypto/tests/hash-test-template.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/crypto/tests/hash-test-template.h b/lib/crypto/tests/hash-test-template.h index ffee1741a1b3..f437a0a9ac6c 100644 --- a/lib/crypto/tests/hash-test-template.h +++ b/lib/crypto/tests/hash-test-template.h @@ -398,7 +398,7 @@ static void run_irq_test(struct kunit *test, bool (*func)(void *), */ hrtimer_setup_on_stack(&state.timer, hash_irq_test_timer_func, CLOCK_MONOTONIC, HRTIMER_MODE_REL_HARD); - INIT_WORK(&state.bh_work, hash_irq_test_bh_work_func); + INIT_WORK_ONSTACK(&state.bh_work, hash_irq_test_bh_work_func); /* Run for up to max_iterations or 1 second, whichever comes first. */ end_jiffies = jiffies + HZ; |