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

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

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,

-- 
Otavio Salvador                  O.S. Systems
E-mail: otavio@xxxxxxxxxxxxxxxx  http://www.ossystems.com.br
Mobile: +55 53 9981-7854         http://projetos.ossystems.com.br

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