Thanks! On 23 maj 2014, at 16:26, Daniel D. Daugherty <daniel.daughe...@oracle.com> wrote:
> > 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 >>>> >
