diff options
| author | Petr Mladek <pmladek@suse.com> | 2025-05-12 16:38:02 +0200 |
|---|---|---|
| committer | Paul E. McKenney <paulmck@kernel.org> | 2025-06-24 05:47:35 -0700 |
| commit | 5a5c5a3de1c7af579a9670b7d4b0169120ec92a7 (patch) | |
| tree | 5cb7df217bbd3c1ea7a314ae9ed2d86207fc2558 | |
| parent | d19e9fa61f609978af9b03f1d20fa59dd8c9997d (diff) | |
lib: Make the ratelimit test more reliable
The selftest fails most of the times when running in qemu with
a kernel configured with CONFIG_HZ = 250:
> test_ratelimit_smoke: 1 callbacks suppressed
> # test_ratelimit_smoke: ASSERTION FAILED at lib/tests/test_ratelimit.c:28
> Expected ___ratelimit(&testrl, "test_ratelimit_smoke") == (false), but
> ___ratelimit(&testrl, "test_ratelimit_smoke") == 1 (0x1)
> (false) == 0 (0x0)
Try to make the test slightly more reliable by calling the problematic
ratelimit in the middle of the interval.
Signed-off-by: Petr Mladek <pmladek@suse.com>
Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
| -rw-r--r-- | lib/tests/test_ratelimit.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/tests/test_ratelimit.c b/lib/tests/test_ratelimit.c index 0374107f5ea8..5d6ec8854600 100644 --- a/lib/tests/test_ratelimit.c +++ b/lib/tests/test_ratelimit.c @@ -24,19 +24,19 @@ static void test_ratelimit_smoke(struct kunit *test) test_ratelimited(test, true); test_ratelimited(test, false); - schedule_timeout_idle(TESTRL_INTERVAL - 40); + schedule_timeout_idle(TESTRL_INTERVAL / 2); test_ratelimited(test, false); - schedule_timeout_idle(50); + schedule_timeout_idle(TESTRL_INTERVAL * 3 / 4); test_ratelimited(test, true); schedule_timeout_idle(2 * TESTRL_INTERVAL); test_ratelimited(test, true); test_ratelimited(test, true); - schedule_timeout_idle(TESTRL_INTERVAL - 40); + schedule_timeout_idle(TESTRL_INTERVAL / 2 ); test_ratelimited(test, true); - schedule_timeout_idle(50); + schedule_timeout_idle(TESTRL_INTERVAL * 3 / 4); test_ratelimited(test, true); test_ratelimited(test, true); test_ratelimited(test, true); |