hg: hsx/hotspot-rt/langtools: 45 new changesets

2013-05-23 Thread john . coomes
Changeset: abd153854f16 Author:jjg Date: 2013-05-03 09:56 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/langtools/rev/abd153854f16 8012728: Normalize @ignore comments on langtools tests Reviewed-by: vromero, mcimadamore ! test/com/sun/javadoc/_template/Template.java ! test/c

hg: hsx/hotspot-rt/nashorn: 19 new changesets

2013-05-23 Thread john . coomes
Changeset: b754fb89367d Author:jlaskey Date: 2013-04-30 10:05 -0300 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/nashorn/rev/b754fb89367d 8006220: Simplify PropertyMaps Reviewed-by: hannesw, lagergren Contributed-by: james.las...@oracle.com ! src/jdk/nashorn/internal/codegen/MapC

hg: hsx/hotspot-rt: 7 new changesets

2013-05-23 Thread john . coomes
Changeset: e2eb6bc06621 Author:mduigou Date: 2013-05-08 21:42 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/rev/e2eb6bc06621 8014269: Add missing .PHONY targets to Main.gmk Reviewed-by: mchung, tbell ! common/makefiles/Main.gmk Changeset: 49ea9293fa49 Author:lana Date:

hg: hsx/hotspot-rt/jaxws: Added tag jdk8-b91 for changeset 0bb1a9fa56b0

2013-05-23 Thread john . coomes
Changeset: a0f604766ca1 Author:katleman Date: 2013-05-23 10:47 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/jaxws/rev/a0f604766ca1 Added tag jdk8-b91 for changeset 0bb1a9fa56b0 ! .hgtags

hg: hsx/hotspot-rt/corba: Added tag jdk8-b91 for changeset 8f7ffb296385

2013-05-23 Thread john . coomes
Changeset: 717aa26f8e0a Author:katleman Date: 2013-05-23 10:47 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/corba/rev/717aa26f8e0a Added tag jdk8-b91 for changeset 8f7ffb296385 ! .hgtags

hg: hsx/hotspot-rt/jaxp: 8 new changesets

2013-05-23 Thread john . coomes
Changeset: 452e1a182907 Author:dfuchs Date: 2013-05-06 18:50 +0200 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/jaxp/rev/452e1a182907 8008738: Issue in com.sun.org.apache.xml.internal.serializer.Encodings causes some JCK tests to fail intermittently Summary: Encodings.java someti

Re: Review Request (S) 8014288: perf regression in nashorn JDK-8008448.js test after 8008511 changes

2013-05-23 Thread serguei.spit...@oracle.com
Hi David, On 5/23/13 5:15 PM, David Holmes wrote: Hi Serguei, I have two queries: In MemberNameTable::add_member_name why do you not have assert_locked_or_safepoint(MemberNameTable_lock) ? Nice catch, thanks. It has to be there but I accidentally removed it. Will fix it. In MethodHandles

Re: Review Request (S) 8014288: perf regression in nashorn JDK-8008448.js test after 8008511 changes

2013-05-23 Thread David Holmes
Hi Serguei, I have two queries: In MemberNameTable::add_member_name why do you not have assert_locked_or_safepoint(MemberNameTable_lock) ? In MethodHandles::init_field_MemberName you deleted: 304 InstanceKlass::cast(field_holder())->add_member_name(mname); apart from this seeming to be un

Re: Review Request (S) 8014288: perf regression in nashorn JDK-8008448.js test after 8008511 changes

2013-05-23 Thread serguei.spit...@oracle.com
Vladimir and Christian, Thank you for the reviews, Serguei On 5/23/13 4:56 PM, Vladimir Kozlov wrote: This is good. thanks, Vladimir On 5/23/13 4:19 PM, serguei.spit...@oracle.com wrote: Please, review the fix for: bug: http://bugs.sun.com/view_bug.do?bug_id=8014288 jbs: https://jbs.or

Re: Review Request (S) 8014288: perf regression in nashorn JDK-8008448.js test after 8008511 changes

2013-05-23 Thread Vladimir Kozlov
This is good. thanks, Vladimir On 5/23/13 4:19 PM, serguei.spit...@oracle.com wrote: Please, review the fix for: bug: http://bugs.sun.com/view_bug.do?bug_id=8014288 jbs: https://jbs.oracle.com/bugs/browse/JDK-8014288 Open webrev: http://cr.openjdk.java.net/~sspitsyn/webrevs/2013/hotspot/

Re: Review Request (S) 8014288: perf regression in nashorn JDK-8008448.js test after 8008511 changes

2013-05-23 Thread Christian Thalinger
Looks good. -- Chris On May 23, 2013, at 4:19 PM, serguei.spit...@oracle.com wrote: > Please, review the fix for: > bug: http://bugs.sun.com/view_bug.do?bug_id=8014288 > jbs: https://jbs.oracle.com/bugs/browse/JDK-8014288 > > Open webrev: > http://cr.openjdk.java.net/~sspitsyn/webrevs/2013/ho

Review Request (S) 8014288: perf regression in nashorn JDK-8008448.js test after 8008511 changes

