Re: review(M): 7144296: PS: Optimize nmethods processing

Igor Veresov (igor.veresov@...) wrote:
> Thanks, John! 
> Here the updated webrev: http://cr.openjdk.java.net/~iveresov/7144296/webrev.01/

Looks good to me.  And thanks for fixing my typos .

-John

> On Friday, February 10, 2012 at 4:50 PM, John Coomes wrote:
> 
> > Igor Veresov (igor.veresov@...
(mailto:igor.veresov@...)) wrote:
> > > This change does two things: 
> > > 1. Prunes the "scavenge roots in code" list after each minor GC.
> > > 2. Promotes objects pointed by code directly into the old gen, instead of ping-ponging them in the
survivor spaces.
> > > 
> > > Webrev: http://cr.openjdk.java.net/~iveresov/7144296/webrev.00/
> > 
> > Looks good. Minor suggestion - you could use a typedef and get
> > meaningful names for the closures, e.g.,
> > 
> > typedef PSRootsClosure<true> PSScavengeRootsClosure;
> > typedef PSRootsClosure<false> PSPromoteRootsClosure;
> > 
> > To keep the existing name (PSScavengeRootsClosure) the same at the use
> > points, the template class has to be renamed. I used PSRootsClosure,
> > but any other name would work.
> > 
> > -John 

review(M): 7144296: PS: Optimize nmethods processing

hg: hsx/hotspot-gc/jdk: 47 new changesets

Changeset: ad9f1c8970da
Author:    prr
Date:      2012-01-19 12:41 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/ad9f1c8970da

7131153: GetDC called way too many times - causes bad performance.
Reviewed-by: igor, jgodinez

! src/windows/native/sun/font/fontpath.c

Changeset: f7dda4bbf1f9
Author:    lana
Date:      2012-01-28 22:47 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/f7dda4bbf1f9

Merge

- test/java/io/File/BlockIsDirectory.java

Changeset: 84b153cd9bd4
Author:    denis
Date:      2012-01-19 14:59 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/84b153cd9bd4

7121761: creation of java.awt.DataFlavour fails for turkish locale
Reviewed-by: anthony

! src/share/classes/java/awt/datatransfer/MimeType.java

Changeset: e32db6535c05
Author:    alexsch
Date:      2012-01-23 13:05 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/e32db6535c05

7112854: [macosx] closed/javax/swing/JPopupMenu/Test6827786.java unstable on MacOS
Reviewed-by: rupashka

+ test/javax/swing/JPopupMenu/6827786/bug6827786.java

Changeset: cc88a9c0474f
Author:    alexsch
Date:      2012-01-23 13:53 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/cc88a9c0474f

7116634: [macosx] closed/javax/swing/JTree/6263446/bug6263446Tree.java fails on MacOS
Reviewed-by: rupashka

+ test/javax/swing/JTree/6263446/bug6263446.java

Changeset: 19431d07bc19
Author:    denis
Date:      2012-01-23 17:26 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/19431d07bc19

7130140: using horizontal scroll button on mouse causes a message to be printed on stdout
Reviewed-by: art

! src/share/classes/java/awt/event/MouseEvent.java

Changeset: 5255fd5b0418
Author:    denis
Date:      2012-01-24 18:46 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/5255fd5b0418

7078460: JDialog is shown as separate icon on the taskbar
Reviewed-by: anthony

! src/solaris/classes/sun/awt/X11/XWindowPeer.java

Changeset: b4589ff4457c
Author:    malenkov
Date:      2012-01-24 19:40 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/b4589ff4457c

7121905: grammatically incorrect apostrophe in BeanInfo javadoc
Reviewed-by: rupashka

! src/share/classes/java/beans/BeanInfo.java

Changeset: 4f2a2bf0ce84
Author:    rupashka
Date:      2012-01-26 17:38 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/4f2a2bf0ce84

7010561: Tab text position with Synth based LaF is different to Java 5/6
Reviewed-by: alexp

! src/share/classes/javax/swing/plaf/basic/BasicTabbedPaneUI.java
+ test/javax/swing/JTabbedPane/7010561/bug7010561.java

Changeset: cc9ff174a1c3
Author:    alexsch
Date:      2012-01-27 16:32 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/cc9ff174a1c3

7122173: [macosx] Several Regression tests fail on MacOS
Reviewed-by: rupashka

+ test/javax/swing/SwingUtilities/4917669/bug4917669.java
+ test/javax/swing/plaf/basic/BasicHTML/4251579/bug4251579.java
+ test/javax/swing/text/html/CSS/4530474/bug4530474.java
+ test/javax/swing/text/html/CSS/4530474/test.css
+ test/javax/swing/text/html/CSS/4530474/test.html

Changeset: 96b5999af66b
Author:    alexsch
Date:      2012-01-27 17:00 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/96b5999af66b

7109962: [macosx] closed/javax/swing/JList/6462008/bug6462008.java fails on MacOS
Reviewed-by: rupashka

