Re: RFR: 8000489: older builds of hsdis don't work anymore after 6879063

[Yumin Qi]

Serguei and Krys

  If you are OK with the second version, I will do integration into 
hotspot_rt.
http://cr.openjdk.java.net/~minqi/8000489
Thanks
Yumin

On 10/24/2012 2:50 PM, serguei.spit...@oracle.com wrote:

|| Yumin,

*src/share/tools/hsdis/hsdis.h:
*

A typo in the comment:

   62 /* This is the compatability interface for older version of hotspot */
     =>
   62 /* This is the compatibility interface for older versions of hotspot */

||*|src|/share/tools/hsdis/hsdis-demo.c:*

Would it make sense to change this to avoid using literal name? :

219   printf("Decoding from %p to %p...with decode_instructions_virtual\n", 
from, to);
   =>
219   printf("Decoding from %p to %p...with %s\n", from, 
to,DECODE_INSTRUCTIONS_NAME);

   and

238   printf("Decoding from %p to %p...with old decode_instructions\n", from, 
to);
   =>
238   printf("Decoding from %p to %p...with old %s\n", from, 
to,DECODE_INSTRUCTIONS);

The (raw&&  xml) is a special case of the (raw) so that the 4 lines:

  225   if (raw&&  xml) {
  226     res = (*decode_instructions_v)(from, to, (unsigned char*)from, to - 
from, simple_handle_event, stdout, NULL, stdout, options);
  227   } else if (raw) {
  228     res = (*decode_instructions_v)(from, to, (unsigned char*)from, to - 
from, simple_handle_event, stdout, NULL, stdout, options);
can be replaced with just 2 lines:
  225   if (raw) {
  226     res = (*decode_instructions_v)(from, to, (unsigned char*)from, to - 
from, simple_handle_event, stdout, NULL, stdout, options);


Thanks,
Serguei




Thanks,
Serguei


On 10/24/12 2:15 PM, yumin...@oracle.com wrote:
Krystal,

  Thanks. Updated in same webrev.
  I checked b10, the output looks OK

/Yumin

On 10/24/2012 12:24 PM, Krystal Mok wrote:
Hi Yumin,

minor typos:
s/instrurctions/instructions/

By the way, I tried to use a build of hsdis plugin after 6879063 in 
an earlier JDK build (e.g. JDK8b59) and it was working strangely. A 
rough look at the output looks like it's missing some newlines, I 
didn't go into the details. Could you please check if this fix makes 
the hsdis plugin work properly on earlier JDK builds as well?

Thanks,
Kris

On Thu, Oct 25, 2012 at 1:01 AM, <yumin...@oracle.com 
<mailto:yumin...@oracle.com>> wrote:

     Hi, all

      Can I have your codereview of 8000489: older builds of hsdis
    don't work anymore after 6879063
      It caused old build broke the disassembler.

      Webrev: http://cr.openjdk.java.net/~minqi/8000489
    <http://cr.openjdk.java.net/%7Eminqi/8000489>

    Thanks
    Yumin