hg: jigsaw/jake/nashorn: 2 new changesets

2016-09-08 Thread mandy . chung
Changeset: 61a07a6d70e4
Author:lana
Date:  2016-09-08 21:11 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/61a07a6d70e4

Added tag jdk-9+135 for changeset cb00d5ef023a

! .hgtags

Changeset: ba56297704e6
Author:mchung
Date:  2016-09-08 17:42 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/nashorn/rev/ba56297704e6

Merge

! .hgtags



hg: jigsaw/jake/jdk: 73 new changesets

2016-09-08 Thread mandy . chung
Changeset: 5e0ff917eab1
Author:amlu
Date:  2016-08-30 09:31 +0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/5e0ff917eab1

8163934: Remove intermittent key from java/lang/ProcessBuilder/Zombies.java
Reviewed-by: darcy

! test/java/lang/ProcessBuilder/Zombies.java

Changeset: 7f730240145e
Author:amlu
Date:  2016-08-30 09:36 +0800
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/7f730240145e

8164545: Mark java/net/URLPermission/nstest/lookup.sh as intermittently failing
Reviewed-by: dfuchs

! test/java/net/URLPermission/nstest/lookup.sh

Changeset: f7026b128452
Author:okutsu
Date:  2016-08-30 14:16 +0900
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/f7026b128452

8157792: After Integrating tzdata2016d the 
test/sun/util/calendar/zi/TestZoneInfo310.java fails for "Asia/Oral" and 
"Asia/Qyzylorda" Timezones
Reviewed-by: peytoia

! test/sun/util/calendar/zi/TestZoneInfo310.java
! test/sun/util/calendar/zi/Zoneinfo.java

Changeset: 556549260289
Author:igerasim
Date:  2016-08-30 15:31 +0300
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/556549260289

6474807: (smartcardio) CardTerminal.connect() throws CardException instead of 
CardNotPresentException
Reviewed-by: valeriep

! src/java.smartcardio/share/classes/sun/security/smartcardio/TerminalImpl.java

Changeset: d55a21a736b7
Author:shurailine
Date:  2016-08-30 14:30 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/d55a21a736b7

8164859: Fix module dependences in java/text tests
Reviewed-by: naoto

! test/java/text/Bidi/BidiConformance.java
! test/java/text/Bidi/BidiEmbeddingTest.java
! test/java/text/Bidi/Bug7042148.java
! test/java/text/Bidi/Bug7051769.java
! test/java/text/BreakIterator/NewVSOld_th_TH.java
! test/java/text/Collator/APITest.java
! test/java/text/Collator/CollationKeyTest.java
! test/java/text/Collator/DanishTest.java
! test/java/text/Collator/FinnishTest.java
! test/java/text/Collator/FrenchTest.java
! test/java/text/Collator/G7Test.java
! test/java/text/Collator/JapaneseTest.java
! test/java/text/Collator/KoreanTest.java
! test/java/text/Collator/Regression.java
! test/java/text/Collator/ThaiTest.java
! test/java/text/Collator/TurkishTest.java
! test/java/text/Collator/VietnameseTest.java
! test/java/text/Format/DateFormat/Bug4823811.java
! test/java/text/Format/DateFormat/Bug6683975.java
! test/java/text/Format/DateFormat/Bug8139572.java
! test/java/text/Format/DateFormat/ContextMonthNamesTest.java
! test/java/text/Format/DateFormat/DateFormatTest.java
! test/java/text/Format/DateFormat/LocaleDateFormats.java
! test/java/text/Format/DateFormat/NonGregorianFormatTest.java
! test/java/text/Format/DateFormat/bug4117335.java
! test/java/text/Format/MessageFormat/LargeMessageFormat.java
! test/java/text/Format/NumberFormat/Bug8132125.java
! test/java/text/Format/NumberFormat/CurrencyFormat.java
! test/java/text/Format/NumberFormat/IntlTestNumberFormatAPI.java
! test/java/text/Format/NumberFormat/NumberRegression.java
! test/java/text/Format/NumberFormat/NumberTest.java

Changeset: ea6b16200114
Author:mchung
Date:  2016-08-30 17:48 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/ea6b16200114

8160851: Remove old launcher module-related options
Reviewed-by: jjg, alanb

! src/java.base/share/native/libjli/java.c

Changeset: 000459da7aa8
Author:redestad
Date:  2016-08-31 14:20 +0200
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/000459da7aa8

8164858: Enable build-time use of java.lang.invoke resolve tracing
Reviewed-by: erikj, vlivanov

! make/GenerateClasslist.gmk
! 
src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/GenerateJLIClassesPlugin.java
! src/jdk.jlink/share/classes/jdk/tools/jlink/resources/plugins.properties
! test/tools/jlink/plugins/GenerateJLIClassesPluginTest.java

