RFR: 8346572: Check is_reserved() before using ReservedSpace instances [v2]

Thomas Schatzl tschatzl at openjdk.org
Fri Jan 24 09:26:54 UTC 2025


On Thu, 23 Jan 2025 12:40:06 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> There are a number of places where we reserve memory and create a ReservedSpace, and after the use the created instance without checking if the memory actually got reserved and the instance got initialized. This mostly affects code paths during JVM initialization and fixing this will mostly give better error handling and tracing.
>> 
>> The patch also includes some minor restructuring to get early returns and remove redundant null checks after calls to new.
>> 
>> Tested tier1 and GHA, but will run more tests when/if this gets accepted.
>
> Stefan Karlsson has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
> 
>  - Merge remote-tracking branch 'upstream/master' into 8346572_check_is_reserved
>  - 8346572: Check is_reserved() before using ReservedSpace instances

Marked as reviewed by tschatzl (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/22825#pullrequestreview-2572119144


More information about the hotspot-gc-dev mailing list