RFR: 8332105: Exploded JDK does not include CDS

David Holmes dholmes at openjdk.org
Mon May 27 06:14:02 UTC 2024


On Sat, 11 May 2024 06:13:29 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> An exploded JDK cannot be used with either -Xshare:on or -Xshare:auto. That causes tests like runtime/CompressedOops/CompressedCPUSpecificClassSpaceReservation.java to fail when running on an exploded JDK.
> 
> Since an exploded JDK cannot use CDS, we should - for tests - treat it as if CDS had not been included.
> 
> 
> ----
> 
> Note that I was torn between two ways to fix this:
> 
> - either this fix, which is rather simple and automatically updates the "vm.cds" `@requires` property
> - or to expose "exploded-ness" as a boolean property via `WhiteBox` and `VMProps`(`jdk.exploded`). See this draft PR: https://github.com/openjdk/jdk/pull/19178 . 
> 
> The latter is cleaner and clearer, conveying the message of exploded-ness without muddling it with the CDS aspect. But OTOH the complexity may not be required. 
> 
> I can go either way, though I have a slight preference for this PR, which is why I posted it.

Seems okay. This test should have had  `requires vm.cds` anyway.

Just out of curiosity why is CDS not compatible with an exploded build?

-------------

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19188#pullrequestreview-2080060349


More information about the hotspot-dev mailing list