RFR: 8277481: Obsolete seldom used CDS flags
Calvin Cheung
ccheung at openjdk.java.net
Fri Dec 10 19:41:16 UTC 2021
On Fri, 10 Dec 2021 15:01:29 GMT, Harold Seigel <hseigel at openjdk.org> wrote:
> Please review this change to obsolete deprecated CDS options UseSharedSpaces, RequireSharedSpaces, DynamicDumpSharedSpaces, and DumpSharedSpaces. The change was tested by running Mach5 tiers 1-2 on Linux, Mac OS, and Windows and Mach5 tiers 3-5 on Linux x64 and Windows x64.
>
> The use of UseSharedSpaces in ps_core_common.c was tested on Mac OS x64 by temporarily removing serviceability/sa/ClhsdbPmap.java#core from the problem list.
>
> Thanks! Harold
Looks good. Just one nit.
src/jdk.hotspot.agent/share/native/libsaproc/ps_core_common.c line 303:
> 301: useSharedSpacesAddr = lookup_symbol(ph, jvm_name, USE_SHARED_SPACES_SYM);
> 302: if (useSharedSpacesAddr == 0) {
> 303: print_debug("can't lookup 'UseSharedSpaces' symbol\n");
Maybe the `print_debug` at line 311 should also be updated from "flag" to "symbol"?
-------------
Marked as reviewed by ccheung (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/6800
More information about the serviceability-dev
mailing list