Re: RFR(xxs): 8200384: jcmd help output should be sorted

2018-04-11 Thread Thomas Stüfe
Hi Serguei,

On Wed, Apr 11, 2018 at 7:51 PM, serguei.spit...@oracle.com
 wrote:
> Hi Thomas,
>
> Sorry, I did not reply to this.

no problem.

> Thank you a lot for providing the output!
> I agree, the sorted one looks much better.
>
> Probably, it is late to say but I'm Ok with a push. :)
>

Thanks :) Have not yet pushed but will do it today or tomorrow.

..Thomas

> Thanks,
> Serguei
>
>
>
> On 4/9/18 02:07, Thomas Stüfe wrote:
>
> Hi Sergey, Christoph,
>
> thanks for the review!
>
> Sure, here you go:
>
> Old output, unsorted:
>
> thomas@mainframe /shared/projects/openjdk/jdk-submit-hs/output-fastdebug $
> ./images/jdk/bin/jcmd test3.Example2 help
> 24278:
> The following commands are available:
> VM.log
> VM.native_memory
> ManagementAgent.status
> ManagementAgent.stop
> ManagementAgent.start_local
> ManagementAgent.start
> Compiler.directives_clear
> Compiler.directives_remove
> Compiler.directives_add
> Compiler.directives_print
> Compiler.CodeHeap_Analytics
> VM.print_touched_methods
> Compiler.codecache
> Compiler.codelist
> Compiler.queue
> VM.classloader_stats
> Thread.print
> JVMTI.data_dump
> JVMTI.agent_load
> VM.metaspace
> VM.stringtable
> VM.symboltable
> VM.class_hierarchy
> VM.systemdictionary
> GC.class_stats
> GC.class_histogram
> GC.heap_dump
> GC.finalizer_info
> GC.heap_info
> GC.run_finalization
> GC.run
> VM.info
> VM.uptime
> VM.dynlibs
> VM.set_flag
> VM.flags
> VM.system_properties
> VM.command_line
> VM.version
> help
>
> New output, sorted:
>
> thomas@mainframe /shared/projects/openjdk/jdk-submit-hs/output-fastdebug $
> ./images/jdk/bin/jcmd test3.Example2 help
> 30230:
> The following commands are available:
> Compiler.CodeHeap_Analytics
> Compiler.codecache
> Compiler.codelist
> Compiler.directives_add
> Compiler.directives_clear
> Compiler.directives_print
> Compiler.directives_remove
> Compiler.queue
> GC.class_histogram
> GC.class_stats
> GC.finalizer_info
> GC.heap_dump
> GC.heap_info
> GC.run
> GC.run_finalization
> JVMTI.agent_load
> JVMTI.data_dump
> ManagementAgent.start
> ManagementAgent.start_local
> ManagementAgent.status
> ManagementAgent.stop
> Thread.print
> VM.class_hierarchy
> VM.classloader_stats
> VM.command_line
> VM.dynlibs
> VM.flags
> VM.info
> VM.log
> VM.metaspace
> VM.native_memory
> VM.print_touched_methods
> VM.set_flag
> VM.stringtable
> VM.symboltable
> VM.system_properties
> VM.systemdictionary
> VM.uptime
> VM.version
> help
>
>
> I'm running submit tests now, if they pass I'll push.
>
> Best Regards, Thomas
>
>
> On Tue, Apr 3, 2018 at 3:52 AM, serguei.spit...@oracle.com
>  wrote:
>>
>> Hi Thomas,
>>
>> Added the serviceability-dev mailing list as it is a Serviceability area.
>>
>> The fix looks good to me.
>> One question:
>>  Could you, please, post the sorted help output?
>>  It is interesting how does it look like when sorted.
>>
>> Thanks,
>> Serguei
>>
>>
>>
>> On 3/28/18 13:08, Thomas Stüfe wrote:
>>>
>>> Hi all,
>>>
>>> may I get reviews for this tiny trivial change which causes jcmd help
>>> output (the command list) to be sorted?
>>>
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8200384
>>> webrev:
>>>
>>> http://cr.openjdk.java.net/~stuefe/webrevs/8200384-jcmd-help-sorted/webrev.00/webrev/
>>>
>>> Thanks!
>>>
>>> Best Regards, Thomas
>>
>>
>
>


Re: RFR(xxs): 8200384: jcmd help output should be sorted

2018-04-11 Thread serguei.spit...@oracle.com

  
  
