[LTP] [PATCH v2 2/2] Add clock_settime04 test
Andrea Cervesato
andrea.cervesato@suse.de
Mon Jun 30 13:56:35 CEST 2025
From: Andrea Cervesato <andrea.cervesato@suse.com>
Test that changing the value of the CLOCK_MONOTONIC clock via
clock_settime(2) shall have no effect on a thread that is blocked
on a relative/absolute clock_nanosleep().
Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
---
runtest/syscalls | 1 +
testcases/kernel/syscalls/clock_settime/.gitignore | 1 +
.../syscalls/clock_settime/clock_settime04.c | 153 +++++++++++++++++++++
3 files changed, 155 insertions(+)
diff --git a/runtest/syscalls b/runtest/syscalls
index d5582ca8da01f11adba75a56d22cb6e5c4996c68..2dc734c5537f1d5477f4c98d08b9717fb89ac05c 100644
--- a/runtest/syscalls
+++ b/runtest/syscalls
@@ -112,6 +112,7 @@ leapsec01 leapsec01
clock_settime01 clock_settime01
clock_settime02 clock_settime02
clock_settime03 clock_settime03
+clock_settime04 clock_settime04
clone01 clone01
clone02 clone02
diff --git a/testcases/kernel/syscalls/clock_settime/.gitignore b/testcases/kernel/syscalls/clock_settime/.gitignore
index b66169b3eb7b4d8c8ea95e9e689b612d8da37b11..8bcc83d6fc9162087e99193a00b8d3d784d4737d 100644
--- a/testcases/kernel/syscalls/clock_settime/.gitignore
+++ b/testcases/kernel/syscalls/clock_settime/.gitignore
@@ -1,3 +1,4 @@
clock_settime01
clock_settime02
clock_settime03
+clock_settime04
diff --git a/testcases/kernel/syscalls/clock_settime/clock_settime04.c b/testcases/kernel/syscalls/clock_settime/clock_settime04.c
new file mode 100644
index 0000000000000000000000000000000000000000..bea42dd8d75a83b346952fc7d54d19e811ac742e
--- /dev/null
+++ b/testcases/kernel/syscalls/clock_settime/clock_settime04.c
@@ -0,0 +1,153 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Copyright (c) 2025 Andrea Cervesato <andrea.cervesato@suse.com>
+ */
+
+/*\
+ * Verify that changing the value of the CLOCK_MONOTONIC clock via
+ * clock_settime() shall have no effect on a thread that is blocked on
+ * absolute/relative clock_nanosleep().
+ */
+
+#include "tst_test.h"
+#include "tst_timer.h"
+#include "tst_safe_clocks.h"
+#include "time64_variants.h"
+
+#define SEC_TO_US(x) (x * 1000 * 1000)
+
+#define CHILD_SLEEP_US SEC_TO_US(5)
+#define PARENT_SLEEP_US SEC_TO_US(2)
+#define DELTA_US SEC_TO_US(1)
+
+static struct tst_ts *begin, *sleep_child, *sleep_parent, *end;
+
+static struct time64_variants variants[] = {
+ {
+ .clock_gettime = libc_clock_gettime,
+ .clock_settime = libc_clock_settime,
+ .clock_nanosleep = libc_clock_nanosleep,
+ .ts_type = TST_LIBC_TIMESPEC,
+ .desc = "vDSO or syscall with libc spec"
+ },
+
+#if (__NR_clock_nanosleep != __LTP__NR_INVALID_SYSCALL)
+ {
+ .clock_gettime = sys_clock_gettime,
+ .clock_settime = sys_clock_settime,
+ .clock_nanosleep = sys_clock_nanosleep,
+ .ts_type = TST_KERN_OLD_TIMESPEC,
+ .desc = "syscall with old kernel spec"
+ },
+#endif
+
+#if (__NR_clock_nanosleep_time64 != __LTP__NR_INVALID_SYSCALL)
+ {
+ .clock_gettime = sys_clock_gettime64,
+ .clock_settime = sys_clock_settime64,
+ .clock_nanosleep = sys_clock_nanosleep64,
+ .ts_type = TST_KERN_TIMESPEC,
+ .desc = "syscall time64 with kernel spec"
+ },
+#endif
+};
+
+static void child_nanosleep(struct time64_variants *tv, const int flags)
+{
+ long long delta;
+
+ TEST(tv->clock_gettime(CLOCK_MONOTONIC, tst_ts_get(begin)));
+ if (TST_RET)
+ tst_brk(TBROK | TERRNO, "clock_gettime() error");
+
+ if (flags & TIMER_ABSTIME) {
+ tst_res(TINFO, "Using absolute time sleep");
+
+ *sleep_child = tst_ts_add_us(*begin, CHILD_SLEEP_US);
+ } else {
+ tst_res(TINFO, "Using relative time sleep");
+
+ tst_ts_set_sec(sleep_child, 0);
+ tst_ts_set_nsec(sleep_child, 0);
+
+ *sleep_child = tst_ts_add_us(*sleep_child, CHILD_SLEEP_US);
+ }
+
+ TST_CHECKPOINT_WAKE(0);
+
+ TEST(tv->clock_nanosleep(CLOCK_MONOTONIC, flags, tst_ts_get(sleep_child), NULL));
+ if (TST_RET)
+ tst_brk(TBROK | TERRNO, "clock_nanosleep() error");
+
+ TEST(tv->clock_gettime(CLOCK_MONOTONIC, tst_ts_get(end)));
+ if (TST_RET)
+ tst_brk(TBROK | TERRNO, "clock_gettime() error");
+
+ if (tst_ts_lt(*end, *begin)) {
+ tst_res(TFAIL, "clock_settime() didn't sleep enough. "
+ "begin: %lld ms >= end: %lld ms",
+ tst_ts_to_ms(*begin),
+ tst_ts_to_ms(*end));
+ return;
+ }
+
+ delta = tst_ts_abs_diff_us(*begin, *end) - CHILD_SLEEP_US;
+ if (delta > DELTA_US) {
+ tst_res(TFAIL, "parent clock_settime() affected child sleep. "
+ "begin: %lld ms, end: %lld ms",
+ tst_ts_to_ms(*begin),
+ tst_ts_to_ms(*end));
+ return;
+ }
+
+ tst_res(TPASS, "parent clock_settime() didn't affect child sleep "
+ "(delta time: %lld us)", delta);
+}
+
+static void run(unsigned int tc_index)
+{
+ struct time64_variants *tv = &variants[tst_variant];
+
+ if (!SAFE_FORK()) {
+ child_nanosleep(tv, tc_index ? TIMER_ABSTIME : 0);
+ exit(0);
+ }
+
+ SAFE_CLOCK_GETTIME(CLOCK_REALTIME, tst_ts_get(begin));
+
+ TST_CHECKPOINT_WAIT(0);
+
+ SAFE_CLOCK_NANOSLEEP(CLOCK_REALTIME, 0, tst_ts_get(sleep_parent), NULL);
+ SAFE_CLOCK_SETTIME(CLOCK_REALTIME, tst_ts_get(begin));
+}
+
+static void setup(void)
+{
+ begin->type = end->type = sleep_child->type = sleep_parent->type =
+ variants[tst_variant].ts_type;
+
+ tst_ts_set_sec(sleep_parent, 0);
+ tst_ts_set_nsec(sleep_parent, 0);
+
+ *sleep_parent = tst_ts_add_us(*sleep_parent, PARENT_SLEEP_US);
+
+ tst_res(TINFO, "Testing variant: %s", variants[tst_variant].desc);
+}
+
+static struct tst_test test = {
+ .test = run,
+ .setup = setup,
+ .tcnt = 2,
+ .needs_root = 1,
+ .forks_child = 1,
+ .needs_checkpoints = 1,
+ .restore_wallclock = 1,
+ .test_variants = ARRAY_SIZE(variants),
+ .bufs = (struct tst_buffers []) {
+ {&sleep_child, .size = sizeof(struct tst_ts)},
+ {&sleep_parent, .size = sizeof(struct tst_ts)},
+ {&begin, .size = sizeof(struct tst_ts)},
+ {&end, .size = sizeof(struct tst_ts)},
+ {},
+ }
+};
--
2.50.0
More information about the ltp
mailing list