diff options
| author | Richard Hughes <richard@hughsie.com> | 2017-04-23 20:22:02 +0100 |
|---|---|---|
| committer | Richard Hughes <richard@hughsie.com> | 2017-04-23 20:23:27 +0100 |
| commit | 798588a480eaae50368bed75fc78f8314523b2a3 (patch) | |
| tree | 19c2b4fc459a259e641491a4df95efbb71a04e4c | |
| parent | eb6b1b62a2733e380bdce7e3f7ef3726dc778c60 (diff) | |
Do not spin in a loop when /proc/timer_stats cannot be written
Only set up the polling once the write has succeeded.
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=100626
| -rw-r--r-- | src/up-wakeups.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/up-wakeups.c b/src/up-wakeups.c index 1bc3c7b..99433dd 100644 --- a/src/up-wakeups.c +++ b/src/up-wakeups.c @@ -626,6 +626,13 @@ up_wakeups_timerstats_enable (UpWakeups *wakeups) g_debug ("enabling timer stats"); + /* enable timer stats */ + file = fopen (UP_WAKEUPS_SOURCE_USERSPACE, "w"); + if (file == NULL) + return FALSE; + fprintf (file, "1\n"); + fclose (file); + /* setup polls */ wakeups->priv->poll_kernel_id = g_timeout_add_seconds (UP_WAKEUPS_POLL_INTERVAL_KERNEL, @@ -637,11 +644,6 @@ up_wakeups_timerstats_enable (UpWakeups *wakeups) (GSourceFunc) up_wakeups_poll_userspace_cb, wakeups); g_source_set_name_by_id (wakeups->priv->poll_userspace_id, "[upower] up_wakeups_poll_userspace_cb"); - file = fopen (UP_WAKEUPS_SOURCE_USERSPACE, "w"); - if (file == NULL) - return FALSE; - fprintf (file, "1\n"); - fclose (file); wakeups->priv->polling_enabled = TRUE; return TRUE; } |