Messages in this thread | | | From | "Cef (LKML)" <> | Subject | Re: [announce/OT] kerneltop ver. 0.7 | Date | Mon, 24 May 2004 15:53:42 +1000 |
| |
On Mon, 24 May 2004 14:50, Randy.Dunlap wrote: > just a little note about kerneltop... > > 'kerneltop' is similar to 'top', but shows only kernel function usage > (modules not included).
Looks good! A few suggestions....
Might want to add support for /boot/System.map containing the version number (eg: /boot/System.map-`uname -r` ), if /boot/System.map doesn't exist, before just dropping out with an error.
For clarity, you might want to invert the "address function ....." line to separate the header from the actual displayed data (like top does).
Having the total up in the header instead of at the end of the data might also be useful (eg: next to ticks perhaps?) as it's really a waste of a display line for 1 number.
Also a little more verbosity in the error messages would be good.
eg: /proc/profile not found: You need to enable profiling support in your kernel to use kerneltop /proc/profile Permission Denied: You need to be root to run kerneltop
Hope these comments are useful.
-- Stuart Young (aka Cef) cef-lkml@optusnet.com.au is for LKML and related email only - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |