8191788: add jdk.internal.vm.compiler to --limit-modules if -Djvmci.Compiler=graal is in the command line
Summary: skip tests which use --limit-modules when Graal is used as JIT compiler.
Reviewed-by: alanb, mchung, dholmes, ccheung, dnsimon
--- a/.hgignore Wed Dec 13 19:32:48 2017 +0300
+++ b/.hgignore Wed Dec 13 11:59:55 2017 -0800
@@ -11,3 +11,5 @@
test/nashorn/script/external
test/nashorn/lib
NashornProfile.txt
+.*/JTreport/.*
+.*/JTwork/.*
--- a/test/hotspot/jtreg/runtime/SharedArchiveFile/BootAppendTests.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/hotspot/jtreg/runtime/SharedArchiveFile/BootAppendTests.java Wed Dec 13 11:59:55 2017 -0800
@@ -23,7 +23,7 @@
/**
* @test
- * @requires vm.cds
+ * @requires vm.cds & !vm.graal.enabled
* @summary Testing -Xbootclasspath/a support for CDS
* @requires (vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)
* @library /test/lib
--- a/test/hotspot/jtreg/runtime/appcds/jigsaw/JigsawOptionsCombo.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/hotspot/jtreg/runtime/appcds/jigsaw/JigsawOptionsCombo.java Wed Dec 13 11:59:55 2017 -0800
@@ -27,7 +27,7 @@
* @summary Test combinations of jigsaw options that affect the use of AppCDS
*
* AppCDS does not support uncompressed oops
- * @requires (vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)
+ * @requires ((vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)) & !vm.graal.enabled
* @library /test/lib ..
* @modules java.base/jdk.internal.misc
* java.management
--- a/test/hotspot/jtreg/runtime/appcds/jigsaw/classpathtests/BootAppendTests.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/hotspot/jtreg/runtime/appcds/jigsaw/classpathtests/BootAppendTests.java Wed Dec 13 11:59:55 2017 -0800
@@ -25,7 +25,7 @@
/**
* @test
* @summary AppCDS tests for testing -Xbootclasspath/a
- * @requires (vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)
+ * @requires ((vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)) & !vm.graal.enabled
* @library /test/lib /test/hotspot/jtreg/runtime/appcds
* @modules java.base/jdk.internal.misc
* java.management
--- a/test/hotspot/jtreg/runtime/appcds/jigsaw/classpathtests/ClassPathTests.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/hotspot/jtreg/runtime/appcds/jigsaw/classpathtests/ClassPathTests.java Wed Dec 13 11:59:55 2017 -0800
@@ -24,7 +24,7 @@
/**
* @test
- * @requires (vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)
+ * @requires ((vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)) & !vm.graal.enabled
* @library ../..
* @library /test/lib
* @modules java.base/jdk.internal.misc
--- a/test/hotspot/jtreg/runtime/appcds/jigsaw/classpathtests/EmptyClassInBootClassPath.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/hotspot/jtreg/runtime/appcds/jigsaw/classpathtests/EmptyClassInBootClassPath.java Wed Dec 13 11:59:55 2017 -0800
@@ -29,7 +29,7 @@
* 2) app loader will load the class from the jimage by default;
* app loader will load the class from the bootclasspath if the
* "--limit-modules java.base" option is specified
- * @requires (vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)
+ * @requires ((vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)) & !vm.graal.enabled
* @library /test/lib /test/hotspot/jtreg/runtime/appcds
* @modules java.base/jdk.internal.misc
* java.management
--- a/test/hotspot/jtreg/runtime/appcds/jigsaw/limitmods/LimitModsTests.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/hotspot/jtreg/runtime/appcds/jigsaw/limitmods/LimitModsTests.java Wed Dec 13 11:59:55 2017 -0800
@@ -24,7 +24,7 @@
/**
* @test
- * @requires (vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)
+ * @requires ((vm.opt.UseCompressedOops == null) | (vm.opt.UseCompressedOops == true)) & !vm.graal.enabled
* @library ../..
* @library /test/lib
* @modules java.base/jdk.internal.misc
--- a/test/jdk/TEST.ROOT Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/TEST.ROOT Wed Dec 13 11:59:55 2017 -0800
@@ -36,6 +36,7 @@
requires.properties= \
sun.arch.data.model \
java.runtime.name \
+ vm.graal.enabled \
vm.cds
# Minimum jtreg version
--- a/test/jdk/com/sun/tools/attach/modules/Driver.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/com/sun/tools/attach/modules/Driver.java Wed Dec 13 11:59:55 2017 -0800
@@ -23,6 +23,7 @@
/**
* @test
+ * @requires !vm.graal.enabled
* @modules jdk.attach
* @build m/* Agent
* @run main/othervm -Djdk.attach.allowAttachSelf m/p.Main jmx javaagent
--- a/test/jdk/java/lang/String/concat/WithSecurityManager.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/java/lang/String/concat/WithSecurityManager.java Wed Dec 13 11:59:55 2017 -0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2016, 2017, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -27,6 +27,7 @@
* @test
* @summary String concatenation fails with a custom SecurityManager that uses concatenation
* @bug 8155090 8158851
+ * @requires !vm.graal.enabled
*
* @compile WithSecurityManager.java
*
--- a/test/jdk/java/lang/System/LoggerFinder/LoggerFinderAPI/LoggerFinderAPI.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/java/lang/System/LoggerFinder/LoggerFinderAPI/LoggerFinderAPI.java Wed Dec 13 11:59:55 2017 -0800
@@ -46,6 +46,7 @@
* throwing NullPointerException. The test uses --limit-module
* to force the selection of one or the other.
* @author danielfuchs
+ * @requires !vm.graal.enabled
* @build LoggerFinderAPI
* @run main/othervm --limit-modules java.base,java.logging
* -Djava.util.logging.SimpleFormatter.format=LOG-%4$s:-[%2$s]-%5$s%6$s%n
--- a/test/jdk/java/lang/instrument/TestAgentWithLimitMods.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/java/lang/instrument/TestAgentWithLimitMods.java Wed Dec 13 11:59:55 2017 -0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2016, 2017, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -24,6 +24,7 @@
/**
*
* @test
+ * @requires !vm.graal.enabled
* @summary Tests that the -javaagent option adds the java.instrument into
* the module graph
*
--- a/test/jdk/java/lang/management/ManagementFactory/DefaultManagementProviderTest.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/java/lang/management/ManagementFactory/DefaultManagementProviderTest.java Wed Dec 13 11:59:55 2017 -0800
@@ -26,6 +26,7 @@
* @bug 8151099
* @summary Verify platform MXBeans initialized properly with java.management
* module only. No other management provider
+ * @requires !vm.graal.enabled
* @modules java.management
* @run main/othervm --limit-modules=java.management DefaultManagementProviderTest
*/
--- a/test/jdk/java/net/SocketOption/OptionsTest.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/java/net/SocketOption/OptionsTest.java Wed Dec 13 11:59:55 2017 -0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2014, 2016, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2014, 2017, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -24,6 +24,7 @@
/*
* @test
* @bug 8036979 8072384 8044773
+ * @requires !vm.graal.enabled
* @run main/othervm -Xcheck:jni OptionsTest
* @run main/othervm -Xcheck:jni -Djava.net.preferIPv4Stack=true OptionsTest
* @run main/othervm --limit-modules=java.base OptionsTest
--- a/test/jdk/java/net/SocketOption/UnsupportedOptionsTest.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/java/net/SocketOption/UnsupportedOptionsTest.java Wed Dec 13 11:59:55 2017 -0800
@@ -32,6 +32,7 @@
* @bug 8143554 8044773
* @summary Test checks that UnsupportedOperationException for unsupported
* SOCKET_OPTIONS is thrown by both getOption() and setOption() methods.
+ * @requires !vm.graal.enabled
* @run main UnsupportedOptionsTest
* @run main/othervm --limit-modules=java.base UnsupportedOptionsTest
*/
--- a/test/jdk/java/nio/channels/DatagramChannel/SocketOptionTests.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/java/nio/channels/DatagramChannel/SocketOptionTests.java Wed Dec 13 11:59:55 2017 -0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2007, 2016, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2007, 2017, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -24,6 +24,7 @@
/* @test
* @bug 4640544 8044773
* @summary Unit test for setOption/getOption/options methods
+ * @requires !vm.graal.enabled
* @run main SocketOptionTests
* @run main/othervm --limit-modules=java.base SocketOptionTests
*/
--- a/test/jdk/java/nio/channels/ServerSocketChannel/SocketOptionTests.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/java/nio/channels/ServerSocketChannel/SocketOptionTests.java Wed Dec 13 11:59:55 2017 -0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2007, 2016, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2007, 2017, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -25,6 +25,7 @@
* @bug 4640544 8044773
* @summary Unit test for ServerSocketChannel setOption/getOption/options
* methods.
+ * @requires !vm.graal.enabled
* @run main SocketOptionTests
* @run main/othervm --limit-modules=java.base SocketOptionTests
*/
--- a/test/jdk/java/nio/channels/SocketChannel/SocketOptionTests.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/java/nio/channels/SocketChannel/SocketOptionTests.java Wed Dec 13 11:59:55 2017 -0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2007, 2011, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2007, 2017, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -25,6 +25,7 @@
* @bug 4640544 8044773
* @summary Unit test to check SocketChannel setOption/getOption/options
* methods.
+ * @requires !vm.graal.enabled
* @run main SocketOptionTests
* @run main/othervm --limit-modules=java.base SocketOptionTests
*/
--- a/test/jdk/tools/launcher/modules/limitmods/LimitModsTest.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/tools/launcher/modules/limitmods/LimitModsTest.java Wed Dec 13 11:59:55 2017 -0800
@@ -23,6 +23,7 @@
/**
* @test
+ * @requires !vm.graal.enabled
* @library /lib/testlibrary /test/lib
* @modules java.desktop java.logging jdk.compiler
* @build LimitModsTest jdk.test.lib.compiler.CompilerUtils jdk.testlibrary.*
--- a/test/jdk/tools/launcher/modules/listmods/ListModsTest.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/tools/launcher/modules/listmods/ListModsTest.java Wed Dec 13 11:59:55 2017 -0800
@@ -23,6 +23,7 @@
/**
* @test
+ * @requires !vm.graal.enabled
* @library /lib/testlibrary /test/lib
* @modules java.se
* @build ListModsTest jdk.test.lib.compiler.CompilerUtils jdk.testlibrary.*
--- a/test/jdk/tools/launcher/modules/showmoduleresolution/ShowModuleResolutionTest.java Wed Dec 13 19:32:48 2017 +0300
+++ b/test/jdk/tools/launcher/modules/showmoduleresolution/ShowModuleResolutionTest.java Wed Dec 13 11:59:55 2017 -0800
@@ -23,6 +23,7 @@
/**
* @test
+ * @requires !vm.graal.enabled
* @modules jdk.jdeps jdk.zipfs
* @library /lib/testlibrary
* @build ShowModuleResolutionTest jdk.testlibrary.*