RFR (S): 8252034: G1: Remove *g1_reserved* methods
Kim Barrett
kim.barrett at oracle.com
Wed Aug 19 23:55:06 UTC 2020
> On Aug 19, 2020, at 4:51 PM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>
> Hi all,
>
> can I have reviews for this change that removes the G1CollectedHeap::*g1_reserved* methods?
>
> They duplicate existing functionality actually just right below them.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8252034
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8252034/webrev/
> Testing:
> tier1
>
> Thanks,
> Thomas
I dislike that we still have two places (CollectedHeap::_reserved and
HeapRegionManager) that have the reserved region information, and it seems
hard to *prove* they are the same, even though they must be.
But okay, looks good.
More information about the hotspot-gc-dev
mailing list