public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Robert Richter <robert.richter@amd.com>
To: Andi Kleen <andi@firstfloor.org>
Cc: Arnd Bergmann <arnd@arndb.de>, Jiri Olsa <jolsa@redhat.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"oprofile-list@lists.sourceforge.net" 
	<oprofile-list@lists.sourceforge.net>,
	"tdm.rhbz@gmail.com" <tdm.rhbz@gmail.com>
Subject: Re: [PATCH] oprofile: Add Support for Intel CPU Family 6 / Model 29
Date: Wed, 22 Sep 2010 09:26:33 +0200	[thread overview]
Message-ID: <20100922072633.GV13563@erda.amd.com> (raw)
In-Reply-To: <38667ac18ee6604649af9396cf156182.squirrel@www.firstfloor.org>

On 21.09.10 10:38:04, Andi Kleen wrote:
> > On 21.09.10 07:21:05, Arnd Bergmann wrote:
> >> Just curious about the other missing IDs: this now supports all Core
> >> microarchitecture CPUs, but only some Nehalem/Westmere ones.
> >>
> >> There is support for 0x1a (Bloomfield/Gainestown) and 0x2e (Beckton),
> >> but
> >> not for the much more common 0x1e (Lynnfield/Clarksfield/Jasper Forest),
> >> 0x25 (Clarkdale/Arrandale) and 0x26 (Gulftown/Westmere-EP).

I also found reports for 0x2c (Gulftown/Westmere-EP?):

vendor_id       : GenuineIntel
cpu family      : 6
model           : 44
model name      : Intel(R) Xeon(R) CPU           E5620  @ 2.40GHz

-Robert

> >>
> >> Shouldn't those be added as well?
> 
> Westmere I still hold off because the oprofile user land
> doesn't have the correct event list yet.
> 
> BTW I was thinking in this case to only add the model
> numbers to the user code and just let it handle arch_perfmon
> from the kernel.
> 
> This should work currently fine and avoid the problem
> that you need to update user land and kernel in parallel
> for a new cpu type.
> 
> >>
> >> When you're there, it probably also wouldn't hurt to fill in the blanks
> >> of some of the older CPUs like Tolopai (0x15) and newer Atoms (0x1c).
> >
> > Yes, would be nice to have also an update for the remaining
> > unsupported models. Andi?
> 
> Yes would make sense. I'll ask around if there is a consolidated
> list somewhere that could be used as a base.
> 
> -Andi
> 
> 

-- 
Advanced Micro Devices, Inc.
Operating System Research Center


  reply	other threads:[~2010-09-22  7:29 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-21  7:26 [PATCH] oprofile: Add Support for Intel CPU Family 6 / Model 29 Jiri Olsa
2010-09-21  8:20 ` Robert Richter
2010-09-21 10:09   ` Andi Kleen
2010-09-21 10:33 ` Robert Richter
2010-09-27  7:39   ` [GIT PULL] oprofile fixes Robert Richter
2010-09-27  7:50     ` Ingo Molnar
2010-09-21 11:21 ` [PATCH] oprofile: Add Support for Intel CPU Family 6 / Model 29 Arnd Bergmann
2010-09-21 14:13   ` Robert Richter
2010-09-21 14:38     ` Andi Kleen
2010-09-22  7:26       ` Robert Richter [this message]
2010-09-22  8:17         ` Arnd Bergmann

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=20100922072633.GV13563@erda.amd.com \
    --to=robert.richter@amd.com \
    --cc=andi@firstfloor.org \
    --cc=arnd@arndb.de \
    --cc=jolsa@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=oprofile-list@lists.sourceforge.net \
    --cc=tdm.rhbz@gmail.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