RE: [Xen-devel] Re: [PATCH] Enable Core 2 Duo PerformanceCountersinHVM guest

["Shan, Haitao" <haitao.shan@xxxxxxxxx>]

Hi, Keir,
Â
I have baked an updated patch. In this patch, I changed some logicÂto shorten the lengthy part of core2_vpmu_do_wrmsr. And also, I did a rebase, since thereÂisÂoneÂnew changeset recently which made changes to xenoprofile.

Can you have a look?
<<pmu.patch>>
Best Regards
Haitao Shan

Shan, Haitao wrote:
> Hi, Salvador,
>
> Do you mean the strange indentation around CTRL_WRITE? That is
> because there are three hard TABs before CTRL_WRITE. I am not sure
> whether I should fix the original code, so I just leave them there
> untouched.  
>
> Haitao Shan
>
> -----Original Message-----
> From: Otavio Salvador [mailto:otavio@xxxxxxxxxxxxxxxx]
> Sent: 2007å12æ15æ 3:05
> To: Shan, Haitao
> Cc: Keir Fraser; xen-devel@xxxxxxxxxxxxxxxxxxx; Jiang, Yunhong
> Subject: Re: [Xen-devel] Re: [PATCH] Enable Core 2 Duo
> PerformanceCounters inHVM guest
>
> "Shan, Haitao" <haitao.shan@xxxxxxxxx> writes:
>
> Hello Haitao,
>
> I've done a look on the patch and found two places with wrong coding
> style. Take a look bellow:
>
>> diff -r 8f0cbfc478d6 xen/arch/x86/oprofile/op_model_ppro.c
>> --- a/xen/arch/x86/oprofile/op_model_ppro.c  Thu Dec 13 09:31:03 2007
>> +0000 +++ b/xen/arch/x86/oprofile/op_model_ppro.c    Fri Dec 14
>> 07:21:43 2007 +0800 @@ -126,6 +127,11 @@ static void
>>      ppro_start(struct op_msrs co CTRL_READ(low, high, msrs, 0);
>>      CTRL_SET_ACTIVE(low);
>>      CTRL_WRITE(low, high, msrs, 0);
>> +    /* Global Control MSR is enabled by default when system power
>> on. +     * However, this may not hold true when xenoprof starts to
>> run. +     */ +    if ( ppro_has_global_ctrl )
>> +        wrmsrl(MSR_CORE_PERF_GLOBAL_CTRL, 1);
>>  }
>>
>>
>> @@ -135,6 +141,8 @@ static void ppro_stop(struct op_msrs con
>>      CTRL_READ(low, high, msrs, 0);
>>      CTRL_SET_INACTIVE(low);
>>      CTRL_WRITE(low, high, msrs, 0);
>> +    if ( ppro_has_global_ctrl )
>> +        wrmsrl(MSR_CORE_PERF_GLOBAL_CTRL, 0);
>>  }
>
> Cheers,

Attachment: pmu.patch
Description: pmu.patch

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel