RFR: 8041923 Command line output is missing from jinfo

[Staffan Larsen]

Thanks!

On 23 maj 2014, at 16:26, Daniel D. Daugherty <daniel.daugherty at 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.bachorik at 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
>>>> 
>