From: Thomas Gleixner <tglx@linutronix.de>
To: Len Brown <lenb@kernel.org>
Cc: x86@kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/9] x86: TSC calibration update
Date: Fri, 10 Jun 2016 15:20:29 +0200 (CEST) [thread overview]
Message-ID: <alpine.DEB.2.11.1606101519020.5839@nanos> (raw)
In-Reply-To: <1459397635-4871-1-git-send-email-lenb@kernel.org>
Len,
On Thu, 31 Mar 2016, Len Brown wrote:
> cpu_khz and tsc_khz initialization can be unreliable and expensive.
> They are initialized in tsc_init()/native_calibrate_tsc(), which prints:
>
> pr_info("Detected %lu.%03lu MHz processor\n", cpu_khz...)
>
> native_calibrate_cpu() first tries quick_pit_calibrate(),
> which can take over 50.0M cycles to succeed,
> or as few as 2.4M cycles to fail.
>
> On failure, pit_calibrate_tsc() is attempted, which can succeed
> in as few as 20M cycles, but may consume over 240M cycles
> before it fails.
>
> By comparison, on many processors, tsc frequency can be discovered by
> table and MSR or CPUID in under 0.002M cycles.
I found this in my backlog. Sorry for missing it.
The series does not apply anymore and the hardcoded family constants want to
be replaced by the new model defines.
Could you please resend?
Thanks,
tglx
prev parent reply other threads:[~2016-06-10 13:22 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-31 4:13 Len Brown
2016-03-31 4:13 ` [PATCH 1/9] x86 tsc_msr: Identify Intel-specific code Len Brown
2016-03-31 4:13 ` [PATCH 2/9] x86 tsc_msr: Remove debugging messages Len Brown
2016-03-31 4:13 ` [PATCH 3/9] x86 tsc_msr: Update comments, expand definitions Len Brown
2016-03-31 4:13 ` [PATCH 4/9] x86 tsc_msr: Correct Silvermont reference clock values Len Brown
2016-03-31 4:13 ` [PATCH 5/9] x86 tsc_msr: Add Airmont " Len Brown
2016-03-31 4:13 ` [PATCH 6/9] x86 tsc_msr: Extend to include Intel Core Architecture Len Brown
2016-03-31 4:13 ` [PATCH 7/9] x86 tsc_msr: Remove irqoff around MSR-based TSC enumeration Len Brown
2016-03-31 4:13 ` [PATCH 8/9] x86 tsc: enumerate SKL cpu_khz and tsc_khz via CPUID Len Brown
2016-03-31 4:13 ` [PATCH 9/9] x86 tsc: enumerate BXT " Len Brown
2016-06-10 13:20 ` Thomas Gleixner [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.2.11.1606101519020.5839@nanos \
--to=tglx@linutronix.de \
--cc=lenb@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®