mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Vegard Nossum" <vegard.nossum@gmail.com>
To: "Andi Kleen" <andi@firstfloor.org>
Cc: "Ingo Molnar" <mingo@elte.hu>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"Andreas Herrmann" <andreas.herrmann3@amd.com>,
	"\"S.Çağlar Onur\"" <caglar@pardus.org.tr>,
	Valdis.Kletnieks@vt.edu, "Matt Mackall" <mpm@selenic.com>,
	linux-kernel@vger.kernel.org, arjan@infradead.org
Subject: Re: [BISECTED] Lots of "rescheduling IPIs" in powertop
Date: Wed, 14 May 2008 08:00:06 +0200	[thread overview]
Message-ID: <19f34abd0805132300p65527be7v46ee206d8b2b57d3@mail.gmail.com> (raw)
In-Reply-To: <482A05F3.7020408@firstfloor.org>

On Tue, May 13, 2008 at 11:19 PM, Andi Kleen <andi@firstfloor.org> wrote:
>
>  > The desktop is a P4:
>  >
>  > processor       : 0
>  > vendor_id       : GenuineIntel
>  > cpu family      : 15
>  > model           : 6
>  > model name      : Intel(R) Pentium(R) 4 CPU 3.00GHz
>  > stepping        : 5
>  > cpu MHz         : 2992.624
>  > cache size      : 2048 KB
>
>
> > flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge
>  > mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe nx
>  > lm constant_tsc pebs bts pni monitor ds_cpl est tm2 cid cx16 xtpr
>  > lahf_lm
>  > bogomips        : 5990.81
>  > clflush size    : 64
>  >
>  > (similar for processor 1)
>  >
>  > # msr
>  > 0
>
>  Ok the CPU reports it doesn't support any C states in MWAIT. If that is
>  correct then it would be correct to not use MWAIT idle and might
>  actually save more power to not use it.
>
>  I don't know if that's true or not. Do you have a power meter perhaps?
>  If yes can you measure if there's a difference between mwait=idle /
>  default on your box when it is idle?
>
>  [cc Arjan he might now if that CPU is supposed to support C1 in MWAIT]

No, sorry, no power meter :-/

>  > The laptop is a Pentium Dual-Core:
>  >
>  > processor       : 0
>  > vendor_id       : GenuineIntel
>  > cpu family      : 6
>  > model           : 15
>  > model name      : Intel(R) Pentium(R) Dual  CPU  T2310  @ 1.46GHz
>  > stepping        : 13
>  > cpu MHz         : 800.000
>  > cache size      : 1024 KB
>  ...
>
> > flags           : fpu vme de pse tsc msr pae mce cx8 apic mtrr pge mca
>  > cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe lm
>  > constant_tsc arch_perfmon pebs bts pni monitor ds_cpl est tm2 ssse3
>  > cx16 xtpr lahf_lm
>  > bogomips        : 2930.23
>  > clflush size    : 64
>  >
>  > (similar for processor 1)
>  >
>  > # ./msr
>  > 1110
>
>  CPU reports it supports C1/C2/C3. Are you sure there is a difference on
>  that box? The code should have kept using MWAIT because it checks C1.
>  Please double check.

Yes, sorry, you are correct. I tested the idle=mwait only on the
desktop machine (P4, msr = 0), and it improved the IPI problem. (I
even rechecked right now, and it really does.)

Now I tested it on the laptop as well, and here it makes no difference.


Vegard

-- 
"The animistic metaphor of the bug that maliciously sneaked in while
the programmer was not looking is intellectually dishonest as it
disguises that the error is the programmer's own creation."
	-- E. W. Dijkstra, EWD1036

  parent reply	other threads:[~2008-05-14  6:00 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-13 20:42 Vegard Nossum
2008-05-13 20:54 ` Andi Kleen
2008-05-13 21:09   ` Vegard Nossum
2008-05-13 21:19     ` Andi Kleen
2008-05-14  4:02       ` Arjan van de Ven
2008-05-14  6:58         ` Andi Kleen
2008-05-14 13:55           ` Arjan van de Ven
2008-05-14 15:21             ` Andi Kleen
2008-05-14  6:00       ` Vegard Nossum [this message]
2008-05-14  6:56 ` Ingo Molnar
2008-05-14  7:11   ` Vegard Nossum
2008-05-14  9:09   ` Andreas Herrmann
2008-05-14 11:42     ` Andi Kleen
2008-05-15 16:10       ` Andreas Herrmann
2008-05-16  8:40     ` Ingo Molnar

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=19f34abd0805132300p65527be7v46ee206d8b2b57d3@mail.gmail.com \
    --to=vegard.nossum@gmail.com \
    --cc=Valdis.Kletnieks@vt.edu \
    --cc=andi@firstfloor.org \
    --cc=andreas.herrmann3@amd.com \
    --cc=arjan@infradead.org \
    --cc=caglar@pardus.org.tr \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=mpm@selenic.com \
    --cc=tglx@linutronix.de \
    /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®