Lejun Zhu wrote: > Hi all, > > Here's the design document for LatencyTop. Comments are welcome. Thanks. >
Hi Lejun The design doc looks good, here are my comments. Sec 2.2: Is there a limit to the number of rules the user can specify in the configuration file? Sec 2.3: I'd suggest changing the title of the top half from "Cause" to "System wide latencies", to emphasize the differences between the two halfs. I wonder if a line or some other visual separation would be helpful. I'd also suggest removing the "(C) 2008 Intel Copyright" part of the title - per what was done with PowerTOP, no other OpenSolaris utility displays a copyright notice during execution. I tried LatencyTOP on a Linux box and have to confess that I wasn't very pleased with the process list at the bottom and how the user navigates it. At the very least I'd suggest adding arrows on each side so the user quickly understands how to navigate. I'm far from a human-computer interaction specialist, but I think the interface presents some challenges. I think there's a lot of room for extending the features of LatencyTOP as proposed, you exemplified this on the last two paragraphs. I'm looking forward to reading the code and trying it on OpenSolaris. Thanks, Rafael _______________________________________________ perf-discuss mailing list perf-discuss@opensolaris.org