Re: Change top's notion of idle processes / threads

On 24/05/2014 7:22 AM, Allan Jude wrote:
> On 2014-05-23 16:05, John Baldwin wrote:
>> Right now, when top is set to not display idle processes or threads, it only 
>> displays processes or threads that are currently in a runnable state or have a 
>> non-zero %cpu.  However, our %cpu is quite imprecise.  I have patch to change 
>> top to instead compare the thread or processes runtime (ki_runtime in 
>> kinfo_proc) against the runtime of the thread or process the last time data 
>> was fetched.  In essence, top will consider any thread that has run on a CPU 
>> since the last update as non-idle.  The end result is that mostly-idle threads 
>> and processes will now be visible in top's idle display.  Personally, I find 
>> this more useful (and find the current implementation completely useless).  
>> The patch is at http://people.freebsd.org/~jhb/patches/top_idle.patch
>>
>> Comments?
>>
> 
> I think this makes good sense. I would definitely prefer it. Would it
> make sense to maybe preserve the old behaviour behind a command line flag?
> 

And an update to top(8) reflecting the algo :) I know these little
esoteric things could always do with more obvious breadcrumbs (like load
average calcs, etc) for our future selves and others.

+1 on the behavior change, not sure about retaining the old under a
flag. Who might benefit from it? How do other OS top implementations
calculate their idle? If there's other examples out there with the same
(current) algo, then retaining compat might be worth it, such as for
newly converted users

--
Koobs