Changeset: b46447382c84
Author:ssahoo
Date:  2016-08-31 08:34 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/b46447382c84

8015595: Test sun/security/krb5/auto/Unreachable.java fails with Timeout error
Summary: Unreachable.java was getting timeout due to PortUnreachableException 
was not thrown
Reviewed-by: weijun

! test/ProblemList.txt
! test/sun/security/krb5/auto/Unreachable.java
- test/sun/security/krb5/auto/unreachable.krb5.conf

Changeset: 077a8cd3df68
Author:ssahoo
Date:  2016-08-31 08:44 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/077a8cd3df68

8164922: sun/security/provider/SecureRandom/AutoReseed.java failed with timeout 
in Ubuntu Linux.
Summary: The test timeout waiting to get seed in an exhausted Linux platform.
Reviewed-by: weijun

! test/sun/security/provider/SecureRandom/AutoReseed.java

Changeset: 6f390eafc676
Author:shurailine
Date:  2016-08-31 09:46 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/6f390eafc676

8164982: Fix legal notices in java/lang, java/net, java/util tests.
Reviewed-by: darcy, iris

! 

hg: jigsaw/jake/jaxws: 2 new changesets

2016-09-08 Thread mandy . chung
Changeset: 09ec13a99f50
Author:lana
Date:  2016-09-08 21:11 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/09ec13a99f50

Added tag jdk-9+135 for changeset 22631824f551

! .hgtags

Changeset: 09dff41baed2
Author:mchung
Date:  2016-09-08 17:40 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxws/rev/09dff41baed2

Merge




hg: jigsaw/jake/hotspot: 52 new changesets

2016-09-08 Thread mandy . chung
Changeset: 65a32a1c9bf9
Author:mchung
Date:  2016-08-30 17:48 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/65a32a1c9bf9

8160851: Remove old launcher module-related options
Reviewed-by: alanb

! test/runtime/Unsafe/NestedUnsafe.java

Changeset: e3a19a55f062
Author:dsamersoff
Date:  2016-08-18 12:10 +0300
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/e3a19a55f062

8151345: compiler/codecache/jmx/PeakUsageTest.java is failing on jdk9/dev for 
JPRT  -testset hotspot
Reviewed-by: sla, dsamersoff

! test/compiler/codecache/jmx/CodeCacheUtils.java
! test/compiler/codecache/jmx/PeakUsageTest.java

Changeset: 1657775e680a
Author:dsamersoff
Date:  2016-08-18 14:07 +0300
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/1657775e680a

8157236: attach on ARMv7 fails with 
com.sun.tools.attach.AttachNotSupportedException: Unable to open socket file
Summary: Add more diagnostic to attach code
Reviewed-by: dholmes, alanb

! src/os/aix/vm/attachListener_aix.cpp
! src/os/bsd/vm/attachListener_bsd.cpp
! src/os/linux/vm/attachListener_linux.cpp
! src/os/solaris/vm/attachListener_solaris.cpp
! src/share/vm/logging/logTag.hpp

Changeset: e99e410e78e1
Author:dsamersoff
Date:  2016-08-18 11:11 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/e99e410e78e1

Merge


Changeset: 0aadda927071
Author:coleenp
Date:  2016-08-18 10:47 -0400
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/0aadda927071

8037138: x86: problem with JVMTI breakpoint
Summary: do aload(0) after rewriting aload bytecodes to fast version for 
frequent pairs.
Reviewed-by: dlong, dholmes, dcubed

! src/cpu/aarch64/vm/templateTable_aarch64.cpp
! src/cpu/sparc/vm/templateTable_sparc.cpp
! src/cpu/x86/vm/templateTable_x86.cpp

Changeset: 6c29c7f73ca1
Author:dholmes
Date:  2016-08-18 21:37 -0400
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/6c29c7f73ca1

8152849: share/vm/runtime/mutex.cpp:1161 
assert(((uintptr_t(_owner))|(uintptr_t(_LockWord.FullWord))|(uintptr_t(_EntryList))|(uintptr_t(_WaitSet))|(uintptr_t(_OnDeck)))
 == 0) failed
Reviewed-by: dcubed

! src/share/vm/runtime/mutex.cpp

Changeset: af4f7418af3e
Author:ysuenaga
Date:  2016-08-19 01:20 -0400
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/af4f7418af3e

8164319: CLHSDB dumpcodecache throws StackOverflowError
Reviewed-by: dholmes, dsamersoff

! src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/code/CodeBlob.java

Changeset: b2c07de32ca7
Author:erikj
Date:  2016-08-19 16:02 +0200
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/b2c07de32ca7

8164297: Jtreg test exeinvoke fails to link on Ubuntu
Reviewed-by: tbell, dholmes

! make/test/JtregNative.gmk

