]> git.ipfire.org Git - thirdparty/systemd.git/commit
timer: Adjust calendar timers based on monotonic timer instead of realtime
authorFilipe Brandenburger <filbranden@gmail.com>
Fri, 10 Jul 2020 21:24:00 +0000 (14:24 -0700)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 15 Jul 2020 07:23:09 +0000 (09:23 +0200)
commit26698337f3842842af51cd007485f1dcd7c43cf2
tree1e343fcc6f691b370122a25359f70162826f7382
parentd3e0662c7d531748dd3626861a21c684857fc52e
timer: Adjust calendar timers based on monotonic timer instead of realtime

When the RTC time at boot is off in the future by a few days, OnCalendar=
timers will be scheduled based on the time at boot. But if the time has been
adjusted since boot, the timers will end up scheduled way in the future, which
may cause them not to fire as shortly or often as expected.

Update the logic so that the time will be adjusted based on monotonic time.
We do that by calculating the adjusted manager startup realtime from the
monotonic time stored at that time, by comparing that time with the realtime
and monotonic time of the current time.

Added a test case to validate this works as expected. The test case creates a
QEMU virtual machine with the clock 3 days in the future. Then we adjust the
clock back 3 days, and test creating a timer with an OnCalendar= for every 15
minutes. We also check the manager startup timestamp from both `systemd-analyze
dump` and from D-Bus.

Test output without the corresponding code changes that fix the issue:

  Timer elapse outside of the expected 20 minute window.
    next_elapsed=1594686119
    now=1594426921
    time_delta=259198

With the code changes in, the test passes as expected.
src/core/timer.c
test/TEST-53-ISSUE-16347/Makefile [new symlink]
test/TEST-53-ISSUE-16347/test.sh [new file with mode: 0755]
test/units/testsuite-53.service [new file with mode: 0644]
test/units/testsuite-53.sh [new file with mode: 0755]