On Thu, Dec 23, 2010 at 02:42:20PM +0000, Matthew Garrett wrote: > We've generally been assuming (rightly or wrongly) that getting into > deep C states is preferable to being active (even at a lower frequency), > and so the current behaviour of tending to rapidly switch to the maximum > P state isn't inherently a problem. What kind of power savings are you > benchmarking with this, and do you still see a saving if you just > disable turbo mode? Thanks a lot! Matthew. Running the well-known power and performance benchmark, performance/watts improve around 10%, the performance without drop. Other benchmarks: kernel buiding and compress-7zip, power saving 2%~5%, the performance also without drop. Exception was: apache benchmark, it will let the performance drop without much power save. These say that it depends on workload itself to save how much power. I also consider that this patchset is effective to save power when workload intermediately be idle, during the idle period, the workload is purely idle not waiting for something happened. Because the low frequency will try to fill up these idle periods while Turbo frequency execute quickly consume much more power than low frequency, then be purely idle. In this situation, use low frequency will save power. But if the workload is not purely idle,we low the frequency to execute, it will sacrifice the performance while it also do not save power. In this situation, I will try to decrease to sampling window to samping rate (10ms), it will roll back and keep the same behaviour as original ondemand does. I need more investigation about how to identify out these purely idle or not. Do you have idea about it? I run benchmark at two situations: one is userspace governor, set all cpu frequency P1 and other is set to powersave, there is no much different between these two result. So it say that there is not much value to tuning between Pn to P1(no-turbo mode). While I compare result of userspace with all P1 frequency and Performance(Turbo Mode), there are much room to tuning. It is the original drive for me to do this patchset. Thanks -Youquan -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html