From a87c1d3a979f8c2641471bed93577558a1027a24 Mon Sep 17 00:00:00 2001 From: Taro Yamada Date: Tue, 19 Feb 2019 21:01:50 +0900 Subject: [PATCH] core: delay persistent timers by "RandomizedDelaySec=" at boot. Fixes #5659. Currently, if Persistent=true and the machine is off at the scheduled time of the timer unit, the timer will be triggered immediately at the next boot even if RandomizedDelaySec= is specified. As a result, if multiple timers meet that condition, they will be triggered at the same time and too much CPU/IO work makes boot slow down. With this commit, if the scheduled time of the persistent timer has already elapsed at boot, set the time when systemd first started as the scheduled time and RandomizedDelaySec= is applied to it. --- src/core/timer.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/core/timer.c b/src/core/timer.c index d9ba2f7..0c1ca3e 100644 --- a/src/core/timer.c +++ b/src/core/timer.c @@ -380,6 +380,13 @@ static void timer_enter_waiting(Timer *t, bool time_change) { if (r < 0) continue; + /* To make the delay due to RandomizedDelaySec= work even at boot, + * if the scheduled time has already passed, set the time when systemd + * first started as the scheduled time. + * Also, we don't have to check t->persistent since the logic implicitly express true. */ + if (v->next_elapse < UNIT(t)->manager->timestamps[MANAGER_TIMESTAMP_USERSPACE].realtime) + v->next_elapse = UNIT(t)->manager->timestamps[MANAGER_TIMESTAMP_USERSPACE].realtime; + if (!found_realtime) t->next_elapse_realtime = v->next_elapse; else -- 2.7.4