DragonFly BSD
DragonFly submit List (threaded) for 2009-11
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

RE: DragonFly patch for top-3.8b1


From: "Alex Hornung" <ahornung@xxxxxxxxx>
Date: Thu, 19 Nov 2009 07:39:14 -0000

Hi,

First off, indentation looks rather inconsistent: see for example lines 233
- 257 of the diff, it seems to use a random number of spaces each time
instead of tabs.
For the rest, the code looks ok and it would be definitely good to get it
into upstream top.

As far as we are concerned, WCPU is not of interest and there has been some
discussion on completely removing it or replacing it by something useful;
see issue1484 [1]. Matt's suggestion of showing a column with the
accumulated children CPU time would be a nice thing to have. While this may
not be acceptable for the upstream top, I'd definitely welcome a step in
that direction.

Keep up the good work!

Cheers,
Alex Hornung


[1]: http://bugs.dragonflybsd.org/issue1484





[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]