[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] oprofile: Add X7542 and E7-8837 to the list of supported cpus


  • To: Jan Beulich <JBeulich@xxxxxxxx>
  • From: Dietmar Hahn <dietmar.hahn@xxxxxxxxxxxxxx>
  • Date: Tue, 04 Dec 2012 15:36:05 +0100
  • Cc: Jun Nakajima <jun.nakajima@xxxxxxxxx>, Donald D Dugger <donald.d.dugger@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxx
  • Delivery-date: Tue, 04 Dec 2012 14:37:19 +0000
  • Domainkey-signature: s=s1536a; d=ts.fujitsu.com; c=nofws; q=dns; h=X-SBRSScore:X-IronPort-AV:Received:X-IronPort-AV: Received:Received:From:To:Cc:Subject:Date:Message-ID: User-Agent:In-Reply-To:References:MIME-Version: Content-Transfer-Encoding:Content-Type; b=R5NCw/5Rq2L7hZTPi+/bW4seubN92ZLBDgxDW8gfOd+SSeH1/0t3LIb9 GpT2OzB3iV22P/9LBa76A14wr4PrvtKEPP9Rjyy9MIOvO6Unb2h8zvpJS Fb8Xfnu4ZZ+VjjWU2yRPFCwTV3cCrXMdlNOLkGJzaYgo2njMVxl/7K+sj guzdk78T348kpGE6xJlXRfXvvXcSgZ1QBBxfGgPpfcRuCG7z52CHWEe2G NGLWoykn4c3vNT41Nya//uGd1vlMR;
  • List-id: Xen developer discussion <xen-devel.lists.xen.org>

Am Dienstag 27 November 2012, 15:46:41 schrieb Jan Beulich:
> >>> On 27.11.12 at 15:21, Dietmar Hahn <dietmar.hahn@xxxxxxxxxxxxxx> wrote:
> > Am Dienstag 27 November 2012, 13:19:34 schrieb Jan Beulich:
> >> >>> On 27.11.12 at 14:04, "Jan Beulich" <JBeulich@xxxxxxxx> wrote:
> >> >>>> On 26.11.12 at 13:52, Dietmar Hahn <dietmar.hahn@xxxxxxxxxxxxxx> 
> >> >>>> wrote:
> >> >> Add intel cpus X7542 and E7-8837 to the list of supported cpus.
> >> >> 
> >> >> Thanks.
> >> >> Dietmar.
> >> >> 
> >> >> Signed-off-by: Dietmar Hahn <dietmar.hahn@xxxxxxxxxxxxxx>
> >> >> 
> >> >> diff -r 0049de3827bc -r 6fb0129600cd xen/arch/x86/oprofile/nmi_int.c
> >> >> --- a/xen/arch/x86/oprofile/nmi_int.c   Fri Nov 23 11:06:15 2012 +0000
> >> >> +++ b/xen/arch/x86/oprofile/nmi_int.c   Mon Nov 26 13:36:00 2012 +0100
> >> >> @@ -366,6 +366,8 @@ static int __init ppro_init(char ** cpu_
> >> >>                 ppro_has_global_ctrl = 1;
> >> >>                 break;
> >> >>         case 26:
> >> >> +       case 46:
> >> >> +       case 47:
> >> >>                 arch_perfmon_setup_counters();
> >> >>                 *cpu_type = "i386/core_i7";
> >> >>                 ppro_has_global_ctrl = 1;
> >> > 
> >> > Actually, and apart from the patch being white space damaged,
> >> > after a closer look I think this is wrong - these newer CPUs
> >> > shouldn't get be handled here, but instead should be covered by
> >> > arch_perfmon_init(). Are you observing X86_FEATURE_ARCH_PERFMON
> >> > not getting set on these CPUs by init_intel()?
> >> 
> >> I.e. the below would be the patch I'd expect when merely
> >> taking the SDM as reference (with the "todo remove?" ones
> >> also fully removed of course).
> > 
> > Yes, looks much cleaner.
> 
> Question is - does it work for you? And if it does, why would it not
> have worked without any change? After all, the patch in this form,
> apart from the setting of ppro_has_global_ctrl in
> arch_perfmon_init(), only removes code.

After one week of vacation I can't reproduce the thing.
I only remember that trying oprofile didn't work on my machine and the problem
went away after the fix. But now I looked deeper and see that something else
must have been wrong I cant reproduce. Sorry I didn't think enough!
Please forget my patch and thank you for better looking than me!

Dietmar.

> Jan
> 
> 
-- 
Company details: http://ts.fujitsu.com/imprint.html

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.