Changeset: 1cbffa2beba6
Author:ctornqvi
Date:  2016-08-19 10:06 -0400
URL:   http://hg.openjdk.java.net/jigsaw/jake/hotspot/rev/1cbffa2beba6

8157957: ClassNotFoundException: jdk.test.lib.JDKToolFinder
Reviewed-by: coleenp, gtriantafill, mseledtsov, iignatyev, dholmes, dsamersoff

! test/compiler/arguments/BMISupportedCPUTest.java
! test/compiler/arguments/BMIUnsupportedCPUTest.java
! test/compiler/arguments/CheckCICompilerCount.java
! test/compiler/arguments/CheckCompileThresholdScaling.java
! test/compiler/arguments/TestUseBMI1InstructionsOnSupportedCPU.java
! test/compiler/arguments/TestUseBMI1InstructionsOnUnsupportedCPU.java
! test/compiler/arguments/TestUseCountLeadingZerosInstructionOnSupportedCPU.java
! 
test/compiler/arguments/TestUseCountLeadingZerosInstructionOnUnsupportedCPU.java
! 
test/compiler/arguments/TestUseCountTrailingZerosInstructionOnSupportedCPU.java
! 
test/compiler/arguments/TestUseCountTrailingZerosInstructionOnUnsupportedCPU.java
! test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java
! test/compiler/c2/PolynomialRoot.java
! test/compiler/c2/Test5057225.java
! test/compiler/c2/Test6603011.java
! test/compiler/c2/Test6800154.java
! test/compiler/c2/Test6805724.java
! test/compiler/c2/Test6857159.java
! test/compiler/c2/Test7068051.java
! test/compiler/c2/Test7177917.java
! test/compiler/c2/cr6589834/Test_ia32.java
! test/compiler/c2/stemmer/Stemmer.java
! test/compiler/calls/fromCompiled/CompiledInvokeDynamic2CompiledTest.java
! test/compiler/calls/fromCompiled/CompiledInvokeDynamic2InterpretedTest.java
! test/compiler/calls/fromCompiled/CompiledInvokeDynamic2NativeTest.java
! test/compiler/calls/fromCompiled/CompiledInvokeInterface2CompiledTest.java
! test/compiler/calls/fromCompiled/CompiledInvokeInterface2InterpretedTest.java
! test/compiler/calls/fromCompiled/CompiledInvokeInterface2NativeTest.java
! test/compiler/calls/fromCompiled/CompiledInvokeSpecial2CompiledTest.java
! test/compiler/calls/fromCompiled/CompiledInvokeSpecial2InterpretedTest.java
! test/compiler/calls/fromCompiled/CompiledInvokeSpecial2NativeTest.java
! test/compiler/calls/fromCompiled/CompiledInvokeStatic2CompiledTest.java
! 

hg: jigsaw/jake/jaxp: 7 new changesets

2016-09-08 Thread mandy . chung
Changeset: 2bb277192648
Author:mchung
Date:  2016-08-30 17:49 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/2bb277192648

8160851: Remove old launcher module-related options
Reviewed-by: jjg, alanb

! test/javax/xml/jaxp/module/ServiceProviderTest/BasicModularXMLParserTest.java

Changeset: 4561cb852f38
Author:joehw
Date:  2016-08-31 14:33 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/4561cb852f38

8162431: CatalogUriResolver with circular/self referencing catalog file is not 
throwing CatalogException as expected
Reviewed-by: lancea

! src/java.xml/share/classes/javax/xml/catalog/CatalogImpl.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogManager.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogReader.java
! src/java.xml/share/classes/javax/xml/catalog/CatalogResolverImpl.java
! src/java.xml/share/classes/javax/xml/catalog/GroupEntry.java
! test/javax/xml/jaxp/functional/catalog/DeferFeatureTest.java
! test/javax/xml/jaxp/unittest/catalog/CatalogTest.java
+ test/javax/xml/jaxp/unittest/catalog/catalogReferCircle-itself.xml
+ test/javax/xml/jaxp/unittest/catalog/catalogReferCircle-left.xml
+ test/javax/xml/jaxp/unittest/catalog/catalogReferCircle-right.xml

Changeset: 68d7d15cf44a
Author:aefimov
Date:  2016-09-01 17:12 +0300
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/68d7d15cf44a

8150145: javax/xml/jaxp/unittest/common/TransformationWarningsTest.java and 
ValidationWarningsTest.java failed intermittently without any error message
Reviewed-by: joehw, clanger

! test/ProblemList.txt
! test/javax/xml/jaxp/unittest/common/TransformationWarningsTest.java
! test/javax/xml/jaxp/unittest/common/ValidationWarningsTest.java
! test/javax/xml/jaxp/unittest/common/WarningsTestBase.java

