RFR: 8356807: Change log_info(cds) to `MetaspaceShared::report_loading_error()` [v3]

Matias Saavedra Silva matsaave at openjdk.org
Tue Jul 15 15:56:40 UTC 2025


On Tue, 15 Jul 2025 04:24:56 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> The patch changes some of the `aot_log_info(aot)()` to `MetaspaceShared::report_loading_error()` for those referring to an error condition during loading of AOT cache or CDS archive.
>> 
>> Passed tiers 1 - 4 testing.
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
> 
>   update ArchivedModuleWithCustomImageTest.java

LGTM! Just one nit below

src/hotspot/share/cds/filemap.cpp line 2021:

> 2019:   if (prop != nullptr) {
> 2020:     if (has_aot_linked_classes()) {
> 2021:       MetaspaceShared::report_loading_error("%s has aot-linked classes. It cannot be used when the "

The alignment was changed here.

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

Marked as reviewed by matsaave (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26253#pullrequestreview-3021074032
PR Review Comment: https://git.openjdk.org/jdk/pull/26253#discussion_r2207905581


More information about the hotspot-runtime-dev mailing list