[intel-platform-dev] Code review for several MONITOR/MWAIT idle loop CRs

Bill Holler Bill.Holler at Sun.COM
Tue Aug 28 19:12:27 PDT 2007


http://cr.opensolaris.org/~bholler/mwait_fixes

Here is a webrev code review for the following idle loop MONITOR/MWAIT
CRs (Change Requests):

6577948 <http://monaco.sfbay.sun.com/detail.jsp?cr=6577948> 
mach_alloc_mwait leaks memory when a CPU fails to start
6588054 <http://monaco.sfbay.sun.com/detail.jsp?cr=6588054> panic() in 
mach_alloc_mwait() should be changed to degraded operation...
6596141 <http://monaco.sfbay.sun.com/detail.jsp?cr=6596141> Solaris 
should not use an unmodified MWAIT idle loop on AMD 10h due to increased 
power consumption


While these are relatively low priority, I want to get these in soon for
maximum S10U5 soak time.

Testing completed:
1) The x86 build of this kernel has passed ON PIT DIY.  The SPARC build
     is currently in ON PIT DIY and looks ok.  (There are no common changes
     that should effect SPARC.)
2) libmicro did not regress.
    PERF PIT is not planned as these are enable/disable changes which should
    not change performance in either state.
3) The memory leak is fiked in unit testing with all non-boot cpus forced to
    fail to online using cpufailset debug hook.
4) My desktop has been running this for 2 weeks without incident.

Ongoing testing:
1) Barcelona testing is ongoing.
2) 6588054 error injection testing is ongoing.

Thank you,
Bill Holler




More information about the intel-platform-dev mailing list