+ test/javax/swing/JList/6462008/bug6462008.java

Changeset: 6a7109f52966
Author:    alexsch
Date:      2012-01-27 17:36 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/6a7109f52966

7105040: [macosx] closed/javax/swing/JPopupMenu/4966112/bug4966112.java deadlocks on MacOS
Reviewed-by: rupashka

+ test/javax/swing/JPopupMenu/4966112/bug4966112.java

Changeset: bc1c20ac8676
Author:    chegar
Date:      2012-01-27 13:48 +0000
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/bc1c20ac8676

7110002: Rename xawt/libmawt.so and headless/libmawt.so so they can be colocated with libawt
Reviewed-by: art, prr, dholmes, alanb

! make/common/Release-embedded.gmk
! make/sun/font/Makefile
! make/sun/font/t2k/Makefile
! make/sun/headless/Makefile
! make/sun/jawt/Makefile
! make/sun/xawt/Makefile
! src/solaris/native/java/lang/java_props_md.c
! src/solaris/native/sun/awt/awt_LoadLibrary.c

Changeset: 5dab2d55bc5b
Author:    lana
Date:      2012-01-28 22:21 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/5dab2d55bc5b

Merge

- test/java/io/File/BlockIsDirectory.java

Changeset: 39b661c5867a
Author:    alexsch
Date:      2012-01-30 12:52 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/39b661c5867a

7122149: [macosx] closed/javax/swing/UITest/UITest.java fails on MacOS
Reviewed-by: rupashka

! src/share/classes/sun/awt/OSInfo.java
+ test/javax/swing/UITest/UITest.java

Changeset: 7d6c7dd72e25
Author:    malenkov
Date:      2012-01-31 14:20 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/7d6c7dd72e25

7122138: IAE thrown because Introspector ignores synthetic methods
Reviewed-by: rupashka

! src/share/classes/java/beans/Introspector.java
! src/share/classes/java/beans/PropertyDescriptor.java
+ test/java/beans/Introspector/7122138/Test7122138.java
+ test/java/beans/Introspector/7122138/pack/Sub.java
+ test/java/beans/Introspector/7122138/pack/Super.java

Changeset: c5c78f293ff8
Author:    rupashka
Date:      2012-01-31 17:30 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/c5c78f293ff8

7082443: JComboBox not backward compatible (with Java 6)
Reviewed-by: alexp

! src/share/classes/javax/swing/plaf/synth/SynthComboBoxUI.java
+ test/javax/swing/JComboBox/7082443/bug7082443.java

Changeset: 363086137375
Author:    lana
Date:      2012-01-31 19:06 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/363086137375

Merge

Changeset: 313da5d059bf
Author:    valeriep
Date:      2012-01-19 12:01 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/313da5d059bf

7092825: javax.crypto.Cipher.Transform.patternCache is synchronizedMap and became scalability bottleneck.
Summary: Changed patternCache from synchronizedMap to ConcurrentHashMap.
Reviewed-by: mullan

! src/share/classes/javax/crypto/Cipher.java

Changeset: 71200c517524
Author:    darcy
Date:      2012-01-20 17:56 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/71200c517524

4504839: Java libraries should provide support for unsigned integer arithmetic
4215269: Some Integer.toHexString(int) results cannot be decoded back to an int
6322074: Converting integers to string as if unsigned
Reviewed-by: mduigou, emcmanus, flar

! src/share/classes/java/lang/Byte.java
! src/share/classes/java/lang/Integer.java
! src/share/classes/java/lang/Long.java
! src/share/classes/java/lang/Short.java
+ test/java/lang/Integer/Unsigned.java
+ test/java/lang/Long/Unsigned.java

Changeset: d383b5d128e3
Author:    xuelei
Date:      2012-01-23 04:44 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/d383b5d128e3

7132248:
sun/security/ssl/sun/net/www/protocol/https/HttpsURLConnection/CookieHttpsClientTest.java failing
Reviewed-by: alanb

! test/sun/security/ssl/sun/net/www/protocol/https/HttpsURLConnection/CookieHttpsClientTest.java

Changeset: 3df0bd3ed880
Author:    mullan
Date:      2012-01-23 12:17 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/3df0bd3ed880

7131084: XMLDSig XPathFilter2Transform regression involving intersect filter
Reviewed-by: xuelei

! src/share/classes/com/sun/org/apache/xml/internal/security/transforms/implementations/TransformXPath2Filter.java
! test/javax/xml/crypto/dsig/KeySelectors.java
! test/javax/xml/crypto/dsig/ValidationTests.java
! test/javax/xml/crypto/dsig/X509KeySelector.java
+ test/javax/xml/crypto/dsig/data/xmldsig-xfilter2.xml

Changeset: 5e1ad6ad41b7
Author:    mullan
Date:      2012-01-23 13:23 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/5e1ad6ad41b7

Merge

