public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Burt Triplett <burt@pbjtriplett.org>
To: Thomas Gleixner <tglx@linutronix.de>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	"H. Peter Anvin" <hpa@zytor.com>,
	Arjan van de Ven <arjan@infradead.org>,
	Venkatesh Pallipadi <venki@google.com>,
	Len Brown <lenb@kernel.org>
Subject: Re: Performance/resume issues on Toshiba NB305
Date: Fri, 04 Mar 2011 21:55:52 -0800	[thread overview]
Message-ID: <4D71D068.2090803@pbjtriplett.org> (raw)
In-Reply-To: <20110228151008.GA29142@thinkpad-t410>

On 02/28/2011 07:10 AM, Seth Forshee wrote:
> On Sun, Feb 27, 2011 at 12:17:29PM -0800, Burt Triplett wrote:
>>> The new release detects the Atom processor okay, but in bits-327 the
>>> system resets during the SMI frequency/latency test, whereas it didn't
>>> with bits-316.
>>
>> The SMI Frequency and Latency test should be fixed in bits-329.
>
> Yes, it's fixed now, thanks.

Glad to hear it.

>> To get more information on how the MSRs differ between CPUs, you can
>> turn on verbose mode.  Go to the GRUB command line by hitting 'c',
>> and enter "test_options -v 2".  Then hit Esc to go back to the menu,
>> and re-run the MSR consistency check.  It will now show you the
>> value of the MSR on each CPU.
>>
>> MSR 0x1a0 is IA32_MISC_ENABLE, and generally any inconsistency in
>> that MSR represents a BIOS bug: it didn't enable features
>> identically on all CPUs.  Once you see which bits differ, you can
>> look at the Intel Software Developer's Manual (specifically volume
>> 3B) to find out what those bits mean and which specific features the
>> BIOS inconsistently enabled.
>
>   (MSR 0x1a0 consistent) FAIL (2 different values)
>   1 CPUs 0x0000000364950089: 0x0
>   1 CPUs 0x0000000364950081: 0x1
>
>   Bit 3: Automatic Thermal Control Circuit Enable
>
> I'm not sure of the impact of this, since there's really only one core
> in the CPU. One of the others differs has bit 3 inconsistent as well. On
> the third bit 3 is the same but bit 0 (fast-strings enable) is
> inconsistent.

0x1a0 bit 3 definitely shouldn't differ between CPUs; that's a BIOS bug.

Fast-strings should *always* be turned on, and should always be 
consistent across processors.  That's also a BIOS bug, which will cause 
suboptimal performance of the string instructions on any CPU with that 
bit disabled.

>> MSR 0x199 is IA32_PERF_CTL.  A difference in that MSR usually means
>> the BIOS left CPUs in different P-states when booting, which it
>> really shouldn't do.  Again, you can decode the result with the
>> Intel SDM.
>
>   (MSR 0x199 consistent): FAIL (2 different values)
>   1 CPUs 0x0000000000000a1f: 0x0
>   1 CPUs 0x0000000000000613: 0x1
>
> So yes, the cores are in different P-states. The results from the other
> machines are similar, but the values for CPU1 are slightly different for
> each machine. CPU1 has the same value on all three machines.

Intel recommends that BIOSes set all CPUs to the same P-state before 
booting (partly for the case where the OS never touches P-states 
itself).  So, that's a BIOS bug, though it'll go away the moment Linux 
starts using P-states itself.

>> Looking into MSR 0x39.
>
> Yeah, this one isn't in the SDM at all. Here are the values I got:
>
>   (MSR 0x39 consistent): FAIL (2 different values)
>   1 CPUs 0x0000000000000000: 0x0
>   1 CPUs 0x0000000000000001: 0x1
>
> Values are identical for all three machines.

After investigating this MSR, I've blacklisted it from the MSR 
consistency check as of bits-332.  This isn't a BIOS bug, and the 
difference is expected.

> I'm not really getting the idea that any of these is contributing to the
> problems I'm seeing with this machine though.

No, none of the issues BITS has detected on your system should affect 
the original problem you reported.  Thanks for providing the additional 
information, though.

- Burt Triplett

  reply	other threads:[~2011-03-05  5:56 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-25 16:42 Performance/resume issues on Toshiba NB305 Seth Forshee
2011-02-25 20:17 ` Steven Rostedt
2011-02-25 20:27   ` Seth Forshee
2011-02-25 22:11     ` Thomas Gleixner
2011-02-25 22:33       ` Seth Forshee
2011-02-25 20:37 ` Thomas Gleixner
2011-02-25 21:21   ` Seth Forshee
2011-02-25 21:47     ` Thomas Gleixner
2011-02-25 22:29       ` Seth Forshee
2011-02-25 22:40         ` Thomas Gleixner
2011-02-26  5:58           ` Burt Triplett
2011-02-26 15:00             ` Seth Forshee
2011-02-27 20:17               ` Burt Triplett
2011-02-28 15:10                 ` Seth Forshee
2011-03-05  5:55                   ` Burt Triplett [this message]
2011-02-26  8:49       ` [tip:timers/urgent] clockevents: Prevent oneshot mode when broadcast device is periodic tip-bot for Thomas Gleixner
2011-03-01 20:04       ` Performance/resume issues on Toshiba NB305 Seth Forshee
2011-03-01 20:22         ` Thomas Gleixner

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=4D71D068.2090803@pbjtriplett.org \
    --to=burt@pbjtriplett.org \
    --cc=arjan@infradead.org \
    --cc=hpa@zytor.com \
    --cc=lenb@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tglx@linutronix.de \
    --cc=venki@google.com \
    /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