Hi Staffan,
Changes look good. However it looks like this test has the same issue, right? http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/file/59d3a69564dc/test/gc/g1/plab/TestPLABResize.java It is currently ignored, but we should probably fix it too... Thanks, Bengt On 2016-03-09 10:47, Staffan Larsen wrote:
Please review this small fix to some tests with the wrong paths in them. Tested locally with jtreg 4.1 b12 and b13. Thanks, /Staffan $ hg diff diff --git a/test/gc/g1/plab/TestPLABPromotion.java b/test/gc/g1/plab/TestPLABPromotion.java --- a/test/gc/g1/plab/TestPLABPromotion.java +++ b/test/gc/g1/plab/TestPLABPromotion.java @@ -27,7 +27,7 @@ * @summary Test PLAB promotion * @requires vm.gc=="G1" | vm.gc=="null" * @requires vm.opt.FlightRecorder != true - * @library /testlibrary /../../test/lib / + * @library /testlibrary /test/lib / * @modules java.management * @build ClassFileInstaller * sun.hotspot.WhiteBox diff --git a/test/serviceability/dcmd/gc/HeapDumpAllTest.java b/test/serviceability/dcmd/gc/HeapDumpAllTest.java --- a/test/serviceability/dcmd/gc/HeapDumpAllTest.java +++ b/test/serviceability/dcmd/gc/HeapDumpAllTest.java @@ -35,7 +35,7 @@ * @build jdk.test.lib.hprof.* * @build jdk.test.lib.hprof.model.* * @build jdk.test.lib.hprof.parser.* - * @build jdk.test.lib.hprof.utils.* + * @build jdk.test.lib.hprof.util.* * @build HeapDumpTest * @run testng HeapDumpAllTest */ diff --git a/test/serviceability/dcmd/gc/HeapDumpTest.java b/test/serviceability/dcmd/gc/HeapDumpTest.java --- a/test/serviceability/dcmd/gc/HeapDumpTest.java +++ b/test/serviceability/dcmd/gc/HeapDumpTest.java @@ -51,7 +51,7 @@ * @build jdk.test.lib.hprof.* * @build jdk.test.lib.hprof.model.* * @build jdk.test.lib.hprof.parser.* - * @build jdk.test.lib.hprof.utils.* + * @build jdk.test.lib.hprof.util.* * @run testng HeapDumpTest */ public class HeapDumpTest {