Subject : Clocksource tsc is always unstable with 2.6.25-* kernels and CONFIG_NO_HZ=y on my box
Submitter : Gabriel C <firstname.lastname@example.org>
Date : 2008-02-24 01:31
References : http://lkml.org/lkml/2008/2/23/380
References : http://lkml.org/lkml/2008/2/24/281
Handled-By : Thomas Gleixner <email@example.com>
This entry is being used for tracking a regression from 2.6.24. Please don't
close it until the problem is fixed in the mainline.
References : http://lkml.org/lkml/2008/3/18/1
Author: Andi Kleen <firstname.lastname@example.org>
Date: Wed Jan 30 13:30:02 2008 +0100
clocksource: make clocksource watchdog cycle through online CPUs
Well if the cycling watchdog detects an inconsistency the clock perhaps
_ought_ to be marked unstable? If it is really not consistent over CPUs
marking it unstable is the right thing to do, short of finding out what
makes it inconsistent.
Fixed by: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=3150e63df41450a795bbd0bd98a8e70da74e0285
Hmm why exactly was it reverted? It is unclear from the lkml reference
and the git commit message is also not very enlightening. Also nobody cc
ed me on anything.
Unless there is some bug in the watchdog itself reverting it will just hide
whatever problem it showed so would be absolutely the wrong fix.
I suspect his machine really had some inconsistency between CPUs and you
just shot the messenger.
My recommendation would be to reopen the bug, but I don't have the rights for that.
Okay, but I'm removing it from the list of recent regressions.
(In reply to comment #4)
> Hmm why exactly was it reverted?
The reporter observed undesired behavior that was not present with 2.6.24 and identified the commit that caused it to happen, AFAICS. Still, the revert was from Andrew, so you should better ask him.
Ok Andrew can you tell us why you reverted it? I think you just shot the messenger who exposed a previously hidden problem.
I think it is better assigned to Thomas. If he determines that my original
patch for the watchdog was broken I would be happy to take a look at that but it would surprise me if that was the case.
Andi I've CC'ed you on that.. ( Andi Kleen <email@example.com> ) Is that the wrong email ?
Also from here on you got CC'ed on each email I've send http://lkml.org/lkml/2008/3/18/1
we definitely want to look deeper into this. The patch looks pretty innocent and I definitely want to know why exactly it triggers on your machine.
Re #9: I meant I was not cc'ed on whatever thread discussed the decision to revert the watchdog patch.
(In reply to comment #11)
> Re #9: I meant I was not cc'ed on whatever thread discussed the decision to
> revert the watchdog patch.
Ach ok , I got that wrong sorry
(In reply to comment #10)
> we definitely want to look deeper into this. The patch looks pretty innocent
> and I definitely want to know why exactly it triggers on your machine.
As said on LKML , I can test any sort patches on that box at any time ( oh well when I'm home : ) )
Are fixing the root cause of the problem.