[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