Changeset: 1defcc790c15
Author:joehw
Date:  2016-09-01 17:03 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/1defcc790c15

8161454: Fails to Load external Java method from inside of a XSL stylesheet if 
SecurityManager is present
Reviewed-by: aefimov, lancea, dfuchs

! src/java.xml/share/classes/jdk/xml/internal/JdkXmlFeatures.java
! test/javax/xml/jaxp/unittest/transform/XSLTFunctionsTest.java

Changeset: f695240370c7
Author:lana
Date:  2016-09-02 02:41 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/f695240370c7

Merge


Changeset: 38f97dc17141
Author:lana
Date:  2016-09-08 21:11 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/38f97dc17141

Added tag jdk-9+135 for changeset f695240370c7

! .hgtags

Changeset: 7352361f6243
Author:mchung
Date:  2016-09-08 17:40 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/jaxp/rev/7352361f6243

Merge




hg: jigsaw/jake/corba: 2 new changesets

2016-09-08 Thread mandy . chung
Changeset: aa053a3faf26
Author:lana
Date:  2016-09-08 21:11 +
URL:   http://hg.openjdk.java.net/jigsaw/jake/corba/rev/aa053a3faf26

Added tag jdk-9+135 for changeset 094d0db606db

! .hgtags

Changeset: d9b3cf2dd4d2
Author:mchung
Date:  2016-09-08 17:40 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/corba/rev/d9b3cf2dd4d2

Merge




hg: jigsaw/jake: 13 new changesets

2016-09-08 Thread mandy . chung
Changeset: f60c6d0cc585
Author:mchung
Date:  2016-08-30 17:49 -0700
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/f60c6d0cc585

8160851: Remove old launcher module-related options
Reviewed-by: jjg, alanb

! make/CompileJavaModules.gmk
! make/common/SetupJavaCompilers.gmk

Changeset: b6f21e99011e
Author:redestad
Date:  2016-08-31 14:20 +0200
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/b6f21e99011e

8164858: Enable build-time use of java.lang.invoke resolve tracing
Reviewed-by: erikj, vlivanov

! make/Images.gmk

Changeset: dcce309ab6a6
Author:erikj
Date:  2016-08-19 16:02 +0200
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/dcce309ab6a6

8164297: Jtreg test exeinvoke fails to link on Ubuntu
Reviewed-by: tbell, dholmes

! make/common/TestFilesCompilation.gmk

Changeset: 2a8815d86b93
Author:ctornqvi
Date:  2016-08-19 10:09 -0400
URL:   http://hg.openjdk.java.net/jigsaw/jake/rev/2a8815d86b93

8157957: ClassNotFoundException: jdk.test.lib.JDKToolFinder
Reviewed-by: coleenp, gtriantafill, mseledtsov, iignatyev, dholmes, dsamersoff

