RFR: 8214118: HeapRegions marked as archive even if CDS mapping fails
Thomas Schatzl
thomas.schatzl at oracle.com
Wed Nov 21 20:29:23 UTC 2018
Hi,
On Wed, 2018-11-21 at 15:43 +0100, Stefan Johansson wrote:
> Hi,
>
> Please review this fix to avoid leaving heap metadata in an
> inconsistent state when CDS archive mapping fails.
>
> Issue: https://bugs.openjdk.java.net/browse/JDK-8214118
> Webrev: http://cr.openjdk.java.net/~sjohanss/8214118/00/
the change seems good. I have two comments on this though:
- would it be useful to clear both maps in
G1ArchiveAllocator::clear_range_archive()? I mean in the failure case
these regions can't be used for one purpose or the other as soon as we
fail the mapping, and saves passing the bool flag.
- this is a cleanup request for the future: is there a reason to have
separate tables for open and closed archives? I understand that the
code happened this way because these types of archives were added at
different times, but in my understanding a region in that table can
either be one or the other, and by using a uint8_t as per-region entry
we can even save space.
Thanks,
Thomas
More information about the hotspot-dev
mailing list