Request for Review: 7120511: Add diagnostic commands
serguei.spitsyn at oracle.com
serguei.spitsyn at oracle.com
Thu Jan 5 12:05:07 PST 2012
Looks good.
Some minor comments.
share/vm/services/diagnosticCommand.hpp
Format of description() return is not consistent:
- Some have dot at the end, some - not.
- Some has an extra space.
72 return "Print the command line used to start this VM instance.";
86 return "Print system properties";
102 return "Print VM flag options and their current values. ";
166 return "Generate a HPROF format dump of the Java heap";
185 return "Provide statistics about the Java heap usage";
Should the impact() function result be formatted the same way as
description()
- to have or not have dot at the end ?
The following lines are incorrectly indented:
85 static const char* description() {
86 return "Print system properties";
87 }
88 static const char* impact() {
89 return "Low:";
90 }
131 static const char* description() {
132 return "Call java.lang.System.gc().";
133 }
134 static const char* impact() {
135 return "Medium: Depends on Java heap size and content";
136 }
145 static const char* description() {
146 return "Call java.lang.System.runFinalization().";
147 }
148 static const char* impact() {
149 return "Medium: Depends on Java content";
150 }
Thanks,
Serguei
On 1/5/12 7:19 AM, Frederic Parain wrote:
> This changeset aims to add a first set of diagnostic commands
> to the HotSpot JVM. It also includes minor modifications to
> the diagnostic command framework implementation to ease
> development of new diagnostic commands.
>
> The webrev is here:
>
> http://cr.openjdk.java.net/~fparain/7120511/webrev.00/
>
>
> Here's the list of new diagnostic commands:
>
> Thread.print
> Print all threads with stacktraces.
>
> GC.class_histogram
> Provides statistics about the Java heap usage
>
> GC.heap_dump
> Generate a HPROF format dump of the Java heap
>
> GC.run_finalization
> Call java.lang.System.runFinalization().
>
> GC.run
> Call java.lang.System.gc().
>
> VM.uptime
> Print VM uptime.
>
> VM.flags
> Print VM flag options and their current values.
>
> VM.system_properties
> Print system properties
>
> VM.command_line
> Print the command line used to start this VM instance.
>
>
> Thanks,
>
> Fred
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20120105/3c2106d2/attachment.html
More information about the serviceability-dev
mailing list