Re: [PATCH] nohz: add missing handling of clocksource watchdog

From: R. J. Wysocki
Date: Sun Dec 07 2008 - 19:02:52 EST


On Monday, 8 of December 2008, Bartlomiej Zolnierkiewicz wrote:
>
> Fixes "Clocksource tsc unstable (delta = -974982308 ns)" problem.

Where can I find the description of the problem?

Rafael

> [ IDE was unlucky to be initialized at the same time that
> clocksource watchdog triggers and was blamed for the issue. ]
>
> Cc: Sergei Shtylyov <sshtylyov@xxxxxxxxxxxxx>
> Cc: Lars Winterfeld <lars.winterfeld@xxxxxxxxxxxxx>
> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@xxxxxxxxx>
> ---
> kernel/time/tick-sched.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> Index: b/kernel/time/tick-sched.c
> ===================================================================
> --- a/kernel/time/tick-sched.c
> +++ b/kernel/time/tick-sched.c
> @@ -21,6 +21,7 @@
> #include <linux/sched.h>
> #include <linux/tick.h>
> #include <linux/module.h>
> +#include <linux/clocksource.h>
>
> #include <asm/irq_regs.h>
>
> @@ -153,6 +154,7 @@ void tick_nohz_update_jiffies(void)
> local_irq_restore(flags);
>
> touch_softlockup_watchdog();
> + clocksource_touch_watchdog();
> }
>
> static void tick_nohz_stop_idle(int cpu)
>



--
Rafael J. Wysocki, Ph.D.
Institute of Theoretical Physics
Faculty of Physics of Warsaw University
ul. Hoza 69, 00-681 Warsaw
[tel: +48 22 55 32 263]
[mob: +48 60 50 53 693]
----------------------------
One should not increase, beyond what is necessary,
the number of entities required to explain anything.
-- William of Ockham
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/