65878a
From adc1b71d84ab8b618c623749f43dc6350d75de6b Mon Sep 17 00:00:00 2001
65878a
From: Michal Sekletar <msekleta@redhat.com>
65878a
Date: Tue, 4 Mar 2014 17:00:54 +0100
65878a
Subject: [PATCH] logind: given that we can now relatively safely shutdown
65878a
 sessions copes without working cgroup empty notifications there's no need to
65878a
 set the stop timeout of sessions scopes low
65878a
65878a
Based-on: a50df72b37ce2a7caf7775c70d18c3f9504b9e80
65878a
---
65878a
 src/login/logind-dbus.c | 11 +----------
65878a
 1 file changed, 1 insertion(+), 10 deletions(-)
65878a
65878a
diff --git a/src/login/logind-dbus.c b/src/login/logind-dbus.c
65878a
index a68e9b4..6cabb7b 100644
65878a
--- a/src/login/logind-dbus.c
65878a
+++ b/src/login/logind-dbus.c
65878a
@@ -2549,10 +2549,9 @@ int manager_start_scope(
65878a
                 DBusError *error,
65878a
                 char **job) {
65878a
 
65878a
-        const char *timeout_stop_property = "TimeoutStopUSec", *send_sighup_property = "SendSIGHUP", *pids_property = "PIDs", *after_property = "After";
65878a
+        const char *send_sighup_property = "SendSIGHUP", *pids_property = "PIDs", *after_property = "After";
65878a
         _cleanup_dbus_message_unref_ DBusMessage *m = NULL, *reply = NULL;
65878a
         DBusMessageIter iter, sub, sub2, sub3, sub4;
65878a
-        uint64_t timeout = 500 * USEC_PER_MSEC;
65878a
         dbus_bool_t send_sighup = true;
65878a
         const char *fail = "fail";
65878a
         uint32_t u;
65878a
@@ -2632,14 +2631,6 @@ int manager_start_scope(
65878a
          * stop timeout for sessions, so that we don't wait
65878a
          * forever. */
65878a
 
65878a
-        if (!dbus_message_iter_open_container(&sub, DBUS_TYPE_STRUCT, NULL, &sub2) ||
65878a
-            !dbus_message_iter_append_basic(&sub2, DBUS_TYPE_STRING, &timeout_stop_property) ||
65878a
-            !dbus_message_iter_open_container(&sub2, DBUS_TYPE_VARIANT, "t", &sub3) ||
65878a
-            !dbus_message_iter_append_basic(&sub3, DBUS_TYPE_UINT64, &timeout) ||
65878a
-            !dbus_message_iter_close_container(&sub2, &sub3) ||
65878a
-            !dbus_message_iter_close_container(&sub, &sub2))
65878a
-                return log_oom();
65878a
-
65878a
         /* Make sure that the session shells are terminated with
65878a
          * SIGHUP since bash and friends tend to ignore SIGTERM */
65878a
         if (!dbus_message_iter_open_container(&sub, DBUS_TYPE_STRUCT, NULL, &sub2) ||