RFR: 8351040: [REDO] Protection zone for easier detection of accidental zero-nKlass use [v3]
Thomas Stuefe
stuefe at openjdk.org
Sun Mar 30 09:11:20 UTC 2025
On Thu, 27 Mar 2025 13:42:14 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
>> Thomas Stuefe has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
>>
>> - Merge branch 'openjdk:master' into JDK-8351040-REDO-Protection-zone-for-easier-detection-of-accidental-zero-nKlass-use
>> - skip test if we have no COH archive
>> - Merge branch 'openjdk:master' into JDK-8351040-REDO-Protection-zone-for-easier-detection-of-accidental-zero-nKlass-use
>> - aix fix
>> - test and aix exclusion
>> - Fix windows when ArchiveRelocationMode=0 or 2
>> - original
>
> src/hotspot/share/cds/archiveBuilder.cpp line 329:
>
>> 327: if (CDSConfig::is_dumping_static_archive()) {
>> 328: _current_dump_region = &_pz_region;
>> 329: _current_dump_region->init(&_shared_rs, &_shared_vs);
>
> Second line in 'if' and 'else' seems to be identical ?
fixed
> src/hotspot/share/cds/archiveUtils.cpp line 85:
>
>> 83:
>> 84: // The number of bits used by the rw/ro ptrmaps. We might have lots of zero
>> 85: // bits at the bottom and top of rrw/ro ptrmaps, but these zeros will be
>
> What means rrw here ?
typo; foxed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23912#discussion_r2020104442
PR Review Comment: https://git.openjdk.org/jdk/pull/23912#discussion_r2020104299
More information about the hotspot-dev
mailing list