On Fri, May 21, 2010 at 10:05 AM, Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> wrote: > Mike Chan <mike@xxxxxxxxxxx> writes: > >> On Thu, May 20, 2010 at 2:01 PM, Thomas Renninger <trenn@xxxxxxx> wrote: >>> Hi Mike, >>> >>> On Thursday 20 May 2010 08:42:21 pm Mike Chan wrote: >>>> v2: >>>> Rebased off of Thomas Renninger's patch for cgroups_cpuacct refactoring, >>> thanks. >>> A general comment: >>> I don't know much about the cgroup stuff. >> >> Perhaps Paul Menage or Balbir Singh can look and sign off on the API's? >> >>> I am also not sure how exactly power can be measured on this arch based on >> >> If you know how much time was spent at each frequency executing code, >> you can calculate how much power was consumed if the platform (with >> hooks) provide power numbers (in milliWatts) for the power at >> frequency X. >> >> I did some initial testing on Motorola Droid comparing to a power >> meter and I got within 2% variance. >> >>> frequency accounting (there also were some threads I was not aware of?) >>> A signed-off-by or reviewed-by from someone who is more involved in this omap >>> stuff would probably not that bad. >>> >> >> OMAP was the closest with mainline support I could provide an example >> how to use these hooks. >> >> I'm hoping for some blessing from some people on the linux-omap list >> for that. However can we possibly just stack the first two patches to >> get the API in? This will make it easier to fixup the omap hooks if >> they don't get in. > > This looks like a great enhancement to me. > > Speaking for OMAP PM... I'd suggest getting the generic stuff upstream > (or into -next) soon and then work out the OMAP specifics after. > > Since the OMAP OPP layer is going through some churn (but stabilizing > and will be submitted for 2.6.36), I'd suggest we queue the > OMAP-specific parts of this along with the OPP layer changes. > So it looks like there is no objections to this API and I'm OK with dropping the omap hooks for now until things are settled in 2.6.36. So are things good with Thomas' re-factoring patch for cpuacct as well as the first 2 patches? -- Mike > Kevin > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html