In extraordinay circumstances (MCA init/ debugger invocation, hardware
problems) the
system may not be able to process timer ticks for an extended period of time.
The timer interrupt will compensate as soon as the system becomes functional
again by
calling do_timer for each missed tick. This will cause time to race forward in
a very
fast way. Device drivers that wait for timeouts will find that the system times
out
on everything and thus device drivers will conclude that the devices are not in
a functional state disabling them. The system then cannot continue from the
frozen
state because the device drivers have given up.
This patch fixes that issue by checking if more than half a second has passed
since the last tick. If more than half a second has passed then we would need
to do
around 500 calls to do_timer to compensate. So in order to avoid these timeouts
we act as if time has been frozen with the system and do not compensate for
lost time.
Device drivers may still find that their outstanding requests have failed but
they
will be able to reinitialize the device and the system can hopefully continue.
A consequence of this patch is that the wall clock will stand still if the no
ticks
can be processed for more than half a second.
Signed-off-by: Christoph Lameter <[EMAIL PROTECTED]>
Index: linux-2.6.13/arch/ia64/kernel/time.c
===================================================================
--- linux-2.6.13.orig/arch/ia64/kernel/time.c 2005-08-28 16:41:01.000000000
-0700
+++ linux-2.6.13/arch/ia64/kernel/time.c 2005-09-09 14:45:37.000000000
-0700
@@ -55,6 +55,7 @@ static irqreturn_t
timer_interrupt (int irq, void *dev_id, struct pt_regs *regs)
{
unsigned long new_itm;
+ unsigned long itc;
if (unlikely(cpu_is_offline(smp_processor_id()))) {
return IRQ_HANDLED;
@@ -64,10 +65,25 @@ timer_interrupt (int irq, void *dev_id,
new_itm = local_cpu_data->itm_next;
- if (!time_after(ia64_get_itc(), new_itm))
+ itc = ia64_get_itc();
+ if (!time_after(itc, new_itm))
printk(KERN_ERR "Oops: timer tick before it's due
(itc=%lx,itm=%lx)\n",
ia64_get_itc(), new_itm);
+ /*
+ * If more than half a second has passed since the last timer interrupt
then
+ * something significant froze the system. Skip the time adjustments
+ * otherwise repeated calls to do_timer will trigger timeouts by
devices.
+ */
+ if (unlikely(time_after(itc, new_itm + HZ /2 *
local_cpu_data->itm_delta))) {
+ new_itm = itc;
+ if (smp_processor_id() == TIME_KEEPER_ID) {
+ time_interpolator_reset();
+ printk(KERN_ERR "Oops: more than 0.5 seconds since last
tick."
+ "Skipping time adjustments in order to avoid
timeouts.\n");
+ }
+ }
+
profile_tick(CPU_PROFILING, regs);
while (1) {
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
