hg: jdk8/build/hotspot: 20 new changesets
Changeset: b6b9a5d4cda0 Author:amurillo Date: 2013-11-29 11:20 -0800 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/b6b9a5d4cda0 8029367: new hotspot build - hs25-b62 Reviewed-by: jcoomes ! make/hotspot_version Changeset: 77b028ba548c Author:jprovino Date: 2013-11-19 16:26 -0500 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/77b028ba548c 8028396: Minimal VM: undefined symbol: _ZN23JvmtiCurrentBreakpoints11metadata_doEPFvP8MetadataE Summary: Minimal VM doesn't run Reviewed-by: coleenp, dholmes ! src/share/vm/prims/jvmtiImpl.hpp Changeset: 3fbb71fdc6e5 Author:vladidan Date: 2013-12-01 22:35 -0500 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/3fbb71fdc6e5 Merge Changeset: 8a42e81e2f9d Author:dsamersoff Date: 2013-11-27 14:26 +0400 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/8a42e81e2f9d 7050685: jsdbproc64.sh has a typo in the package name Summary: fixed typeo Reviewed-by: sla, kmo, sspitsyn ! agent/make/jsdbproc64.sh Changeset: 6ce6a0d23467 Author:mgronlun Date: 2013-12-02 11:42 +0100 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/6ce6a0d23467 Merge - test/compiler/jsr292/methodHandleExceptions/C.java - test/compiler/jsr292/methodHandleExceptions/I.java Changeset: 7a58803b5069 Author:acorn Date: 2013-12-03 08:36 -0800 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/7a58803b5069 8026066: ICCE for invokeinterface static Reviewed-by: coleenp, lfoltan, hseigel ! src/share/vm/interpreter/linkResolver.cpp ! src/share/vm/interpreter/linkResolver.hpp ! test/TEST.groups ! test/runtime/8024804/RegisterNatives.java Changeset: 379f11bc04fc Author:acorn Date: 2013-12-03 11:13 -0800 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/379f11bc04fc 8028438: static superclass method masks default methods Reviewed-by: hseigel, lfoltan, coleenp ! src/share/vm/classfile/defaultMethods.cpp ! src/share/vm/interpreter/linkResolver.cpp ! src/share/vm/oops/instanceKlass.cpp ! src/share/vm/oops/instanceKlass.hpp ! src/share/vm/oops/klassVtable.cpp Changeset: c8c2d6b82499 Author:sspitsyn Date: 2013-12-03 15:41 -0800 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/c8c2d6b82499 8028126: nsk/jvmti/scenarios/hotswap/HS101/hs101t006 Crashed the vm on Solaris-sparc64 fastdebug builds: only current thread can flush its registers Summary: Fix a race between VMOp_GetCurrentLocation reaching a safepoint and arget thread exiting from Java execution Reviewed-by: sla, dholmes, dsamersoff Contributed-by: serguei.spit...@oracle.com ! src/share/vm/prims/jvmtiEnvThreadState.cpp Changeset: e84d2afb2fb0 Author:sspitsyn Date: 2013-12-03 13:56 -0800 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/e84d2afb2fb0 Merge Changeset: 55a0da3d420b Author:sjohanss Date: 2013-11-26 14:35 +0100 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/55a0da3d420b 8027675: Full collections with Serial slower in JDK 8 compared to 7u40 Summary: Reduced the number of calls to follow_class_loader and instead marked and pushed the klass holder directly. Also removed unneeded calls to adjust_klass. Reviewed-by: coleenp, jmasa, mgerdin, tschatzl ! src/share/vm/gc_implementation/shared/markSweep.cpp ! src/share/vm/gc_implementation/shared/markSweep.hpp ! src/share/vm/gc_implementation/shared/markSweep.inline.hpp ! src/share/vm/oops/instanceKlass.cpp ! src/share/vm/oops/instanceMirrorKlass.cpp ! src/share/vm/oops/objArrayKlass.cpp Changeset: 9fc985481d78 Author:ehelin Date: 2013-12-02 15:43 +0100 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/9fc985481d78 Merge ! src/share/vm/oops/instanceKlass.cpp - test/compiler/jsr292/methodHandleExceptions/C.java - test/compiler/jsr292/methodHandleExceptions/I.java Changeset: 50287b659eb8 Author:sjohanss Date: 2013-12-03 12:01 +0100 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/50287b659eb8 8029329: tmtools tests fail with NPE (in the tool) when run with G1 and FlightRecorder Summary: Now iterating over all committed (used) G1 regions instead of all reserved. Reviewed-by: brutisso, dsamersoff, mgerdin ! agent/src/share/classes/sun/jvm/hotspot/gc_implementation/g1/G1HeapRegionTable.java ! agent/src/share/classes/sun/jvm/hotspot/gc_implementation/g1/HeapRegionSeq.java Changeset: 816c89d5957d Author:ehelin Date: 2013-12-05 17:49 +0100 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/816c89d5957d Merge ! src/share/vm/oops/instanceKlass.cpp Changeset: 9949533a8623 Author:rbackman Date: 2013-11-22 14:14 +0100 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/9949533a8623 8028997: mathexact intrinsics are unstable Reviewed-by: iveresov, kvn ! src/share/vm/opto/c2_globals.hpp ! test/compiler/intrinsics/mathexact/AddExactICondTest.java ! test/compiler/intrinsics/mathexact/AddExactIConstantTest.java !
Re: How to increment JDK_MINOR_VERSION for JDK 9?
On 12/10/2013 3:33 PM, Joseph Darcy wrote: Hello, Looking ahead to JDK 9, a build 01 change that is needed is updating the JDK_MINOR_VERSION setting in the build from "8" to "9": JDK-8000962 Update JDK_MINOR_VERSION for JDK 9 https://bugs.openjdk.java.net/browse/JDK-8000962 I poked around a bit, but didn't see where this needs to be set. The value of this variable gets used in numerous places throughout the JDK, including in system properties. How can this be updating in the new build? That would be in common/autoconf/version-numbers Dave
How to increment JDK_MINOR_VERSION for JDK 9?
Hello, Looking ahead to JDK 9, a build 01 change that is needed is updating the JDK_MINOR_VERSION setting in the build from "8" to "9": JDK-8000962 Update JDK_MINOR_VERSION for JDK 9 https://bugs.openjdk.java.net/browse/JDK-8000962 I poked around a bit, but didn't see where this needs to be set. The value of this variable gets used in numerous places throughout the JDK, including in system properties. How can this be updating in the new build? Thanks, -Joe
hg: jdk8/build/jdk: 8029616: Update jdeps man page to include a new -jdkinternals option
Changeset: d31cd980e1da Author:rgallard Date: 2013-12-10 15:20 -0800 URL: http://hg.openjdk.java.net/jdk8/build/jdk/rev/d31cd980e1da 8029616: Update jdeps man page to include a new -jdkinternals option Reviewed-by: mchung ! src/bsd/doc/man/jdeps.1 ! src/linux/doc/man/jdeps.1 ! src/solaris/doc/sun/man/man1/jdeps.1
hg: jdk8/build/nashorn: 3 new changesets
Changeset: e0b4483668a7 Author:jlaskey Date: 2013-11-26 11:58 -0400 URL: http://hg.openjdk.java.net/jdk8/build/nashorn/rev/e0b4483668a7 8029173: Debugger support doesn't handle ConsString Reviewed-by: lagergren, hannesw, sundar Contributed-by: james.las...@oracle.com ! src/jdk/nashorn/internal/runtime/DebuggerSupport.java Changeset: c14fe3f90616 Author:sundar Date: 2013-12-04 14:37 +0530 URL: http://hg.openjdk.java.net/jdk8/build/nashorn/rev/c14fe3f90616 Merge Changeset: 55cbc2d00c93 Author:lana Date: 2013-12-05 10:34 -0800 URL: http://hg.openjdk.java.net/jdk8/build/nashorn/rev/55cbc2d00c93 Merge
hg: jdk8/build/langtools: 6 new changesets
Changeset: a746587a1ff1
Author:jlahoda
Date: 2013-12-03 18:50 +0100
URL: http://hg.openjdk.java.net/jdk8/build/langtools/rev/a746587a1ff1
8028699: Compiler crash during speculative attribution of annotated type
Summary: Moving the checkForDeclarationAnnotations check into
Attr.TypeAnnotationsValidator
Reviewed-by: jjg
Contributed-by: wdi...@gmail.com
! src/share/classes/com/sun/tools/javac/comp/Attr.java
! src/share/classes/com/sun/tools/javac/comp/Check.java
+ test/tools/javac/annotations/typeAnnotations/failures/CheckForDeclAnnoNPE.java
!
test/tools/javac/annotations/typeAnnotations/failures/common/arrays/DeclarationAnnotation.java
!
test/tools/javac/annotations/typeAnnotations/failures/common/arrays/DeclarationAnnotation.out
Changeset: fb8c59cf26c8
Author:vromero
Date: 2013-12-03 18:13 +
URL: http://hg.openjdk.java.net/jdk8/build/langtools/rev/fb8c59cf26c8
8029179: javac produces a compile error for valid boolean expressions
Reviewed-by: jjg, jlahoda
! src/share/classes/com/sun/tools/javac/parser/JavacParser.java
+ test/tools/javac/T8029179/CompileErrorForValidBooleanExpTest.java
Changeset: 4cb9de4dd420
Author:bpatel
Date: 2013-12-03 14:21 -0800
URL: http://hg.openjdk.java.net/jdk8/build/langtools/rev/4cb9de4dd420
8025416: doclet not substituting {@docRoot} in some cases
Reviewed-by: jjg
! src/share/classes/com/sun/tools/doclets/formats/html/HtmlDocletWriter.java
! src/share/classes/com/sun/tools/doclets/formats/html/TagletWriterImpl.java
! test/com/sun/javadoc/testDocRootLink/TestDocRootLink.java
! test/com/sun/javadoc/testDocRootLink/pkg1/C1.java
! test/com/sun/javadoc/testDocRootLink/pkg1/package.html
! test/com/sun/javadoc/testDocRootLink/pkg2/C2.java
! test/com/sun/javadoc/testDocRootLink/pkg2/package.html
Changeset: 1b69023743be
Author:lana
Date: 2013-12-03 23:10 -0800
URL: http://hg.openjdk.java.net/jdk8/build/langtools/rev/1b69023743be
Merge
Changeset: 4a2ed1900428
Author:mchung
Date: 2013-12-04 15:39 -0800
URL: http://hg.openjdk.java.net/jdk8/build/langtools/rev/4a2ed1900428
8029216: (jdeps) Provide a specific option to report JDK internal APIs
Reviewed-by: alanb
! src/share/classes/com/sun/tools/jdeps/JdepsTask.java
! src/share/classes/com/sun/tools/jdeps/resources/jdeps.properties
! test/tools/jdeps/APIDeps.java
Changeset: b3d7e86a0647
Author:lana
Date: 2013-12-05 10:34 -0800
URL: http://hg.openjdk.java.net/jdk8/build/langtools/rev/b3d7e86a0647
Merge
hg: jdk8/build/jaxp: 2 new changesets
Changeset: aed9ca4d33ec Author:joehw Date: 2013-12-04 00:17 -0800 URL: http://hg.openjdk.java.net/jdk8/build/jaxp/rev/aed9ca4d33ec 8027973: Error in the documentation for newFactory method of the javax.xml.stream factories Reviewed-by: alanb, dfuchs, lancea, rriggs ! src/javax/xml/stream/FactoryFinder.java ! src/javax/xml/stream/XMLEventFactory.java ! src/javax/xml/stream/XMLInputFactory.java ! src/javax/xml/stream/XMLOutputFactory.java Changeset: 64d8b228a72c Author:lana Date: 2013-12-05 10:32 -0800 URL: http://hg.openjdk.java.net/jdk8/build/jaxp/rev/64d8b228a72c Merge
hg: jdk8/build: 2 new changesets
Changeset: c009462c1e92 Author:erikj Date: 2013-12-04 12:45 +0100 URL: http://hg.openjdk.java.net/jdk8/build/rev/c009462c1e92 8027963: Create unlimited policy jars. Reviewed-by: wetmore, ihse ! common/autoconf/spec.gmk.in Changeset: f204455b60cc Author:lana Date: 2013-12-05 10:32 -0800 URL: http://hg.openjdk.java.net/jdk8/build/rev/f204455b60cc Merge
Re: 8029805/8029806: Remove XXX addPropertyChangeListener and removePropertyChangeListener methods
Looks good. Happy to see these methods finally removed for a clean jdk
modularization!
Mandy
On 12/10/2013 3:06 AM, Alan Bateman wrote:
(This one is for the jdk9-dev forest once it is created)
The addPropertyChangeListener and removePropertyChangeListener methods
defined by Pack200.{Packer,Unpacker} and LogManager methods are
deprecated and flagged with a warning that they "will be removed in a
future release". This is highlighted in the EDR and Public Review of
JSR 337 and also flagged in JEP 162.
I'd like to swing the axe early in JDK 9 so as to give every
opportunity for anything that might have a dependency. There are a few
other modularity related changes that need to go in early too but this
is the only one that involves the removal of methods.
The webrev with the changes is here:
http://cr.openjdk.java.net/~alanb/8029805%2b8029806/webrev/
I've cc'ed build-dev as there are make file changes to remove the
de-beaning rules (these methods do not exist in subset Profiles of
Java SE and were removed as part of the build).
-Alan.
Re: 8029805/8029806: Remove XXX addPropertyChangeListener and removePropertyChangeListener methods
Hi Alan:
(This one is for the jdk9-dev forest once it is created)
The addPropertyChangeListener and removePropertyChangeListener methods
defined by Pack200.{Packer,Unpacker} and LogManager methods are
deprecated and flagged with a warning that they "will be removed in a
future release". This is highlighted in the EDR and Public Review of
JSR 337 and also flagged in JEP 162.
I'd like to swing the axe early in JDK 9 so as to give every
opportunity for anything that might have a dependency. There are a few
other modularity related changes that need to go in early too but this
is the only one that involves the removal of methods.
The webrev with the changes is here:
http://cr.openjdk.java.net/~alanb/8029805%2b8029806/webrev/
I've cc'ed build-dev as there are make file changes to remove the
de-beaning rules (these methods do not exist in subset Profiles of
Java SE and were removed as part of the build).
The code deletions in the make files look good.
Tim
Re: 8029805/8029806: Remove XXX addPropertyChangeListener and removePropertyChangeListener methods
On 12/10/13 12:06 PM, Alan Bateman wrote:
(This one is for the jdk9-dev forest once it is created)
The addPropertyChangeListener and removePropertyChangeListener methods
defined by Pack200.{Packer,Unpacker} and LogManager methods are
deprecated and flagged with a warning that they "will be removed in a
future release". This is highlighted in the EDR and Public Review of JSR
337 and also flagged in JEP 162.
I'd like to swing the axe early in JDK 9 so as to give every opportunity
for anything that might have a dependency. There are a few other
modularity related changes that need to go in early too but this is the
only one that involves the removal of methods.
The webrev with the changes is here:
http://cr.openjdk.java.net/~alanb/8029805%2b8029806/webrev/
Hi Alan
I had a look at the j.u.l part of the fix - test included, and that
looks fine. I am glad to see these go :-)
I applaud at doing this early in JDK 9 - hopefully we will get feedback
early if some kind of replacement appears to be needed.
I've cc'ed build-dev as there are make file changes to remove the
de-beaning rules (these methods do not exist in subset Profiles of Java
SE and were removed as part of the build).
Wahou. Debeaning - I didn't know about that :-) - don't count me as
reviewer for these :-)
-- daniel
-Alan.
8029805/8029806: Remove XXX addPropertyChangeListener and removePropertyChangeListener methods
(This one is for the jdk9-dev forest once it is created)
The addPropertyChangeListener and removePropertyChangeListener methods
defined by Pack200.{Packer,Unpacker} and LogManager methods are
deprecated and flagged with a warning that they "will be removed in a
future release". This is highlighted in the EDR and Public Review of JSR
337 and also flagged in JEP 162.
I'd like to swing the axe early in JDK 9 so as to give every opportunity
for anything that might have a dependency. There are a few other
modularity related changes that need to go in early too but this is the
only one that involves the removal of methods.
The webrev with the changes is here:
http://cr.openjdk.java.net/~alanb/8029805%2b8029806/webrev/
I've cc'ed build-dev as there are make file changes to remove the
de-beaning rules (these methods do not exist in subset Profiles of Java
SE and were removed as part of the build).
-Alan.
Re: [8] WXP minor fixes for a cleaner compile of c code
Francis,
I suggest you post this to the hotspot-dev mailing list instead, since
these are changes to hotspot source code, not the build system.
/Magnus
On 2013-12-07 18:49, Francis ANDRE wrote:
Hi
Below are some warnings produced by the build of jdk8.
Z:/JDK/jdk8/jdk/src/share/native/java/lang/Throwable.c(48) : warning
C4028: paramétre formel 3 différent de la déclaration
Z:/JDK/jdk8/jdk/src/windows/native/java/io/WinNTFileSystem_md.c(363) :
warning C4101: 'pathlen': variable locale non référencée
Z:/JDK/jdk8/jdk/src/windows/native/common/jdk_util_md.c(45) : warning
C4101: 'ret': variable locale non référencée
Z:/JDK/jdk8/jdk/src/share/bin/java.c(1253) : warning C4101: 'result':
variable locale nonréférencée
Z:/JDK/jdk8/jdk/src/share/bin/parse_manifest.c(196) : warning C4244:
'fonction': conversion de 'jlong' en 'unsigned int', perte possible de
données
And here are the fixes
diff --git a/src/share/bin/java.c b/src/share/bin/java.c
--- a/src/share/bin/java.c
+++ b/src/share/bin/java.c
@@ -1250,7 +1250,6 @@
GetApplicationClass(JNIEnv *env)
{
jmethodID mid;
-jobject result;
jclass cls = GetLauncherHelperClass(env);
NULL_CHECK0(cls);
NULL_CHECK0(mid = (*env)->GetStaticMethodID(env, cls,
diff --git a/src/share/bin/parse_manifest.c
b/src/share/bin/parse_manifest.c
--- a/src/share/bin/parse_manifest.c
+++ b/src/share/bin/parse_manifest.c
@@ -193,7 +193,7 @@
return (-1);
if ((buffer = malloc(END_MAXLEN)) == NULL)
return (-1);
-if ((bytes = read(fd, buffer, len)) < 0) {
+if ((bytes = read(fd, buffer, (size_t)len)) < 0) {
free(buffer);
return (-1);
}
diff --git a/src/share/native/java/lang/Throwable.c
b/src/share/native/java/lang/Throwable.c
--- a/src/share/native/java/lang/Throwable.c
+++ b/src/share/native/java/lang/Throwable.c
@@ -44,7 +44,7 @@
* `this' so you can write 'throw e.fillInStackTrace();'
*/
JNIEXPORT jobject JNICALL
-Java_java_lang_Throwable_fillInStackTrace(JNIEnv *env, jobject
throwable, int dummy)
+Java_java_lang_Throwable_fillInStackTrace(JNIEnv *env, jobject
throwable, jint dummy)
{
JVM_FillInStackTrace(env, throwable);
return throwable;
diff --git a/src/windows/native/common/jdk_util_md.c
b/src/windows/native/common/jdk_util_md.c
--- a/src/windows/native/common/jdk_util_md.c
+++ b/src/windows/native/common/jdk_util_md.c
@@ -42,7 +42,6 @@
JNIEXPORT HMODULE JDK_LoadSystemLibrary(const char* name) {
HMODULE handle = NULL;
char path[MAX_PATH];
-int ret;
if (GetSystemDirectory(path, sizeof(path)) != 0) {
strcat(path, "\\");
diff --git a/src/windows/native/java/io/WinNTFileSystem_md.c
b/src/windows/native/java/io/WinNTFileSystem_md.c
--- a/src/windows/native/java/io/WinNTFileSystem_md.c
+++ b/src/windows/native/java/io/WinNTFileSystem_md.c
@@ -360,7 +360,6 @@
jobject file)
{
jint rv = 0;
-jint pathlen;
WCHAR *pathbuf = fileToNTPath(env, file, ids.path);
if (pathbuf == NULL)
