public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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


       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