> webrev: http://cr.openjdk.java.net/~sla/8041923/webrev.00/

src/share/classes/sun/tools/jinfo/JInfo.java
    No comments.

Thumbs up.

Dan


On 5/23/14 8:03 AM, Staffan Larsen wrote:
Peter, Jaroslav: Thanks!

Any Reviewer out there willing to take a look?

/Staffan


On 23 maj 2014, at 15:51, Jaroslav Bachorik <jaroslav.bacho...@oracle.com> 
wrote:

Looks fine!

-JB-

On 05/09/2014 04:30 PM, Staffan Larsen wrote:
Before the fix of JDK-8036599 the jinfo -flags output contained:

Non-default VM flags: -XX:InitialHeapSize=31457280 -XX:MaxHeapSize=31457280 
-XX:MaxNewSize=10485760 -XX:MinHeapDeltaBytes=524288 -XX:NewSize=1572864 
-XX:OldSize=20971520 -XX:+UseCompressedClassPointers -XX:+UseCompressedOops 
-XX:+UseParallelGC
Command line: -Xmx30m

After the fix it says only:

VM Flags:
-XX:CICompilerCount=3 -XX:InitialHeapSize=31457280 -XX:MaxHeapSize=31457280 
-XX:MaxNewSize=10485760 -XX:MinHeapDeltaBytes=524288 -XX:NewSize=10485760 
-XX:OldSize=20971520 -XX:+UseCompressedClassPointers -XX:+UseCompressedOops 
-XX:+UseParallelGC

To restore the old output sun.tools.jinfo.JInfo has been modified to include output from 
vm.executeJCmd("VM.command_line"); as well:

VM Flags:
-XX:CICompilerCount=3 -XX:InitialHeapSize=31457280 -XX:MaxHeapSize=31457280 
-XX:MaxNewSize=10485760 -XX:MinHeapDeltaBytes=524288 -XX:NewSize=10485760 
-XX:OldSize=20971520 -XX:+UseCompressedClassPointers -XX:+UseCompressedOops 
-XX:+UseParallelGC

VM Arguments:
jvm_args: -Xmx30m
java_command: Sleeper 10000
java_class_path (initial): /Users/staffan/dev
Launcher Type: SUN_STANDARD


webrev: http://cr.openjdk.java.net/~sla/8041923/webrev.00/
bug: https://bugs.openjdk.java.net/browse/JDK-8041923

Thanks,
/Staffan


Reply via email to