slice, scope: IgnoreOnIsolate=yes is already the default

`IgnoreOnIsolate=yes` is the default for slices and scopes.  So it's not
essential to set it on root.slice or init.scope.

We don't need to worry about a bad unit file configuration.  Any attempt
to stop these unit should fail, since we mark them as `perpetual`.

Also since init.scope cannot be stopped, there is no point setting
`KillSignal=SIGRTMIN+14`.  According to both documentation and testing,
KillSignal= does not affect the behaviour of `systemctl kill`.

diff --git a/src/core/scope.c b/src/core/scope.c
index 10454d56b04651e7fe45c71b3774ccae849388ff..c49708ae4c6c859de05bb0b29afdc88788e8846c 100644 (file)
--- a/src/core/scope.c
+++ b/src/core/scope.c
@@ -163,9 +163,6 @@ static int scope_load_init_scope(Unit *u) {
          * synthesize it here, instead of relying on the unit file on disk. */
 
         u->default_dependencies = false;
-        u->ignore_on_isolate = true;
-
-        SCOPE(u)->kill_context.kill_signal = SIGRTMIN+14;
 
         /* Prettify things, if we can. */
         if (!u->description)

diff --git a/src/core/slice.c b/src/core/slice.c
index fef47b04fee1d2dd08d15651daa8fe744ccd56cb..0bbc0aedc57d76de57caef02075d344ec0040627 100644 (file)
--- a/src/core/slice.c
+++ b/src/core/slice.c
@@ -137,7 +137,6 @@ static int slice_load_root_slice(Unit *u) {
          * special semantics we synthesize it here, instead of relying on the unit file on disk. */
 
         u->default_dependencies = false;
-        u->ignore_on_isolate = true;
 
         if (!u->description)
                 u->description = strdup("Root Slice");