[PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-23 Thread John Stultz
From: Stephen Boyd During suspend we call sched_clock_poll() to update the epoch and accumulated time and reprogram the sched_clock_timer to fire before the next wrap-around time. Unfortunately, sched_clock_poll() doesn't restart the timer, instead it relies on the hrtimer layer to do that and du

Re: [PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-22 Thread Stephen Boyd
On 07/18/14 17:14, John Stultz wrote: > On 07/18/2014 04:24 PM, Stephen Boyd wrote: >> On 07/18/14 15:42, John Stultz wrote: >>> If its a regression (and needs -stable backports) it needs to go in via >>> tip/timers/urgent, and not via the regular merge window. >>> >>> Whats the additional risk -st

Re: [PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-18 Thread John Stultz
On 07/18/2014 04:24 PM, Stephen Boyd wrote: > On 07/18/14 15:42, John Stultz wrote: >> If its a regression (and needs -stable backports) it needs to go in via >> tip/timers/urgent, and not via the regular merge window. >> >> Whats the additional risk -stable wise for canceling the timer during >> s

Re: [PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-18 Thread Stephen Boyd
p during resume? > I'd say close to zero given that we'd only be making the timer run a little bit later and we have slack in there already. Here's that version. 8<- Subject: [PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend During suspend we call sched_

Re: [PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-18 Thread John Stultz
On 07/18/2014 03:38 PM, Stephen Boyd wrote: > On 07/18/14 15:25, John Stultz wrote: >> On 07/18/2014 03:09 PM, Stephen Boyd wrote: >>> During suspend we call sched_clock_poll() to update the epoch and >>> accumulated time and reprogram the sched_clock_timer to fire >>> before the next wrap-around t

Re: [PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-18 Thread Stephen Boyd
On 07/18/14 15:25, John Stultz wrote: > On 07/18/2014 03:09 PM, Stephen Boyd wrote: >> During suspend we call sched_clock_poll() to update the epoch and >> accumulated time and reprogram the sched_clock_timer to fire >> before the next wrap-around time. Unfortunately, >> sched_clock_poll() doesn't

Re: [PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-18 Thread John Stultz
On 07/18/2014 03:09 PM, Stephen Boyd wrote: > During suspend we call sched_clock_poll() to update the epoch and > accumulated time and reprogram the sched_clock_timer to fire > before the next wrap-around time. Unfortunately, > sched_clock_poll() doesn't restart the timer, instead it relies > on th

[PATCH] sched_clock: Avoid corrupting hrtimer tree during suspend

2014-07-18 Thread Stephen Boyd
During suspend we call sched_clock_poll() to update the epoch and accumulated time and reprogram the sched_clock_timer to fire before the next wrap-around time. Unfortunately, sched_clock_poll() doesn't restart the timer, instead it relies on the hrtimer layer to do that and during suspend we aren'