Hi Thomas,
  
  Sorry, I did not reply to this.
  Thank you a lot for providing the output!
  I agree, the sorted one looks much better.
  
  Probably, it is late to say but I'm Ok with a push. :)
  
  Thanks,
  Serguei
  
  
  On 4/9/18 02:07, Thomas Stüfe wrote:


  
  Hi Sergey, Christoph,


thanks for the review! 


Sure, here you go:


Old output, unsorted:



  thomas@mainframe
/shared/projects/openjdk/jdk-submit-hs/output-fastdebug $
./images/jdk/bin/jcmd test3.Example2 help
  24278:
  The following commands are available:
  VM.log
  VM.native_memory
  ManagementAgent.status
  ManagementAgent.stop
  ManagementAgent.start_local
  ManagementAgent.start
  Compiler.directives_clear
  Compiler.directives_remove
  Compiler.directives_add
  Compiler.directives_print
  Compiler.CodeHeap_Analytics
  VM.print_touched_methods
  Compiler.codecache
  Compiler.codelist
  Compiler.queue
  VM.classloader_stats
  Thread.print
  JVMTI.data_dump
  JVMTI.agent_load
  VM.metaspace
  VM.stringtable
  VM.symboltable
  VM.class_hierarchy
  VM.systemdictionary
  GC.class_stats
  GC.class_histogram
  GC.heap_dump
  GC.finalizer_info
  GC.heap_info
  GC.run_finalization
  GC.run
  VM.info
  VM.uptime
  VM.dynlibs
  VM.set_flag
  VM.flags
  VM.system_properties
  VM.command_line
  VM.version
  help



New output, sorted:



  thomas@mainframe
/shared/projects/openjdk/jdk-submit-hs/output-fastdebug $
./images/jdk/bin/jcmd test3.Example2 help
  30230:
  The following commands are available:
  Compiler.CodeHeap_Analytics
  Compiler.codecache
  Compiler.codelist
  Compiler.directives_add
  Compiler.directives_clear
  Compiler.directives_print
  Compiler.directives_remove
  Compiler.queue
  GC.class_histogram
  GC.class_stats
  GC.finalizer_info
  GC.heap_dump
  GC.heap_info
  GC.run
  GC.run_finalization
  JVMTI.agent_load
  JVMTI.data_dump
  ManagementAgent.start
  ManagementAgent.start_local
  ManagementAgent.status
  ManagementAgent.stop
  Thread.print
  VM.class_hierarchy
  VM.classloader_stats
  VM.command_line
  VM.dynlibs
  VM.flags
  VM.info
  VM.log
  VM.metaspace
  VM.native_memory
  VM.print_touched_methods
  VM.set_flag
  VM.stringtable
  VM.symboltable
  VM.system_properties
  VM.systemdictionary
  VM.uptime
  VM.version
  help





I'm running submit tests now, if they pass I'll push.


Best Regards, Thomas


  
  
On Tue, Apr 3, 2018 at 3:52 AM, serguei.spit...@oracle.com 
  wrote:
  Hi Thomas,

Added the serviceability-dev mailing list as it is a
Serviceability area.

The fix looks good to me.
One question:
 Could you, please, post the sorted help output?
 It is interesting how does it look like when sorted.

Thanks,
Serguei

  


On 3/28/18 13:08, Thomas Stüfe wrote:

  Hi all,
  
  may I get reviews for this tiny trivial change which
  causes jcmd help
  output (the command list) to be sorted?
  
  bug: https://bugs.openjdk.java.net/browse/JDK-8200384
  webrev:
  http://cr.openjdk.java.net/~stuefe/webrevs/8200384-jcmd-help-sorted/webrev.00/webrev/
  
  Thanks!
  
  Best Regards, Thomas


  

  


  


  



Re: RFR(xxs): 8200384: jcmd help output should be sorted

2018-04-11 Thread serguei.spit...@oracle.com

  
  
Hi Thomas,
  
  Sorry, I did not reply to this.
  Thank you a lot for providing the output!
  I agree, the sorted one looks much better.
  
  Probably, it is late to say but I'm Ok with a push. :)
  
  Thanks,
  Serguei
  
  
  On 4/9/18 02:07, Thomas Stüfe wrote:


  
  Hi Sergey, Christoph,


thanks for the review! 


Sure, here you go:


Old output, unsorted:



  thomas@mainframe
