Eric Saxe wrote: > Hi Aubrey, > > That fixed the issue with the percentages being non-sensical, but the > tool reports almost no time being spent in C1 on an otherwise > idle system.
Please don't use this patch, use "hg pull" to get the tip. Let me know the result. Thanks, -Aubrey > > Eric Saxe wrote: >> 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 >>> >>> >> >> _______________________________________________ >> tesla-dev mailing list >> tesla-dev at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/tesla-dev
