Re: [PATCH v16 12/13] arm, tile: turn off timer tick for oneshot_stopped state

2017-11-03 Thread Chris Metcalf
On 11/3/2017 1:18 PM, Mark Rutland wrote: Hi Chris, On Fri, Nov 03, 2017 at 01:04:51PM -0400, Chris Metcalf wrote: diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c index fd4b7f684bd0..61ea7f907c56 100644 --- a/drivers/clocksource/arm_arch_timer.c +++ b/d

Re: [PATCH v16 12/13] arm, tile: turn off timer tick for oneshot_stopped state

2017-11-03 Thread Mark Rutland
Hi Chris, On Fri, Nov 03, 2017 at 01:04:51PM -0400, Chris Metcalf wrote: > diff --git a/drivers/clocksource/arm_arch_timer.c > b/drivers/clocksource/arm_arch_timer.c > index fd4b7f684bd0..61ea7f907c56 100644 > --- a/drivers/clocksource/arm_arch_timer.c > +++ b/drivers/clocksource/arm_arch_timer.c

[PATCH v16 12/13] arm, tile: turn off timer tick for oneshot_stopped state

2017-11-03 Thread Chris Metcalf
When the schedule tick is disabled in tick_nohz_stop_sched_tick(), we call hrtimer_cancel(), which eventually calls down into __remove_hrtimer() and thus into hrtimer_force_reprogram(). That function's call to tick_program_event() detects that we are trying to set the expiration to KTIME_MAX and ca