8004840: Jstack seems to output unnecessary information in 7u9

Thu, 14 Feb 2013 13:08:15 +0100

author
sla
date
Thu, 14 Feb 2013 13:08:15 +0100
changeset 4600
49618582fc5b
parent 4599
2394a89e89f4
child 4601
3a531d40ad93

8004840: Jstack seems to output unnecessary information in 7u9
Reviewed-by: dholmes, coleenp, sspitsyn, rbackman

agent/src/share/classes/sun/jvm/hotspot/memory/CMSCollector.java file | annotate | diff | comparison | revisions
agent/src/share/classes/sun/jvm/hotspot/memory/CompactibleFreeListSpace.java file | annotate | diff | comparison | revisions
agent/src/share/classes/sun/jvm/hotspot/oops/MethodData.java file | annotate | diff | comparison | revisions
agent/src/share/classes/sun/jvm/hotspot/oops/ObjectHeap.java file | annotate | diff | comparison | revisions
     1.1 --- a/agent/src/share/classes/sun/jvm/hotspot/memory/CMSCollector.java	Wed Feb 13 09:46:19 2013 +0100
     1.2 +++ b/agent/src/share/classes/sun/jvm/hotspot/memory/CMSCollector.java	Thu Feb 14 13:08:15 2013 +0100
     1.3 @@ -61,15 +61,13 @@
     1.4      CMSBitMap markBitMap = markBitMap();
     1.5      long addressSize = VM.getVM().getAddressSize();
     1.6      if ( markBitMap.isMarked(addr) &&  markBitMap.isMarked(addr.addOffsetTo(1*addressSize)) ) {
     1.7 -       System.err.println("Printezis bits are set...");
     1.8        Address nextOneAddr = markBitMap.getNextMarkedWordAddress(addr.addOffsetTo(2*addressSize));
     1.9        //return size in bytes
    1.10        long size =  (nextOneAddr.addOffsetTo(1*addressSize)).minus(addr);
    1.11        return size;
    1.12      } else {
    1.13 -     //missing Printezis marks
    1.14 -     System.err.println("Missing Printszis marks...");
    1.15 -     return -1;
    1.16 +      //missing Printezis marks
    1.17 +      return -1;
    1.18      }
    1.19  
    1.20    }
     2.1 --- a/agent/src/share/classes/sun/jvm/hotspot/memory/CompactibleFreeListSpace.java	Wed Feb 13 09:46:19 2013 +0100
     2.2 +++ b/agent/src/share/classes/sun/jvm/hotspot/memory/CompactibleFreeListSpace.java	Thu Feb 14 13:08:15 2013 +0100
     2.3 @@ -191,7 +191,6 @@
     2.4              //Find the object size using Printezis bits and skip over
     2.5              long size = collector().blockSizeUsingPrintezisBits(cur);
     2.6              if (size == -1) {
     2.7 -              System.err.println("Printezis bits not set...");
     2.8                break;
     2.9              }
    2.10              cur = cur.addOffsetTo(adjustObjectSizeInBytes(size));
     3.1 --- a/agent/src/share/classes/sun/jvm/hotspot/oops/MethodData.java	Wed Feb 13 09:46:19 2013 +0100
     3.2 +++ b/agent/src/share/classes/sun/jvm/hotspot/oops/MethodData.java	Thu Feb 14 13:08:15 2013 +0100
     3.3 @@ -184,7 +184,6 @@
     3.4        if (trapReasonName[index] == null) {
     3.5          throw new InternalError("missing reason for " + index);
     3.6        }
     3.7 -      System.out.println(trapReasonName[index]);
     3.8      }
     3.9    }
    3.10  
     4.1 --- a/agent/src/share/classes/sun/jvm/hotspot/oops/ObjectHeap.java	Wed Feb 13 09:46:19 2013 +0100
     4.2 +++ b/agent/src/share/classes/sun/jvm/hotspot/oops/ObjectHeap.java	Thu Feb 14 13:08:15 2013 +0100
     4.3 @@ -335,7 +335,6 @@
     4.4            }
     4.5            if (obj == null) {
     4.6               //Find the object size using Printezis bits and skip over
     4.7 -             System.err.println("Finding object size using Printezis bits and skipping over...");
     4.8               long size = 0;
     4.9  
    4.10               if ( (cmsSpaceOld != null) && cmsSpaceOld.contains(handle) ){

mercurial