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.