RFR: 8301715: CDS should be disabled in exploded JDK [v2]

Matias Saavedra Silva matsaave at openjdk.org
Wed Mar 1 21:38:12 UTC 2023


On Tue, 28 Feb 2023 00:46:32 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Matias Saavedra Silva has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Replaced condition with assert
>
> src/hotspot/share/runtime/arguments.cpp line 2927:
> 
>> 2925:     UseSharedSpaces = false;
>> 2926:     RequireSharedSpaces = false;
>> 2927:   }
> 
> I think you should call `no_shared_spaces()` instead. That way, the VM will print an error message and exit if `-Xshare:on` is specified.

Adding `no_shared_spaces()` here makes sense but it results in a duplicate output. I will move this check up to the caller of this method to avoid redundant logs.

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

PR: https://git.openjdk.org/jdk/pull/12705


More information about the hotspot-runtime-dev mailing list