Date: Mon, 28 Jul 2008 17:35:38 +0200
From: Ingo Molnar <mingo@elte.hu>
To: "Daniel K." <dk@uw.no>
Cc: Robert Richter <robert.richter@amd.com>,
       Barry Kasindorf <barry.kasindorf@amd.com>,
       Thomas Gleixner <tglx@linutronix.de>,
       oprofile-list <oprofile-list@lists.sourceforge.net>,
       LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 03/24] oprofile: Add support for AMD Family 11h
Message-ID: <20080728153538.GB28495@elte.hu>
References: <1216753748-11261-1-git-send-email-robert.richter@amd.com> <1216753748-11261-4-git-send-email-robert.richter@amd.com> <488B5F93.3060600@uw.no>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <488B5F93.3060600@uw.no>
User-Agent: Mutt/1.5.18 (2008-05-17)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 911
Lines: 27


* Daniel K. <dk@uw.no> wrote:

> Robert Richter wrote:
>> diff --git a/arch/x86/oprofile/nmi_int.c b/arch/x86/oprofile/nmi_int.c
>> index 7f3329b..cf28a02 100644
>> --- a/arch/x86/oprofile/nmi_int.c
>> +++ b/arch/x86/oprofile/nmi_int.c
>> @@ -422,6 +422,10 @@ int __init op_nmi_init(struct oprofile_operations *ops)
>>  			model = &op_athlon_spec;
>>  			cpu_type = "x86-64/family10";
>>  			break;
>> +		case 0x11:
>> +			model = &op_athlon_spec;
>> +			cpu_type = "x86-64/family11h";
>
> Should the 'h' be dropped, as for family10 above, to keep them similar.

'family10' is a typo we've got to live with - but there's no need to 
repeat it for family11h.

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/