# HG changeset patch # User kvn # Date 1513195195 28800 # Node ID ead47ddf5844ac423ec6be69fa05188bc9519dcc # Parent 3c9975e46464f6969a9af5c411f5d5db128be89f 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 diff -r 3c9975e46464 -r ead47ddf5844 .hgignore --- 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/.* diff -r 3c9975e46464 -r ead47ddf5844 test/hotspot/jtreg/runtime/SharedArchiveFile/BootAppendTests.java --- 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 diff -r 3c9975e46464 -r ead47ddf5844 test/hotspot/jtreg/runtime/appcds/jigsaw/JigsawOptionsCombo.java --- 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 diff -r 3c9975e46464 -r ead47ddf5844 test/hotspot/jtreg/runtime/appcds/jigsaw/classpathtests/BootAppendTests.java --- 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 diff -r 3c9975e46464 -r ead47ddf5844 test/hotspot/jtreg/runtime/appcds/jigsaw/classpathtests/ClassPathTests.java --- 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 diff -r 3c9975e46464 -r ead47ddf5844 test/hotspot/jtreg/runtime/appcds/jigsaw/classpathtests/EmptyClassInBootClassPath.java --- 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 diff -r 3c9975e46464 -r ead47ddf5844 test/hotspot/jtreg/runtime/appcds/jigsaw/limitmods/LimitModsTests.java --- 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 diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/TEST.ROOT --- 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 diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/com/sun/tools/attach/modules/Driver.java --- 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 diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/java/lang/String/concat/WithSecurityManager.java --- 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 * diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/java/lang/System/LoggerFinder/LoggerFinderAPI/LoggerFinderAPI.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 diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/java/lang/instrument/TestAgentWithLimitMods.java --- 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 * diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/java/lang/management/ManagementFactory/DefaultManagementProviderTest.java --- 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 */ diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/java/net/SocketOption/OptionsTest.java --- 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 diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/java/net/SocketOption/UnsupportedOptionsTest.java --- 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 */ diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/java/nio/channels/DatagramChannel/SocketOptionTests.java --- 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 */ diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/java/nio/channels/ServerSocketChannel/SocketOptionTests.java --- 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 */ diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/java/nio/channels/SocketChannel/SocketOptionTests.java --- 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 */ diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/tools/launcher/modules/limitmods/LimitModsTest.java --- 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.* diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/tools/launcher/modules/listmods/ListModsTest.java --- 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.* diff -r 3c9975e46464 -r ead47ddf5844 test/jdk/tools/launcher/modules/showmoduleresolution/ShowModuleResolutionTest.java --- 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.*