Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1505212
| From | Thomas Gleixner <tglx@linutronix.de> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v3] x86/tsc: add X86_FEATURE_TSC_KNOWN_FREQ flag |
| Date | 2016-10-20 21:50 +0200 |
| Message-ID | <survH-3sZ-11@gated-at.bofh.it> (permalink) |
| References | (5 earlier) <srctb-3jq-9@gated-at.bofh.it> <srmCe-1kH-7@gated-at.bofh.it> <srXs5-1E4-1@gated-at.bofh.it> <suisq-68t-15@gated-at.bofh.it> <suiC5-6ej-1@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Thu, 20 Oct 2016, Peter Zijlstra wrote: > On Thu, Oct 20, 2016 at 11:57:03AM +0200, Thomas Gleixner wrote: > > On Thu, 13 Oct 2016, Bin Gao wrote: > > > @@ -702,6 +702,15 @@ unsigned long native_calibrate_tsc(void) > > > } > > > } > > > > > > + setup_force_cpu_cap(X86_FEATURE_TSC_KNOWN_FREQ); > > > + > > > + /* > > > + * For Atom SoCs TSC is the only reliable clocksource. > > > + * Mark TSC reliable so no watchdog on it. > > > + */ > > > + if (boot_cpu_data.x86_model == INTEL_FAM6_ATOM_GOLDMONT) > > > + setup_force_cpu_cap(X86_FEATURE_TSC_RELIABLE); > > > + > > AFAICT setting TSC_RELIABLE also skips the check_tsc_warp() tests in > tsc_sync.c. > > This means that if someone does a Goldmont BIOS with 'features', we'll > never detect the wreckage :-/ Well, we have the same issue on other platforms/models which set the reliable flag. So one sanity check we can do is to read the IA32_TSC_ADJUST MSR on all cores. They should all have the same value (usually 0) or at least have a very minimal delta. If that's off by more than 1us then something is fishy especially on single socket systems. We could at least WARN about it. We could do this in idle occasionally as well, so we can detect the dreaded "SMI wants to hide the cycles" crapola. Thanks, tglx
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v3] x86/tsc: add X86_FEATURE_TSC_KNOWN_FREQ flag Bin Gao <bin.gao@linux.intel.com> - 2016-10-14 01:20 +0200
Re: [PATCH v3] x86/tsc: add X86_FEATURE_TSC_KNOWN_FREQ flag Thomas Gleixner <tglx@linutronix.de> - 2016-10-20 12:10 +0200
Re: [PATCH v3] x86/tsc: add X86_FEATURE_TSC_KNOWN_FREQ flag Peter Zijlstra <peterz@infradead.org> - 2016-10-20 12:20 +0200
Re: [PATCH v3] x86/tsc: add X86_FEATURE_TSC_KNOWN_FREQ flag Thomas Gleixner <tglx@linutronix.de> - 2016-10-20 21:50 +0200
Re: [PATCH v3] x86/tsc: add X86_FEATURE_TSC_KNOWN_FREQ flag Peter Zijlstra <peterz@infradead.org> - 2016-10-21 07:50 +0200
Re: [PATCH v3] x86/tsc: add X86_FEATURE_TSC_KNOWN_FREQ flag Thomas Gleixner <tglx@linutronix.de> - 2016-10-21 10:10 +0200
csiph-web