-
Bug
-
Resolution: Fixed
-
P4
-
18
-
b07
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8281446 | 17.0.4-oracle | Adithya Haradi Gopal | P4 | Resolved | Fixed | b01 |
JDK-8280285 | 17.0.3 | Goetz Lindenmaier | P4 | Resolved | Fixed | b01 |
If the ergonomically selected GC doesn't support string deduplication, all tests in gc/stringdedup are failing. This is the case for client VMs.
I think the problem is that the requires "vm.gc == "null"" basically means, that the test case should be executed if a GC has been selected ergonomically, but "gc.gc == "G1"" only means that, the G1 gc is present and can be selected. So the test case is executed (and fails) if, for instance, the ergo GC is Serial, and G1 is compiled into the JVM.
https://git.openjdk.java.net/jdk/blob/master/test/hotspot/jtreg/gc/stringdedup/TestStringDeduplicationAgeThreshold.java
Since certain GCs are required. I think the test case should be executed in othervm mode with the specific GC enabled, if the GC is available.
I think the problem is that the requires "vm.gc == "null"" basically means, that the test case should be executed if a GC has been selected ergonomically, but "gc.gc == "G1"" only means that, the G1 gc is present and can be selected. So the test case is executed (and fails) if, for instance, the ergo GC is Serial, and G1 is compiled into the JVM.
https://git.openjdk.java.net/jdk/blob/master/test/hotspot/jtreg/gc/stringdedup/TestStringDeduplicationAgeThreshold.java
Since certain GCs are required. I think the test case should be executed in othervm mode with the specific GC enabled, if the GC is available.
- backported by
-
JDK-8280285 Stringdedup tests are failing if the ergonomically select GC does not support it
-
- Resolved
-
-
JDK-8281446 Stringdedup tests are failing if the ergonomically select GC does not support it
-
- Resolved
-
- relates to
-
JDK-8271834 TestStringDeduplicationAgeThreshold intermittent failures on Shenandoah
-
- Resolved
-
- links to
-
Commit openjdk/jdk17u-dev/3ca676cd
-
Commit openjdk/jdk/3fc761d9
-
Review openjdk/jdk17u-dev/105
-
Review openjdk/jdk/4603
(2 links to)