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


Bill Holler <bill.holler at sun.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bill.holler at sun.com




--- Comment #3 from Bill Holler <bill.holler at sun.com>  2008-12-16 19:49:38 
---
Changing all of the defines (CSTATE_INTERVAL and watermarks) to variables
would allow tuning of this for different policies.

It might be better to remove the if (mcpu->cstate_interval++ == 0) check?
The case is only true the first time through or after a long time.
Instead cstate_history and cstate_ep can be initialized to values to allow
processors to only go into C1 until they have gone through one complete
interval.

+       /*
+        * record the time stamp of the first time cstate entry point
+        */
+       if (mcpu->cstate_interval++ == 0) {
+               mcpu->cstate_ep = start;

-- 
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