Hi Frank, Sorry for the delay. I've unexpectedly been out dealing with a family emergency last week, so there hasn't been much much progress on my end. Feel free to commit this under usr/src/cmd/powertop (which would be a new subdirectory), and we'll proceed from there.
Thanks, -Eric Wang, Frank wrote: > Has there been any feedback? If no, we will assume that looks good and > proceed to next steps. > > Frank > > Li, Aubrey wrote: > >> On Wednesday, October 31, 2007 6:26 AM, Eric Saxe wrote: >> >> >>> Hi Aubrey, >>> >>> Here is what I have so far for powertop.c. >>> I added some new data structures to track the C/P states and events. >>> >>> Also attached is a DTrace script that uses the new cstate_transition >>> probe to report on how long we're spending in cstates > 0. >>> The intent here is to use libdtrace() to use these probes to extract >>> out this sort of information, and fill in the data structures at the >>> top of powertop.c (or that's what i've been thinking so far)... :) >>> >>> >> Hi Eric, >> >> The attachment is what I have so far. The patch has already included >> your powertop.c. >> It can be compiled and run properly on my side. And it's just a >> framework and no function is implemented. >> You'll see a black window drawn like linuxpowertop after 5 seconds. >> Please feel free to modify it and welcome your any comments or >> suggestions. >> When you think the draft is ok, I'll commit it into powertop repo and >> we can start our work under track. >> >> Thanks, >> -Aubrey >> >> >> > ======================================================================== > >> == >> Makefile | 67 +++++++++++ >> Makefile.com | 61 ++++++++++ >> amd64/Makefile | 32 +++++ >> battery.c | 41 +++++++ >> cpufreq.c | 38 ++++++ >> display.c | 271 ++++++++++++++++++++++++++++++++++++++++++++++ >> i386/Makefile | 32 +++++ >> powertop.c | 333 >> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++ >> >> powertop.h | 119 ++++++++++++++++++++ >> 9 files changed, 994 insertions(+) >>