2013-05-23 Thread serguei.spit...@oracle.com
Please, review the fix for: bug: http://bugs.sun.com/view_bug.do?bug_id=8014288 jbs: https://jbs.oracle.com/bugs/browse/JDK-8014288 Open webrev: http://cr.openjdk.java.net/~sspitsyn/webrevs/2013/hotspot/8014288-JVMTI-JSR292.1/ Summary: The fix of the 8008511 introduced a performance regres

hg: jdk8/tl/jdk: 8014076: Arrays parallel and serial sorting improvements

2013-05-23 Thread chris . hegarty
Changeset: 3b23e3529ab3 Author:dl Date: 2013-05-23 18:34 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3b23e3529ab3 8014076: Arrays parallel and serial sorting improvements Reviewed-by: chegar, mduigou ! src/share/classes/java/util/Arrays.java ! src/share/classes/java/util/

hg: hsx/hotspot-rt/hotspot: 8009576: Test returns ClassNotFoundException

2013-05-23 Thread christian . tornqvist
Changeset: 699d9df07e59 Author:ctornqvi Date: 2013-05-23 17:39 +0200 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/699d9df07e59 8009576: Test returns ClassNotFoundException Summary: Small classpath fix and move tests into open Reviewed-by: mgerdin, zgu + test/runtime/M

Re: review request for JDK-8013461 There is a symbol AsyncGetCallTrace in libjvm.symbols that does not exist in minimal/libjvm.a when DEBUG_LEVEL == release

2013-05-23 Thread Joseph Provino
David, thanks for finding that! joe On 05/22/2013 08:09 PM, David Holmes wrote: Joe, On 23/05/2013 5:27 AM, Joseph Provino wrote: Is there a consensus what is in the webrev is okay? Yes I am now okay with it. I found this in 4889433: "AsyncGetCallTrace() requires the CLASS_LOAD event to be

hg: jdk8/tl/jdk: 8014048: Online user guide of jconsole points incorrect link

2013-05-23 Thread nils . loodin
Changeset: 0da6485cf656 Author:nloodin Date: 2013-05-23 15:50 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/0da6485cf656 8014048: Online user guide of jconsole points incorrect link Reviewed-by: mchung, sla, jbachorik ! src/share/classes/sun/tools/jconsole/AboutDialog.java

Re: RFR: 8010278 SA: provide mechanism for using an alternative SA debugger back-end.

2013-05-23 Thread Kevin Walls
Forgot to mention: I moved the ClassDump Tool around a little so it was usable via a route other than calling main(), and added the constructor that takes a String for the pkgList, which saves using the system property to communicate what classes you want to dump (sun.jvm.hotspot.tools.jcore

hg: jdk8/tl/nashorn: 9 new changesets

2013-05-23 Thread james . laskey
Changeset: 833a9a584b64 Author:attila Date: 2013-05-21 13:40 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/833a9a584b64 8014953: Have NativeJavaPackage throw a ClassNotFoundException when invoked Reviewed-by: jlaskey, sundar ! src/jdk/nashorn/internal/runtime/NativeJava

RFR: 8010278 SA: provide mechanism for using an alternative SA debugger back-end.

2013-05-23 Thread Kevin Walls
Hi, As the Serviceability Agent has been _the_ new and interesting way to find things post-mortem in the JVM [1], I'd like to propose an update which continues that tradition. 8010278 SA: provide mechanism for using an alternative SA debugger back-end. https://jbs.oracle.com/bugs/browse/JDK

Re: Still need a reviewer please! Re: RR(S): 8014420 Default JDP address does not match the one assigned by IANA

2013-05-23 Thread David Holmes
Hi Dmitry, This looks fine to me. Thanks, and sorry for the delay :) David On 23/05/2013 5:12 PM, Dmitry Samersoff wrote: On 2013-05-15 23:08, Dmitry Samersoff wrote: Please, Review a small patch, changing defaults for JDP to one assigned by IANA. All credential belongs to Florian Weimer

Re: review request for JDK-8013461 There is a symbol AsyncGetCallTrace in libjvm.symbols that does not exist in minimal/libjvm.a when DEBUG_LEVEL == release

2013-05-23 Thread Staffan Larsen
Ok! /Staffan On 22 maj 2013, at 21:27, Joseph Provino wrote: > Is there a consensus what is in the webrev is okay? > > The change is to include forte.cpp in the minimal jvm but to > conditionalize the code so that only AsyncGetCallTrace() > is defined with the minimal jvm. > > Webrev is here:

Still need a reviewer please! Re: RR(S): 8014420 Default JDP address does not match the one assigned by IANA

2013-05-23 Thread Dmitry Samersoff
On 2013-05-15 23:08, Dmitry Samersoff wrote: > Please, > > Review a small patch, changing defaults for JDP to one assigned by IANA. > > All credential belongs to Florian Weimer > > http://cr.openjdk.java.net/~dsamersoff/8014420.JDP/webrev.01/ > > -Dmitry > -- Dmitry Samersoff Oracle Java de