Attachment 130886 Details for Bug 87791 – fix for the bug

[patch] fix for the bug

0001-daemon-Fix-critical-action-after-resume-from-hiberna.patch (text/plain), 4.24 KB, created by Miroslav Šustek on 2017-04-17 23:21:10 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Miroslav Šustek

Created: 2017-04-17 23:21:10 UTC

Size: 4.24 KB

patch

obsolete

>From c8ce6b542150e4c53c78a26af1bebb37baaaa6fa Mon Sep 17 00:00:00 2001
>From: Miroslav Sustek <sustmidown@centrum.cz>
>Date: Tue, 18 Apr 2017 00:54:42 +0200
>Subject: [PATCH] daemon: Fix critical action after resume from hibernate
>
>The take_action_timeout_cb() function returns G_SOURCE_REMOVE which
>makes GLib destroy the timeout. However the action_timeout_id
>stayed != 0 so when warning level turned to "action" again the daemon
>assumed that the timeout is already set and did nothing.
>(It only logged: "Not taking action, timeout id already set")
>
>https://bugs.freedesktop.org/show_bug.cgi?id=87791
>---
> src/linux/integration-test | 57 ++++++++++++++++++++++++++++++++++++++++++++++
> src/up-daemon.c            |  4 ++++
> 2 files changed, 61 insertions(+)
>
>diff --git a/src/linux/integration-test b/src/linux/integration-test
>index b864772..d3af756 100755
>--- a/src/linux/integration-test
>+++ b/src/linux/integration-test
>@@ -252,6 +252,10 @@ class Tests(dbusmock.DBusTestCase):
>         with open(self.log.name) as f:
>             return text in f.read()
> 
>+    def have_text_in_log_n_times(self, text, n):
>+        with open(self.log.name) as f:
>+            return f.read().count(text) == n
>+
>     def assertEventually(self, condition, message=None, timeout=50):
>         '''Assert that condition function eventually returns True.
> 
>@@ -744,6 +748,59 @@ class Tests(dbusmock.DBusTestCase):
> 
>         self.stop_daemon()
> 
>+    def test_critical_action_is_taken_repeatedly(self):
>+        '''check that critical action works repeatedly (eg. after resume)'''
>+
>+        bat0 = self.testbed.add_device('power_supply', 'BAT0', None,
>+                                       ['type', 'Battery',
>+                                        'present', '1',
>+                                        'status', 'Discharging',
>+                                        'energy_full', '60000000',
>+                                        'energy_full_design', '80000000',
>+                                        'energy_now', '50000000',
>+                                        'voltage_now', '12000000'], [])
>+
>+        config = tempfile.NamedTemporaryFile(delete=False, mode='w')
>+        config.write("[UPower]\n")
>+        config.write("UsePercentageForPolicy=true\n")
>+        config.write("PercentageAction=2\n")
>+        config.write("CriticalPowerAction=Hibernate\n")
>+        config.close()
>+
>+        self.start_logind()
>+        self.start_daemon(cfgfile=config.name)
>+
>+        devs = self.proxy.EnumerateDevices()
>+        self.assertEqual(len(devs), 1)
>+        bat0_up = devs[0]
>+
>+        # simulate that battery has 1% (less than PercentageAction)
>+        self.testbed.set_attribute(bat0, 'energy_now', '600000')
>+        self.testbed.uevent(bat0, 'change')
>+        time.sleep(0.5)
>+
>+        self.assertEqual(self.get_dbus_display_property('WarningLevel'), UP_DEVICE_LEVEL_ACTION)
>+        self.assertEventually(lambda: self.have_text_in_log_n_times("About to call logind method Hibernate", 1), timeout=300)
>+
>+        # simulate that battery was charged to 100% during sleep
>+        self.testbed.set_attribute(bat0, 'energy_now', '60000000')
>+        self.testbed.uevent(bat0, 'change')
>+        time.sleep(0.5)
>+
>+        self.assertEqual(self.get_dbus_display_property('WarningLevel'), UP_DEVICE_LEVEL_NONE)
>+
>+        # simulate that battery was drained to 1% again
>+        self.testbed.set_attribute(bat0, 'energy_now', '600000')
>+        self.testbed.uevent(bat0, 'change')
>+        time.sleep(0.5)
>+
>+        self.assertEqual(self.get_dbus_display_property('WarningLevel'), UP_DEVICE_LEVEL_ACTION)
>+        self.assertEventually(lambda: self.have_text_in_log_n_times("About to call logind method Hibernate", 2), timeout=300)
>+
>+        self.stop_daemon()
>+
>+        os.unlink(config.name)
>+
>     def test_no_poll_batteries(self):
>         ''' setting NoPollBatteries option should disable polling'''
> 
>diff --git a/src/up-daemon.c b/src/up-daemon.c
>index 37f66ab..1a5dddc 100644
>--- a/src/up-daemon.c
>+++ b/src/up-daemon.c
>@@ -587,6 +587,10 @@ static gboolean
> take_action_timeout_cb (UpDaemon *daemon)
> {
> 	up_backend_take_action (daemon->priv->backend);
>+
>+	g_debug ("Backend was notified to take action. The timeout will be removed.");
>+	daemon->priv->action_timeout_id = 0;
>+
> 	return G_SOURCE_REMOVE;
> }
> 
>-- 
>2.9.3
>

Actions: View | Splinter Review

Attachments on bug 87791: 113012 | 130886 | 132983

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.