diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2008-09-03 21:37:14 +0000 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2008-09-05 11:11:53 +0200 |
commit | 1fb9b7d29d8e85ba3196eaa7ab871bf76fc98d36 (patch) | |
tree | 1fbefa54e16e76f76616202d8abf9607725afeff /kernel/time/tick-internal.h | |
parent | 9c17bcda991000351cb2373f78be7e4b1c44caa3 (diff) | |
download | lwn-1fb9b7d29d8e85ba3196eaa7ab871bf76fc98d36.tar.gz lwn-1fb9b7d29d8e85ba3196eaa7ab871bf76fc98d36.zip |
clockevents: prevent endless loop lockup
The C1E/HPET bug reports on AMDX2/RS690 systems where tracked down to a
too small value of the HPET minumum delta for programming an event.
The clockevents code needs to enforce an interrupt event on the clock event
device in some cases. The enforcement code was stupid and naive, as it just
added the minimum delta to the current time and tried to reprogram the device.
When the minimum delta is too small, then this loops forever.
Add a sanity check. Allow reprogramming to fail 3 times, then print a warning
and double the minimum delta value to make sure, that this does not happen again.
Use the same function for both tick-oneshot and tick-broadcast code.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/time/tick-internal.h')
-rw-r--r-- | kernel/time/tick-internal.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/time/tick-internal.h b/kernel/time/tick-internal.h index f13f2b7f4fd4..0ffc2918ea6f 100644 --- a/kernel/time/tick-internal.h +++ b/kernel/time/tick-internal.h @@ -17,6 +17,8 @@ extern void tick_handle_periodic(struct clock_event_device *dev); extern void tick_setup_oneshot(struct clock_event_device *newdev, void (*handler)(struct clock_event_device *), ktime_t nextevt); +extern int tick_dev_program_event(struct clock_event_device *dev, + ktime_t expires, int force); extern int tick_program_event(ktime_t expires, int force); extern void tick_oneshot_notify(void); extern int tick_switch_to_oneshot(void (*handler)(struct clock_event_device *)); |