Yeah, I saw that a couple of hours ago. It's a minor fix at the end of the main loop. I'm pushing the fix in a few minutes.
As far as pushing 6834110, if this next push doesn't sell you on the freeze option, I'll remove it. Thanks, Rafael Aubrey Li wrote: > Hi Rafael, > > Sorry for the delay, please hold on this changeset if it is being put > back into ON, > Press "I" once, everything works good, I saw the string "I - Idle" in > the status window. > Press "I" the second time, "I - Idle" is gone, but everything is frozen > forever. > > And I still saw 100% C0 residency. I'll send the screenshot > offline(it's larger than the size > the mailing list allow), the screenshot is captured when I pressed "I" twice. > > I don't want you to take much effort on 6836685, for me it's not good > enough to put back > into ON, Can we put back 6834110 first? ;) > > Thanks, > -Aubrey > > > On Sun, Jun 14, 2009 at 6:34 AM, <rafael.vanoni at sun.com> wrote: >> Author: Rafael Vanoni <rafael.vanoni at sun.com> >> Repository: /hg/tesla/ptop-full >> Latest revision: 70f27d2d4b60bf7ebbae6f9464ba59393b79487d >> Total changesets: 1 >> Log message: >> 6834110 PowerTOP's display.c needs some lovin' >> 6836685 PowerTOP should allow the user to freeze its subwindows >> >> Files: >> update: usr/src/cmd/powertop/amd64/pt_amd64.c >> update: usr/src/cmd/powertop/common/battery.c >> update: usr/src/cmd/powertop/common/cpufreq.c >> update: usr/src/cmd/powertop/common/cpuidle.c >> update: usr/src/cmd/powertop/common/display.c >> update: usr/src/cmd/powertop/common/events.c >> update: usr/src/cmd/powertop/common/powertop.c >> update: usr/src/cmd/powertop/common/powertop.h >> update: usr/src/cmd/powertop/common/suggestions.c >> update: usr/src/cmd/powertop/i386/pt_i386.c >> update: usr/src/cmd/powertop/sparcv9/pt_sparcv9.c >> _______________________________________________ >> tesla-dev mailing list >> tesla-dev at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/tesla-dev >>