Changeset: 914711cccc60
Author:    darcy
Date:      2012-01-23 12:17 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/914711cccc60

7132338: Use @code friendly idiom for '\' in javadoc
Reviewed-by: alanb

! src/share/classes/java/io/DataInput.java
! src/share/classes/java/io/LineNumberInputStream.java
! src/share/classes/java/io/RandomAccessFile.java
! src/share/classes/java/io/StreamTokenizer.java
! src/share/classes/java/lang/AbstractStringBuilder.java
! src/share/classes/java/lang/Byte.java
! src/share/classes/java/lang/Double.java
! src/share/classes/java/lang/Float.java
! src/share/classes/java/lang/Integer.java
! src/share/classes/java/lang/Long.java
! src/share/classes/java/lang/Short.java
! src/share/classes/java/lang/String.java
! src/share/classes/java/util/Properties.java

Changeset: 237319a01a9a
Author:    alanb
Date:      2012-01-24 09:09 +0000
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/237319a01a9a

7132204: Default testset in JPRT should not run all tests
Reviewed-by: ohair

! make/jprt.properties
! test/ProblemList.txt

Changeset: 718bca4e685f
Author:    rbackman
Date:      2012-01-17 16:20 +0100
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/718bca4e685f

7132386: makefile support for tracing/Java Flight Recorder framework phase I
Reviewed-by: ohair, dholmes, rottenha
Contributed-by: Markus Gronlund <markus.gronlund@...>,
Erik Gahlin <erik.gahlin@...>, Nils Loodin
<nils.loodin@...>, Rickard Backman
<rickard.backman@...>, Staffan Larsen <staffan.larsen@...>

! make/com/oracle/Makefile
+ make/com/oracle/jfr/Makefile
! make/common/Defs.gmk
! make/common/Release.gmk

Changeset: f64ea40293db
Author:    ksrini
Date:      2012-01-24 09:58 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/f64ea40293db

7132270: tools/launcher/DefaultLocaleTestRun.java failing (win)
Reviewed-by: alanb, chegar

! test/tools/launcher/DefaultLocaleTestRun.java
! test/tools/launcher/TestHelper.java

Changeset: 303b67074666
Author:    lancea
Date:      2012-01-24 15:13 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/303b67074666

7132879: address Findbugs issue in WebRowSetXmlWriter
Reviewed-by: forax

! src/share/classes/com/sun/rowset/internal/WebRowSetXmlWriter.java

Changeset: ceab7e149581
Author:    peytoia
Date:      2012-01-26 17:06 +0900
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/ceab7e149581

7017458: (cal) Multithreaded deserialization of Calendar leads to ClassCastException
Reviewed-by: okutsu

! src/share/classes/java/util/Calendar.java
+ test/java/util/Calendar/Bug7017458.java

Changeset: 350971f50949
Author:    rbackman
Date:      2012-01-26 09:51 +0100
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/350971f50949

7133124: Remove redundant packages from JAR command line
Reviewed-by: acorn, alanb, dholmes, rottenha

! make/common/Release.gmk

Changeset: b518b160607f
Author:    lancea
Date:      2012-01-26 19:41 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/b518b160607f

7133815: address the findbug errors in CachedRowSetImpl,  SerialStruct, BaseRow, SerialInputImpl, SerialOutputImpl
Reviewed-by: forax

! src/share/classes/com/sun/rowset/CachedRowSetImpl.java
! src/share/classes/com/sun/rowset/internal/BaseRow.java
! src/share/classes/javax/sql/rowset/serial/SQLInputImpl.java
! src/share/classes/javax/sql/rowset/serial/SQLOutputImpl.java
! src/share/classes/javax/sql/rowset/serial/SerialStruct.java

Changeset: 5ee30ab905db
Author:    wetmore
Date:      2012-01-26 17:16 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/5ee30ab905db

7126889: Incorrect SSLEngine debug output
Reviewed-by: xuelei

! src/share/classes/sun/security/ssl/EngineArgs.java
! src/share/classes/sun/security/ssl/SSLEngineImpl.java
+ test/sun/security/ssl/com/sun/net/ssl/internal/ssl/EngineArgs/DebugReportsOneExtraByte.java
+ test/sun/security/ssl/com/sun/net/ssl/internal/ssl/EngineArgs/DebugReportsOneExtraByte.sh

Changeset: 7aa5ddfa3c9d
Author:    okutsu
Date:      2012-01-27 14:58 +0900
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/7aa5ddfa3c9d

7130335: Problem with timezone in a SimpleDateFormat
Reviewed-by: peytoia

! src/share/classes/java/text/SimpleDateFormat.java
+ test/java/text/Format/DateFormat/Bug7130335.java

Changeset: ff24779c147f
Author:    valeriep
Date:      2012-01-27 15:25 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/ff24779c147f

7136538: typo in test/Makefile under the jdk_security3 target
Summary: Fixed the typo of "secrity".
Reviewed-by: wetmore

! test/Makefile

