From Kelly.Ohair at Sun.COM Wed Mar 3 17:42:36 2010 From: Kelly.Ohair at Sun.COM (Kelly O'Hair) Date: Wed, 03 Mar 2010 09:42:36 -0800 Subject: Need reviewer - cygwin problems with egrep Message-ID: <4B8E9F8C.6090708@sun.com> Need a reviewer on this small change. 6931763: sanity checks broken with latest cygwin, newer egrep -i option problems Either the latest cygwin or the latest egrep has broken the -i option. This is causing all the sanity checks on full path checks to fail on windows. A workaround is to replace the patterns $(EGREP) -ci '^([a-z]:)?/' with $(EGREP) -c '^([a-zA-Z]:)?/' Here is the patch: diff --git a/make/common/shared/Sanity.gmk b/make/common/shared/Sanity.gmk --- a/make/common/shared/Sanity.gmk +++ b/make/common/shared/Sanity.gmk @@ -687,13 +687,13 @@ ###################################################### sane-alt_outputdir: ifdef ALT_OUTPUTDIR - @if [ `$(ECHO) $(subst \,/,$(ALT_OUTPUTDIR)) | $(EGREP) -ci '^([a-z]:)?/'` -ne 1 ]; then \ + @if [ `$(ECHO) $(subst \,/,$(ALT_OUTPUTDIR)) | $(EGREP) -c '^([A-Za-z]:)?/'` -ne 1 ]; then \ $(ECHO) "ERROR: ALT_OUTPUTDIR must be an Absolute Path Name, \n" \ " not a Relative Path Name. \n" \ "" >> $(ERROR_FILE) ; \ fi ifeq ($(PLATFORM), windows) - @if [ `$(ECHO) $(subst \,/,$(ALT_OUTPUTDIR)) | $(EGREP) -ci '^([a-z]:)'` -ne 1 ]; then \ + @if [ `$(ECHO) $(subst \,/,$(ALT_OUTPUTDIR)) | $(EGREP) -c '^([A-Za-z]:)'` -ne 1 ]; then \ $(ECHO) "ERROR: On windows, ALT_OUTPUTDIR must contain the drive letter. \n" \ "" >> $(ERROR_FILE) ; \ fi @@ -742,7 +742,7 @@ ###################################################### sane-alt_bootdir: ifdef ALT_BOOTDIR - @if [ `$(ECHO) $(subst \,/,$(ALT_BOOTDIR)) | $(EGREP) -ci '^([a-z]:)?/'` -ne 1 ]; then \ + @if [ `$(ECHO) $(subst \,/,$(ALT_BOOTDIR)) | $(EGREP) -c '^([A-Za-z]:)?/'` -ne 1 ]; then \ $(ECHO) "ERROR: ALT_BOOTDIR must be an Absolute Path Name, \n" \ " not a Relative Path Name. \n" \ " The current value of ALT_BOOTDIR is \n" \ @@ -772,7 +772,7 @@ ###################################################### sane-local-bootdir: ifeq ($(PLATFORM), windows) - @if [ `$(ECHO) $(BOOTDIR) | $(EGREP) -ci '^J:'` -ne 0 ]; then \ + @if [ `$(ECHO) $(BOOTDIR) | $(EGREP) -c '^[jJ]:'` -ne 0 ]; then \ $(ECHO) "WARNING: Your BOOTDIR is located on the J: drive. Often the J:\n" \ " drive is mapped over a network. Using a mapped drive for\n" \ " the BOOTDIR may significantly slow down the build process.\n" \ @@ -789,7 +789,7 @@ ###################################################### sane-cacerts: ifdef ALT_CACERTS_FILE - @if [ `$(ECHO) $(subst \,/,$(ALT_CACERTS_FILE)) | $(EGREP) -ci '^([a-z]:)?/'` -ne 1 ]; then \ + @if [ `$(ECHO) $(subst \,/,$(ALT_CACERTS_FILE)) | $(EGREP) -c '^([A-Za-z]:)?/'` -ne 1 ]; then \ $(ECHO) "ERROR: ALT_CACERTS_FILE must be an Absolute Path Name, \n" \ " not a Relative Path Name. \n" \ " The current value of ALT_CACERTS_FILE is \n" \ @@ -1538,7 +1538,7 @@ sane-mozilla: ifeq ($(ARCH_DATA_MODEL), 32) ifdef ALT_MOZILLA_HEADERS_PATH - @if [ `$(ECHO) $(subst \,/,$(ALT_MOZILLA_HEADERS_PATH)) | $(EGREP) -ci '^([a-z]:)?/'` -ne 1 ]; then \ + @if [ `$(ECHO) $(subst \,/,$(ALT_MOZILLA_HEADERS_PATH)) | $(EGREP) -c '^([A-Za-z]:)?/'` -ne 1 ]; then \ $(ECHO) "ERROR: ALT_MOZILLA_HEADERS_PATH must be an Absolute Path Name, \n" \ " not a Relative Path Name. \n" \ " The current value of ALT_MOZILLA_HEADERS_PATH is \n" \ *** (#1 of 1): 2010-03-03 09:30:54 PST kelly.ohair at sun.com From Jonathan.Gibbons at Sun.COM Wed Mar 3 17:54:28 2010 From: Jonathan.Gibbons at Sun.COM (Jonathan Gibbons) Date: Wed, 03 Mar 2010 09:54:28 -0800 Subject: Need reviewer - cygwin problems with egrep In-Reply-To: <4B8E9F8C.6090708@sun.com> References: <4B8E9F8C.6090708@sun.com> Message-ID: <4B8EA254.2010209@sun.com> Looks good to me. -- Jon Kelly O'Hair wrote: > Need a reviewer on this small change. > > 6931763: sanity checks broken with latest cygwin, newer egrep -i > option problems > > Either the latest cygwin or the latest egrep has broken the -i option. > This is causing all the sanity checks on full path checks to fail on > windows. > > A workaround is to replace the patterns > $(EGREP) -ci '^([a-z]:)?/' > with > $(EGREP) -c '^([a-zA-Z]:)?/' > > Here is the patch: > > diff --git a/make/common/shared/Sanity.gmk > b/make/common/shared/Sanity.gmk > --- a/make/common/shared/Sanity.gmk > +++ b/make/common/shared/Sanity.gmk > @@ -687,13 +687,13 @@ > ###################################################### > sane-alt_outputdir: > ifdef ALT_OUTPUTDIR > - @if [ `$(ECHO) $(subst \,/,$(ALT_OUTPUTDIR)) | $(EGREP) -ci > '^([a-z]:)?/'` -ne 1 ]; then \ > + @if [ `$(ECHO) $(subst \,/,$(ALT_OUTPUTDIR)) | $(EGREP) -c > '^([A-Za-z]:)?/'` -ne 1 ]; then \ > $(ECHO) "ERROR: ALT_OUTPUTDIR must be an Absolute Path Name, > \n" \ > " not a Relative Path Name. \n" \ > "" >> $(ERROR_FILE) ; \ > fi > ifeq ($(PLATFORM), windows) > - @if [ `$(ECHO) $(subst \,/,$(ALT_OUTPUTDIR)) | $(EGREP) -ci > '^([a-z]:)'` -ne 1 ]; then \ > + @if [ `$(ECHO) $(subst \,/,$(ALT_OUTPUTDIR)) | $(EGREP) -c > '^([A-Za-z]:)'` -ne 1 ]; then \ > $(ECHO) "ERROR: On windows, ALT_OUTPUTDIR must contain the > drive letter. \n" \ > "" >> $(ERROR_FILE) ; \ > fi > @@ -742,7 +742,7 @@ > ###################################################### > sane-alt_bootdir: > ifdef ALT_BOOTDIR > - @if [ `$(ECHO) $(subst \,/,$(ALT_BOOTDIR)) | $(EGREP) -ci > '^([a-z]:)?/'` -ne 1 ]; then \ > + @if [ `$(ECHO) $(subst \,/,$(ALT_BOOTDIR)) | $(EGREP) -c > '^([A-Za-z]:)?/'` -ne 1 ]; then \ > $(ECHO) "ERROR: ALT_BOOTDIR must be an Absolute Path Name, > \n" \ > " not a Relative Path Name. \n" \ > " The current value of ALT_BOOTDIR is \n" \ > @@ -772,7 +772,7 @@ > ###################################################### > sane-local-bootdir: > ifeq ($(PLATFORM), windows) > - @if [ `$(ECHO) $(BOOTDIR) | $(EGREP) -ci '^J:'` -ne 0 ]; then \ > + @if [ `$(ECHO) $(BOOTDIR) | $(EGREP) -c '^[jJ]:'` -ne 0 ]; then \ > $(ECHO) "WARNING: Your BOOTDIR is located on the J: drive. > Often the J:\n" \ > " drive is mapped over a network. Using a mapped > drive for\n" \ > " the BOOTDIR may significantly slow down the build > process.\n" \ > @@ -789,7 +789,7 @@ > ###################################################### > sane-cacerts: > ifdef ALT_CACERTS_FILE > - @if [ `$(ECHO) $(subst \,/,$(ALT_CACERTS_FILE)) | $(EGREP) -ci > '^([a-z]:)?/'` -ne 1 ]; then \ > + @if [ `$(ECHO) $(subst \,/,$(ALT_CACERTS_FILE)) | $(EGREP) -c > '^([A-Za-z]:)?/'` -ne 1 ]; then \ > $(ECHO) "ERROR: ALT_CACERTS_FILE must be an Absolute Path > Name, \n" \ > " not a Relative Path Name. \n" \ > " The current value of ALT_CACERTS_FILE is \n" \ > @@ -1538,7 +1538,7 @@ > sane-mozilla: > ifeq ($(ARCH_DATA_MODEL), 32) > ifdef ALT_MOZILLA_HEADERS_PATH > - @if [ `$(ECHO) $(subst \,/,$(ALT_MOZILLA_HEADERS_PATH)) | > $(EGREP) -ci '^([a-z]:)?/'` -ne 1 ]; then \ > + @if [ `$(ECHO) $(subst \,/,$(ALT_MOZILLA_HEADERS_PATH)) | > $(EGREP) -c '^([A-Za-z]:)?/'` -ne 1 ]; then \ > $(ECHO) "ERROR: ALT_MOZILLA_HEADERS_PATH must be an Absolute > Path Name, \n" \ > " not a Relative Path Name. \n" \ > " The current value of ALT_MOZILLA_HEADERS_PATH is \n" \ > *** (#1 of 1): 2010-03-03 09:30:54 PST kelly.ohair at sun.com > > From michael.wilkerson at sun.com Thu Mar 4 00:42:42 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 04 Mar 2010 00:42:42 +0000 Subject: hg: jdk7/build/hotspot: 26 new changesets Message-ID: <20100304004402.3432843CB7@hg.openjdk.java.net> Changeset: 745c853ee57f Author: johnc Date: 2010-01-29 14:51 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/745c853ee57f 6885297: java -XX:RefDiscoveryPolicy=2 or -XX:TLABWasteTargetPercent=0 cause VM crash Summary: Interval checking is now being performed on the values passed in for these two flags. The current acceptable range for RefDiscoveryPolicy is [0..1], and for TLABWasteTargetPercent it is [1..100]. Reviewed-by: apetrusenko, ysr ! src/share/vm/includeDB_core ! src/share/vm/memory/referenceProcessor.hpp ! src/share/vm/runtime/arguments.cpp ! src/share/vm/runtime/arguments.hpp Changeset: 6484c4ee11cb Author: ysr Date: 2010-02-01 17:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/6484c4ee11cb 6904516: More object array barrier fixes, following up on 6906727 Summary: Fixed missing pre-barrier calls for G1, modified C1 to call pre- and correct post-barrier interfaces, deleted obsolete interface, (temporarily) disabled redundant deferred barrier in BacktraceBuilder. Reviewed-by: coleenp, jmasa, kvn, never ! src/share/vm/c1/c1_Runtime1.cpp ! src/share/vm/classfile/javaClasses.cpp ! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp ! src/share/vm/memory/barrierSet.hpp ! src/share/vm/memory/barrierSet.inline.hpp ! src/share/vm/runtime/stubRoutines.cpp Changeset: deada8912c54 Author: johnc Date: 2010-02-02 18:39 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/deada8912c54 6914402: G1: assert(!is_young_card(cached_ptr),"shouldn't get a card in young region") Summary: Invalid assert. Filter cards evicted from the card count cache instead. Reviewed-by: apetrusenko, tonyp ! src/share/vm/gc_implementation/g1/concurrentG1Refine.cpp Changeset: 230fac611b50 Author: johnc Date: 2010-02-08 09:58 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/230fac611b50 Merge ! src/share/vm/c1/c1_Runtime1.cpp ! src/share/vm/includeDB_core Changeset: 455df1b81409 Author: kamg Date: 2010-02-08 13:49 -0500 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/455df1b81409 6587322: dtrace probe object__alloc doesn't fire in some situations on amd64 Summary: Fix misplaced probe point Reviewed-by: rasbold, phh Contributed-by: neojia at gmail.com ! src/cpu/x86/vm/templateTable_x86_64.cpp Changeset: 95d21201c29a Author: apangin Date: 2010-02-11 10:48 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/95d21201c29a Merge Changeset: 3f5b7efb9642 Author: never Date: 2010-02-05 11:07 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/3f5b7efb9642 6920293: OptimizeStringConcat causing core dumps Reviewed-by: kvn, twisti ! src/os_cpu/solaris_x86/vm/os_solaris_x86.cpp ! src/share/vm/code/nmethod.cpp ! src/share/vm/opto/stringopts.cpp ! src/share/vm/runtime/sharedRuntime.cpp Changeset: 576e77447e3c Author: kvn Date: 2010-02-07 12:15 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/576e77447e3c 6923002: assert(false,"this call site should not be polymorphic") Summary: Clear the total count when a receiver information is cleared. Reviewed-by: never, jrose ! src/cpu/sparc/vm/c1_LIRAssembler_sparc.cpp ! src/cpu/sparc/vm/interp_masm_sparc.cpp ! src/cpu/sparc/vm/sharedRuntime_sparc.cpp ! src/cpu/x86/vm/c1_LIRAssembler_x86.cpp ! src/cpu/x86/vm/interp_masm_x86_32.cpp ! src/cpu/x86/vm/interp_masm_x86_64.cpp ! src/share/vm/ci/ciMethod.cpp ! src/share/vm/oops/methodDataOop.hpp ! src/share/vm/opto/doCall.cpp ! src/share/vm/opto/runtime.cpp ! src/share/vm/runtime/arguments.cpp Changeset: f516d5d7a019 Author: kvn Date: 2010-02-08 12:20 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/f516d5d7a019 6910605: C2: NullPointerException/ClassCaseException is thrown when C2 with DeoptimizeALot is used Summary: Set the reexecute bit for runtime calls _new_array_Java when they used for _multianewarray bytecode. Reviewed-by: never ! src/share/vm/code/pcDesc.cpp ! src/share/vm/opto/graphKit.cpp ! src/share/vm/opto/parse3.cpp + test/compiler/6910605/Test.java Changeset: f70b0d9ab095 Author: kvn Date: 2010-02-09 01:31 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/f70b0d9ab095 6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop") Summary: Mark in PcDesc call sites which return oop and save the result oop across objects reallocation during deoptimization. Reviewed-by: never ! src/share/vm/c1/c1_IR.hpp ! src/share/vm/code/debugInfoRec.cpp ! src/share/vm/code/debugInfoRec.hpp ! src/share/vm/code/nmethod.cpp ! src/share/vm/code/pcDesc.hpp ! src/share/vm/code/scopeDesc.cpp ! src/share/vm/code/scopeDesc.hpp ! src/share/vm/includeDB_core ! src/share/vm/opto/output.cpp ! src/share/vm/prims/jvmtiCodeBlobEvents.cpp ! src/share/vm/runtime/deoptimization.cpp + test/compiler/6910618/Test.java Changeset: 4ee1c645110e Author: kvn Date: 2010-02-09 10:21 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/4ee1c645110e 6924097: assert((_type == Type::MEMORY) == (_adr_type != 0),"adr_type for memory phis only") Summary: Use PhiNode::make_blank(r, n) method to construct the phi. Reviewed-by: never ! src/share/vm/opto/loopopts.cpp Changeset: e3a4305c6bc3 Author: kvn Date: 2010-02-12 08:54 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/e3a4305c6bc3 6925249: assert(last_sp < (intptr_t*) interpreter_frame_monitor_begin(),"bad tos") Summary: Fix assert since top deoptimized frame has last_sp == interpreter_frame_monitor_begin if there are no expressions. Reviewed-by: twisti ! src/cpu/x86/vm/frame_x86.inline.hpp ! src/share/vm/runtime/deoptimization.cpp ! src/share/vm/runtime/frame.cpp ! src/share/vm/runtime/vframeArray.cpp Changeset: c09ee209b65c Author: kvn Date: 2010-02-12 10:34 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/c09ee209b65c 6926048: Improve Zero performance Summary: Make Zero figure out result types in a similar way to C++ interpreter implementation. Reviewed-by: kvn Contributed-by: gbenson at redhat.com ! src/cpu/zero/vm/cppInterpreter_zero.cpp ! src/cpu/zero/vm/cppInterpreter_zero.hpp Changeset: 7b4415a18c8a Author: kvn Date: 2010-02-12 15:27 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/7b4415a18c8a Merge ! src/cpu/sparc/vm/c1_LIRAssembler_sparc.cpp ! src/cpu/x86/vm/c1_LIRAssembler_x86.cpp ! src/share/vm/includeDB_core ! src/share/vm/opto/graphKit.cpp ! src/share/vm/opto/runtime.cpp ! src/share/vm/runtime/arguments.cpp ! src/share/vm/runtime/sharedRuntime.cpp Changeset: 38836cf1d8d2 Author: tonyp Date: 2010-02-05 11:05 -0500 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/38836cf1d8d2 6920977: G1: guarantee(k == probe->klass(),"klass should be in dictionary") fails Summary: the guarantee is too strict and the test will fail (incorrectly) if the class is not in the system dictionary but in the placeholders. Reviewed-by: acorn, phh ! src/share/vm/classfile/loaderConstraints.cpp ! src/share/vm/classfile/loaderConstraints.hpp ! src/share/vm/classfile/systemDictionary.cpp ! src/share/vm/includeDB_core Changeset: 9eee977dd1a9 Author: tonyp Date: 2010-02-08 14:23 -0500 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/9eee977dd1a9 6802453: G1: hr()->is_in_reserved(from),"Precondition." Summary: The operations of re-using a RSet component and expanding the same RSet component were not mutually exlusive, and this could lead to RSets getting corrupted and entries being dropped. Reviewed-by: iveresov, johnc ! src/share/vm/gc_implementation/g1/heapRegionRemSet.cpp Changeset: 8859772195c6 Author: johnc Date: 2010-02-09 13:56 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/8859772195c6 6782663: Data produced by PrintGCApplicationConcurrentTime and PrintGCApplicationStoppedTime is not accurate. Summary: Update and display the timers associated with these flags for all safepoints. Reviewed-by: ysr, jcoomes ! src/share/vm/runtime/vmThread.cpp ! src/share/vm/services/runtimeService.cpp Changeset: 0414c1049f15 Author: iveresov Date: 2010-02-11 15:52 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/0414c1049f15 6923991: G1: improve scalability of RSet scanning Summary: Implemented block-based work stealing. Moved copying during the rset scanning phase to the main copying phase. Made the size of rset table depend on the region size. Reviewed-by: apetrusenko, tonyp ! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp ! src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp ! src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp ! src/share/vm/gc_implementation/g1/g1OopClosures.hpp ! src/share/vm/gc_implementation/g1/g1OopClosures.inline.hpp ! src/share/vm/gc_implementation/g1/g1RemSet.cpp ! src/share/vm/gc_implementation/g1/g1_globals.hpp ! src/share/vm/gc_implementation/g1/g1_specialized_oop_closures.hpp ! src/share/vm/gc_implementation/g1/heapRegionRemSet.cpp ! src/share/vm/gc_implementation/g1/heapRegionRemSet.hpp ! src/share/vm/gc_implementation/g1/sparsePRT.cpp ! src/share/vm/gc_implementation/g1/sparsePRT.hpp ! src/share/vm/memory/cardTableModRefBS.hpp ! src/share/vm/utilities/globalDefinitions.hpp Changeset: 58add740c4ee Author: johnc Date: 2010-02-16 14:11 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/58add740c4ee Merge ! src/share/vm/includeDB_core Changeset: e7b1cc79bd25 Author: kvn Date: 2010-02-16 16:17 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/e7b1cc79bd25 6926697: "optimized" VM build failed: The type "AdapterHandlerTableIterator" is incomplete Summary: Define AdapterHandlerTableIterator class as non product instead of debug. Reviewed-by: never ! src/share/vm/runtime/sharedRuntime.cpp Changeset: 106f41e88c85 Author: never Date: 2010-02-16 20:07 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/106f41e88c85 6877221: Endless deoptimizations in OSR nmethod Reviewed-by: kvn ! src/share/vm/opto/parse1.cpp Changeset: b4b440360f1e Author: twisti Date: 2010-02-18 11:35 +0100 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/b4b440360f1e 6926782: CodeBuffer size too small after 6921352 Summary: After 6921352 the CodeBuffer size was too small. Reviewed-by: kvn, never ! src/share/vm/opto/callGenerator.cpp ! src/share/vm/opto/compile.cpp ! src/share/vm/opto/compile.hpp ! src/share/vm/opto/output.cpp Changeset: 3b687c53c266 Author: twisti Date: 2010-02-18 06:54 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/3b687c53c266 6927165: Zero S/390 fixes Summary: Fixes two failures on 31-bit S/390. Reviewed-by: twisti Contributed-by: Gary Benson ! src/cpu/zero/vm/globals_zero.hpp ! src/os_cpu/linux_zero/vm/os_linux_zero.hpp Changeset: 72f1840531a4 Author: twisti Date: 2010-02-18 10:44 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/72f1840531a4 Merge Changeset: 1f341bb67b5b Author: trims Date: 2010-02-18 22:15 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/1f341bb67b5b Merge Changeset: 6c9796468b91 Author: trims Date: 2010-02-18 22:16 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/6c9796468b91 6927886: Bump the HS17 build number to 10 Summary: Update the HS17 build number to 10 Reviewed-by: jcoomes ! make/hotspot_version From michael.wilkerson at sun.com Thu Mar 4 00:45:04 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 04 Mar 2010 00:45:04 +0000 Subject: hg: jdk7/build/jdk: 12 new changesets Message-ID: <20100304004848.9644943CB9@hg.openjdk.java.net> Changeset: 2ba381560071 Author: dcherepanov Date: 2010-02-12 19:58 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/2ba381560071 6705345: Enable multiple file selection in AWT FileDialog Reviewed-by: art, anthony, alexp ! src/share/classes/java/awt/FileDialog.java ! src/share/classes/sun/awt/AWTAccessor.java ! src/solaris/classes/sun/awt/X11/XFileDialogPeer.java ! src/windows/classes/sun/awt/windows/WFileDialogPeer.java ! src/windows/native/sun/windows/awt_FileDialog.cpp ! src/windows/native/sun/windows/awt_FileDialog.h + test/java/awt/FileDialog/MultipleMode/MultipleMode.html + test/java/awt/FileDialog/MultipleMode/MultipleMode.java Changeset: d6d2de6ee2d1 Author: lana Date: 2010-02-19 15:13 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/d6d2de6ee2d1 Merge Changeset: 83c34a6b1458 Author: mchung Date: 2010-02-08 23:02 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/83c34a6b1458 6924497: HotSpotDiagnosticsMXBean.getDiagnosticOptions throws NPE Summary: Check if the element in the flags array is non-null to filter unsupported flags Reviewed-by: dcubed ! src/share/classes/sun/management/Flag.java ! src/share/native/sun/management/Flag.c Changeset: ec438f2b6886 Author: chegar Date: 2010-02-10 13:23 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/ec438f2b6886 6693244: Java Web Start app fails on 6u10 beta w/ AssertionError in AuthenticationInfo.requestCompleted Reviewed-by: michaelm ! src/share/classes/sun/net/www/protocol/http/AuthenticationInfo.java ! test/ProblemList.txt ! test/java/net/Authenticator/B4769350.java Changeset: 784e52734b8d Author: mchung Date: 2010-02-10 17:51 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/784e52734b8d 6915413: Module build: building of specified jdk components instead of all Summary: Define new SUBDIRS_* variables for specifying components for one group Reviewed-by: ohair ! make/Makefile ! make/com/Makefile ! make/com/sun/Makefile ! make/com/sun/demo/Makefile ! make/com/sun/demo/jvmti/Makefile ! make/com/sun/inputmethods/Makefile ! make/com/sun/java/Makefile ! make/com/sun/java/browser/Makefile ! make/com/sun/jmx/Makefile ! make/com/sun/jndi/Makefile ! make/com/sun/jndi/rmi/Makefile ! make/com/sun/nio/Makefile ! make/com/sun/org/Makefile ! make/com/sun/org/apache/Makefile ! make/com/sun/security/Makefile ! make/com/sun/tools/Makefile ! make/com/sun/tracing/Makefile ! make/common/Defs.gmk ! make/common/Sanity.gmk + make/common/Subdirs.gmk ! make/common/shared/Sanity.gmk ! make/java/Makefile ! make/java/hpi/Makefile ! make/java/java/Makefile ! make/java/java/genlocales.gmk ! make/java/main/Makefile ! make/java/nio/FILES_java.gmk ! make/java/nio/Makefile + make/java/nio/mxbean/Makefile ! make/java/redist/Makefile - make/java/text/FILES_java.gmk ! make/java/text/Makefile + make/java/text/base/FILES_java.gmk + make/java/text/base/Makefile + make/java/text/bidi/Makefile ! make/javax/Makefile ! make/javax/rmi/Makefile ! make/javax/sound/Makefile ! make/javax/swing/Makefile ! make/jpda/Makefile ! make/jpda/transport/Makefile ! make/mkdemo/Makefile ! make/mkdemo/applets/Makefile ! make/mkdemo/jfc/Makefile ! make/mkdemo/jni/Makefile ! make/mkdemo/jvmti/Makefile ! make/mkdemo/management/Makefile ! make/mkdemo/scripting/Makefile ! make/mksample/Makefile ! make/mksample/jmx/Makefile ! make/mksample/nio/Makefile ! make/mksample/scripting/Makefile ! make/mksample/webservices/Makefile ! make/org/Makefile ! make/org/ietf/Makefile ! make/sun/Makefile ! make/sun/cmm/Makefile ! make/sun/image/Makefile ! make/sun/management/Makefile ! make/sun/net/Makefile ! make/sun/net/spi/Makefile ! make/sun/net/spi/nameservice/Makefile ! make/sun/nio/Makefile ! make/sun/org/Makefile ! make/sun/org/mozilla/Makefile ! make/sun/rmi/Makefile ! make/sun/security/Makefile ! make/sun/tracing/Makefile ! make/tools/Makefile Changeset: d7d8807fca86 Author: weijun Date: 2010-02-12 10:24 +0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/d7d8807fca86 6925639: keytool -genkeypair -help missing dname option Reviewed-by: mullan ! src/share/classes/sun/security/tools/KeyTool.java Changeset: 74f493fae483 Author: mchung Date: 2010-02-12 11:33 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/74f493fae483 6925868: Eliminate pack200's dependency on logging Summary: Replace j.u.l.Logger with sun.util.logging.PlatformLogger Reviewed-by: alanb, forax ! src/share/classes/com/sun/java/util/jar/pack/PackageReader.java ! src/share/classes/com/sun/java/util/jar/pack/PackageWriter.java ! src/share/classes/com/sun/java/util/jar/pack/Utils.java Changeset: 328c5d3974fe Author: mchung Date: 2010-02-15 10:18 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/328c5d3974fe Merge Changeset: 84792500750c Author: lana Date: 2010-02-17 10:24 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/84792500750c Merge Changeset: e83d9c0d5e95 Author: chegar Date: 2010-02-22 15:27 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/e83d9c0d5e95 6912868: "java.net.useSystemProxies" behavior fails to check "use_same_proxy" in GNOME Reviewed-by: alanb, chegar Contributed-by: damjan.jov at gmail.com ! src/solaris/native/sun/net/spi/DefaultProxySelector.c + test/java/net/ProxySelector/SystemProxies.java Changeset: c96d6cb31723 Author: chegar Date: 2010-02-23 17:08 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/c96d6cb31723 6365587: Proxy-Connection header sent through tunnel Reviewed-by: michaelm ! src/share/classes/sun/net/www/protocol/http/HttpURLConnection.java Changeset: b396584a3e64 Author: lana Date: 2010-02-23 10:17 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/b396584a3e64 Merge - make/java/text/FILES_java.gmk From michael.wilkerson at sun.com Thu Mar 4 00:53:15 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 04 Mar 2010 00:53:15 +0000 Subject: hg: jdk7/build/langtools: 10 new changesets Message-ID: <20100304005343.0049443CBC@hg.openjdk.java.net> Changeset: 7d9e3a15d2b3 Author: jjg Date: 2010-02-15 16:09 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/7d9e3a15d2b3 6926555: 6921979 breaks TreePosTest Reviewed-by: darcy ! test/tools/javac/treepostests/TreePosTest.java Changeset: af18e3956985 Author: darcy Date: 2010-02-15 18:20 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/af18e3956985 6634138: Source generated in last round not compiled Reviewed-by: jjg ! src/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java ! test/tools/javac/T6403466.java + test/tools/javac/processing/6634138/Dummy.java + test/tools/javac/processing/6634138/ExerciseDependency.java + test/tools/javac/processing/6634138/T6634138.java Changeset: fe17a9dbef03 Author: darcy Date: 2010-02-15 20:06 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/fe17a9dbef03 6926699: Annotation processing regression tests should typically return SourceVersion.latest Reviewed-by: jjg ! test/tools/javac/6341866/Anno.java ! test/tools/javac/T6406771.java ! test/tools/javac/T6411379.java ! test/tools/javac/T6423583.java ! test/tools/javac/T6855236.java ! test/tools/javac/api/6421111/T6421111.java ! test/tools/javac/api/6468404/T6468404.java ! test/tools/javac/api/T6412669.java ! test/tools/javac/enum/6424358/T6424358.java ! test/tools/javac/processing/6348499/A.java ! test/tools/javac/processing/6414633/A.java ! test/tools/javac/processing/6430209/T6430209.java ! test/tools/javac/processing/6430209/b6341534.java ! test/tools/javac/processing/T6439826.java ! test/tools/javac/processing/model/element/TypeParamBounds.java ! test/tools/javac/processing/model/type/MirroredTypeEx/OverEager.java ! test/tools/javac/processing/model/type/NoTypes.java ! test/tools/javac/processing/model/util/GetTypeElemBadArg.java ! test/tools/javac/processing/model/util/OverridesSpecEx.java Changeset: 631a273dac0f Author: darcy Date: 2010-02-15 20:17 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/631a273dac0f 6926703: apt tests should run with assertions enabled Reviewed-by: jjg ! src/share/classes/com/sun/mirror/util/SourceOrderDeclScanner.java Changeset: 16b9b7f45933 Author: darcy Date: 2010-02-17 14:30 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/16b9b7f45933 6927061: Refactor apt implemenation to use code from JSR 269 Reviewed-by: jjg ! src/share/classes/com/sun/mirror/util/SourceOrderDeclScanner.java ! src/share/classes/com/sun/tools/apt/comp/Apt.java ! src/share/classes/com/sun/tools/apt/main/Main.java ! src/share/classes/com/sun/tools/javac/file/Paths.java ! src/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java ! src/share/classes/com/sun/tools/javadoc/DocletInvoker.java Changeset: 67f0e05098fa Author: lana Date: 2010-02-17 10:25 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/67f0e05098fa Merge Changeset: 0fce6b64c258 Author: lana Date: 2010-02-17 16:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/0fce6b64c258 Merge Changeset: a3be81d385ee Author: jjg Date: 2010-02-18 15:41 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/a3be81d385ee 6927797: langtools/test/tools/javac/EarlyAssert.java fails when run with assertions enabled (-ea) Reviewed-by: darcy ! test/tools/javac/EarlyAssert.java + test/tools/javac/EarlyAssertWrapper.java Changeset: f25efdb55c99 Author: andrew Date: 2010-02-22 21:37 +0000 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/f25efdb55c99 6928623: Behaviour of VERBOSE=true on langtools build Summary: VERBOSE=true causes -diagnostics to be passed to ant rather than -debug Reviewed-by: jjg ! make/Makefile Changeset: 136bfc679462 Author: lana Date: 2010-02-23 10:17 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/136bfc679462 Merge From michael.wilkerson at sun.com Thu Mar 11 04:03:26 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 11 Mar 2010 04:03:26 +0000 Subject: hg: jdk7/build: Added tag jdk7-b85 for changeset cf26288a114b Message-ID: <20100311040327.0A90344679@hg.openjdk.java.net> Changeset: 3ddf90b39176 Author: mikejwre Date: 2010-03-04 13:50 -0800 URL: http://hg.openjdk.java.net/jdk7/build/rev/3ddf90b39176 Added tag jdk7-b85 for changeset cf26288a114b ! .hgtags From michael.wilkerson at sun.com Thu Mar 11 04:03:33 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 11 Mar 2010 04:03:33 +0000 Subject: hg: jdk7/build/corba: Added tag jdk7-b85 for changeset c67a9df7bc0c Message-ID: <20100311040334.7CA574467A@hg.openjdk.java.net> Changeset: 6253e28826d1 Author: mikejwre Date: 2010-03-04 13:50 -0800 URL: http://hg.openjdk.java.net/jdk7/build/corba/rev/6253e28826d1 Added tag jdk7-b85 for changeset c67a9df7bc0c ! .hgtags From michael.wilkerson at sun.com Thu Mar 11 04:05:11 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 11 Mar 2010 04:05:11 +0000 Subject: hg: jdk7/build/hotspot: 2 new changesets Message-ID: <20100311040525.933634467C@hg.openjdk.java.net> Changeset: 418bc80ce139 Author: mikejwre Date: 2010-03-04 13:50 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/418bc80ce139 Added tag jdk7-b85 for changeset 6c9796468b91 ! .hgtags Changeset: bf823ef06b4f Author: trims Date: 2010-03-08 15:50 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/bf823ef06b4f Added tag hs17-b10 for changeset 418bc80ce139 ! .hgtags From michael.wilkerson at sun.com Thu Mar 11 04:07:24 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 11 Mar 2010 04:07:24 +0000 Subject: hg: jdk7/build/jaxp: Added tag jdk7-b85 for changeset 6c0ccabb430d Message-ID: <20100311040724.D131D4467D@hg.openjdk.java.net> Changeset: 81c0f115bbe5 Author: mikejwre Date: 2010-03-04 13:50 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jaxp/rev/81c0f115bbe5 Added tag jdk7-b85 for changeset 6c0ccabb430d ! .hgtags From michael.wilkerson at sun.com Thu Mar 11 04:07:30 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 11 Mar 2010 04:07:30 +0000 Subject: hg: jdk7/build/jaxws: Added tag jdk7-b85 for changeset 8424512588ff Message-ID: <20100311040730.929CA4467E@hg.openjdk.java.net> Changeset: 512b0e924a5a Author: mikejwre Date: 2010-03-04 13:50 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jaxws/rev/512b0e924a5a Added tag jdk7-b85 for changeset 8424512588ff ! .hgtags From michael.wilkerson at sun.com Thu Mar 11 04:07:38 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 11 Mar 2010 04:07:38 +0000 Subject: hg: jdk7/build/jdk: Added tag jdk7-b85 for changeset b396584a3e64 Message-ID: <20100311040815.01CF04467F@hg.openjdk.java.net> Changeset: 03cd9e62961f Author: mikejwre Date: 2010-03-04 13:50 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/03cd9e62961f Added tag jdk7-b85 for changeset b396584a3e64 ! .hgtags From michael.wilkerson at sun.com Thu Mar 11 04:10:06 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 11 Mar 2010 04:10:06 +0000 Subject: hg: jdk7/build/langtools: Added tag jdk7-b85 for changeset 136bfc679462 Message-ID: <20100311041009.8B9C344680@hg.openjdk.java.net> Changeset: b816baf594e3 Author: mikejwre Date: 2010-03-04 13:50 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/b816baf594e3 Added tag jdk7-b85 for changeset 136bfc679462 ! .hgtags From michael.wilkerson at sun.com Thu Mar 18 02:46:00 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 18 Mar 2010 02:46:00 +0000 Subject: hg: jdk7/build: 3 new changesets Message-ID: <20100318024600.BC19844014@hg.openjdk.java.net> Changeset: 4d7419e4b759 Author: ohair Date: 2010-03-06 15:00 -0800 URL: http://hg.openjdk.java.net/jdk7/build/rev/4d7419e4b759 6928700: Configure top repo for JPRT testing Reviewed-by: alanb, jjg ! make/jprt.properties + test/Makefile Changeset: f3664d6879ab Author: ohair Date: 2010-03-06 15:01 -0800 URL: http://hg.openjdk.java.net/jdk7/build/rev/f3664d6879ab Merge Changeset: 433a60a9c0bf Author: lana Date: 2010-03-09 15:28 -0800 URL: http://hg.openjdk.java.net/jdk7/build/rev/433a60a9c0bf Merge From michael.wilkerson at sun.com Thu Mar 18 02:49:04 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 18 Mar 2010 02:49:04 +0000 Subject: hg: jdk7/build/jdk: 39 new changesets Message-ID: <20100318030145.B9C3F44019@hg.openjdk.java.net> Changeset: 840601ac5ab7 Author: rkennke Date: 2010-03-03 15:50 +0100 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/840601ac5ab7 6892485: Deadlock in SunGraphicsEnvironment / FontManager Summary: Synchronize on correct monitor in SunFontManager. Reviewed-by: igor, prr ! src/share/classes/sun/font/SunFontManager.java Changeset: 1d7db2d5c4c5 Author: minqi Date: 2010-03-08 11:35 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/1d7db2d5c4c5 6918065: Crash in Java2D blit loop (IntArgbToIntArgbPreSrcOverMaskBlit) in 64bit mode Reviewed-by: igor, bae ! src/share/classes/java/awt/AlphaComposite.java + test/java/awt/AlphaComposite/TestAlphaCompositeForNaN.java Changeset: 494f5e4f24da Author: lana Date: 2010-03-09 15:26 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/494f5e4f24da Merge Changeset: e64331144648 Author: rupashka Date: 2010-02-10 15:15 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/e64331144648 6848475: JSlider does not display the correct value of its BoundedRangeModel Reviewed-by: peterz ! src/share/classes/javax/swing/plaf/basic/BasicSliderUI.java + test/javax/swing/JSlider/6848475/bug6848475.java Changeset: f81c8041ccf4 Author: peytoia Date: 2010-02-11 15:58 +0900 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/f81c8041ccf4 6909002: Remove indicim.jar and thaiim.jar from JRE and move to samples if needed Reviewed-by: okutsu ! make/com/sun/Makefile Changeset: e2b58a45a426 Author: peytoia Date: 2010-02-12 14:38 +0900 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/e2b58a45a426 6921289: (tz) Support tzdata2010b Reviewed-by: okutsu ! make/sun/javazic/tzdata/VERSION ! make/sun/javazic/tzdata/antarctica ! make/sun/javazic/tzdata/asia ! make/sun/javazic/tzdata/australasia ! make/sun/javazic/tzdata/europe ! make/sun/javazic/tzdata/northamerica ! make/sun/javazic/tzdata/zone.tab ! src/share/classes/sun/util/resources/TimeZoneNames.java ! src/share/classes/sun/util/resources/TimeZoneNames_de.java ! src/share/classes/sun/util/resources/TimeZoneNames_es.java ! src/share/classes/sun/util/resources/TimeZoneNames_fr.java ! src/share/classes/sun/util/resources/TimeZoneNames_it.java ! src/share/classes/sun/util/resources/TimeZoneNames_ja.java ! src/share/classes/sun/util/resources/TimeZoneNames_ko.java ! src/share/classes/sun/util/resources/TimeZoneNames_sv.java ! src/share/classes/sun/util/resources/TimeZoneNames_zh_CN.java ! src/share/classes/sun/util/resources/TimeZoneNames_zh_TW.java Changeset: e8340332745e Author: malenkov Date: 2010-02-18 17:46 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/e8340332745e 4498236: RFE: Provide a toString method for PropertyChangeEvent and other classes Reviewed-by: peterz ! src/share/classes/java/beans/BeanDescriptor.java ! src/share/classes/java/beans/EventSetDescriptor.java ! src/share/classes/java/beans/FeatureDescriptor.java ! src/share/classes/java/beans/IndexedPropertyChangeEvent.java ! src/share/classes/java/beans/IndexedPropertyDescriptor.java ! src/share/classes/java/beans/MethodDescriptor.java ! src/share/classes/java/beans/PropertyChangeEvent.java ! src/share/classes/java/beans/PropertyDescriptor.java + test/java/beans/Introspector/Test4498236.java Changeset: 5c03237838e1 Author: rupashka Date: 2010-02-27 14:26 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/5c03237838e1 6913758: Specification for SynthViewportUI.paintBorder(...) should mention that this method is never called Reviewed-by: peterz ! src/share/classes/javax/swing/plaf/synth/SynthViewportUI.java Changeset: 96205ed1b196 Author: rupashka Date: 2010-02-27 14:47 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/96205ed1b196 6918447: SynthToolBarUI.setBorderToXXXX() methods don't correspond inherited spec. They do nothing. Reviewed-by: peterz ! src/share/classes/javax/swing/plaf/synth/SynthToolBarUI.java Changeset: 621e921a14cd Author: rupashka Date: 2010-02-27 15:09 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/621e921a14cd 6918861: SynthSliderUI.uninstallDefaults() is not called when UI is uninstalled Reviewed-by: malenkov ! src/share/classes/javax/swing/plaf/basic/BasicSliderUI.java ! src/share/classes/javax/swing/plaf/synth/SynthSliderUI.java + test/javax/swing/JSlider/6918861/bug6918861.java Changeset: 28741de0bb4a Author: rupashka Date: 2010-02-27 16:03 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/28741de0bb4a 6923305: SynthSliderUI paints the slider track when the slider's "paintTrack" property is set to false Reviewed-by: alexp ! src/share/classes/javax/swing/plaf/synth/SynthSliderUI.java + test/javax/swing/JSlider/6923305/bug6923305.java Changeset: 2bf137beb9bd Author: rupashka Date: 2010-02-27 16:14 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/2bf137beb9bd 6929298: The SynthSliderUI#calculateTickRect method should be removed Reviewed-by: peterz ! src/share/classes/javax/swing/plaf/synth/SynthSliderUI.java Changeset: d6b3a07c8752 Author: rupashka Date: 2010-03-03 17:57 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/d6b3a07c8752 6924059: SynthScrollBarUI.configureScrollBarColors() should have spec different from the overridden method Reviewed-by: peterz ! src/share/classes/javax/swing/plaf/synth/SynthScrollBarUI.java + test/javax/swing/JScrollBar/6924059/bug6924059.java Changeset: 30c520bd148f Author: rupashka Date: 2010-03-03 20:08 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/30c520bd148f 6913768: With default SynthLookAndFeel instance installed new JTable creation leads to throwing NPE Reviewed-by: peterz ! src/share/classes/javax/swing/JTable.java ! src/share/classes/javax/swing/plaf/synth/SynthTableUI.java + test/javax/swing/JTable/6913768/bug6913768.java Changeset: f13fc955be62 Author: rupashka Date: 2010-03-03 20:53 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/f13fc955be62 6917744: JScrollPane Page Up/Down keys do not handle correctly html tables with different cells contents Reviewed-by: peterz, alexp ! src/share/classes/javax/swing/text/DefaultEditorKit.java + test/javax/swing/JEditorPane/6917744/bug6917744.java + test/javax/swing/JEditorPane/6917744/test.html Changeset: 0622086d82ac Author: malenkov Date: 2010-03-04 21:17 +0300 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/0622086d82ac 6921644: XMLEncoder generates invalid XML Reviewed-by: peterz ! src/share/classes/java/beans/XMLEncoder.java + test/java/beans/XMLEncoder/Test5023550.java + test/java/beans/XMLEncoder/Test5023557.java + test/java/beans/XMLEncoder/Test6921644.java Changeset: 79a509ac8f35 Author: lana Date: 2010-03-01 18:30 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/79a509ac8f35 Merge ! make/com/sun/Makefile - make/java/text/FILES_java.gmk Changeset: 90248595ec35 Author: lana Date: 2010-03-04 13:07 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/90248595ec35 Merge Changeset: 2fe4e72288ce Author: lana Date: 2010-03-09 15:28 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/2fe4e72288ce Merge Changeset: 38fbb2353a6a Author: alanb Date: 2010-02-23 17:56 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/38fbb2353a6a 6925977: (file) test/java/nio/file/Path/CheckPermissions.java fails if test.src on read-only file system Reviewed-by: chegar ! test/java/nio/file/Path/CheckPermissions.java Changeset: 00abf8c232be Author: alanb Date: 2010-02-23 17:58 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/00abf8c232be 6925932: (file) Path.endsWith can throw ArrayIndexOutOfBoundsException (unx) Reviewed-by: chegar ! src/solaris/classes/sun/nio/fs/UnixPath.java ! test/java/nio/file/Path/PathOps.java Changeset: be5db597f3be Author: alanb Date: 2010-02-23 18:19 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/be5db597f3be 6928960: make modules fails to build class analyzer Reviewed-by: mchung ! make/modules/tools/Makefile Changeset: e94b296b53b4 Author: alanb Date: 2010-02-23 18:21 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/e94b296b53b4 6926800: TEST_BUG: java/nio/file/Files/walk_file_tree.sh fails with newer versions of find(1) Reviewed-by: forax ! test/java/nio/file/Files/PrintFileTree.java ! test/java/nio/file/Files/walk_file_tree.sh Changeset: e842e99b514a Author: darcy Date: 2010-02-24 10:48 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/e842e99b514a 6929382: Various core classes in util and elsewhere are missing @param tags Reviewed-by: dholmes, martin ! src/share/classes/java/lang/Iterable.java ! src/share/classes/java/util/Collection.java ! src/share/classes/java/util/Iterator.java ! src/share/classes/java/util/List.java Changeset: 9929203a8b98 Author: xuelei Date: 2010-02-25 13:32 +0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/9929203a8b98 6916202: More cases of invalid ldap filters accepted and processed Reviewed-by: vinnie, weijun ! src/share/classes/com/sun/jndi/ldap/Filter.java + test/com/sun/jndi/ldap/InvalidLdapFilters.java Changeset: 77beb60b39c6 Author: alanb Date: 2010-02-27 18:18 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/77beb60b39c6 6929532: (file) WatchService should avoid queuing new modify events when lots of files are changing Reviewed-by: alanb Contributed-by: sebastian.sickelmann at gmx.de ! src/share/classes/sun/nio/fs/AbstractWatchKey.java + test/java/nio/file/WatchService/LotsOfEvents.java - test/java/nio/file/WatchService/OverflowEventIsLoner.java Changeset: b77e94f5a601 Author: alanb Date: 2010-02-27 19:15 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/b77e94f5a601 6929259: Remove double spaces from Dual-pivot quicksort Reviewed-by: alanb Contributed-by: vladimir.yaroslavskiy at sun.com ! src/share/classes/java/util/DualPivotQuicksort.java Changeset: 529d2da0aee2 Author: alanb Date: 2010-02-27 19:26 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/529d2da0aee2 6815768: File.getxxxSpace() methods fail for very large file systems under 32bit Java Reviewed-by: ohair ! src/solaris/native/java/io/UnixFileSystem_md.c Changeset: f7a6eae6e1eb Author: alanb Date: 2010-02-27 19:29 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/f7a6eae6e1eb 6921374: java.lang.String::hashCode() should check for count == 0 to avoid repeated stores hash = 0 Reviewed-by: darcy, ohair ! src/share/classes/java/lang/String.java Changeset: 78d91c4223cb Author: vinnie Date: 2010-03-01 17:54 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/78d91c4223cb 6921001: api/java_security/IdentityScope/IdentityScopeTests.html#getSystemScope fails starting from b78 JDK7 Reviewed-by: mullan ! src/share/classes/java/security/IdentityScope.java ! src/share/lib/security/java.security + test/java/security/IdentityScope/NoDefaultSystemScope.java Changeset: 893034df4ec2 Author: vinnie Date: 2010-03-01 18:00 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/893034df4ec2 Merge - test/java/nio/file/WatchService/OverflowEventIsLoner.java Changeset: cddb43b12d28 Author: alanb Date: 2010-03-03 16:09 +0000 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/cddb43b12d28 6931216: TEST_BUG: test/java/nio/file/WatchService/LotsOfEvents.java failed with NPE Reviewed-by: chegar ! test/java/nio/file/WatchService/LotsOfEvents.java Changeset: 507159d8d143 Author: ohair Date: 2010-03-03 11:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/507159d8d143 6931763: sanity checks broken with latest cygwin, newer egrep -i option problems Reviewed-by: jjg ! make/common/shared/Sanity.gmk Changeset: 61c298558549 Author: weijun Date: 2010-03-04 10:37 +0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/61c298558549 6844909: support allow_weak_crypto in krb5.conf Reviewed-by: valeriep ! src/share/classes/sun/security/krb5/internal/crypto/EType.java + test/sun/security/krb5/etype/WeakCrypto.java + test/sun/security/krb5/etype/weakcrypto.conf Changeset: 0f383673ce31 Author: weijun Date: 2010-03-04 10:38 +0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/0f383673ce31 6923681: Jarsigner crashes during timestamping Reviewed-by: vinnie ! src/share/classes/sun/security/tools/TimestampedSigner.java Changeset: 5e15b70e6d27 Author: weijun Date: 2010-03-04 10:38 +0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/5e15b70e6d27 6880321: sun.security.provider.JavaKeyStore abuse of OOM Exception handling Reviewed-by: xuelei ! src/share/classes/sun/security/provider/JavaKeyStore.java Changeset: c2d29e5695c2 Author: lana Date: 2010-03-04 13:40 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/c2d29e5695c2 Merge Changeset: 58b44ac0b10d Author: ohair Date: 2010-03-06 14:59 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/58b44ac0b10d 6915983: testing problems, adjusting list of tests, needs some investigation Reviewed-by: alanb ! test/Makefile ! test/ProblemList.txt Changeset: eae6e9ab2606 Author: lana Date: 2010-03-09 15:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/eae6e9ab2606 Merge - test/java/nio/file/WatchService/OverflowEventIsLoner.java From michael.wilkerson at sun.com Thu Mar 18 03:05:49 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 18 Mar 2010 03:05:49 +0000 Subject: hg: jdk7/build/langtools: 18 new changesets Message-ID: <20100318030642.BD6844401B@hg.openjdk.java.net> Changeset: 6eca0895a644 Author: jjg Date: 2010-02-23 18:43 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/6eca0895a644 6511613: javac unexpectedly doesn't fail in some cases if an annotation processor specified Reviewed-by: darcy ! src/share/classes/com/sun/tools/javac/comp/Attr.java ! src/share/classes/com/sun/tools/javac/main/JavaCompiler.java ! src/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java ! src/share/classes/com/sun/tools/javac/util/Log.java + test/tools/javac/processing/6511613/DummyProcessor.java + test/tools/javac/processing/6511613/clss41701.java Changeset: 87eb6edd4f21 Author: jjg Date: 2010-02-25 09:42 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/87eb6edd4f21 4880220: Add a warning when accessing a static method via an reference Reviewed-by: darcy ! make/build.properties ! src/share/classes/com/sun/tools/javac/code/Lint.java ! src/share/classes/com/sun/tools/javac/comp/Attr.java ! src/share/classes/com/sun/tools/javac/comp/Check.java ! src/share/classes/com/sun/tools/javac/resources/compiler.properties + test/tools/javac/4880220/T4880220.empty.out + test/tools/javac/4880220/T4880220.error.out + test/tools/javac/4880220/T4880220.java + test/tools/javac/4880220/T4880220.warn.out Changeset: 85242c273d31 Author: darcy Date: 2010-02-25 11:04 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/85242c273d31 6929645: Address various findbugs warnings in langtools Reviewed-by: jjg ! src/share/classes/com/sun/tools/apt/comp/Apt.java ! src/share/classes/com/sun/tools/apt/mirror/declaration/AnnotationProxyMaker.java ! src/share/classes/com/sun/tools/apt/mirror/declaration/DeclarationImpl.java ! src/share/classes/com/sun/tools/javac/model/AnnotationProxyMaker.java ! src/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java Changeset: dbcba45123cd Author: jjg Date: 2010-02-25 12:26 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/dbcba45123cd 6929544: langtools source code uses statics qualified by instance variables Reviewed-by: darcy ! make/build.properties ! src/share/classes/com/sun/tools/apt/main/CommandLine.java ! src/share/classes/com/sun/tools/apt/mirror/type/TypeMirrorImpl.java ! src/share/classes/com/sun/tools/doclets/standard/Standard.java ! src/share/classes/com/sun/tools/javac/Launcher.java ! src/share/classes/com/sun/tools/javac/api/JavacTool.java ! src/share/classes/com/sun/tools/javac/code/Types.java ! src/share/classes/com/sun/tools/javac/jvm/Gen.java ! src/share/classes/com/sun/tools/javac/jvm/Items.java ! src/share/classes/com/sun/tools/javac/main/CommandLine.java ! src/share/classes/com/sun/tools/javac/main/JavaCompiler.java Changeset: af75fd6155de Author: jjg Date: 2010-02-25 13:32 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/af75fd6155de 6893943: exit code from javah with no args is 0 Reviewed-by: darcy ! src/share/classes/com/sun/tools/javah/JavahTask.java + test/tools/javah/T6893943.java Changeset: b030706da5b4 Author: jjg Date: 2010-02-26 08:42 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/b030706da5b4 6881645: Unchecked method call on a method declared inside anonymous inner causes javac to crash Reviewed-by: mcimadamore ! src/share/classes/com/sun/tools/javac/code/Symbol.java + test/tools/javac/T6881645.java Changeset: 72833a8a6086 Author: jjg Date: 2010-02-26 15:26 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/72833a8a6086 6930076: "null" can incorrectly appear in error message compiler.err.error.reading.file Reviewed-by: darcy ! src/share/classes/com/sun/tools/javac/file/JavacFileManager.java ! src/share/classes/com/sun/tools/javac/file/Paths.java ! src/share/classes/com/sun/tools/javac/main/JavaCompiler.java Changeset: 7b69c7083a97 Author: jjg Date: 2010-02-26 15:30 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/7b69c7083a97 6930032: fix findbugs errors in com.sun.tools.javac.comp Reviewed-by: darcy ! src/share/classes/com/sun/tools/javac/comp/Enter.java ! src/share/classes/com/sun/tools/javac/comp/TransTypes.java Changeset: 7c23bbbe0dbd Author: darcy Date: 2010-03-02 14:06 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/7c23bbbe0dbd 6931130: Remove unused AnnotationCollector code from JavacProcessingEnvironment Reviewed-by: jjg ! src/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java Changeset: 6e1e2738c530 Author: jjg Date: 2010-03-02 16:40 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/6e1e2738c530 6931482: minor findbugs fixes Reviewed-by: darcy ! src/share/classes/com/sun/tools/classfile/ConstantPool.java ! src/share/classes/com/sun/tools/javadoc/DocEnv.java ! src/share/classes/com/sun/tools/javadoc/SeeTagImpl.java Changeset: 235135d61974 Author: jjg Date: 2010-03-02 16:43 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/235135d61974 6931127: strange test class files Reviewed-by: darcy ! test/tools/javac/annotations/neg/Constant.java ! test/tools/javac/generics/Casting.java ! test/tools/javac/generics/Casting3.java ! test/tools/javac/generics/Casting4.java ! test/tools/javac/generics/InnerInterface1.java ! test/tools/javac/generics/InnerInterface2.java ! test/tools/javac/generics/Multibound1.java ! test/tools/javac/generics/MultipleInheritance.java ! test/tools/javac/generics/NameOrder.java ! test/tools/javac/generics/PermuteBound.java ! test/tools/javac/generics/PrimitiveVariant.java Changeset: fc7132746501 Author: darcy Date: 2010-03-03 16:05 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/fc7132746501 6449781: TypeElement.getQualifiedName for anonymous classes returns null instead of an empty name Reviewed-by: jjg ! src/share/classes/com/sun/tools/javac/jvm/ClassReader.java + test/tools/javac/processing/model/element/TestAnonClassNames.java + test/tools/javac/processing/model/element/TestAnonSourceNames.java Changeset: 7f5db2e8b423 Author: jjg Date: 2010-03-03 17:22 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/7f5db2e8b423 6931927: position issues with synthesized anonymous class Reviewed-by: darcy ! src/share/classes/com/sun/tools/javac/parser/JavacParser.java + test/tools/javac/tree/TestAnnotatedAnonClass.java + test/tools/javac/tree/TreePosTest.java - test/tools/javac/treepostests/TreePosTest.java Changeset: 117c95448ab9 Author: jjg Date: 2010-03-03 19:34 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/117c95448ab9 6931126: jtreg tests not Windows friendly Reviewed-by: darcy ! test/tools/javac/ThrowsIntersection_1.java ! test/tools/javac/ThrowsIntersection_2.java ! test/tools/javac/ThrowsIntersection_3.java ! test/tools/javac/ThrowsIntersection_4.java ! test/tools/javac/generics/NameOrder.java Changeset: c55733ceed61 Author: lana Date: 2010-03-04 13:40 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/c55733ceed61 Merge Changeset: a23282f17d0b Author: jjg Date: 2010-03-05 16:12 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/a23282f17d0b 6930108: IllegalArgumentException in AbstractDiagnosticFormatter for tools/javac/api/TestJavacTaskScanner.jav Reviewed-by: darcy ! src/share/classes/com/sun/tools/javac/util/BasicDiagnosticFormatter.java ! test/tools/javac/api/TestJavacTaskScanner.java + test/tools/javac/api/TestResolveError.java Changeset: a4f3b97c8028 Author: jjg Date: 2010-03-05 16:13 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/a4f3b97c8028 Merge Changeset: ef07347428f2 Author: lana Date: 2010-03-09 15:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/ef07347428f2 Merge - test/tools/javac/treepostests/TreePosTest.java From kelly.ohair at sun.com Fri Mar 19 21:47:09 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 19 Mar 2010 21:47:09 +0000 Subject: hg: jdk7/build: Added tag jdk7-b86 for changeset 433a60a9c0bf Message-ID: <20100319214711.60A7C442D6@hg.openjdk.java.net> Changeset: 6b1069f53fbc Author: mikejwre Date: 2010-03-18 13:52 -0700 URL: http://hg.openjdk.java.net/jdk7/build/rev/6b1069f53fbc Added tag jdk7-b86 for changeset 433a60a9c0bf ! .hgtags From kelly.ohair at sun.com Fri Mar 19 21:47:19 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 19 Mar 2010 21:47:19 +0000 Subject: hg: jdk7/build/corba: Added tag jdk7-b86 for changeset 6253e28826d1 Message-ID: <20100319214722.79E1E442D7@hg.openjdk.java.net> Changeset: 09a41111a401 Author: mikejwre Date: 2010-03-18 13:52 -0700 URL: http://hg.openjdk.java.net/jdk7/build/corba/rev/09a41111a401 Added tag jdk7-b86 for changeset 6253e28826d1 ! .hgtags From kelly.ohair at sun.com Fri Mar 19 21:48:52 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 19 Mar 2010 21:48:52 +0000 Subject: hg: jdk7/build/hotspot: Added tag jdk7-b86 for changeset bf823ef06b4f Message-ID: <20100319214906.94DF2442D8@hg.openjdk.java.net> Changeset: 4b58861a3570 Author: mikejwre Date: 2010-03-18 13:52 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/4b58861a3570 Added tag jdk7-b86 for changeset bf823ef06b4f ! .hgtags From kelly.ohair at sun.com Fri Mar 19 21:53:10 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 19 Mar 2010 21:53:10 +0000 Subject: hg: jdk7/build/jaxp: Added tag jdk7-b86 for changeset 81c0f115bbe5 Message-ID: <20100319215310.9819B442DA@hg.openjdk.java.net> Changeset: 8b493f1aa136 Author: mikejwre Date: 2010-03-18 13:52 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jaxp/rev/8b493f1aa136 Added tag jdk7-b86 for changeset 81c0f115bbe5 ! .hgtags From kelly.ohair at sun.com Fri Mar 19 21:53:17 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 19 Mar 2010 21:53:17 +0000 Subject: hg: jdk7/build/jaxws: Added tag jdk7-b86 for changeset 512b0e924a5a Message-ID: <20100319215318.51784442DB@hg.openjdk.java.net> Changeset: 3febd6fab2ac Author: mikejwre Date: 2010-03-18 13:52 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jaxws/rev/3febd6fab2ac Added tag jdk7-b86 for changeset 512b0e924a5a ! .hgtags From kelly.ohair at sun.com Fri Mar 19 21:53:29 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 19 Mar 2010 21:53:29 +0000 Subject: hg: jdk7/build/jdk: Added tag jdk7-b86 for changeset eae6e9ab2606 Message-ID: <20100319215420.17C6B442DD@hg.openjdk.java.net> Changeset: 2cafbbe9825e Author: mikejwre Date: 2010-03-18 13:53 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/2cafbbe9825e Added tag jdk7-b86 for changeset eae6e9ab2606 ! .hgtags From kelly.ohair at sun.com Fri Mar 19 21:57:45 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 19 Mar 2010 21:57:45 +0000 Subject: hg: jdk7/build/langtools: Added tag jdk7-b86 for changeset ef07347428f2 Message-ID: <20100319215754.20407442DF@hg.openjdk.java.net> Changeset: 409db93d19c0 Author: mikejwre Date: 2010-03-18 13:53 -0700 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/409db93d19c0 Added tag jdk7-b86 for changeset ef07347428f2 ! .hgtags From michael.wilkerson at sun.com Thu Mar 25 02:42:53 2010 From: michael.wilkerson at sun.com (michael.wilkerson at sun.com) Date: Thu, 25 Mar 2010 02:42:53 +0000 Subject: hg: jdk7/build/hotspot: 37 new changesets Message-ID: <20100325024413.8C48B44219@hg.openjdk.java.net> Changeset: 877a14af58e1 Author: never Date: 2010-02-18 15:05 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/877a14af58e1 6663854: assert(n != __null,"Bad immediate dominator info.") in C2 with -Xcomp Reviewed-by: kvn ! src/share/vm/opto/split_if.cpp + test/compiler/6663854/Test6663854.java Changeset: 2883969d09e7 Author: kvn Date: 2010-02-19 10:04 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/2883969d09e7 6910664: C2: java/util/Arrays/Sorting.java fails with DeoptimizeALot flag Summary: Matcher::float_in_double should be true only when FPU is used for floats. Reviewed-by: never, twisti ! src/cpu/sparc/vm/sparc.ad ! src/cpu/x86/vm/x86_32.ad ! src/cpu/x86/vm/x86_64.ad ! src/share/vm/opto/matcher.hpp ! src/share/vm/opto/output.cpp Changeset: b71f13525cc8 Author: never Date: 2010-02-19 13:06 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/b71f13525cc8 6927049: assert(is_Loop(),"invalid node class") Reviewed-by: kvn ! src/share/vm/opto/loopTransform.cpp Changeset: 8b38237bae55 Author: kvn Date: 2010-02-22 16:56 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/8b38237bae55 6928717: HS17 fails to build with SS11 C++ Summary: Add missing handles.inline.hpp for codeCache.cpp. Reviewed-by: never ! src/share/vm/includeDB_core Changeset: 855c5171834c Author: twisti Date: 2010-02-23 17:46 +0100 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/855c5171834c 6928839: JSR 292 typo in x86 _adapter_check_cast Summary: There is a small typo in methodHandles_x86.cpp. Reviewed-by: kvn ! src/cpu/x86/vm/methodHandles_x86.cpp Changeset: da9559b49b84 Author: never Date: 2010-02-25 11:38 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/da9559b49b84 6915557: assert(_gvn.type(l)->higher_equal(type),"must constrain OSR typestate") with debug build Reviewed-by: kvn ! src/share/vm/opto/parse1.cpp Changeset: 2432acbee618 Author: kvn Date: 2010-02-25 15:55 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/2432acbee618 6930035: C2 type system incorrectly handles case j.l.Object->meet(constant AryPtr) Summary: Add missing code. Reviewed-by: never ! src/share/vm/opto/type.cpp Changeset: 336c6c200f5f Author: kvn Date: 2010-02-25 22:58 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/336c6c200f5f 6930116: loop predication code does not handle If nodes with only one projection Summary: Add check for iff->outcnt() < 2. Reviewed-by: never ! src/share/vm/opto/loopTransform.cpp Changeset: 7d236a9688c5 Author: never Date: 2010-03-01 12:12 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/7d236a9688c5 6930398: fix for return address locals in OSR entries uses wrong test Reviewed-by: kvn ! src/share/vm/opto/parse1.cpp Changeset: b81f3572f355 Author: tonyp Date: 2010-02-23 23:13 -0500 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/b81f3572f355 6928059: G1: command line parameter renaming Summary: Rename G1 parameters to make them more consistent. Reviewed-by: jmasa, johnc ! src/share/vm/gc_implementation/g1/concurrentG1Refine.cpp ! src/share/vm/gc_implementation/g1/concurrentG1Refine.hpp ! src/share/vm/gc_implementation/g1/concurrentG1RefineThread.cpp ! src/share/vm/gc_implementation/g1/concurrentMark.cpp ! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp ! src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp ! src/share/vm/gc_implementation/g1/g1MMUTracker.cpp ! src/share/vm/gc_implementation/g1/g1MMUTracker.hpp ! src/share/vm/gc_implementation/g1/g1RemSet.cpp ! src/share/vm/gc_implementation/g1/g1_globals.hpp Changeset: 1c72304f1885 Author: tonyp Date: 2010-02-23 23:14 -0500 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/1c72304f1885 6928073: G1: use existing command line parameters for marking cycle initiation Summary: replace the combination of the G1SteadyStateUsed / G1SteadyStateUsedDelta parameteres to decide the marking initiation threshold and instead use InitiatingHeapOccupancyPercent. Reviewed-by: ysr, johnc ! src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp ! src/share/vm/gc_implementation/g1/g1_globals.hpp ! src/share/vm/runtime/arguments.cpp ! src/share/vm/runtime/globals.hpp Changeset: 5f1f51edaff6 Author: jmasa Date: 2010-02-24 07:00 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/5f1f51edaff6 6928081: G1: rename parameters common with CMS Summary: Rename marking stack sizing flags to be common between G1 and CMS Reviewed-by: ysr, tonyp ! src/share/vm/gc_implementation/concurrentMarkSweep/cmsAdaptiveSizePolicy.cpp ! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp ! src/share/vm/gc_implementation/g1/concurrentMark.cpp ! src/share/vm/gc_implementation/g1/g1_globals.hpp ! src/share/vm/includeDB_core ! src/share/vm/runtime/arguments.cpp ! src/share/vm/runtime/arguments.hpp ! src/share/vm/runtime/globals.hpp ! src/share/vm/utilities/taskqueue.hpp Changeset: a1c410de27e4 Author: tonyp Date: 2010-02-24 14:56 -0500 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/a1c410de27e4 6928065: G1: use existing command line parameters to set the young generation size Summary: see synopsis Reviewed-by: johnc, jmasa ! src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp ! src/share/vm/gc_implementation/g1/g1_globals.hpp Changeset: ab75c83d7c37 Author: johnc Date: 2010-03-02 13:57 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/ab75c83d7c37 Merge ! src/share/vm/includeDB_core Changeset: 8911d8c0596f Author: phh Date: 2010-02-26 16:40 -0500 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/8911d8c0596f 6923123: Hotspot refuses to start when -Xmx4m or -Xms4m is specified Summary: Reduce NewSize from 4m to 1m. Reviewed-by: tonyp, jmasa ! src/share/vm/runtime/globals.hpp Changeset: c76ca382971b Author: johnc Date: 2010-03-02 13:59 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/c76ca382971b Merge ! src/share/vm/runtime/globals.hpp Changeset: d47555d7aca8 Author: jmasa Date: 2010-03-03 08:10 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/d47555d7aca8 6910182: CMS: assert(_cursor[j] == _survivor_plab_array[j].end(),"Ctl pt invariant") Summary: Calculation of the slicing of survivor spaces for MT was incorrect. Reviewed-by: ysr ! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp Changeset: c8a467bf56ad Author: coleenp Date: 2010-03-02 12:09 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/c8a467bf56ad 6914050: jvm assertion "guard pages must be in use" in -Xcomp mode Summary: Move creating stack guard pages in jni attach thread before potential java call rather than after. Also cleanup stack guard pages when jni attach fails Reviewed-by: never, dholmes ! src/share/vm/prims/jni.cpp ! src/share/vm/runtime/thread.cpp Changeset: 4b0f2f4918ed Author: xlu Date: 2010-03-10 21:42 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/4b0f2f4918ed 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time Summary: Improve the usability of safepoint statistics data. See bug evaluation for more details. Reviewed-by: ysr, dholmes ! src/share/vm/runtime/safepoint.cpp ! src/share/vm/runtime/safepoint.hpp Changeset: 12d91eb0f579 Author: acorn Date: 2010-03-11 14:41 -0500 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/12d91eb0f579 Merge Changeset: 51db1e4b379d Author: twisti Date: 2010-03-08 04:46 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/51db1e4b379d 6932536: JSR 292 modified JDK MethodHandlesTest fails on x86_64 Summary: A modified MethodHandlesTest revealed two bugs on x86_64. Reviewed-by: never, jrose ! src/cpu/x86/vm/methodHandles_x86.cpp Changeset: 7de45b5044c3 Author: never Date: 2010-03-09 11:02 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/7de45b5044c3 6932270: Allow Java's ELF symtab reader to use separate debuginfo files Reviewed-by: never Contributed-by: Andrew Haley ! agent/src/os/linux/libproc_impl.c ! agent/src/os/linux/symtab.c ! agent/src/os/linux/symtab.h + make/linux/makefiles/build_vm_def.sh ! make/linux/makefiles/mapfile-vers-debug ! make/linux/makefiles/mapfile-vers-product ! make/linux/makefiles/vm.make Changeset: 3cf667df43ef Author: twisti Date: 2010-03-09 20:16 +0100 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/3cf667df43ef 6919934: JSR 292 needs to support x86 C1 Summary: This implements JSR 292 support for C1 x86. Reviewed-by: never, jrose, kvn ! src/cpu/sparc/vm/c1_CodeStubs_sparc.cpp ! src/cpu/sparc/vm/c1_LIRAssembler_sparc.cpp ! src/cpu/sparc/vm/c1_MacroAssembler_sparc.cpp ! src/cpu/sparc/vm/c1_Runtime1_sparc.cpp ! src/cpu/sparc/vm/interp_masm_sparc.cpp ! src/cpu/sparc/vm/interp_masm_sparc.hpp ! src/cpu/sparc/vm/stubGenerator_sparc.cpp ! src/cpu/sparc/vm/templateInterpreter_sparc.cpp ! src/cpu/x86/vm/c1_CodeStubs_x86.cpp ! src/cpu/x86/vm/c1_LIRAssembler_x86.cpp ! src/cpu/x86/vm/c1_MacroAssembler_x86.cpp ! src/cpu/x86/vm/c1_Runtime1_x86.cpp ! src/cpu/x86/vm/stubGenerator_x86_32.cpp ! src/cpu/x86/vm/stubGenerator_x86_64.cpp ! src/cpu/x86/vm/templateInterpreter_x86_32.cpp ! src/cpu/x86/vm/templateInterpreter_x86_64.cpp ! src/share/vm/c1/c1_Canonicalizer.cpp ! src/share/vm/c1/c1_CodeStubs.hpp ! src/share/vm/c1/c1_GraphBuilder.cpp ! src/share/vm/c1/c1_IR.cpp ! src/share/vm/c1/c1_IR.hpp ! src/share/vm/c1/c1_Instruction.cpp ! src/share/vm/c1/c1_Instruction.hpp ! src/share/vm/c1/c1_LIR.cpp ! src/share/vm/c1/c1_LIR.hpp ! src/share/vm/c1/c1_LIRAssembler.cpp ! src/share/vm/c1/c1_LIRAssembler.hpp ! src/share/vm/c1/c1_LIRGenerator.cpp ! src/share/vm/c1/c1_MacroAssembler.hpp ! src/share/vm/ci/ciCPCache.cpp ! src/share/vm/ci/ciCPCache.hpp ! src/share/vm/includeDB_compiler1 ! src/share/vm/includeDB_core ! src/share/vm/opto/runtime.cpp ! src/share/vm/runtime/sharedRuntime.cpp ! src/share/vm/runtime/sharedRuntime.hpp ! src/share/vm/runtime/vframeArray.cpp Changeset: d8e270c4f609 Author: twisti Date: 2010-03-09 23:57 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/d8e270c4f609 Merge Changeset: c466efa608d5 Author: roland Date: 2010-03-05 13:58 +0100 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/c466efa608d5 6932496: c1: deoptimization of jsr subroutine fails on sparcv9 Summary: store jsr ret bci as intptr constant in c1 debug info Reviewed-by: never ! src/cpu/sparc/vm/c1_LIRAssembler_sparc.cpp ! src/cpu/x86/vm/c1_LIRAssembler_x86.cpp ! src/share/vm/c1/c1_LIR.cpp ! src/share/vm/c1/c1_LIR.hpp ! src/share/vm/c1/c1_LinearScan.cpp + test/compiler/6932496/Test6932496.java Changeset: da06d1795d84 Author: twisti Date: 2010-03-11 05:09 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/da06d1795d84 6934089: Zero 32-bit/64kb page fix Summary: The fix for 6927165 increased the number of shadow pages for 32-bit platforms and this causes a problem on systems with 64kb pages. Reviewed-by: twisti Contributed-by: Gary Benson ! src/os_cpu/linux_zero/vm/globals_linux_zero.hpp Changeset: 9eba43136cb5 Author: twisti Date: 2010-03-16 11:52 +0100 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/9eba43136cb5 6934494: JSR 292 MethodHandles adapters should be generated into their own CodeBlob Summary: Passing a null pointer to an InvokeDynamic function call should lead to a NullPointerException. Reviewed-by: kvn, never ! src/cpu/sparc/vm/stubRoutines_sparc.hpp ! src/cpu/x86/vm/stubGenerator_x86_32.cpp ! src/cpu/x86/vm/stubGenerator_x86_64.cpp ! src/cpu/x86/vm/stubRoutines_x86_32.hpp ! src/cpu/x86/vm/stubRoutines_x86_64.hpp ! src/share/vm/code/codeBlob.cpp ! src/share/vm/code/codeBlob.hpp ! src/share/vm/includeDB_core ! src/share/vm/prims/methodHandles.cpp ! src/share/vm/prims/methodHandles.hpp ! src/share/vm/runtime/init.cpp ! src/share/vm/runtime/sharedRuntime.cpp ! src/share/vm/runtime/sharedRuntime.hpp Changeset: 428a9c451986 Author: kvn Date: 2010-03-16 15:35 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/428a9c451986 6935466: new CodeCache flushing code is not guarded by the flag Summary: Add missing guard. Reviewed-by: never ! src/share/vm/compiler/compileBroker.cpp Changeset: fc2c71045ada Author: twisti Date: 2010-03-17 10:22 +0100 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/fc2c71045ada 6934966: JSR 292 add C1 logic for saved SP over MethodHandle calls Summary: The logic for x86 C1 to save the SP over MH calls is pretty straight forward but SPARC handles that differently. Reviewed-by: never, jrose ! src/cpu/sparc/vm/c1_FrameMap_sparc.hpp ! src/cpu/sparc/vm/c1_LIRAssembler_sparc.cpp ! src/cpu/x86/vm/c1_FrameMap_x86.hpp ! src/cpu/x86/vm/c1_LIRAssembler_x86.cpp ! src/share/vm/c1/c1_LIR.cpp ! src/share/vm/c1/c1_LIRAssembler.cpp ! src/share/vm/c1/c1_LIRAssembler.hpp Changeset: 2484f4d6a54e Author: kvn Date: 2010-03-17 10:47 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/2484f4d6a54e 6935535: String.indexOf() returns incorrect result on x86 with SSE4.2 Summary: Added missing counter decrement when substring search restarted. Reviewed-by: twisti ! src/cpu/x86/vm/assembler_x86.cpp + test/compiler/6935535/Test.java Changeset: c047da02984c Author: never Date: 2010-03-17 16:40 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/c047da02984c 6930043: C2: SIGSEGV in javasoft.sqe.tests.lang.arr017.arr01702.arr01702.loop_forw(II)I Reviewed-by: kvn ! src/share/vm/opto/loopTransform.cpp ! src/share/vm/opto/loopnode.hpp + test/compiler/6930043/Test6930043.java Changeset: 76c1d7d13ec5 Author: twisti Date: 2010-03-18 09:56 +0100 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/76c1d7d13ec5 6932091: JSR 292 x86 code cleanup Summary: Some code cleanups found during the JSR 292 SPARC port. Reviewed-by: kvn, never ! src/cpu/x86/vm/methodHandles_x86.cpp ! src/cpu/x86/vm/templateTable_x86_32.cpp ! src/cpu/x86/vm/templateTable_x86_64.cpp ! src/share/vm/c1/c1_LIRGenerator.cpp ! src/share/vm/prims/methodHandles.hpp ! src/share/vm/runtime/arguments.cpp Changeset: 97fe2cc98b1d Author: twisti Date: 2010-03-18 06:36 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/97fe2cc98b1d Merge Changeset: 6c94fe3c8df3 Author: trims Date: 2010-03-18 16:06 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/6c94fe3c8df3 Merge Changeset: 07226e9eab8f Author: trims Date: 2010-03-18 17:21 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/07226e9eab8f 6936366: Fork HS17 to HS18 - renumber Major and build numbers of JVM Summary: Update the Major and build numbers for HS18 fork Reviewed-by: jcoomes ! make/hotspot_version Changeset: 3deb84ecd19d Author: trims Date: 2010-03-23 20:28 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/3deb84ecd19d Added tag hs18-b01 for changeset 07226e9eab8f ! .hgtags Changeset: e7e7e36ccdb5 Author: trims Date: 2010-03-23 20:37 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/e7e7e36ccdb5 Merge ! .hgtags From kelly.ohair at sun.com Fri Mar 26 17:31:58 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 26 Mar 2010 17:31:58 +0000 Subject: hg: jdk7/build: Added tag jdk7-b87 for changeset 6b1069f53fbc Message-ID: <20100326173158.92E8B44491@hg.openjdk.java.net> Changeset: 82135c848d5f Author: mikejwre Date: 2010-03-25 15:05 -0700 URL: http://hg.openjdk.java.net/jdk7/build/rev/82135c848d5f Added tag jdk7-b87 for changeset 6b1069f53fbc ! .hgtags From kelly.ohair at sun.com Fri Mar 26 17:32:03 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 26 Mar 2010 17:32:03 +0000 Subject: hg: jdk7/build/corba: Added tag jdk7-b87 for changeset 09a41111a401 Message-ID: <20100326173205.E966A44492@hg.openjdk.java.net> Changeset: 39e14d2da687 Author: mikejwre Date: 2010-03-25 15:05 -0700 URL: http://hg.openjdk.java.net/jdk7/build/corba/rev/39e14d2da687 Added tag jdk7-b87 for changeset 09a41111a401 ! .hgtags From kelly.ohair at sun.com Fri Mar 26 17:33:49 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 26 Mar 2010 17:33:49 +0000 Subject: hg: jdk7/build/hotspot: Added tag jdk7-b87 for changeset e7e7e36ccdb5 Message-ID: <20100326173357.A592444493@hg.openjdk.java.net> Changeset: 465c39e1fb46 Author: mikejwre Date: 2010-03-25 15:05 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/465c39e1fb46 Added tag jdk7-b87 for changeset e7e7e36ccdb5 ! .hgtags From kelly.ohair at sun.com Fri Mar 26 17:39:39 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 26 Mar 2010 17:39:39 +0000 Subject: hg: jdk7/build/jaxp: Added tag jdk7-b87 for changeset 8b493f1aa136 Message-ID: <20100326173940.1663944496@hg.openjdk.java.net> Changeset: d8ebd1591003 Author: mikejwre Date: 2010-03-25 15:05 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jaxp/rev/d8ebd1591003 Added tag jdk7-b87 for changeset 8b493f1aa136 ! .hgtags From kelly.ohair at sun.com Fri Mar 26 17:39:44 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 26 Mar 2010 17:39:44 +0000 Subject: hg: jdk7/build/jaxws: Added tag jdk7-b87 for changeset 3febd6fab2ac Message-ID: <20100326173944.AE8FD44497@hg.openjdk.java.net> Changeset: 8c666f8f3565 Author: mikejwre Date: 2010-03-25 15:05 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jaxws/rev/8c666f8f3565 Added tag jdk7-b87 for changeset 3febd6fab2ac ! .hgtags From kelly.ohair at sun.com Fri Mar 26 17:39:50 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 26 Mar 2010 17:39:50 +0000 Subject: hg: jdk7/build/jdk: Added tag jdk7-b87 for changeset 2cafbbe9825e Message-ID: <20100326174018.DFAE044499@hg.openjdk.java.net> Changeset: b3c69282f6d3 Author: mikejwre Date: 2010-03-25 15:05 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/b3c69282f6d3 Added tag jdk7-b87 for changeset 2cafbbe9825e ! .hgtags From kelly.ohair at sun.com Fri Mar 26 17:46:05 2010 From: kelly.ohair at sun.com (kelly.ohair at sun.com) Date: Fri, 26 Mar 2010 17:46:05 +0000 Subject: hg: jdk7/build/langtools: Added tag jdk7-b87 for changeset 409db93d19c0 Message-ID: <20100326174612.C6A984449B@hg.openjdk.java.net> Changeset: f9b5d4867a26 Author: mikejwre Date: 2010-03-25 15:05 -0700 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/f9b5d4867a26 Added tag jdk7-b87 for changeset 409db93d19c0 ! .hgtags From kelly.ohair at oracle.com Fri Mar 26 16:06:37 2010 From: kelly.ohair at oracle.com (Kelly O'Hair) Date: Fri, 26 Mar 2010 09:06:37 -0700 Subject: ant 1.7.1 and make 3.81 Message-ID: <9DB28096-E0DB-4776-8324-E23FBBE83230@oracle.com> It appears that a problem with ant 1.7.1 can potentially impact the open jdk builds, see https://issues.apache.org/bugzilla/show_bug.cgi?id=43114 for more information. The potential for missing package-info.class files seems high, and the latest jaxws sources run into this problem. Since I hate to move backwards (to 1.7.0) I'd like to propose that the jdk builds move to ant 1.8.0 as soon as possible, and that the sanity check be changed to require 1.8.0. (openjdk6, openjdk7 and jdk7). While I'm there, I'd like to also advance the 'make' version required to 3.81. I suspect that the make 3.81 requirement is no big deal for the Linux people, but may complicate lives a little for Windows and Solaris people. The ant 1.8.0 issue may be more complicated for Linux users, since it likely will not be available in rpm/deb package form right away, but the download and install is trivial: http://archive.apache.org/dist/ant/binaries/apache-ant-1.8.0-bin.zip Sorry about this disturbance, but this seems like the best course. -kto From ahughes at redhat.com Sun Mar 28 13:03:04 2010 From: ahughes at redhat.com (Andrew John Hughes) Date: Sun, 28 Mar 2010 13:03:04 +0000 Subject: ant 1.7.1 and make 3.81 In-Reply-To: <9DB28096-E0DB-4776-8324-E23FBBE83230@oracle.com> References: <9DB28096-E0DB-4776-8324-E23FBBE83230@oracle.com> Message-ID: <17c6771e1003280603m37561c92l9f40aac9afdd0962@mail.gmail.com> On 26 March 2010 16:06, Kelly O'Hair wrote: > > It appears that a problem with ant 1.7.1 can potentially impact the open jdk > builds, see > https://issues.apache.org/bugzilla/show_bug.cgi?id=43114 for more > information. > The potential for missing package-info.class files seems high, and the > latest jaxws > sources run into this problem. > We've seen this issue on Fedora 10 and 11. > Since I hate to move backwards (to 1.7.0) I'd like to propose that the jdk > builds move > to ant 1.8.0 as soon as possible, and that the sanity check be changed to > require 1.8.0. > (openjdk6, openjdk7 and jdk7). > While I'm there, I'd like to also advance the 'make' version required to > 3.81. Can we not just blacklist 1.7.1 as suggested in the bug? Personally, you may not want to move to 1.7.0 but forcing 1.8 is going to be difficult as it's not yet in enough distros. That said, by the time this filters through to OpenJDK6, it may be. Is there no way of working around 1.7.1 on our side? > > I suspect that the make 3.81 requirement is no big deal for the Linux > people, but may > complicate lives a little for Windows and Solaris people. > GNU Make 3.81 Copyright (C) 2006 Free Software Foundation, Inc. So pretty old in computing terms. > The ant 1.8.0 issue may be more complicated for Linux users, since it likely > will not > be available in rpm/deb package form right away, but the download and > install is trivial: > ?http://archive.apache.org/dist/ant/binaries/apache-ant-1.8.0-bin.zip > Picking up some random binary from the web is not possible for building distro packages. Ant 1.8 needs to be packaged and available as a dependency. As I said, the issue may be moot by the time this filters through to something actually being packaged though. For OpenJDK6, I think it should wait until after the next build drop (b19) and go in b20. > Sorry about this disturbance, but this seems like the best course. > > -kto > -- Andrew :-) Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Support Free Java! Contribute to GNU Classpath and the OpenJDK http://www.gnu.org/software/classpath http://openjdk.java.net PGP Key: 94EFD9D8 (http://subkeys.pgp.net) Fingerprint: F8EF F1EA 401E 2E60 15FA 7927 142C 2591 94EF D9D8 From Dalibor.Topic at Sun.COM Sun Mar 28 16:34:16 2010 From: Dalibor.Topic at Sun.COM (Dalibor Topic) Date: Sun, 28 Mar 2010 18:34:16 +0200 Subject: ant 1.7.1 and make 3.81 In-Reply-To: <17c6771e1003280603m37561c92l9f40aac9afdd0962@mail.gmail.com> References: <9DB28096-E0DB-4776-8324-E23FBBE83230@oracle.com> <17c6771e1003280603m37561c92l9f40aac9afdd0962@mail.gmail.com> Message-ID: <4BAF8508.5000504@sun.com> Andrew John Hughes wrote: > On 26 March 2010 16:06, Kelly O'Hair wrote: >> It appears that a problem with ant 1.7.1 can potentially impact the open jdk >> builds, see >> https://issues.apache.org/bugzilla/show_bug.cgi?id=43114 for more >> information. >> The potential for missing package-info.class files seems high, and the >> latest jaxws >> sources run into this problem. >> > > We've seen this issue on Fedora 10 and 11. Since you don't list Fedora 12, I'm curious - has the corresponding patch from ant been backported to the Fedora 12 ant packages? > Can we not just blacklist 1.7.1 as suggested in the bug? Personally, > you may not want to move to 1.7.0 but forcing 1.8 is going to be > difficult as it's not yet in enough distros. That said, by the time > this filters through to OpenJDK6, it may be. > > Is there no way of working around 1.7.1 on our side? I assume the distributions that ship a patched ant 1.7.1 package could change the check to accept 1.7.1. Would that work? cheers, dalibor topic -- ******************************************************************* Dalibor Topic Tel: (+49 40) 23 646 738 Java F/OSS Ambassador AIM: robiladonaim Sun Microsystems GmbH Mobile: (+49 177) 2664 192 Nagelsweg 55 http://openjdk.java.net D-20097 Hamburg mailto:Dalibor.Topic at sun.com Sitz der Gesellschaft: Sonnenallee 1, D-85551 Kirchheim-Heimstetten Amtsgericht M?nchen: HRB 161028 Gesch?ftsf?hrer: Thomas Schr?der From ahughes at redhat.com Sun Mar 28 19:49:19 2010 From: ahughes at redhat.com (Andrew John Hughes) Date: Sun, 28 Mar 2010 19:49:19 +0000 Subject: ant 1.7.1 and make 3.81 In-Reply-To: <4BAF8508.5000504@sun.com> References: <9DB28096-E0DB-4776-8324-E23FBBE83230@oracle.com> <17c6771e1003280603m37561c92l9f40aac9afdd0962@mail.gmail.com> <4BAF8508.5000504@sun.com> Message-ID: <17c6771e1003281249w4cea619enf3cd474db9287729@mail.gmail.com> On 28 March 2010 16:34, Dalibor Topic wrote: > Andrew John Hughes wrote: >> On 26 March 2010 16:06, Kelly O'Hair wrote: >>> It appears that a problem with ant 1.7.1 can potentially impact the open jdk >>> builds, see >>> https://issues.apache.org/bugzilla/show_bug.cgi?id=43114 for more >>> information. >>> The potential for missing package-info.class files seems high, and the >>> latest jaxws >>> sources run into this problem. >>> >> >> We've seen this issue on Fedora 10 and 11. > > Since you don't list Fedora 12, I'm curious - has the corresponding patch > from ant been backported to the Fedora 12 ant packages? > Sorry, I haven't seen the issue myself nor do I know what's in the various Ant packages. I've pointed Pavel at this thread; maybe he'll know more. >> Can we not just blacklist 1.7.1 as suggested in the bug? ?Personally, >> you may not want to move to 1.7.0 but forcing 1.8 is going to be >> difficult as it's not yet in enough distros. ?That said, by the time >> this filters through to OpenJDK6, it may be. >> >> Is there no way of working around 1.7.1 on our side? > > I assume the distributions that ship a patched ant 1.7.1 package could > change the check to accept 1.7.1. Would that work? > 1.8 has its own issues, as have been posted on the list too. I think blocking anything but 1.8 is far too drastic, especially in the immediate future. > cheers, > dalibor topic > > -- > ******************************************************************* > Dalibor Topic ? ? ? ? ? ? ? ? ? Tel: (+49 40) 23 646 738 > Java F/OSS Ambassador ? ? ? ? ? AIM: robiladonaim > Sun Microsystems GmbH ? ? ? ? ? Mobile: (+49 177) 2664 192 > Nagelsweg 55 ? ? ? ? ? ? ? ? ? ?http://openjdk.java.net > D-20097 Hamburg ? ? ? ? ? ? ? ? mailto:Dalibor.Topic at sun.com > Sitz der Gesellschaft: Sonnenallee 1, D-85551 Kirchheim-Heimstetten > Amtsgericht M?nchen: HRB 161028 > Gesch?ftsf?hrer: Thomas Schr?der > > > -- Andrew :-) Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Support Free Java! Contribute to GNU Classpath and the OpenJDK http://www.gnu.org/software/classpath http://openjdk.java.net PGP Key: 94EFD9D8 (http://subkeys.pgp.net) Fingerprint: F8EF F1EA 401E 2E60 15FA 7927 142C 2591 94EF D9D8 From martinrb at google.com Mon Mar 29 22:57:40 2010 From: martinrb at google.com (Martin Buchholz) Date: Mon, 29 Mar 2010 15:57:40 -0700 Subject: Unhappy about test/ProblemList.txt Message-ID: <1ccfd1c11003291557n530535ek51b9130fa44bc6b4@mail.gmail.com> I was surprised to see some of my own favorite tests in test/ProblemList.txt and am not happy about that. I fear that we have regression tests finding actual regressions and that instead of this causing the guilty changeset to be reverted, (or better yet never checked in), innocent tests are just getting added to the blacklist. At least please un-blacklist my pet ProcessBuilder/Basic.java. Martin From ptisnovs at redhat.com Mon Mar 29 16:56:44 2010 From: ptisnovs at redhat.com (Pavel Tisnovsky) Date: Mon, 29 Mar 2010 18:56:44 +0200 Subject: ant 1.7.1 and make 3.81 In-Reply-To: <17c6771e1003281249w4cea619enf3cd474db9287729@mail.gmail.com> References: <9DB28096-E0DB-4776-8324-E23FBBE83230@oracle.com> <17c6771e1003280603m37561c92l9f40aac9afdd0962@mail.gmail.com> <4BAF8508.5000504@sun.com> <17c6771e1003281249w4cea619enf3cd474db9287729@mail.gmail.com> Message-ID: <4BB0DBCC.90101@redhat.com> Andrew John Hughes wrote: > On 28 March 2010 16:34, Dalibor Topic wrote: >> Andrew John Hughes wrote: >>> On 26 March 2010 16:06, Kelly O'Hair wrote: >>>> It appears that a problem with ant 1.7.1 can potentially impact the open jdk >>>> builds, see >>>> https://issues.apache.org/bugzilla/show_bug.cgi?id=43114 for more >>>> information. >>>> The potential for missing package-info.class files seems high, and the >>>> latest jaxws >>>> sources run into this problem. >>>> >>> We've seen this issue on Fedora 10 and 11. >> Since you don't list Fedora 12, I'm curious - has the corresponding patch >> from ant been backported to the Fedora 12 ant packages? >> > > Sorry, I haven't seen the issue myself nor do I know what's in the > various Ant packages. I've pointed Pavel at this thread; maybe he'll > know more. I saw this issue on these combinations of Fedora systems and Ant versions: Fedora 10 (i386 | x86_64 | ppc) Apache Ant version 1.7.1 compiled on October 1 2008 Fedora 11 (i386 | x86_64) Apache Ant version 1.7.1 compiled on February 24 2009 Fedora 12 (i386 | x86_64) Apache Ant version 1.7.1 compiled on August 13 2009 Cheers Pavel Tisnovsky > >>> Can we not just blacklist 1.7.1 as suggested in the bug? Personally, >>> you may not want to move to 1.7.0 but forcing 1.8 is going to be >>> difficult as it's not yet in enough distros. That said, by the time >>> this filters through to OpenJDK6, it may be. >>> >>> Is there no way of working around 1.7.1 on our side? >> I assume the distributions that ship a patched ant 1.7.1 package could >> change the check to accept 1.7.1. Would that work? >> > > 1.8 has its own issues, as have been posted on the list too. I think > blocking anything but 1.8 is far too drastic, especially in the > immediate future. > >> cheers, >> dalibor topic >> >> -- >> ******************************************************************* >> Dalibor Topic Tel: (+49 40) 23 646 738 >> Java F/OSS Ambassador AIM: robiladonaim >> Sun Microsystems GmbH Mobile: (+49 177) 2664 192 >> Nagelsweg 55 http://openjdk.java.net >> D-20097 Hamburg mailto:Dalibor.Topic at sun.com >> Sitz der Gesellschaft: Sonnenallee 1, D-85551 Kirchheim-Heimstetten >> Amtsgericht M?nchen: HRB 161028 >> Gesch?ftsf?hrer: Thomas Schr?der >> >> >> > > > From kelly.ohair at sun.com Tue Mar 30 00:14:43 2010 From: kelly.ohair at sun.com (Kelly O'Hair) Date: Mon, 29 Mar 2010 17:14:43 -0700 Subject: Unhappy about test/ProblemList.txt In-Reply-To: <1ccfd1c11003291557n530535ek51b9130fa44bc6b4@mail.gmail.com> References: <1ccfd1c11003291557n530535ek51b9130fa44bc6b4@mail.gmail.com> Message-ID: <6A87C6B9-8CEC-48A3-A162-30F3B4F1D0EA@sun.com> Jonathan is working on some changes to jtreg that may allow many of the tests on the ProblemList to be removed, I am hopeful anyway. Once we have that new jtreg available, trimming this list down is next. I admit to being a little quick to add some tests to the list, it was a somewhat frustrating experience to isolate these things when looking at all 12 platforms/vms. I think in this case it was timing out on Solaris sparc and appeared to be a stress test. This test is already marked othervm, so I can only assume that it is very close to the default timeout, looks like a fairly complex testcase, but I see no reason to not add it back in, maybe I can bump up the timeout on this test. And I had no idea any of these were pets, I assumed they were all wild. ;^) -kto On Mar 29, 2010, at 3:57 PM, Martin Buchholz wrote: > I was surprised to see some of my own favorite tests > in test/ProblemList.txt and am not happy about that. > I fear that we have regression tests finding actual > regressions and that instead of this causing the > guilty changeset to be reverted, (or better yet > never checked in), innocent tests are just > getting added to the blacklist. > > At least please un-blacklist my pet ProcessBuilder/Basic.java. > > Martin From martinrb at google.com Tue Mar 30 02:02:46 2010 From: martinrb at google.com (Martin Buchholz) Date: Mon, 29 Mar 2010 19:02:46 -0700 Subject: Unhappy about test/ProblemList.txt In-Reply-To: <6A87C6B9-8CEC-48A3-A162-30F3B4F1D0EA@sun.com> References: <1ccfd1c11003291557n530535ek51b9130fa44bc6b4@mail.gmail.com> <6A87C6B9-8CEC-48A3-A162-30F3B4F1D0EA@sun.com> Message-ID: <1ccfd1c11003291902h7f60c67ewb5436422d55697cd@mail.gmail.com> On Mon, Mar 29, 2010 at 17:14, Kelly O'Hair wrote: > > Jonathan is working on some changes to jtreg that may allow many of the > tests on the ProblemList to be removed, I am hopeful anyway. > Once we have that new jtreg available, trimming this list down is next. > > I admit to being a little quick to add some tests to the list, it was a > somewhat > frustrating experience to isolate these things when looking at all 12 > platforms/vms. > > I think in this case it was timing out on Solaris sparc and appeared to be a > stress test. > This test is already marked othervm, so I can only assume that it is very > close > to the default timeout, looks like a fairly complex testcase, but I see > no reason to not add it back in, maybe I can bump up the timeout on this > test. ProcessBuilder/Basic.java has a lot of infrastructure for testing the process infrastructure, so it has grown organically to house many different individual tests. Consider it a test suite! Martin From martinrb at google.com Tue Mar 30 02:04:39 2010 From: martinrb at google.com (Martin Buchholz) Date: Mon, 29 Mar 2010 19:04:39 -0700 Subject: Unhappy about test/ProblemList.txt In-Reply-To: <6A87C6B9-8CEC-48A3-A162-30F3B4F1D0EA@sun.com> References: <1ccfd1c11003291557n530535ek51b9130fa44bc6b4@mail.gmail.com> <6A87C6B9-8CEC-48A3-A162-30F3B4F1D0EA@sun.com> Message-ID: <1ccfd1c11003291904u3ee4fe68kea9ee1c6d198ba36@mail.gmail.com> On Mon, Mar 29, 2010 at 17:14, Kelly O'Hair wrote: > > Jonathan is working on some changes to jtreg that may allow many of the > tests on the ProblemList to be removed, I am hopeful anyway. > Once we have that new jtreg available, trimming this list down is next. > > I admit to being a little quick to add some tests to the list, it was a > somewhat > frustrating experience to isolate these things when looking at all 12 > platforms/vms. > > I think in this case it was timing out on Solaris sparc and appeared to be a > stress test. > This test is already marked othervm, so I can only assume that it is very > close > to the default timeout, If increasing the timeout will make failures go away, then that is perfectly fine. You can reasonably increase the timeout up to 1 hour. Martin From kelly.ohair at sun.com Tue Mar 30 16:11:54 2010 From: kelly.ohair at sun.com (Kelly O'Hair) Date: Tue, 30 Mar 2010 09:11:54 -0700 Subject: Unhappy about test/ProblemList.txt In-Reply-To: <1ccfd1c11003291904u3ee4fe68kea9ee1c6d198ba36@mail.gmail.com> References: <1ccfd1c11003291557n530535ek51b9130fa44bc6b4@mail.gmail.com> <6A87C6B9-8CEC-48A3-A162-30F3B4F1D0EA@sun.com> <1ccfd1c11003291904u3ee4fe68kea9ee1c6d198ba36@mail.gmail.com> Message-ID: On Mar 29, 2010, at 7:04 PM, Martin Buchholz wrote: > On Mon, Mar 29, 2010 at 17:14, Kelly O'Hair > wrote: >> >> Jonathan is working on some changes to jtreg that may allow many of >> the >> tests on the ProblemList to be removed, I am hopeful anyway. >> Once we have that new jtreg available, trimming this list down is >> next. >> >> I admit to being a little quick to add some tests to the list, it >> was a >> somewhat >> frustrating experience to isolate these things when looking at all 12 >> platforms/vms. >> >> I think in this case it was timing out on Solaris sparc and >> appeared to be a >> stress test. >> This test is already marked othervm, so I can only assume that it >> is very >> close >> to the default timeout, > > If increasing the timeout will make failures go away, then that is > perfectly fine. > You can reasonably increase the timeout up to 1 hour. I'm not sure having this one test take an hour is fair to the other 1,000's of tests. It it can't run in under 5min, I'd question the test, maybe it's sleeping or blocking too much if that is the case. Part of the goal in all this was to provide wide solid test coverage in a reasonable amount of time, but if we allow many tests to run for an hour, that's a bit of a problem. Regardless, I don't think this test takes an hour, I'll put a 10min timeout on it and see how that goes. But I suspect it will run in well under 5min. -kto > > Martin From jonathan.gibbons at oracle.com Tue Mar 30 16:26:38 2010 From: jonathan.gibbons at oracle.com (Jonathan Gibbons) Date: Tue, 30 Mar 2010 09:26:38 -0700 Subject: Unhappy about test/ProblemList.txt In-Reply-To: References: <1ccfd1c11003291557n530535ek51b9130fa44bc6b4@mail.gmail.com> <6A87C6B9-8CEC-48A3-A162-30F3B4F1D0EA@sun.com> <1ccfd1c11003291904u3ee4fe68kea9ee1c6d198ba36@mail.gmail.com> Message-ID: <4BB2263E.7050008@oracle.com> Kelly O'Hair wrote: > > On Mar 29, 2010, at 7:04 PM, Martin Buchholz wrote: > >> On Mon, Mar 29, 2010 at 17:14, Kelly O'Hair wrote: >>> >>> Jonathan is working on some changes to jtreg that may allow many of the >>> tests on the ProblemList to be removed, I am hopeful anyway. >>> Once we have that new jtreg available, trimming this list down is next. >>> >>> I admit to being a little quick to add some tests to the list, it was a >>> somewhat >>> frustrating experience to isolate these things when looking at all 12 >>> platforms/vms. >>> >>> I think in this case it was timing out on Solaris sparc and appeared >>> to be a >>> stress test. >>> This test is already marked othervm, so I can only assume that it is >>> very >>> close >>> to the default timeout, >> >> If increasing the timeout will make failures go away, then that is >> perfectly fine. >> You can reasonably increase the timeout up to 1 hour. > > I'm not sure having this one test take an hour is fair to the other > 1,000's of tests. > It it can't run in under 5min, I'd question the test, maybe it's > sleeping or blocking > too much if that is the case. > Part of the goal in all this was to provide wide solid test coverage > in a reasonable amount > of time, but if we allow many tests to run for an hour, that's a bit > of a problem. > > Regardless, I don't think this test takes an hour, I'll put a 10min > timeout on it > and see how that goes. But I suspect it will run in well under 5min. > > -kto > >> >> Martin > I think it is worth noting that just because a test is on the ProblemList does not mean it does not deserve to be run. It just means it needs more TLC than other tests, for one of any number of reasons. Since we have (regrettably) historically had trouble running all the tests and getting all them to pass, the approach is to separate the set of tests into two groups -- those that are well behaved and run in reasonable time with reasonable resources, and reliably pass on a good JDK -- and the rest. The well behaved tests are good candidates for automated running on systems like JPRT or Hudson, and failures should be indicative of a fault in what is being tested. The other tests should still be run, but by their nature, they are harder to run, and may require manual checking of the results to see if failures are real or spurious. The real problem, in all of this, is the dwindling resources to expend on such TLC :-( -- Jon From martinrb at google.com Tue Mar 30 20:17:46 2010 From: martinrb at google.com (Martin Buchholz) Date: Tue, 30 Mar 2010 13:17:46 -0700 Subject: Unhappy about test/ProblemList.txt In-Reply-To: References: <1ccfd1c11003291557n530535ek51b9130fa44bc6b4@mail.gmail.com> <6A87C6B9-8CEC-48A3-A162-30F3B4F1D0EA@sun.com> <1ccfd1c11003291904u3ee4fe68kea9ee1c6d198ba36@mail.gmail.com> Message-ID: <1ccfd1c11003301317k706dfeabofde13e54da008630@mail.gmail.com> On Tue, Mar 30, 2010 at 09:11, Kelly O'Hair wrote: > > On Mar 29, 2010, at 7:04 PM, Martin Buchholz wrote: > >> On Mon, Mar 29, 2010 at 17:14, Kelly O'Hair wrote: >>> >>> Jonathan is working on some changes to jtreg that may allow many of the >>> tests on the ProblemList to be removed, I am hopeful anyway. >>> Once we have that new jtreg available, trimming this list down is next. >>> >>> I admit to being a little quick to add some tests to the list, it was a >>> somewhat >>> frustrating experience to isolate these things when looking at all 12 >>> platforms/vms. >>> >>> I think in this case it was timing out on Solaris sparc and appeared to >>> be a >>> stress test. >>> This test is already marked othervm, so I can only assume that it is very >>> close >>> to the default timeout, >> >> If increasing the timeout will make failures go away, then that is >> perfectly fine. >> You can reasonably increase the timeout up to 1 hour. > > I'm not sure having this one test take an hour is fair to the other 1,000's > of tests. > It it can't run in under 5min, I'd question the test, maybe it's sleeping or > blocking > too much if that is the case. > Part of the goal in all this was to provide wide solid test coverage in a > reasonable amount > of time, but if we allow many tests to run for an hour, that's a bit of a > problem. > > Regardless, I don't think this test takes an hour, I'll put a 10min timeout > on it > and see how that goes. But I suspect it will run in well under 5min. It takes (only) 8 sec to run on my linux box, and tests the entire Process subsystem, which requires actually creating many subprocesses, has regression tests for many bugids, and reports Passed = 2668 Not bad for one test file. Martin