[PATCH v2 11/25] timekeeping: Introduce tkd_basic_setup() to make lock and seqcount init reusable

From: Anna-Maria Behnsen
Date: Wed Oct 09 2024 - 04:31:22 EST


Initialization of lock and seqcount needs to be done for every instance of
timekeeper struct. To be able to easily reuse it, create a separate
function for it.

Signed-off-by: Anna-Maria Behnsen <anna-maria@xxxxxxxxxxxxx>
---
v2: new in v2 - splitted from another patch
---
kernel/time/timekeeping.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/kernel/time/timekeeping.c b/kernel/time/timekeeping.c
index 40c60bb88416..a98f823be6db 100644
--- a/kernel/time/timekeeping.c
+++ b/kernel/time/timekeeping.c
@@ -1734,6 +1734,12 @@ read_persistent_wall_and_boot_offset(struct timespec64 *wall_time,
*boot_offset = ns_to_timespec64(local_clock());
}

+static __init void tkd_basic_setup(struct tk_data *tkd)
+{
+ raw_spin_lock_init(&tkd->lock);
+ seqcount_raw_spinlock_init(&tkd->seq, &tkd->lock);
+}
+
/*
* Flag reflecting whether timekeeping_resume() has injected sleeptime.
*
@@ -1761,8 +1767,7 @@ void __init timekeeping_init(void)
struct timekeeper *tk = &tk_core.timekeeper;
struct clocksource *clock;

- raw_spin_lock_init(&tk_core.lock);
- seqcount_raw_spinlock_init(&tk_core.seq, &tkd->lock);
+ tkd_basic_setup(&tk_core);

read_persistent_wall_and_boot_offset(&wall_time, &boot_offset);
if (timespec64_valid_settod(&wall_time) &&

--
2.39.5