Changeset: 7dbc129d8e5c
Author:    ksrini
Date:      2012-01-28 10:46 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/7dbc129d8e5c

7127906: (launcher) convert the launcher regression tests to java
Reviewed-by: darcy, naoto

! test/tools/launcher/Arrrghs.java
+ test/tools/launcher/ChangeDataModel.java
- test/tools/launcher/ChangeDataModel.sh
- test/tools/launcher/CreatePlatformFile.java
! test/tools/launcher/DefaultLocaleTestRun.java
! test/tools/launcher/ExecutionEnvironment.java
! test/tools/launcher/I18NJarTest.java
+ test/tools/launcher/I18NTest.java
! test/tools/launcher/MiscTests.java
! test/tools/launcher/Settings.java
- test/tools/launcher/SomeException.java
! test/tools/launcher/Test7029048.java
! test/tools/launcher/TestHelper.java
- test/tools/launcher/UnicodeCleanup.java
! test/tools/launcher/UnicodeTest.java
- test/tools/launcher/UnicodeTest.sh
! test/tools/launcher/UnresolvedExceptions.java
- test/tools/launcher/deleteI18n.sh
- test/tools/launcher/i18nTest.sh
- test/tools/launcher/unresolvedExceptions.sh

Changeset: 7a25b72b3644
Author:    lana
Date:      2012-01-28 20:41 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/7a25b72b3644

Merge

Changeset: f9fb8c4b4550
Author:    dl
Date:      2012-01-30 11:44 +0000
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/f9fb8c4b4550

7132378: Race in FutureTask if used with explicit set ( not Runnable )
Reviewed-by: chegar, dholmes

! src/share/classes/java/util/concurrent/FutureTask.java
+ test/java/util/concurrent/FutureTask/DoneTimedGetLoops.java
+ test/java/util/concurrent/FutureTask/ExplicitSet.java

Changeset: 863a20b0bf08
Author:    ngmr
Date:      2012-01-10 00:07 +0000
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/863a20b0bf08

7123229: (coll) EnumMap.containsValue(null) returns true
Summary: java.util.EnumMap.NULL equals() must only be true for itself
Reviewed-by: alanb, mduigou
Contributed-by: Neil Richards <neil.richards@...>

! src/share/classes/java/util/EnumMap.java
+ test/java/util/EnumMap/UniqueNullValue.java

Changeset: 13978750cb87
Author:    ngmr
Date:      2012-01-31 10:31 +0000
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/13978750cb87

7133301: (process) UNIXProcess_md.c should include sys/wait.h rather than wait.h
Reviewed-by: alanb
Contributed-by: Jonathan Lu <luchsh@...>

! src/solaris/native/java/lang/UNIXProcess_md.c

Changeset: 431bc327f34a
Author:    sla
Date:      2012-01-31 10:46 +0100
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/431bc327f34a

7132199: sun/management/jmxremote/bootstrap/JvmstatCountersTest.java failing on all platforms
Summary: Make sure HotSpot and JDK looks for well-known files in the same location
Reviewed-by: dholmes, dsamersoff

! src/solaris/classes/sun/tools/attach/LinuxVirtualMachine.java
! src/solaris/classes/sun/tools/attach/SolarisVirtualMachine.java
! test/ProblemList.txt

Changeset: 663a6333105d
Author:    sla
Date:      2012-01-31 04:57 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/663a6333105d

Merge

Changeset: 533bc0a10233
Author:    lana
Date:      2012-01-31 19:08 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/533bc0a10233

Merge

- test/tools/launcher/ChangeDataModel.sh
- test/tools/launcher/CreatePlatformFile.java
- test/tools/launcher/SomeException.java
- test/tools/launcher/UnicodeCleanup.java
- test/tools/launcher/UnicodeTest.sh
- test/tools/launcher/deleteI18n.sh
- test/tools/launcher/i18nTest.sh
- test/tools/launcher/unresolvedExceptions.sh

Changeset: ce62fb7aa1b8
Author:    lana
Date:      2012-02-07 10:38 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/ce62fb7aa1b8

Merge

- test/tools/launcher/ChangeDataModel.sh
- test/tools/launcher/CreatePlatformFile.java
- test/tools/launcher/SomeException.java
- test/tools/launcher/UnicodeCleanup.java
- test/tools/launcher/UnicodeTest.sh
- test/tools/launcher/deleteI18n.sh
- test/tools/launcher/i18nTest.sh
- test/tools/launcher/unresolvedExceptions.sh

Changeset: 1a99dad28223
Author:    yhuang
Date:      2012-02-06 18:56 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/1a99dad28223

7129382: change minor unit of VND to 0
Reviewed-by: naoto

! src/share/classes/java/util/CurrencyData.properties
! test/java/util/Currency/tablea1.txt

Changeset: 930756e55285
Author:    yhuang
Date:      2012-02-06 18:58 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/930756e55285

Merge

