From: Robert Richter <robert.richter@amd.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: "oprofile-list@lists.sf.net" <oprofile-list@lists.sf.net>,
Andi Kleen <andi@firstfloor.org>,
Patrick Simmons <linuxrocks123@netscape.net>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] Add Support for Intel CPU Family 6 / Model 22 (Intel Celeron 540)
Date: Thu, 9 Sep 2010 20:48:12 +0200 [thread overview]
Message-ID: <20100909184812.GT13563@erda.amd.com> (raw)
In-Reply-To: <20100909123102.GQ13563@erda.amd.com>
On 09.09.10 14:31:02, Robert Richter wrote:
> On 08.09.10 10:34:28, Patrick Simmons wrote:
> > This patch adds CPU type detection for the Intel Celeron 540, which is
> > part of the Core 2 family according to Wikipedia; the family and ID pair
> > is absent from the Volume 3B table referenced in the source code
> > comments. I have tested this patch on an Intel Celeron 540 machine
> > reporting itself as Family 6 Model 22, and OProfile runs on the machine
> > without issue. The patch was made against Linux version 2.6.35.4 but is
> > so trivial that it should apply cleanly to anything relevant. Please CC
> > me with any comments or questions as I am not subscribed to the list.
> >
> > --Patrick
> >
> > Signed-off-by: Patrick Simmons <linuxrocks123@netscape.net>
>
> Applied to oprofile/urgent.
Ingo,
please pull this update for tip/perf/urgent from:
git://git.kernel.org/pub/scm/linux/kernel/git/rric/oprofile.git urgent
Thanks,
-Robert
>
> >
> > --- linux/arch/x86/oprofile/nmi_int.c.orig 2010-09-08
> > 08:01:56.000000000 -0600
> > +++ linux/arch/x86/oprofile/nmi_int.c 2010-09-08 05:57:18.000000000 -0600
> > @@ -664,7 +664,7 @@ static int __init ppro_init(char **cpu_t
> > case 14:
> > *cpu_type = "i386/core";
> > break;
> > - case 15: case 23:
> > + case 15: case 22: case 23:
>
> I slightly modified the change:
>
> - case 15: case 23:
> + case 0x0f:
> + case 0x16:
> + case 0x17:
>
> Thanks Patrick.
>
> -Robert
>
> > *cpu_type = "i386/core_2";
> > break;
> > case 0x1a:
> >
> >
--
Advanced Micro Devices, Inc.
Operating System Research Center
next parent reply other threads:[~2010-09-09 18:53 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <4C879EF4.90509@netscape.net>
[not found] ` <20100909123102.GQ13563@erda.amd.com>
2010-09-09 18:48 ` Robert Richter [this message]
2010-09-09 20:22 ` [PATCH] Add Support for Intel CPU Family 6 / Model 22 (Intel Celeron 540) 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=20100909184812.GT13563@erda.amd.com \
--to=robert.richter@amd.com \
--cc=andi@firstfloor.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxrocks123@netscape.net \
--cc=mingo@elte.hu \
--cc=oprofile-list@lists.sf.net \
/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