Re: [PATCH 4/5] hrtimer: Kick lowres dynticks targets on timer enqueue

2014-06-22 Thread Viresh Kumar
On 22 June 2014 19:06, Thomas Gleixner wrote: > On Sun, 22 Jun 2014, Frederic Weisbecker wrote: >> + wake_up_nohz_cpu(base->cpu_base->cpu); > > The timer is queued on new_base not on base. Oops, thanks for spotting this bug. Will be fixed in next version. -- To unsubscribe from this

Re: [PATCH 4/5] hrtimer: Kick lowres dynticks targets on timer enqueue

2014-06-22 Thread Thomas Gleixner
On Sun, 22 Jun 2014, Frederic Weisbecker wrote: > From: Viresh Kumar > > In lowres mode, hrtimers are serviced by the tick instead of a clock > event. It works well as long as the tick stays periodic but we must also > make sure that the hrtimers are serviced in dynticks mode targets, > pretty mu

[PATCH 4/5] hrtimer: Kick lowres dynticks targets on timer enqueue

2014-06-21 Thread Frederic Weisbecker
From: Viresh Kumar In lowres mode, hrtimers are serviced by the tick instead of a clock event. It works well as long as the tick stays periodic but we must also make sure that the hrtimers are serviced in dynticks mode targets, pretty much like timer list timers do. Note that all dynticks modes