Changeset: ec17fbe5b8fb
Author:    katleman
Date:      2012-02-08 19:13 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/ec17fbe5b8fb

Merge

Changeset: 5aca406e87cb
Author:    katleman
Date:      2012-02-09 12:56 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/5aca406e87cb

Added tag jdk8-b25 for changeset ec17fbe5b8fb

! .hgtags

hg: hsx/hotspot-gc/jaxws: Added tag jdk8-b25 for changeset b376d901e006

Changeset: 3518639eab6c
Author:    katleman
Date:      2012-02-09 12:55 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jaxws/rev/3518639eab6c

Added tag jdk8-b25 for changeset b376d901e006

! .hgtags

hg: hsx/hotspot-gc: 4 new changesets

Changeset: 0f653ee93477
Author:    alanb
Date:      2012-01-24 09:08 +0000
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/rev/0f653ee93477

7132204: Default testset in JPRT should not run all tests
Reviewed-by: ohair

! make/jprt.properties

Changeset: bd3fcc98c5d2
Author:    lana
Date:      2012-01-28 20:36 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/rev/bd3fcc98c5d2

Merge

Changeset: 221a378e06a3
Author:    lana
Date:      2012-02-07 10:36 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/rev/221a378e06a3

Merge

Changeset: 2accafff224a
Author:    katleman
Date:      2012-02-09 12:55 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/rev/2accafff224a

Added tag jdk8-b25 for changeset 221a378e06a3

! .hgtags

hg: hsx/hotspot-gc/jaxp: Added tag jdk8-b25 for changeset bb694c151fc7

Changeset: dbb7283c197b
Author:    katleman
Date:      2012-02-09 12:55 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jaxp/rev/dbb7283c197b

Added tag jdk8-b25 for changeset bb694c151fc7

! .hgtags

hg: hsx/hotspot-gc/corba: Added tag jdk8-b25 for changeset e45d6b406d5f

Changeset: 79f709a099f4
Author:    katleman
Date:      2012-02-09 12:55 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/corba/rev/79f709a099f4

Added tag jdk8-b25 for changeset e45d6b406d5f

! .hgtags

Re: separate ParallelGCThreads ?

UseAdaptiveGCBoundary?

hg: hsx/hotspot-gc/hotspot: 8 new changesets

Changeset: 905945c5913e
Author:    katleman
Date:      2012-02-02 09:39 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/905945c5913e

Added tag jdk8-b24 for changeset a80fd4f45d7a

! .hgtags

Changeset: b22de8247499
Author:    amurillo
Date:      2012-02-03 18:04 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/b22de8247499

Merge

Changeset: 4e9b30938cbf
Author:    amurillo
Date:      2012-02-03 18:04 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/4e9b30938cbf

Added tag hs23-b13 for changeset b22de8247499

! .hgtags

Changeset: 1f22b536808b
Author:    amurillo
Date:      2012-02-03 18:09 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/1f22b536808b

7142393: new hotspot build - hs23-b14
Reviewed-by: jcoomes

! make/hotspot_version

Changeset: 585feefad374
Author:    phh
Date:      2012-02-06 14:01 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/585feefad374

7142852: MAC: Comment out JPRT jbb tests on Mac OS X until 7142850 is resolved
Summary: Comment out JPRT jbb tests on Mac OS X until GUI hang can be fixed
Reviewed-by: dholmes, brutisso, phh
Contributed-by: james.melvin@...

! make/jprt.properties

Changeset: 64b46f975ab8
Author:    phh
Date:      2012-02-06 14:02 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/64b46f975ab8

7142616: MAC: Honor ALT_EXPORT_PATH overrides from JDK control builds
Summary: Fix EXPORT_PATH overrides on Mac OS X and only change default.
Reviewed-by: phh, dcubed
Contributed-by: james.melvin@...

! make/bsd/makefiles/defs.make
! make/bsd/makefiles/universal.gmk

Changeset: 9ad8feb5afbd
Author:    amurillo
Date:      2012-02-06 12:13 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/9ad8feb5afbd

Added tag hs23-b14 for changeset 64b46f975ab8

! .hgtags

Changeset: 3c4621be5149
Author:    amurillo
Date:      2012-02-06 12:18 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/3c4621be5149

7143122: new hotspot build - hs23-b15
Reviewed-by: jcoomes

! make/hotspot_version

hg: hsx/hotspot-gc/hotspot: 36 new changesets

Changeset: af739d5ab23c
Author:    bpittore
Date:      2012-01-21 23:02 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/af739d5ab23c

6972759: Step over not working after thrown exception and Pop
Summary: reset jvmtithreadstate exception state after frame pop and forceearlyreturn processed
Reviewed-by: minqi, dholmes, dlong
Contributed-by: bill.pittore@...

! src/share/vm/prims/jvmtiThreadState.cpp
! src/share/vm/prims/jvmtiThreadState.hpp

Changeset: 583b428aa858
Author:    coleenp
Date:      2012-01-23 17:45 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/583b428aa858

