[PATCH] C-State display fix, readdir to scandir for alpha sort
Woodruff, Richard
r-woodruff2 at ti.com
Fri Apr 11 15:05:42 PDT 2008
Arjan van de Ven wrote:
> Woodruff, Richard wrote:
> > Arjan,
> >
> > I just pulled the 1.10 candidate from svn.
> >
> > This patch to PowerTop:
> > - Allows it to display a definable number of C-States
>
> Hmm I'm not entirely convinced on this one. I was considering going
the
> same way the P-states are done,
> eg picking the top 4 states in terms of usage %age and only show
those.
> Idea was to keep the general UI the same format as it is now (esp for
text
> mode screens it's tight already),
> and just only show the relevant data...
>
> do you see a problem with such an approach?
In my current usage I want to be able to see the distribution across all
states. If there were a hundred maybe its time to be annoyed and only
focus on the top 5 but with only 7 for me its not so bad. There is
still enough room on the screen.
I like to also see that I'm touching all available P and C states. I
use this in lieu of specific state forcing tests. Call it a randomized
test of dynamic transitions.
Regards,
Richard W.
More information about the Power
mailing list