Re: [.32-rc3] scheduler: iwlagn consistently high in "waiting for CPU"

[Frans Pop <elendil@planet.nl>]

On Thursday 08 October 2009, Mike Galbraith wrote:
> Care to try a patch?
>
> latencytop: only account on-runqueue wait time as being scheduler
> latency.
>
> Latencytop was accounting uninterruptible and interruptible sleep time
> up to 5ms, which is not the latency the user is looking for.  Account
> time waiting on-runqueue instead.  Also add a clock update to
> update_curr() for the case where there's nobody home.  When coming out
> of idle with NO_HZ, not updating the clock leads to bogus max latency
> report.

With this patch the high values for iwlagn and phy0 are gone. The values 
now match roughly with the output of your script.

If (un)interruptible sleep is worth displaying in latencytop then maybe it 
should be identified as a separate cause instead of lumped under "waiting 
for CPU"?

> TODO: Figure out where the heck empty braces come from.

I also see these lines with "[]" sometimes; looks to come from 'sleep'.

And I still see lines with only a "." as the cause; this seems specific to 
targets marked with the penguin (work queues?).

> Signed-off-by: Mike Galbraith <efault@gmx.de>
> Arjan van de Ven <arjan@infradead.org>
> Cc: Ingo Molnar <mingo@elte.hu>
> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
> LKML-Reference: <new-submission>

If Arjan is happy with the patch, please add my:

Reported-and-tested-by: Frans Pop <elendil@planet.nl>

Thanks,
FJP