! make/test/BuildTestLib.gmk
+ test/lib/ClassFileInstaller.java
+ test/lib/RedefineClassHelper.java
+ test/lib/jdk/test/lib/Asserts.java
+ test/lib/jdk/test/lib/BuildHelper.java
+ test/lib/jdk/test/lib/ByteCodeLoader.java
+ test/lib/jdk/test/lib/DynamicVMOption.java
+ test/lib/jdk/test/lib/FileInstaller.java
+ test/lib/jdk/test/lib/InMemoryJavaCompiler.java
+ test/lib/jdk/test/lib/JDKToolFinder.java
+ test/lib/jdk/test/lib/JDKToolLauncher.java
+ test/lib/jdk/test/lib/Platform.java
+ test/lib/jdk/test/lib/Utils.java
+ test/lib/jdk/test/lib/apps/LingeredApp.java
+ test/lib/jdk/test/lib/apps/LingeredAppWithDeadlock.java
+ test/lib/jdk/test/lib/cli/CPUSpecificCommandLineOptionTest.java
+ test/lib/jdk/test/lib/cli/CommandLineOptionTest.java
+ test/lib/jdk/test/lib/cli/predicate/AndPredicate.java
+ test/lib/jdk/test/lib/cli/predicate/CPUSpecificPredicate.java
+ test/lib/jdk/test/lib/cli/predicate/NotPredicate.java
+ test/lib/jdk/test/lib/cli/predicate/OrPredicate.java
+ test/lib/jdk/test/lib/dcmd/CommandExecutor.java
+ test/lib/jdk/test/lib/dcmd/CommandExecutorException.java
+ test/lib/jdk/test/lib/dcmd/FileJcmdExecutor.java
+ test/lib/jdk/test/lib/dcmd/JMXExecutor.java
+ test/lib/jdk/test/lib/dcmd/JcmdExecutor.java
+ test/lib/jdk/test/lib/dcmd/MainClassJcmdExecutor.java
+ test/lib/jdk/test/lib/dcmd/PidJcmdExecutor.java
+ test/lib/jdk/test/lib/hprof/HprofParser.java
+ test/lib/jdk/test/lib/hprof/README
+ test/lib/jdk/test/lib/hprof/model/AbstractJavaHeapObjectVisitor.java
+ test/lib/jdk/test/lib/hprof/model/ArrayTypeCodes.java
+ test/lib/jdk/test/lib/hprof/model/HackJavaValue.java
+ test/lib/jdk/test/lib/hprof/model/JavaBoolean.java
+ test/lib/jdk/test/lib/hprof/model/JavaByte.java
+ test/lib/jdk/test/lib/hprof/model/JavaChar.java
+ test/lib/jdk/test/lib/hprof/model/JavaClass.java
+ test/lib/jdk/test/lib/hprof/model/JavaDouble.java
+ test/lib/jdk/test/lib/hprof/model/JavaField.java
+ test/lib/jdk/test/lib/hprof/model/JavaFloat.java
+ test/lib/jdk/test/lib/hprof/model/JavaHeapObject.java
+ test/lib/jdk/test/lib/hprof/model/JavaHeapObjectVisitor.java
+ test/lib/jdk/test/lib/hprof/model/JavaInt.java
+ test/lib/jdk/test/lib/hprof/model/JavaLazyReadObject.java
+ test/lib/jdk/test/lib/hprof/model/JavaLong.java
+ test/lib/jdk/test/lib/hprof/model/JavaObject.java
+ test/lib/jdk/test/lib/hprof/model/JavaObjectArray.java
+ test/lib/jdk/test/lib/hprof/model/JavaObjectRef.java
+ test/lib/jdk/test/lib/hprof/model/JavaShort.java
+ test/lib/jdk/test/lib/hprof/model/JavaStatic.java
+ test/lib/jdk/test/lib/hprof/model/JavaThing.java
+ test/lib/jdk/test/lib/hprof/model/JavaValue.java
+ test/lib/jdk/test/lib/hprof/model/JavaValueArray.java
+ test/lib/jdk/test/lib/hprof/model/ReachableExcludes.java
+ test/lib/jdk/test/lib/hprof/model/ReachableExcludesImpl.java
+ test/lib/jdk/test/lib/hprof/model/ReachableObjects.java
+ test/lib/jdk/test/lib/hprof/model/ReferenceChain.java
+ test/lib/jdk/test/lib/hprof/model/Root.java
+ test/lib/jdk/test/lib/hprof/model/Snapshot.java
+ test/lib/jdk/test/lib/hprof/model/StackFrame.java
+ test/lib/jdk/test/lib/hprof/model/StackTrace.java
+ test/lib/jdk/test/lib/hprof/parser/FileReadBuffer.java
+ test/lib/jdk/test/lib/hprof/parser/HprofReader.java
+ test/lib/jdk/test/lib/hprof/parser/MappedReadBuffer.java
+ test/lib/jdk/test/lib/hprof/parser/PositionDataInputStream.java
+ test/lib/jdk/test/lib/hprof/parser/PositionInputStream.java
+ test/lib/jdk/test/lib/hprof/parser/ReadBuffer.java
+ test/lib/jdk/test/lib/hprof/parser/Reader.java
+ test/lib/jdk/test/lib/hprof/util/ArraySorter.java
+ test/lib/jdk/test/lib/hprof/util/Comparer.java
+ test/lib/jdk/test/lib/hprof/util/CompositeEnumeration.java
+ test/lib/jdk/test/lib/hprof/util/Misc.java
+ test/lib/jdk/test/lib/hprof/util/VectorSorter.java
+ test/lib/jdk/test/lib/process/ExitCode.java
+ test/lib/jdk/test/lib/process/OutputAnalyzer.java
+ 

Re: Using non-parallel custom class loaders for Layer configurations

2016-09-08 Thread Mandy Chung

> On Sep 8, 2016, at 3:29 PM, David M. Lloyd  wrote:
> 
> Would it be possible to include a method like this (pretty old patch I had 
> laying around):
> 
> diff --git a/jdk/src/java.base/share/classes/java/lang/ClassLoader.java 
> b/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
> index 1bb1580..3def10e 100644
> --- a/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
> +++ b/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
> @@ -469,6 +477,15 @@ public abstract class ClassLoader {
> return lock;
> }
> 
> +/**
> + * Determine whether this class loader is parallel-capable.
> + *
> + * @return {@code true} if the class loader is parallel-capable, {@code 
> false} otherwise
> + */
> +protected final boolean isParallelCapable() {
> +return ParallelLoaders.isRegistered(getClass());
> +}
> +

This seems a fine method to add.  I think it can be a public method (I don’t 
see any reason it has to be protected)

Mandy



