Thanks Aubrey, I was out of town Thursday and Fri, and I didn't get a chance to try your changes... I'll let you know if it rectifies the issue I was seeing...
-Eric Li, Aubrey wrote: > Li, Aubrey wrote: > > >> I failed to replicate this issue on my side. >> But from my point of view, we didn't do anything wrong. >> So, is the following patch acceptable for this bug? >> ====================================================== >> diff -r 2fb476052291 usr/src/cmd/powertop/display.c >> --- a/usr/src/cmd/powertop/display.c Tue Feb 12 11:55:53 2008 -0800 >> +++ b/usr/src/cmd/powertop/display.c Tue Feb 19 23:08:25 2008 +0800 >> @@ -185,6 +185,9 @@ void show_cstates(double interval) >> >> for (i=0; i< g_ncpus; i++) >> total_c1 += cstate_info[i].total_time; >> + >> + if (total_c1 > interval * g_ncpus) >> + total_c1 = interval * g_ncpus; >> >> print(cstate_window, 0, 0, "%s", "Cn\t\t\tAvg residency\n"); >> sprintf(c, "C0 (cpu running)\t\t(%.1f%%)\n", >> ======================================================= >> > > I committed this patch since no objection. > > Thanks, > -Aubrey > _______________________________________________ > tesla-dev mailing list > tesla-dev at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/tesla-dev >