Merge

- src/os/bsd/vm/decoder_bsd.cpp

Changeset: d6660fedbab5
Author:    phh
Date:      2012-01-24 14:07 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/d6660fedbab5

7126732: MAC: Require Mac OS X builds/tests for JPRT integrate jobs for HotSpot
Summary: Modify jprt.properties to run OSX builds and tests.
Reviewed-by: dcubed, kamg, ohair, dholmes
Contributed-by: james.melvin@...

! make/jprt.properties

Changeset: bf864f701a4a
Author:    dsamersoff
Date:      2012-01-25 02:29 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/bf864f701a4a

7066129: GarbageCollectorMXBean#getLastGcInfo leaks native memory
Summary: Make GCStatInfo a resource object
Reviewed-by: phh, coleenp

! src/share/vm/services/gcNotifier.cpp
! src/share/vm/services/management.cpp
! src/share/vm/services/memoryManager.cpp
! src/share/vm/services/memoryManager.hpp

Changeset: df88f58f3b61
Author:    dsamersoff
Date:      2012-01-24 20:15 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/df88f58f3b61

Merge

Changeset: e8a4934564b2
Author:    phh
Date:      2012-01-24 19:33 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/e8a4934564b2

7125793: MAC: test_gamma should always work
Summary: Fix gamma launcher on Mac OS X and reconcile test_gamma script on Unix platforms
Reviewed-by: dcubed, ohair, jcoomes, dholmes, ksrini
Contributed-by: james.melvin@...

! make/bsd/Makefile
! make/bsd/makefiles/buildtree.make
! make/bsd/makefiles/defs.make
! make/bsd/makefiles/launcher.make
! make/bsd/makefiles/vm.make
! make/linux/makefiles/buildtree.make
! make/solaris/makefiles/buildtree.make
! src/os/bsd/vm/os_bsd.cpp
! src/os/posix/launcher/java_md.c

Changeset: 78dadb7b16ab
Author:    phh
Date:      2012-01-25 01:16 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/78dadb7b16ab

Merge

Changeset: d708a8cdd022
Author:    kamg
Date:      2012-01-25 10:08 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/d708a8cdd022

Merge

Changeset: 520830f632e7
Author:    fparain
Date:      2012-01-25 10:32 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/520830f632e7

7131346: Parsing of boolean arguments to diagnostic commands is broken
Reviewed-by: dholmes, dcubed

! src/share/vm/services/diagnosticArgument.cpp
! src/share/vm/utilities/globalDefinitions_visCPP.hpp

Changeset: 24ec1a6d6ef3
Author:    fparain
Date:      2012-01-25 16:33 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/24ec1a6d6ef3

Merge

Changeset: a42c07c38c47
Author:    dsamersoff
Date:      2012-01-25 21:10 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/a42c07c38c47

7132515: Add dcmd to manage UnlockingCommercialFeature flag
Summary: Added dcmd to unlock or check status of UnlockingCommercialFeature flag
Reviewed-by: fparain, rottenha

! src/share/vm/services/diagnosticCommand.cpp
! src/share/vm/services/diagnosticCommand.hpp
+ src/share/vm/services/diagnosticCommand_ext.hpp
! src/share/vm/services/diagnosticFramework.hpp
! src/share/vm/services/management.cpp

Changeset: 6d00795f99a1
Author:    dsamersoff
Date:      2012-01-25 15:03 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/6d00795f99a1

Merge

Changeset: 6db63e782d3d
Author:    dsamersoff
Date:      2012-01-25 18:58 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/6db63e782d3d

Merge

Changeset: de268c8a8075
Author:    phh
Date:      2012-01-26 20:06 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/de268c8a8075

7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
Summary: Add CriticalPriority == MaxPriority+1 and enable scheduling class as well as thread priority to
change on Solaris.
Reviewed-by: dholmes, dcubed

! src/os/bsd/vm/os_bsd.cpp
! src/os/linux/vm/os_linux.cpp
! src/os/solaris/vm/osThread_solaris.hpp
! src/os/solaris/vm/os_solaris.cpp
! src/os/windows/vm/os_windows.cpp
! src/share/vm/compiler/compileBroker.cpp
! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepThread.cpp
! src/share/vm/runtime/globals.hpp
! src/share/vm/runtime/os.hpp

Changeset: bf5da1648543
Author:    kamg
Date:      2012-01-27 10:42 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/bf5da1648543

Merge

! src/share/vm/compiler/compileBroker.cpp
! src/share/vm/runtime/globals.hpp

Changeset: 6edfe6e42a68
Author:    katleman
Date:      2012-01-26 18:23 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/6edfe6e42a68

Added tag jdk8-b23 for changeset e850d8e7ea54

! .hgtags

Changeset: 9e177d44b10f
Author:    amurillo
Date:      2012-01-27 14:44 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/9e177d44b10f

Merge

