Messages in this thread |  | | Date | 24 Jul 2006 19:39:20 +0200 | | Date | Mon, 24 Jul 2006 19:39:20 +0200 | | From | Andi Kleen <> | | Subject | Re: REGRESSION: the new i386 timer code fails to sync CPUs |
| |
On Mon, Jul 24, 2006 at 08:58:58AM -0700, john stultz wrote: > On Sun, 2006-07-23 at 05:37 -0700, Andrew Morton wrote: > > On Sun, 23 Jul 2006 14:08:29 +0200 > > Matthias Urlichs <smurf@smurf.noris.de> wrote: > > > > > Hi, > > > > > > Andrew Morton: > > > > - CPU0 and CPU1 share a TSC and CPU2 and CPU3 share another TSC. > > > > > > > That mmakes sense, since they're one dual-core Xeon each. > > > > OK. > > > > > > - Earlier kernels didn't use the TSC as a time source whereas this one > > > > does, hence the problems which you're observing. > > > > > > > Correct; see below. > > > > > > > I assume that booting with clock=pit or clock=pmtmr fixes it? > > > > > > > Testing... yes, both. > > > > > > > It would be useful to check your 2.6.17 boot logs, see if we can work out > > > > what 2.6.17 was using for a clock source. > > > > > > > That's easy: > > > > > > 2.6.17 -Using pmtmr for high-res timesource > > > 2.6.18git +Time: tsc clocksource has been installed. > > > > > > I missed those two lines, as in the boot logs they're not really > > > adjacent, so they got lost in the jumble of other differences. > > > > OK, thanks. Marking the TSC as bad in this case is simple to do - let us > > let John work out the best way. > > > > We must have lost a TSC sanity check somewhere along the way. I wonder > > what it was? > > Well, I changed the TSC vs ACPI PM timer priority ordering to be more > like x86-64 (Andi had a similar patch he was proposing as well). For > awhile suse/redhat kernels have been swapping them, as the TSC gives > such a performance boost, however the ACPI PM timer is usually the safer > option (distro customers are often told to use clock=pmtmr on some > boxes). > > I'll see what we can do to narrow it down, but its been assumed by both > x86-64 and the new i386 code that the TSCs on Intel SMP boxes are > synched, unless we're explicitly told they aren't (Summit, etc).
Or it supports C3. I just had to add that check on 64bit too for Merom.
> With the current code it is trivial to mark the TSC as unstable and the > system will automatically fall back to the next best clocksource. The > difficulty is just making sure we've got all the cases covered without > needlessly disqualifying synced systems. > > Andi: If this is a generic issue, and not specific to Matthias' box, we > may need to re-think the assumption that Intel SMP is synced. You're > thoughts?
I'm missing context. Full log files/full system description?
At least on x86-64 I'm doing it like this for a long time and didn't have any complaints so I would assume that the 64bit capable boxes are near completely ok.
-Andi
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |