Re: CVS commit: src/sys/arch/x86/acpi

[Christoph Egger <Christoph_Egger%gmx.de@localhost>]

On 19.08.10 20:30, Jukka Ruohonen wrote:
> Module Name:  src
> Committed By: jruoho
> Date:         Thu Aug 19 18:30:24 UTC 2010
> 
> Modified Files:
>       src/sys/arch/x86/acpi: acpi_cpu_md.c
> 
> Log Message:
> Add native P-state support for AMD family 10h and 11h processors. Both are
> supported irrespective of XPSS. Family 10h tested by jakllsch@.
> 

[...]

>  
>       case CPUVENDOR_AMD:
>  
> -             if ((sc->sc_flags & ACPICPU_FLAG_P_XPSS) == 0)
> -                     return EOPNOTSUPP;
> +             switch (CPUID2FAMILY(ci->ci_signature)) {

This is not correct. On AMD the cpu family is computed as:

   family = CPUID2FAMILY(ci->ci_signature);
   if (family == 0xf)
       family += CPUID2EXTFAMILY(ci->ci_signature);

> +
> +             case 0x10:
> +             case 0x11:
> +                     msr.ps_control_addr = MSR_10H_CONTROL;
> +                     msr.ps_control_mask = __BITS(0, 2);
> +
> +                     msr.ps_status_addr  = MSR_10H_STATUS;
> +                     msr.ps_status_mask  = __BITS(0, 2);
> +                     break;
> +