Using non-parallel custom class loaders for Layer configurations

2016-09-08 Thread David M. Lloyd
Is it not necessary that any class loader in use by a Layer must be 
parallel-capable?  Otherwise it seems like deadlocks could occur in 
certain situations when there are references that are cyclic with 
respect to class loaders mapped by the mapping function.


On that note, there is in fact no good way for user code to determine 
whether or not a class loader is indeed parallel-capable, which is often 
a critical question in module systems, especially when users can provide 
specialized class loader implementations.  Right now you can only do 
something (awful) like this (e.g. in your base ClassLoader constructor):


if (getClassLoadingLock("$TEST$") == this) {
throw new Error("Cannot instantiate non-parallel subclass");
}

Would it be possible to include a method like this (pretty old patch I 
had laying around):


diff --git a/jdk/src/java.base/share/classes/java/lang/ClassLoader.java 
b/jdk/src/java.base/share/classes/java/lang/ClassLoader.java

index 1bb1580..3def10e 100644
--- a/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
+++ b/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
@@ -469,6 +477,15 @@ public abstract class ClassLoader {
 return lock;
 }

+/**
+ * Determine whether this class loader is parallel-capable.
+ *
+ * @return {@code true} if the class loader is parallel-capable, 
{@code false} otherwise

+ */
+protected final boolean isParallelCapable() {
+return ParallelLoaders.isRegistered(getClass());
+}
+
 // This method is invoked by the virtual machine to load a class.
 private Class loadClassInternal(String name)
 throws ClassNotFoundException

Alternatively, the method could be made static and protected, only 
returning true if the calling class is a class loader that is parallel 
capable, something like this (against a newer branch):


diff --git a/jdk/src/java.base/share/classes/java/lang/ClassLoader.java 
b/jdk/src/java.base/share/classes/java/lang/ClassLoader.java

