http://blog.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel hourly 1 1901-01-01T00:00+00:00 http://gmane.org/img/gmane-25t.png http://gmane.org http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10657 <pre>Changeset: 1cdbd42c3e49 Author: katleman Date: 2013-05-16 12:14 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/1cdbd42c3e49 Added tag jdk8-b90 for changeset 1ae0472ff3a0 ! .hgtags Changeset: 712a1e9c91f3 Author: coleenp Date: 2013-05-07 09:46 -0400 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/712a1e9c91f3 8013063: nsk/jvmti/RetransformClasses/retransform001 failed debug version on os::free Summary: Clear out class_file_bytes so they aren't deallocated twice Reviewed-by: dcubed, sspitsyn ! src/share/vm/prims/jvmtiRedefineClasses.cpp Changeset: 4674e409a9e6 Author: coleenp Date: 2013-05-07 18:51 -0400 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/4674e409a9e6 8014024: NPG: keep compiled ic methods from being deallocated in redefine classes Summary: Walk the compiledIC relocation records to keep Method* from being deallocated. Reviewed-by: dlong, kvn ! src/share/vm/code/nmethod.cpp Changeset: a1cc1d1e7ce5 Autho</pre> alejandro.murillo-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-18T01:25:25 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10655 <pre>Hi all. I've been trying to refactor and cleanup the split code currently located at src/share/vm/opto/reg_split.cpp. Here is the result of that. This fix also fixes bugs https://jbs.oracle.com/bugs/browse/JDK-8005956 and https://jbs.oracle.com/bugs/browse/JDK-8013830. The first bug is resolved by removing rematerialization of reaches to phi nodes. This comes with a slight performance degradation [0]. However, since this rematerialization is causing JVM crashes, I would say it's worth it. When reviewing this fix, please use the existing reg_split.cpp found at src/share/vm/opto/reg_split.cpp as a reference. JBS: https://jbs.oracle.com/bugs/browse/JDK-8014786 WEBREV: http://cr.openjdk.java.net/~adlertz/JDK-8014786/webrev00/ Kind Regards, Niclas Adlertz [0] Results from refworkload on 1) x64 Linux and 2) SPARCV9 Solaris. Baseline and fix with 12 iterations each. 1) x64 Linux ============================================================================ refworkload_base: reference_server Benchmark </pre> Niclas Adlertz 2013-05-17T23:37:03 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10645 <pre>Hi, I profiled virtual calls on a very fast machine (core i7) after Aleksey pointing out some micro benchmark deficiencies in my benchmarking technique.  Corrected those. Nailed down the time of a virtual call on core i7: with or without interfaces involved, if a bimorphic call is enforced on hotspot due to inability to inline, the overhead is very close to 0.5 nanos, whether the method is in the base class or in an interface. Knowing nothing about java's evolving design for interfaces and generics, I do know that C++ compilers can emit much much faster vcalls, especially when the class is not involving multiple inheritance. My wish: That if Java knows that a class does not have interfaces, then theoretically, the bimorphic call thru the vtable of such a class should approach C++ speeds. Is this wish ever achievable? Andy </pre> Andy Nuss 2013-05-17T15:40:03 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10637 <pre>Changeset: ed918a442b83 Author: jlahoda Date: 2013-04-17 15:54 +0200 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/langtools/rev/ed918a442b83 8008174: DocTree API should provide start and end positions for tree nodes Summary: Adding DocSourcePositions to allow access to DocTree starting/ending position Reviewed-by: jjg, darcy Contributed-by: Ralph Benjamin Ruijs &lt;ralphbenjamin-gVrGfPuBK2Jg9hUCZPvPmw&lt; at &gt;public.gmane.org&gt;, Jan Lahoda &lt;jlahoda-gVrGfPuBK2Jg9hUCZPvPmw&lt; at &gt;public.gmane.org&gt; + src/share/classes/com/sun/source/util/DocSourcePositions.java ! src/share/classes/com/sun/source/util/DocTrees.java ! src/share/classes/com/sun/source/util/SourcePositions.java ! src/share/classes/com/sun/tools/javac/api/JavacTrees.java ! src/share/classes/com/sun/tools/javac/parser/DocCommentParser.java ! src/share/classes/com/sun/tools/javac/tree/DCTree.java + test/tools/javac/doctree/positions/TestPosition.java + test/tools/javac/doctree/positions/TestPosition.out + test/tools/javac/doctree/positions/TestPositi</pre> john.coomes-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-17T04:27:09 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10636 <pre>Changeset: aa8170c0dec9 Author: sundar Date: 2013-04-15 20:12 +0530 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/nashorn/rev/aa8170c0dec9 8012240: Array.prototype.map.call({length: -1, get 0(){throw 0}}, function(){}).length does not throw error Reviewed-by: lagergren, jlaskey ! src/jdk/nashorn/internal/runtime/arrays/MapIterator.java + test/script/basic/JDK-8012240.js Changeset: 486d92559c37 Author: sundar Date: 2013-04-17 16:52 +0530 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/nashorn/rev/486d92559c37 8012457: Function.prototype.apply should accept any array-like argument for function arguments Reviewed-by: lagergren, jlaskey ! src/jdk/nashorn/internal/objects/NativeFunction.java + test/script/basic/JDK-8012457.js Changeset: d4468316fe73 Author: jlaskey Date: 2013-04-17 08:48 -0300 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/nashorn/rev/d4468316fe73 Merge Changeset: 04b36c02c0e2 Author: jlaskey Date: 2013-04-17 15:36 -0300 URL: </pre> john.coomes-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-17T04:28:13 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10635 <pre>Changeset: b0c41789f500 Author: jgodinez Date: 2013-04-25 10:14 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/jdk/rev/b0c41789f500 8009199: Printed text become garbage on Mac OSX Reviewed-by: bae, prr ! src/macosx/native/sun/awt/CTextPipe.m Changeset: f4aa34a7a44d Author: jchen Date: 2013-04-29 10:02 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/jdk/rev/f4aa34a7a44d 8005302: [findbugs] public methods return internal arrays; may be private Reviewed-by: bae, prr ! src/share/classes/sun/java2d/pipe/AAShapePipe.java Changeset: 46686202aa23 Author: lana Date: 2013-04-30 22:43 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/jdk/rev/46686202aa23 Merge Changeset: c70346f4c0a9 Author: pchelko Date: 2013-04-18 15:09 +0100 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/jdk/rev/c70346f4c0a9 8011686: AWT accidentally disables the NSApplicationDelegate of SWT, causing loss of OS X integration functionality Reviewed-by: anthony, se</pre> john.coomes-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-17T04:07:07 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10634 <pre>Changeset: fad6560cb32a Author: dfuchs Date: 2013-04-17 15:23 +0200 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/jaxp/rev/fad6560cb32a 8005954: JAXP Plugability Layer should use java.util.ServiceLoader Summary: This fix replaces manual processing of files under META-INF/services in JAXP factories by calls to java.util.ServiceLoader. Reviewed-by: alanb, joehw, mchung ! src/javax/xml/datatype/DatatypeFactory.java ! src/javax/xml/datatype/FactoryFinder.java ! src/javax/xml/parsers/DocumentBuilderFactory.java ! src/javax/xml/parsers/FactoryFinder.java ! src/javax/xml/parsers/SAXParserFactory.java ! src/javax/xml/stream/FactoryFinder.java ! src/javax/xml/stream/XMLEventFactory.java ! src/javax/xml/stream/XMLInputFactory.java ! src/javax/xml/stream/XMLOutputFactory.java ! src/javax/xml/transform/FactoryFinder.java ! src/javax/xml/transform/TransformerFactory.java ! src/javax/xml/validation/SchemaFactory.java + src/javax/xml/validation/SchemaFactoryConfigurationError.java ! src/javax/xml/validat</pre> john.coomes-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-17T04:03:08 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10633 <pre>Changeset: 3e5b9ea5ac35 Author: katleman Date: 2013-05-09 10:04 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/jaxws/rev/3e5b9ea5ac35 Added tag jdk8-b89 for changeset 88838e08e4ef ! .hgtags Changeset: 0bb1a9fa56b0 Author: katleman Date: 2013-05-16 12:14 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/jaxws/rev/0bb1a9fa56b0 Added tag jdk8-b90 for changeset 3e5b9ea5ac35 ! .hgtags </pre> john.coomes-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-17T04:03:35 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10632 <pre>Changeset: e34781a0566b Author: mduigou Date: 2013-04-24 21:46 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/rev/e34781a0566b 8013185: Add java.util.stream to CORE_PKGS.gmk in root repo Reviewed-by: mduigou Contributed-by: Henry Jen &lt;henry.jen-QHcLZuEGTsvQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; ! common/makefiles/javadoc/CORE_PKGS.gmk Changeset: e4794ae1016e Author: mduigou Date: 2013-04-24 21:46 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/rev/e4794ae1016e Merge Changeset: 10775618db00 Author: aharlap Date: 2013-04-26 15:54 -0400 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/rev/10775618db00 8011152: Precision problems on sflt builds Summary: Need to add global flag to the linker Reviewed-by: tbell, dholmes ! common/makefiles/NativeCompilation.gmk Changeset: a7a8302473d3 Author: mduigou Date: 2013-04-29 14:20 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/rev/a7a8302473d3 8008632: Additional JavaDoc tags &lt; at &gt;apiNote, &lt; at &gt;implSpec</pre> john.coomes-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-17T04:02:53 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10631 <pre>Changeset: 8f0a461776a9 Author: dmeetry Date: 2013-04-29 16:44 +0400 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/corba/rev/8f0a461776a9 4504275: CORBA boolean type unions do not generate compilable code from idlj Summary: JLS doesn't allow boolean type in switch statement, hence substituted by if statement. Reviewed-by: lancea ! src/share/classes/com/sun/tools/corba/se/idl/toJavaPortable/UnionGen.java Changeset: 846aaf02e516 Author: dmeetry Date: 2013-04-29 16:51 +0400 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/corba/rev/846aaf02e516 8011986: [corba] idlj generates read/write union helper methods that throw wrong exception in some cases Reviewed-by: lancea ! src/share/classes/com/sun/tools/corba/se/idl/toJavaPortable/UnionGen.java Changeset: ed59110eecdb Author: lana Date: 2013-04-30 17:41 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/corba/rev/ed59110eecdb Merge Changeset: fe4150590ee5 Author: lana Date: 2013-05-06 11:41 -0700 UR</pre> john.coomes-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-17T04:02:57 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10612 <pre>Previously sent to core-libs-dev, apologies for double-coverage. Comments received are appended at the bottom. ----- webrev: http://cr.openjdk.java.net/~drchase/7088419/webrev.01/ problem: Some applications spend a surprising amount of time computing CRC32s (Not sure I'm supposed to be precise on an open mailing list). Recent Intel architectures provide instructions that might be useful in addressing this. See https://jbs.oracle.com/bugs/browse/JDK-7088419 I turned this into a general attack on performance in Adler32 and CRC32, partly because the bug report was not clear on the size of the problematic inputs. The general approach turned out to be useful, because it's tricky to get the small-case overheads down for the accelerated-instruction version of the code. fix: 1) For CRC32 and Adler32, break out the "small" case (single bytes, and up to around 60-80 bytes) to be computed on the Java side, avoiding JNI overheads. 2) For CRC32 and Adler32, figure out the "combine" operations for the checksum </pre> David Chase 2013-05-16T18:12:11 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10600 <pre>Hi all, please review this patch. Many thanks in advance, Albert jbs: https://jbs.oracle.com/bugs/browse/JDK-8012312 webrev: http://cr.openjdk.java.net/~anoll/8012312/webrev.00/ Problem: hsdis fails to compile with binutils 2.23.2 Solution: Added &lt;config.h&gt; With this change, hsdis compiles on my Linux 64-bit system. </pre> Albert Noll 2013-05-16T08:28:36 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10598 <pre>Changeset: cd6f6fccd287 Author: iignatyev Date: 2013-05-15 22:44 +0400 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/cd6f6fccd287 8014068: TEST_BUG: compiler/ciReplay/TestSA.sh fails on Windows: core wasn't generated Reviewed-by: kvn ! test/compiler/ciReplay/TestSA.sh ! test/compiler/ciReplay/common.sh </pre> igor.ignatyev-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-16T01:50:24 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10578 <pre>Folks, Could I get a review for these two small changes in src/solaris/native. This is to fix the nightly CTW testing crashes on Solaris caused by a library SEGV internal to X11 that occurs during class initialization when the display is NULL. I've tested this patch on SfBay with JPRT and with the CTW tests on Solaris x86 and Sparc. Thanks much, --morris JBS - https://jbs.oracle.com/bugs/browse/JDK-7196866 WEBREV - http://cr.openjdk.java.net/~morris/7196866 </pre> Morris Meyer 2013-05-15T20:21:23 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10569 <pre>Hi, thank you for reviewing! Best, Albert jbs: https://jbs.oracle.com/bugs/browse/JDK-8014430 webrev: http://cr.openjdk.java.net/~anoll/8014430/webrev.00/ &lt;http://cr.openjdk.java.net/%7Eanoll/8014430/webrev.00/&gt; Problem: VM failed with the following error message when the specified code cache specified by -XX:ReservedCodeCacheSize=... is too small to startup the VM. # # A fatal error has been detected by the Java Runtime Environment: # # Internal Error (c1_Compiler.cpp:87), pid=25077, tid=3856251760 # guarantee(blob != NULL) failed: must create initial code buffer # # JRE version: Java(TM) SE Runtime Environment (8.0-b89) (build 1.8.0-ea-b89) # Java VM: Java HotSpot(TM) Client VM (25.0-b31 mixed mode linux-x86 ) # Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again # # If you would like to submit a bug report, please visit: # http://bugreport.sun.com/bugreport/crash.jsp # Solution: replace guarantee() with vm_exi</pre> Albert Noll 2013-05-15T09:49:56 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10564 <pre>This review request is for the backport of the fix for 8014189 to hs24. The fix has already been integrated into hs25. Webrev: http://cr.openjdk.java.net/~poonam/8014189/webrev/ Fix: Added NULL checks for Type::make_ptr() returned value. Testing : JPRT Thanks, Poonam </pre> Poonam Bajaj 2013-05-15T01:50:06 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10557 <pre>Changeset: 1da5d70655e9 Author: kvn Date: 2013-05-13 14:36 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/1da5d70655e9 8014286: failed java/lang/Math/DivModTests.java after 6934604 changes Summary: Corrected escape state for the result of boxing method. Added force inlining executed boxing methods. Reviewed-by: twisti ! src/share/vm/opto/bytecodeInfo.cpp ! src/share/vm/opto/escape.cpp </pre> vladimir.kozlov-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-14T02:22:57 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10556 <pre>Changeset: c456f4510385 Author: sla Date: 2013-05-03 12:24 +0200 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/c456f4510385 8008453: JvmtiClassFileReconstituter does not recognize default methods Reviewed-by: acorn, sspitsyn ! src/share/vm/prims/jvmtiClassFileReconstituter.cpp Changeset: 0380df7c3cd0 Author: sla Date: 2013-05-03 12:26 +0200 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/0380df7c3cd0 8013785: Respect EXTRA_CFLAGS on windows Reviewed-by: mgronlun, rbackman, kvn ! make/windows/makefiles/compile.make ! make/windows/makefiles/defs.make Changeset: 31a4e55f8c9d Author: fparain Date: 2013-05-03 05:05 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/31a4e55f8c9d 8004095: Add support for JMX interface to Diagnostic Framework and Commands Reviewed-by: acorn, sla ! src/share/vm/classfile/vmSymbols.hpp ! src/share/vm/runtime/serviceThread.cpp ! src/share/vm/services/attachListener.cpp ! src/share/vm/services/</pre> christian.thalinger-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org 2013-05-13T23:25:12 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10518 <pre>Hi all, Please review patch. Problem: incorrect processing of core location on windows hosts due to using invalid variable. Fix: using of $OS was replaced by using of $VM_OS variable fixed processing core locations for *nix: added core.&lt;pid&gt; also slight cosmetic changes: - added check of OS before 'ulimit' change - not supported by all hosts '--brief' flags was removed webrev: http://cr.openjdk.java.net/~iignatyev/8014068/webrev.00/ jbs: https://jbs.oracle.com/bugs/browse/JDK-8014068 </pre> Igor Ignatyev 2013-05-13T16:28:14 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10515 <pre>Hi, I was profiling various aspects of the JVM and hit a big surprise. * on my corei7, virtual calls are about .5 nanos * when a class has not derived a new behavior for hashCode(), this hashCode call is 1.5 nanos because native * for java.lang.Integer, which just returns the intValue(), hashCode is zero time when hotspot can inline (that is about one clock cycle when testing hits the same Integer instances keeping them in L1 cache) * but when you force HotSpot to go thru the vtable for Integer.hashCode, the call grows to 4 nanos! The last case was a big surprise, as I thought for Integer, a vcall to hashCode would only cost the 0.5 nanos of the vtable. Somehow, native code is involved even when hashCode() has been subclassed to not be native. ... Then I tried mucking with the code in openjdk.  I compiled the sources.  I edited Object.java to be this: public class Object {     public int hashCode ()     {          return System.identityHashCode(this);     } } To me, this seems li</pre> Andy Nuss 2013-05-13T15:44:32 http://comments.gmane.org/gmane.comp.java.openjdk.hotspot.compiler.devel/10506 <pre>WEBREV: http://cr.openjdk.java.net/~drchase/8014362/webrev.00/ PROBLEM: Bug 7088419 Use x86 Hardware CRC32 Instruction with java.util.zip.CRC32 and java.util.zip.Adler32 can profitably (if Intel docs are to be believed) be addressed using the PCLMULQDQ - Carry-Less Multiplication Quadword instruction. However, this instruction is not available on all Intel processors, so the code in java.util.zip, either the Java or the JNI code that it calls, needs to know if the instruction is available, and also whether it has been disabled on the command line. The least-ugly way of doing this appears to be to expose the UseCLMUL flag via an Unsafe intrinsic. I considered instead passing the result of cpu_features(), however this is not affected by the command line flags disabling use of these features by hotspot. FIX: 1) add a bit to cpuFeatureFlags for CLMUL. 2) expose the CLMUL bit in StdCpuid1Ecx 3) add a flag (in the style of UseAES) 4) add an unsafe intrinsic. And connect all those dots. The unsafe inter</pre> David Chase 2013-05-10T20:33:06 Search the mailing list at Gmane query http://search.gmane.org/?group=$group=gmane.comp.java.openjdk.hotspot.compiler.devel