Re: [PATCH 2/2] x86: remove Pentium Pro check

[Andrew Cooper <andrew.cooper3@citrix.com>]

On 30/08/13 03:14, Matt Wilson wrote:
> ... as it's not a supported processor
>
> Signed-off-by: Matt Wilson <msw@amazon.com>
> Cc: Keir Fraser <keir@xen.org>
> Cc: Jan Beulich <jbeulich@suse.com>
> Cc: Andrew Cooper <andrew.cooper3@citrix.com>

Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>

> ---
>  xen/arch/x86/cpu/intel.c |    4 ----
>  1 files changed, 0 insertions(+), 4 deletions(-)
>
> diff --git a/xen/arch/x86/cpu/intel.c b/xen/arch/x86/cpu/intel.c
> index 072ecbc..31b69c9 100644
> --- a/xen/arch/x86/cpu/intel.c
> +++ b/xen/arch/x86/cpu/intel.c
> @@ -198,10 +198,6 @@ static void __devinit init_intel(struct cpuinfo_x86 *c)
>  			set_bit(X86_FEATURE_ARCH_PERFMON, c->x86_capability);
>  	}
>  
> -	/* SEP CPUID bug: Pentium Pro reports SEP but doesn't have it until model 3 mask 3 */
> -	if ((c->x86<<8 | c->x86_model<<4 | c->x86_mask) < 0x633)
> -		clear_bit(X86_FEATURE_SEP, c->x86_capability);
> -
>  	if ( !cpu_has(c, X86_FEATURE_XTOPOLOGY) )
>  	{
>  		c->x86_max_cores = num_cpu_cores(c);