index 0139123..06a5909 100644
--- a/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
+++ b/jdk/src/java.base/share/classes/java/lang/ClassLoader.java
@@ -1421,6 +1421,19 @@ public abstract class ClassLoader {
 }

 /**
+ * Determine whether the calling class is a parallel-capable class 
loader.

+ *
+ * @return true if the caller is a parallel-capable class loader 
and false otherwise

+ *
+ * @since  9
+ */
+@CallerSensitive
+protected static boolean isParallelCapable() {
+final Class callerClass = Reflection.getCallerClass();
+return ClassLoader.class.isAssignableFrom(callerClass) && 
ParallelLoaders.isRegistered(callerClass.asSubclass(ClassLoader.class));

+}
+
+/**
  * Find a resource of the specified name from the search path used 
to load

  * classes.  This method locates the resource through the system class
  * loader (see {@link #getSystemClassLoader()}).

WDYT?

--
- DML


Re: [MRJAR] Entry order matters?

2016-09-08 Thread Robert Scholte
Confirmed there's an issue; jar should have been recognized as a  
multirelease jar.


https://bugs.openjdk.java.net/browse/JDK-8165723


thanks,
Robert

On Mon, 05 Sep 2016 09:10:56 +0200, Alan Bateman   
wrote:





On 04/09/2016 21:56, Robert Scholte wrote:

:


Also if you "unzip   -d " then does everything look okay?



hboutemy$ unzip -t multirelease-0.8-SNAPSHOT_failure.jar | grep  
class

hboutemy testing: base/Base.class OK
hboutemy testing: META-INF/versions/9/mr/A.class OK
hboutemy testing: META-INF/versions/8/mr/A.class OK
hboutemy testing: mr/A.class OK
hboutemy$ unzip -t multirelease-0.8-SNAPSHOT_success.jar | grep  
class

hboutemy testing: base/Base.class OK
hboutemy testing: META-INF/versions/8/mr/A.class OK
hboutemy testing: mr/A.class OK
hboutemy testing: META-INF/versions/9/mr/A.class OK

Looks good to me.
I think I need to see multirelease-0.8-SNAPSHOT_failure.jar to  
understand this as I suspect there is more going on that is obvious  
here. I'll see if I can duplicate it.


-Alan


Re: Possibility to create modular jars out of old non-modular ones

2016-09-08 Thread Alex Buckley

On 9/8/2016 6:11 AM, Waldek Kozaczuk wrote:

Imagine you have pre-jigsaw jar like log4j 1.x that happens to depend on
java.awt.* and swing (it does in fact) but I know for a fact my app does
not use this part of it.

Would it be possible to create a modular jar out of non-modular by simply
repackaging it and adding module-info.java that specifies what specifically
my app depends on (is exposed in log4j jar)


Yes. By all means, add module-info.class to your app JAR.

The following thread from August will be helpful if you want to compile 
just module-info.java for JDK 9: 
http://mail.openjdk.java.net/pipermail/jigsaw-dev/2016-August/009177.html



and what that log4j in my case "requires" from Java 9 in terms of
modules?


The module-info.class file in your app JAR is the declaration of your 
application module. It can't contain dependencies for some other module. 
If you want your application module to say 'requires log4j', then your 
choices are to put log4j on the modulepath as an automatic module, or to 
find a version of log4j that is already a modular JAR.


Alex


Re: RFR 8165697: jlink running on Mac with Windows jmods produces non-runnable image

2016-09-08 Thread Jim Laskey (Oracle)
+1

> On Sep 8, 2016, at 11:32 AM, Sundararajan Athijegannathan 
>  wrote:
> 
> Please review http://cr.openjdk.java.net/~sundar/8165697/webrev.01/ for
> https://bugs.openjdk.java.net/browse/JDK-8165697
> 
> Thanks
> 
> -Sundar
> 



Re: RFR 8165697: jlink running on Mac with Windows jmods produces non-runnable image

2016-09-08 Thread Claes Redestad

Looks good.

/Claes

On 2016-09-08 16:32, Sundararajan Athijegannathan wrote:

Please review http://cr.openjdk.java.net/~sundar/8165697/webrev.01/ for
https://bugs.openjdk.java.net/browse/JDK-8165697

Thanks

-Sundar



RFR 8165697: jlink running on Mac with Windows jmods produces non-runnable image

2016-09-08 Thread Sundararajan Athijegannathan
Please review http://cr.openjdk.java.net/~sundar/8165697/webrev.01/ for
https://bugs.openjdk.java.net/browse/JDK-8165697

Thanks

-Sundar



Possibility to create modular jars out of old non-modular ones

2016-09-08 Thread Waldek Kozaczuk
Imagine you have pre-jigsaw jar like log4j 1.x that happens to depend on
java.awt.* and swing (it does in fact) but I know for a fact my app does
not use this part of it.

Would it be possible to create a modular jar out of non-modular by simply
repackaging it and adding module-info.java that specifies what specifically
my app depends on (is exposed in log4j jar) and what that log4j in my case
"requires" from Java 9 in terms of modules?

If possible then jlink could rely on this information and build according
minimal image.


Re: Can jdeps resolve unnecessary dependencies?

2016-09-08 Thread Sander Mak
Hi Waldek,

I feel the other responses focus too much on your side-remark on reflection.

What you're seeing here with jdeps is not unexpected behavior, I believe. For 
example, log4j *does* reference awt code [0], even though you might not expect 
it. Same holds true for swing classes [1]. It's just when you run it on a full 
JDK, you're not aware of this dependency! Hurray for modularity, making it 
clear that you need awt to use log4j... (and therefore, transitively your 
app.jar if it uses log4j classes). Now this ColorRule [0] class may never be 
loaded in any of your usage scenarios, but that doesn't mean the dependency is 
not there.

Hope this helps,

Sander

[0] 
https://github.com/apache/log4j/blob/c5f4279081091e562c44bb753f6e52dc6be5fa52/src/main/java/org/apache/log4j/rule/ColorRule.java#L20
[1] 
https://github.com/apache/log4j/search?utf8=✓=swing

On 08 Sep 2016, at 00:34, Waldek Kozaczuk 
> wrote:

Thanks for your responses.

So today I did some experimentation with Java 8 jdeps to analyze how my
app.jar depends at class level on other jars and eventually individual JRE
classes. I would run following command:

jdeps -v -P -R -cp lib/* app.jar # where lib/ contains jars that app.jar
depends on to run and libs did not contain app.jar

It turns our that jdeps would find and present dependent classes in all
jars irregardless if any class in app.jar would directly or indirectly
depend on it per "->" references.

The good example of it was log4j jar where jdeps showed number of classes
that depend directly or indirectly on some java.awt.* or javax.swing.*
classes. And I am sure that this app does not requires AWT nor Swing to
execute. Is it because jdeps finds individual class dependencies by
detecting reflection calls in bytecode of log4j?

Or maybe this "greedy" behavior of jdeps is intende and I would need to
build a tool that would process its generated dependency graph starting
from app.jar and that way find the real subset of JRE it depends on? I am
still thinking that jdeps would not be able to show me classes that were
called through java.lang.reflect.* constructs or Class.forName().

Do my findings make sense?

On Wed, Sep 7, 2016 at 5:03 PM, Mandy Chung 
> wrote:


On Sep 7, 2016, at 8:07 AM, Sander Mak 
> wrote:


On 07 Sep 2016, at 16:12, Waldek Kozaczuk 
> wrote:

Will I be able to take advantage of jlink's ability to produce
subset-only
image of the JRE even if I do not use new Java 9 modular jars? In other
words if my app.jar and other dependent jars are old java 8 ones and do
not
have any module descriptors will jlink be able to identify only needed
JRE
modules and produce minimal runtime image?

Jlink will not be able to do that for you without module descriptors,
but there's an alternative solution. First, use jdeps to find all the
platform modules your application needs. Then create an image using jlink
with an --add-modules  argument for each of the identified
platform modules. You can then run your existing application with all
application/library JARs on the classpath on top of the resulting image.

As Sander said, jlink creates custom image from packaged modules only. You
can create a minimal runtime image and your application and other libraries
can be put on the classpath.

The open issue of adding link-time support for automatic modules is
JDK-8139947 [1].

Mandy
[1] https://bugs.openjdk.java.net/browse/JDK-8130047






Re: Upgradeable Modules Question

2016-09-08 Thread Alan Bateman

On 07/09/2016 13:43, Richard Opalka wrote:



but I'm guessing you might have artifacts like 
jaxb-api-.jar, in which case the derived module name isn't 
useful.
I used exploded directory named java.xml.bind (of course without 
module-info.class).
I'm not sure that I understand this as it won't be treated as an 
automatic module in that case.


-Alan


Re: Can jdeps resolve unnecessary dependencies?

2016-09-08 Thread dalibor topic
As Mandy said, jdeps can't do that. There are other tools that perform 
bytecode engineering, optimization and shrinking that could assist you 
in determining a useful subset of all your application's classes to feed 
into jdeps for further analysis.


cheers,
dalibor topic

On 08.09.2016 00:42, Waldek Kozaczuk wrote:

My real question I am in pursuit of answering is this: is it in theory possible 
to build an automated process that would use jdeps to analyze my app to build a 
minimal JRE image it would run on? The latter could be done by rewriting rt.jar 
in Java 8 and using jlink in Java 9.

Sent from my iPhone


On Sep 7, 2016, at 18:34, Waldek Kozaczuk  wrote:

Thanks for your responses.

So today I did some experimentation with Java 8 jdeps to analyze how my app.jar 
depends at class level on other jars and eventually individual JRE classes. I 
would run following command:

jdeps -v -P -R -cp lib/* app.jar # where lib/ contains jars that app.jar 
depends on to run and libs did not contain app.jar

It turns our that jdeps would find and present dependent classes in all jars irregardless if 
any class in app.jar would directly or indirectly depend on it per "->" 
references.

The good example of it was log4j jar where jdeps showed number of classes that 
depend directly or indirectly on some java.awt.* or javax.swing.* classes. And 
I am sure that this app does not requires AWT nor Swing to execute. Is it 
because jdeps finds individual class dependencies by detecting reflection calls 
in bytecode of log4j?

Or maybe this "greedy" behavior of jdeps is intende and I would need to build a 
tool that would process its generated dependency graph starting from app.jar and that way 
find the real subset of JRE it depends on? I am still thinking that jdeps would not be 
able to show me classes that were called through java.lang.reflect.* constructs or 
Class.forName().

Do my findings make sense?


On Wed, Sep 7, 2016 at 5:03 PM, Mandy Chung  wrote:


On Sep 7, 2016, at 8:07 AM, Sander Mak  wrote:



On 07 Sep 2016, at 16:12, Waldek Kozaczuk  wrote:

Will I be able to take advantage of jlink's ability to produce subset-only
image of the JRE even if I do not use new Java 9 modular jars? In other
words if my app.jar and other dependent jars are old java 8 ones and do not
have any module descriptors will jlink be able to identify only needed JRE
modules and produce minimal runtime image?


Jlink will not be able to do that for you without module descriptors, but there's an 
alternative solution. First, use jdeps to find all the platform modules your 
application needs. Then create an image using jlink with an --add-modules 
 argument for each of the identified platform modules. You can 
then run your existing application with all application/library JARs on the classpath 
on top of the resulting image.


As Sander said, jlink creates custom image from packaged modules only. You can 
create a minimal runtime image and your application and other libraries can be 
put on the classpath.

The open issue of adding link-time support for automatic modules is JDK-8139947 
[1].

Mandy
[1] https://bugs.openjdk.java.net/browse/JDK-8130047




--
 Dalibor Topic | Principal Product Manager
Phone: +494089091214  | Mobile: +491737185961


ORACLE Deutschland B.V. & Co. KG | Kühnehöfe 5 | 22761 Hamburg

ORACLE Deutschland B.V. & Co. KG
Hauptverwaltung: Riesstr. 25, D-80992 München
Registergericht: Amtsgericht München, HRA 95603

Komplementärin: ORACLE Deutschland Verwaltung B.V.
Hertogswetering 163/167, 3543 AS Utrecht, Niederlande
Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697
Geschäftsführer: Alexander van der Ven, Jan Schultheiss, Val Maher

 Oracle is committed to developing
practices and products that help protect the environment