RFR: 8292269: Replace FileMapInfo::fail_continue() with Unified Logging [v10]

Matias Saavedra Silva matsaave at openjdk.org
Fri Mar 3 18:10:14 UTC 2023


> The current logging method FileMapInfo::fail_continue() method reports the reason when the CDS archive cannot be mapped and has variable behavior depending on the -Xshare mode. Logging calls are divided primarily between the "warning" and "info" channels, where the "info" channel was used to reduce the number of printed logs for expected and normal failures. 
> 
> Logging will now shift toward Unified Logging, so `fail_continue()` will be replaced with `log_info(cds)` and `log_warning(cds)`. Some genuine failures will be logged to the warning channel for better visibility. Upon actual failures, the VM will exit at MetaspaceShared::initialize_runtime_shared_and_meta_spaces() unless it is otherwise necessary. Relevant tests are updated to accommodate this change. Verified with tier1-4 tests

Matias Saavedra Silva has updated the pull request incrementally with one additional commit since the last revision:

  Added early exit for one case

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12419/files
  - new: https://git.openjdk.org/jdk/pull/12419/files/4887a142..8d7c1d4e

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=12419&range=09
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12419&range=08-09

  Stats: 6 lines in 2 files changed: 5 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/12419.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12419/head:pull/12419

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


More information about the hotspot-runtime-dev mailing list