Changeset: a80fd4f45d7a
Author:    amurillo
Date:      2012-01-27 14:44 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/a80fd4f45d7a

Added tag hs23-b12 for changeset 9e177d44b10f

! .hgtags

Changeset: 9f1c2b7cdfb6
Author:    amurillo
Date:      2012-01-27 14:49 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/9f1c2b7cdfb6

7135385: new hotspot build - hs23-b13
Reviewed-by: jcoomes

! make/hotspot_version

Changeset: 34e2e90e7182
Author:    rbackman
Date:      2012-01-24 14:48 +0100
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/34e2e90e7182

7130476: Remove use of #ifdef TRACE_DEFINE_KLASS_TRACE_ID from klass.hpp
Reviewed-by: kamg, phh, dsamersoff
Contributed-by: Rickard Backman <rickard.backman@...>

! src/share/vm/oops/klass.cpp
! src/share/vm/oops/klass.hpp
! src/share/vm/trace/traceMacros.hpp

Changeset: 26a08cbbf042
Author:    stefank
Date:      2012-01-27 13:46 +0100
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/26a08cbbf042

7022100: Method annotations are incorrectly set when redefining classes
Summary: Changed to the correct annotation arrays
Reviewed-by: kamg, dholmes, sla

! src/share/vm/oops/instanceKlass.hpp

Changeset: f457154eee8b
Author:    brutisso
Date:      2012-01-30 12:36 +0100
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/f457154eee8b

7140882: Don't return booleans from methods returning pointers
Summary: Changed "return false" to "return NULL"
Reviewed-by: dholmes, rottenha
Contributed-by: dbhole@...

! src/share/vm/oops/constantPoolOop.cpp
! src/share/vm/opto/loopnode.cpp

Changeset: d96c130c9399
Author:    brutisso
Date:      2012-01-30 05:08 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/d96c130c9399

Merge

Changeset: b2cd0ee8f778
Author:    acorn
Date:      2012-01-30 23:27 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/b2cd0ee8f778

7114376: Make system dictionary hashtable bucket array size configurable
Summary: 7u4 new experimental flag -XX:PredictedClassLoadedCount=#
Reviewed-by: dholmes, phh, dcubed

! agent/src/share/classes/sun/jvm/hotspot/memory/LoaderConstraintTable.java
! agent/src/share/classes/sun/jvm/hotspot/memory/SystemDictionary.java
! src/share/vm/classfile/dictionary.cpp
! src/share/vm/classfile/systemDictionary.cpp
! src/share/vm/classfile/systemDictionary.hpp
! src/share/vm/runtime/globals.hpp
! src/share/vm/runtime/vmStructs.cpp
! src/share/vm/utilities/hashtable.hpp

Changeset: 481a9443f721
Author:    phh
Date:      2012-02-01 15:01 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/481a9443f721

7123386: RFE: Preserve universal builds of HotSpot on Mac OS X
Summary: Add support for packaging HotSpot JVM builds in universal binaries
Reviewed-by: dholmes, kamg, dcubed, phh
Contributed-by: james.melvin@...

! make/Makefile
! make/bsd/makefiles/defs.make
+ make/bsd/makefiles/universal.gmk
! make/defs.make

Changeset: 527cf36f4a20
Author:    fparain
Date:      2012-02-03 14:04 -0500
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/527cf36f4a20

Merge

! src/share/vm/runtime/globals.hpp

Changeset: 1a2723f7ad8e
Author:    never
Date:      2012-01-29 16:46 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/1a2723f7ad8e

7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
Reviewed-by: kvn, iveresov, dholmes

! src/share/vm/memory/gcLocker.cpp
! src/share/vm/memory/gcLocker.hpp
! src/share/vm/memory/gcLocker.inline.hpp
! src/share/vm/runtime/safepoint.cpp
! src/share/vm/runtime/safepoint.hpp
! src/share/vm/runtime/thread.hpp

Changeset: 5f17b16b3219
Author:    iveresov
Date:      2012-01-30 19:37 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/5f17b16b3219

7141059: 7116795 broke pure c2 builds
Summary: Fix pure c2 builds
Reviewed-by: kvn, brutisso, never

! src/cpu/sparc/vm/c2_globals_sparc.hpp
! src/cpu/sparc/vm/frame_sparc.cpp
! src/cpu/x86/vm/c2_globals_x86.hpp
! src/cpu/x86/vm/frame_x86.cpp
! src/share/vm/runtime/globals.hpp

Changeset: 5ed8f599a788
Author:    kvn
Date:      2012-01-31 07:18 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/5ed8f599a788

7140924: SIGSEGV in compiled code for sun.awt.X11.XDecoratedPeer.updateMinSizeHints
Summary: Use unknown_obj instead of empty_map for NULL or Constant Pool object constants in bytecode
Escape Analyzer.
Reviewed-by: iveresov, never

! src/share/vm/ci/bcEscapeAnalyzer.cpp