/shared/projects/openjdk/jdk-submit-hs/output-fastdebug $
./images/jdk/bin/jcmd test3.Example2 help
  24278:
  The following commands are available:
  VM.log
  VM.native_memory
  ManagementAgent.status
  ManagementAgent.stop
  ManagementAgent.start_local
  ManagementAgent.start
  Compiler.directives_clear
  Compiler.directives_remove
  Compiler.directives_add
  Compiler.directives_print
  Compiler.CodeHeap_Analytics
  VM.print_touched_methods
  Compiler.codecache
  Compiler.codelist
  Compiler.queue
  VM.classloader_stats
  Thread.print
  JVMTI.data_dump
  JVMTI.agent_load
  VM.metaspace
  VM.stringtable
  VM.symboltable
  VM.class_hierarchy
  VM.systemdictionary
  GC.class_stats
  GC.class_histogram
  GC.heap_dump
  GC.finalizer_info
  GC.heap_info
  GC.run_finalization
  GC.run
  VM.info
  VM.uptime
  VM.dynlibs
  VM.set_flag
  VM.flags
  VM.system_properties
  VM.command_line
  VM.version
  help



New output, sorted:



  thomas@mainframe
/shared/projects/openjdk/jdk-submit-hs/output-fastdebug $
./images/jdk/bin/jcmd test3.Example2 help
  30230:
  The following commands are available:
  Compiler.CodeHeap_Analytics
  Compiler.codecache
  Compiler.codelist
  Compiler.directives_add
  Compiler.directives_clear
  Compiler.directives_print
  Compiler.directives_remove
  Compiler.queue
  GC.class_histogram
  GC.class_stats
  GC.finalizer_info
  GC.heap_dump
  GC.heap_info
  GC.run
  GC.run_finalization
  JVMTI.agent_load
  JVMTI.data_dump
  ManagementAgent.start
  ManagementAgent.start_local
  ManagementAgent.status
  ManagementAgent.stop
  Thread.print
  VM.class_hierarchy
  VM.classloader_stats
  VM.command_line
  VM.dynlibs
  VM.flags
  VM.info
  VM.log
  VM.metaspace
  VM.native_memory
  VM.print_touched_methods
  VM.set_flag
  VM.stringtable
  VM.symboltable
  VM.system_properties
  VM.systemdictionary
  VM.uptime
  VM.version
  help





I'm running submit tests now, if they pass I'll push.


Best Regards, Thomas


  
  
On Tue, Apr 3, 2018 at 3:52 AM, serguei.spit...@oracle.com 
  wrote:
  Hi Thomas,

Added the serviceability-dev mailing list as it is a
Serviceability area.

The fix looks good to me.
One question:
 Could you, please, post the sorted help output?
 It is interesting how does it look like when sorted.

Thanks,
Serguei

  


On 3/28/18 13:08, Thomas Stüfe wrote:

  Hi all,
  
  may I get reviews for this tiny trivial change which
  causes jcmd help
  output (the command list) to be sorted?
  
  bug: https://bugs.openjdk.java.net/browse/JDK-8200384
  webrev:
  http://cr.openjdk.java.net/~stuefe/webrevs/8200384-jcmd-help-sorted/webrev.00/webrev/
  
  Thanks!
  
  Best Regards, Thomas


  

  


  


  



Re: RFR(xxs): 8200384: jcmd help output should be sorted

2018-04-09 Thread Thomas Stüfe
Hi Sergey, Christoph,

thanks for the review!

Sure, here you go:

Old output, unsorted:

thomas@mainframe /shared/projects/openjdk/jdk-submit-hs/output-fastdebug $
./images/jdk/bin/jcmd test3.Example2 help
24278:
The following commands are available:
VM.log
VM.native_memory
ManagementAgent.status
ManagementAgent.stop
ManagementAgent.start_local
ManagementAgent.start
Compiler.directives_clear
Compiler.directives_remove
Compiler.directives_add
Compiler.directives_print
Compiler.CodeHeap_Analytics
VM.print_touched_methods
Compiler.codecache
Compiler.codelist
Compiler.queue
VM.classloader_stats
Thread.print
JVMTI.data_dump
JVMTI.agent_load
VM.metaspace
VM.stringtable
VM.symboltable
VM.class_hierarchy
VM.systemdictionary
GC.class_stats
GC.class_histogram
GC.heap_dump
GC.finalizer_info
GC.heap_info
GC.run_finalization
GC.run
VM.info
VM.uptime
VM.dynlibs
VM.set_flag
VM.flags
VM.system_properties
VM.command_line
VM.version
help

New output, sorted:

