Commit 8d71844b authored by Linus Torvalds's avatar Linus Torvalds

Merge branch 'timers-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip

Pull timer fixes from Thomas Gleixner:
 "Two fixes in the timer area:
   - a long-standing lock inversion due to a printk
   - suspend-related hrtimer corruption in sched_clock"

* 'timers-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
  timer: Fix lock inversion between hrtimer_bases.lock and scheduler locks
  sched_clock: Avoid corrupting hrtimer tree during suspend
parents 3f9c08f7 504d5874
...@@ -146,7 +146,8 @@ static int clockevents_increase_min_delta(struct clock_event_device *dev) ...@@ -146,7 +146,8 @@ static int clockevents_increase_min_delta(struct clock_event_device *dev)
{ {
/* Nothing to do if we already reached the limit */ /* Nothing to do if we already reached the limit */
if (dev->min_delta_ns >= MIN_DELTA_LIMIT) { if (dev->min_delta_ns >= MIN_DELTA_LIMIT) {
printk(KERN_WARNING "CE: Reprogramming failure. Giving up\n"); printk_deferred(KERN_WARNING
"CE: Reprogramming failure. Giving up\n");
dev->next_event.tv64 = KTIME_MAX; dev->next_event.tv64 = KTIME_MAX;
return -ETIME; return -ETIME;
} }
...@@ -159,7 +160,8 @@ static int clockevents_increase_min_delta(struct clock_event_device *dev) ...@@ -159,7 +160,8 @@ static int clockevents_increase_min_delta(struct clock_event_device *dev)
if (dev->min_delta_ns > MIN_DELTA_LIMIT) if (dev->min_delta_ns > MIN_DELTA_LIMIT)
dev->min_delta_ns = MIN_DELTA_LIMIT; dev->min_delta_ns = MIN_DELTA_LIMIT;
printk(KERN_WARNING "CE: %s increased min_delta_ns to %llu nsec\n", printk_deferred(KERN_WARNING
"CE: %s increased min_delta_ns to %llu nsec\n",
dev->name ? dev->name : "?", dev->name ? dev->name : "?",
(unsigned long long) dev->min_delta_ns); (unsigned long long) dev->min_delta_ns);
return 0; return 0;
......
...@@ -191,7 +191,8 @@ void __init sched_clock_postinit(void) ...@@ -191,7 +191,8 @@ void __init sched_clock_postinit(void)
static int sched_clock_suspend(void) static int sched_clock_suspend(void)
{ {
sched_clock_poll(&sched_clock_timer); update_sched_clock();
hrtimer_cancel(&sched_clock_timer);
cd.suspended = true; cd.suspended = true;
return 0; return 0;
} }
...@@ -199,6 +200,7 @@ static int sched_clock_suspend(void) ...@@ -199,6 +200,7 @@ static int sched_clock_suspend(void)
static void sched_clock_resume(void) static void sched_clock_resume(void)
{ {
cd.epoch_cyc = read_sched_clock(); cd.epoch_cyc = read_sched_clock();
hrtimer_start(&sched_clock_timer, cd.wrap_kt, HRTIMER_MODE_REL);
cd.suspended = false; cd.suspended = false;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment