NOHZ: prevent multiplication overflow - stop timer for huge timeouts
authorThomas Gleixner <tglx@linutronix.de>
Tue, 29 May 2007 21:47:39 +0000 (23:47 +0200)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Wed, 30 May 2007 01:11:10 +0000 (18:11 -0700)
commiteaad084bb0f3a6259e56400cd45d061dbf040600
tree34a5f5c3e1a849b035510ee28d5aaffd42a36dcc
parent6e98ee75c3ab99db48ecc0615c2246dc193111a9
NOHZ: prevent multiplication overflow - stop timer for huge timeouts

get_next_timer_interrupt() returns a delta of (LONG_MAX > 1) in case
there is no timer pending. On 64 bit machines this results in a
multiplication overflow in tick_nohz_stop_sched_tick().

Reported by: Dave Miller <davem@davemloft.net>

Make the return value a constant and limit the return value to a 32 bit
value.

When the max timeout value is returned, we can safely stop the tick
timer device. The max jiffies delta results in a 12 days timeout for
HZ=1000.

In the long term the get_next_timer_interrupt() code needs to be
reworked to return ktime instead of jiffies, but we have to wait until
the last users of the original NO_IDLE_HZ code are converted.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Acked-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
include/linux/timer.h
kernel/time/tick-sched.c
kernel/timer.c