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

Reply via email to