staging: android-alarm: Update hrtimer if alarm at the head of the queue is reprogrammed
authorArve Hjønnevåg <arve@android.com>
Thu, 9 Feb 2012 22:24:38 +0000 (14:24 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 10 Feb 2012 18:08:00 +0000 (10:08 -0800)
If an alarm was restarted with a value that moved it away from the head
of a queue, the hrtimer would not be updated. This would cause unnecessary
wakeups.

CC: Arve Hjønnevåg <arve@android.com>
CC: Android Kernel Team <kernel-team@android.com>
Change-Id: If379f8dd92b0bdb3173bd8d057adfe0dc1d15259
Signed-off-by: Arve Hjønnevåg <arve@android.com>
Signed-off-by: John Stultz <john.stultz@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/android/alarm.c

index 5370fdc8b016b66c96e35a27d413a9078c2149e6..445c0b12c473c890ac3db02add6c214cd2eb1e9f 100644 (file)
@@ -119,12 +119,15 @@ static void alarm_enqueue_locked(struct android_alarm *alarm)
        struct rb_node *parent = NULL;
        struct android_alarm *entry;
        int leftmost = 1;
+       bool was_first = false;
 
        pr_alarm(FLOW, "added alarm, type %d, func %pF at %lld\n",
                alarm->type, alarm->function, ktime_to_ns(alarm->expires));
 
-       if (base->first == &alarm->node)
+       if (base->first == &alarm->node) {
                base->first = rb_next(&alarm->node);
+               was_first = true;
+       }
        if (!RB_EMPTY_NODE(&alarm->node)) {
                rb_erase(&alarm->node, &base->alarms);
                RB_CLEAR_NODE(&alarm->node);
@@ -144,10 +147,10 @@ static void alarm_enqueue_locked(struct android_alarm *alarm)
                        leftmost = 0;
                }
        }
-       if (leftmost) {
+       if (leftmost)
                base->first = &alarm->node;
-               update_timer_locked(base, false);
-       }
+       if (leftmost || was_first)
+               update_timer_locked(base, was_first);
 
        rb_link_node(&alarm->node, parent, link);
        rb_insert_color(&alarm->node, &base->alarms);