Changeset: 2f5980b127e3
Author:    twisti
Date:      2012-01-31 09:53 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/2f5980b127e3

7132180: JSR 292: C1 JVM crash with ClassValue/MethodHandle
Reviewed-by: never

! src/share/vm/c1/c1_GraphBuilder.cpp

Changeset: f067b4e0e04b
Author:    roland
Date:      2012-02-01 10:36 +0100
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/f067b4e0e04b

7090976: Eclipse/CDT causes a JVM crash while indexing C++ code
Summary: too optimistic inlining decision confuses local value numbering.
Reviewed-by: never

! src/share/vm/c1/c1_GraphBuilder.cpp
! src/share/vm/c1/c1_GraphBuilder.hpp
! src/share/vm/c1/c1_ValueMap.cpp
+ test/compiler/7090976/Test7090976.java

Changeset: aa3d708d67c4
Author:    never
Date:      2012-02-01 07:59 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/aa3d708d67c4

7141200: log some interesting information in ring buffers for crashes
Reviewed-by: kvn, jrose, kevinw, brutisso, twisti, jmasa

! src/os/windows/vm/os_windows.cpp
! src/share/vm/c1/c1_Runtime1.cpp
! src/share/vm/ci/ciEnv.hpp
! src/share/vm/code/compiledIC.cpp
! src/share/vm/code/nmethod.cpp
! src/share/vm/compiler/compileBroker.cpp
! src/share/vm/compiler/compileBroker.hpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/gc_implementation/g1/g1MarkSweep.cpp
! src/share/vm/gc_implementation/parallelScavenge/psMarkSweep.cpp
! src/share/vm/gc_implementation/parallelScavenge/psParallelCompact.cpp
! src/share/vm/gc_implementation/parallelScavenge/psScavenge.cpp
! src/share/vm/gc_interface/collectedHeap.cpp
! src/share/vm/gc_interface/collectedHeap.hpp
! src/share/vm/memory/genCollectedHeap.cpp
! src/share/vm/memory/genMarkSweep.cpp
! src/share/vm/prims/jvm.cpp
! src/share/vm/runtime/deoptimization.cpp
! src/share/vm/runtime/frame.cpp
! src/share/vm/runtime/globals.hpp
! src/share/vm/runtime/init.cpp
! src/share/vm/runtime/mutex.cpp
! src/share/vm/runtime/sharedRuntime.cpp
! src/share/vm/runtime/thread.cpp
! src/share/vm/utilities/debug.cpp
! src/share/vm/utilities/debug.hpp
! src/share/vm/utilities/events.cpp
! src/share/vm/utilities/events.hpp
! src/share/vm/utilities/exceptions.cpp
! src/share/vm/utilities/vmError.cpp

Changeset: 0382d2b469b2
Author:    never
Date:      2012-02-01 16:57 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/0382d2b469b2

7013347: allow crypto functions to be called inline to enhance performance
Reviewed-by: kvn

! src/cpu/sparc/vm/assembler_sparc.hpp
! src/cpu/sparc/vm/assembler_sparc.inline.hpp
! src/cpu/sparc/vm/sharedRuntime_sparc.cpp
! src/cpu/x86/vm/sharedRuntime_x86_32.cpp
! src/cpu/x86/vm/sharedRuntime_x86_64.cpp
! src/share/vm/code/nmethod.cpp
! src/share/vm/code/nmethod.hpp
! src/share/vm/memory/gcLocker.cpp
! src/share/vm/memory/gcLocker.hpp
! src/share/vm/oops/arrayOop.cpp
! src/share/vm/oops/methodOop.cpp
! src/share/vm/oops/methodOop.hpp
! src/share/vm/prims/nativeLookup.cpp
! src/share/vm/prims/nativeLookup.hpp
! src/share/vm/runtime/globals.hpp
! src/share/vm/runtime/safepoint.cpp
! src/share/vm/runtime/safepoint.hpp
! src/share/vm/runtime/sharedRuntime.cpp
! src/share/vm/runtime/sharedRuntime.hpp
! src/share/vm/runtime/thread.cpp
! src/share/vm/runtime/thread.hpp

Changeset: 392a3f07d567
Author:    twisti
Date:      2012-02-02 09:14 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/392a3f07d567

7141637: JSR 292: MH spread invoker crashes with NULL argument on x86_32
Reviewed-by: twisti
Contributed-by: Volker Simonis <volker.simonis@...>

! src/cpu/x86/vm/methodHandles_x86.cpp
+ test/compiler/7141637/SpreadNullArg.java

Changeset: 379b22e03c32
Author:    jcoomes
Date:      2012-02-03 12:08 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/379b22e03c32

Merge

! src/os/windows/vm/os_windows.cpp
! src/share/vm/compiler/compileBroker.cpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/runtime/globals.hpp

Changeset: 5ab44ceb4d57
Author:    jcoomes
Date:      2012-02-03 12:20 -0800
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/5ab44ceb4d57

Merge