http://defect.opensolaris.org/bz/show_bug.cgi?id=4128

           Summary: cpupm_enabled does not have CPUPM_C_STATES bit set
    Classification: Development
           Product: power-mgmt
           Version: unspecified
          Platform: Other
        OS/Version: Solaris
            Status: NEW
          Severity: minor
          Priority: P3
         Component: c-state
        AssignedTo: bill.holler at sun.com
        ReportedBy: bill.holler at sun.com
                CC: tesla-dev at opensolaris.org
   Estimated Hours: 0.0


A NHM system with latest pad-gate tip 7490 does not have the CPUPM_C_STATES
bit set in cpupm_enabled:
> cpupm_enabled ::print
0x3

i86pc/os/cpupm.c does not initialize this bit: 
static uint32_t cpupm_enabled = CPUPM_P_STATES | CPUPM_T_STATES;


It looks like cpudrv_pm_cstate_ready() will fail which will cause the
cpudrv_pm_manage_cstates() change notification handler to return early
without rebuilding new C-State domain info?


I believe the fix is to initialize cpupm_enabled to CPUPM_ALL_STATES:
-        static uint32_t cpupm_enabled = CPUPM_P_STATES | CPUPM_T_STATES;
+        static uint32_t cpupm_enabled = CPUPM_ALL_STATES;

-- 
Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

Reply via email to