thomas@mainframe /shared/projects/openjdk/jdk-submit-hs/output-fastdebug $
./images/jdk/bin/jcmd test3.Example2 help
30230:
The following commands are available:
Compiler.CodeHeap_Analytics
Compiler.codecache
Compiler.codelist
Compiler.directives_add
Compiler.directives_clear
Compiler.directives_print
Compiler.directives_remove
Compiler.queue
GC.class_histogram
GC.class_stats
GC.finalizer_info
GC.heap_dump
GC.heap_info
GC.run
GC.run_finalization
JVMTI.agent_load
JVMTI.data_dump
ManagementAgent.start
ManagementAgent.start_local
ManagementAgent.status
ManagementAgent.stop
Thread.print
VM.class_hierarchy
VM.classloader_stats
VM.command_line
VM.dynlibs
VM.flags
VM.info
VM.log
VM.metaspace
VM.native_memory
VM.print_touched_methods
VM.set_flag
VM.stringtable
VM.symboltable
VM.system_properties
VM.systemdictionary
VM.uptime
VM.version
help


I'm running submit tests now, if they pass I'll push.

Best Regards, Thomas


On Tue, Apr 3, 2018 at 3:52 AM, serguei.spit...@oracle.com <
serguei.spit...@oracle.com> wrote:

> Hi Thomas,
>
> Added the serviceability-dev mailing list as it is a Serviceability area.
>
> The fix looks good to me.
> One question:
>  Could you, please, post the sorted help output?
>  It is interesting how does it look like when sorted.
>
> Thanks,
> Serguei
>
>
>
> On 3/28/18 13:08, Thomas Stüfe wrote:
>
>> Hi all,
>>
>> may I get reviews for this tiny trivial change which causes jcmd help
>> output (the command list) to be sorted?
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8200384
>> webrev:
>> http://cr.openjdk.java.net/~stuefe/webrevs/8200384-jcmd-help
>> -sorted/webrev.00/webrev/
>>
>> Thanks!
>>
>> Best Regards, Thomas
>>
>
>


RE: RFR(xxs): 8200384: jcmd help output should be sorted

2018-04-04 Thread Langer, Christoph
Hi Thomas,

I like the fix, too.

Maybe you can add example output before and after sorting to the bug.

Thanks
Christoph

> -Original Message-
> From: hotspot-runtime-dev [mailto:hotspot-runtime-dev-
> boun...@openjdk.java.net] On Behalf Of serguei.spit...@oracle.com
> Sent: Dienstag, 3. April 2018 03:53
> To: Thomas Stüfe <thomas.stu...@gmail.com>; Hotspot dev runtime
> <hotspot-runtime-...@openjdk.java.net>; serviceability-
> d...@openjdk.java.net
> Subject: Re: RFR(xxs): 8200384: jcmd help output should be sorted
> 
> Hi Thomas,
> 
> Added the serviceability-dev mailing list as it is a Serviceability area.
> 
> The fix looks good to me.
> One question:
>   Could you, please, post the sorted help output?
>   It is interesting how does it look like when sorted.
> 
> Thanks,
> Serguei
> 
> 
> On 3/28/18 13:08, Thomas Stüfe wrote:
> > Hi all,
> >
> > may I get reviews for this tiny trivial change which causes jcmd help
> > output (the command list) to be sorted?
> >
> > bug: https://bugs.openjdk.java.net/browse/JDK-8200384
> > webrev:
> > http://cr.openjdk.java.net/~stuefe/webrevs/8200384-jcmd-help-
> sorted/webrev.00/webrev/
> >
> > Thanks!
> >
> > Best Regards, Thomas



Re: RFR(xxs): 8200384: jcmd help output should be sorted

2018-04-02 Thread serguei.spit...@oracle.com

Hi Thomas,

Added the serviceability-dev mailing list as it is a Serviceability area.

The fix looks good to me.
One question:
 Could you, please, post the sorted help output?
 It is interesting how does it look like when sorted.

Thanks,
Serguei


On 3/28/18 13:08, Thomas Stüfe wrote:

Hi all,

may I get reviews for this tiny trivial change which causes jcmd help
output (the command list) to be sorted?

bug: https://bugs.openjdk.java.net/browse/JDK-8200384
webrev:
http://cr.openjdk.java.net/~stuefe/webrevs/8200384-jcmd-help-sorted/webrev.00/webrev/

Thanks!

Best Regards, Thomas




Re: RFR(xxs): 8200384: jcmd help output should be sorted

2018-03-28 Thread David Holmes

Re-directing to serviceability-dev.

David

On 29/03/2018 6:08 AM, Thomas Stüfe wrote:

Hi all,

may I get reviews for this tiny trivial change which causes jcmd help
output (the command list) to be sorted?

bug: https://bugs.openjdk.java.net/browse/JDK-8200384
webrev:
http://cr.openjdk.java.net/~stuefe/webrevs/8200384-jcmd-help-sorted/webrev.00/webrev/

Thanks!

Best Regards, Thomas