On Thu, Apr 05, 2007 at 04:24:45PM +0200, Andi Kleen wrote: > On Thursday 05 April 2007 16:00:45 Andreas Herrmann wrote: > > > > Commit 991528d7348667924176f3e29addea0675298944 > > introduced mwait_idle which is supposed to work > > for Intel CPUs starting with Core Duo. > > > > AMD Fam10 processors won't enter C1 on mwait. > > Unfortunate. Will this be fixed?
No, it is not planned to change this behavior. In fact mwait does certain power savings but the core won't enter the C1 state. And power savings from entering C1 are greater than power savings caused by mwait (on AMD Fam10). > > > This patch will enable default_idle for non-Intel > > CPUs even if mwait is supported. > > It would be better to clear MONITOR/MWAIT in the AMD specific > CPU initialize code than add workarounds everywhere else. Why is that? MONITOR/MWAIT is usable. And I think this should be indicated by cpuinfo. It's just inappropriate to use it in pm_idle. Regards, Andreas -- AMD Saxony, Dresden, Germany Operating System Research Center - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
