From 3c6a000d5c291f998dba2eb4c8e351f4603ca985 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Sun, 2 Nov 2014 11:39:17 -0500 Subject: [PATCH] unit: do not order timers.target before basic.target Since commit 19f8d037833f2 'timer: order OnCalendar units after timer-sync.target if DefaultDependencies=no' timers might get a dependency on time-sync.target, which does not really belong in early boot. If ntp is enabled, time-sync.target might be delayed until a network connection is established. It turns out that majority of timer units found in the wild do not need to be started in early boot. Out of the timer units available in Fedora 21, only systemd-readahead-done.timer and mdadm-last-resort@.timer should be started early, but they both have DefaultDependencies=no, so are not part of timers.target anyway. All the rest look like they will be fine with being started a bit later (and the majority even much later, since they run daily or weekly). Let timers.target be pulled in by basic.target, but without the temporal dependency. This means timer units are started on a "best effort" schedule. https://bugzilla.redhat.com/show_bug.cgi?id=1158206 (cherry picked from commit 3b0217036040a6013faeab4eb9da7469e3bbcfb3) Conflicts: units/basic.target --- units/basic.target | 4 +++- units/timers.target | 3 +++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/units/basic.target b/units/basic.target index b890d48bbc..abb63ec560 100644 --- a/units/basic.target +++ b/units/basic.target @@ -8,6 +8,8 @@ [Unit] Description=Basic System Documentation=man:systemd.special(7) + Requires=sysinit.target +After=sysinit.target Wants=sockets.target timers.target paths.target slices.target -After=sysinit.target sockets.target timers.target paths.target slices.target +After=sockets.target paths.target slices.target diff --git a/units/timers.target b/units/timers.target index 07fda3d9d0..251fa68065 100644 --- a/units/timers.target +++ b/units/timers.target @@ -8,3 +8,6 @@ [Unit] Description=Timers Documentation=man:systemd.special(7) + +DefaultDependencies=no +Conflicts=shutdown.target