Message ID | 20220804000626.1098734-1-edliaw@google.com |
---|---|
State | Accepted |
Headers | show |
Series | [v1] syscalls/timerfd04: incorrect CLOCK_BOOTIME end | expand |
Hi Edward, > The end time was hardcoded to CLOCK_MONOTONIC and needs to be changed to > the clk_id of the test case. Good catch! This bug has been since ever, probably because reproducible only on bare metal SUT (wrong clock works on VM). Thanks, merged! Kind regards, Petr
diff --git a/testcases/kernel/syscalls/timerfd/timerfd04.c b/testcases/kernel/syscalls/timerfd/timerfd04.c index 4af91d6f8..eb7f98d5b 100644 --- a/testcases/kernel/syscalls/timerfd/timerfd04.c +++ b/testcases/kernel/syscalls/timerfd/timerfd04.c @@ -94,9 +94,9 @@ static void verify_timerfd(unsigned int n) SAFE_WAIT(NULL); - if (tv->clock_gettime(CLOCK_MONOTONIC, tst_ts_get(&end))) { + if (tv->clock_gettime(tc->clk_id, tst_ts_get(&end))) { tst_res(TFAIL | TERRNO, "clock_gettime(2) failed for clock %s", - tst_clock_name(CLOCK_MONOTONIC)); + tst_clock_name(tc->clk_id)); return; }
The end time was hardcoded to CLOCK_MONOTONIC and needs to be changed to the clk_id of the test case. Signed-off-by: Edward Liaw <edliaw@google.com> Change-Id: I28c5b7d0132bf32cae6c50faa7d300ded42cc9f7 --- testcases/kernel/syscalls/timerfd/timerfd04.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)