RFR: 8198509: Move satisfy_failed_metadata_allocation out from CollectorPolicy
stefan.karlsson at oracle.com
Thu Feb 22 15:17:29 UTC 2018
On 2018-02-22 14:50, Per Liden wrote:
> I'm not too sure about moving this into MetaspaceGC, since Metaspace
> would then make assumptions about how the GC works. I think I'd rather
> keep this in CollectedHeap for now. That's also far from perfect, but
> I'd rather contain the "uglyness" in CollectedHeap for now, and work on
> getting the right abstraction in place later.
OK. I moved it as you suggested:
> On 02/21/2018 02:34 PM, Stefan Karlsson wrote:
>> Hi all,
>> Please review this patch to move satisfy_failed_metadata_allocation
>> out from CollectorPolicy.
>> This is done in preparation for the work to remove CollectorPolicy:
>> The function and the code moves to MetaspaceGC, and CollectedHeap gets
>> a virtual function that calls
>> MetaspaceGC::satisfy_failed_metadata_allocation. MetaspaceGC seems
>> like a good place to put this function, but I'm open for suggestions.
>> The current allocation code would then be:
>> virtual CollectedHeap::satisfy_failed_metadata_allocation
>> And GCs that want to provide alternative implementations to
>> CollectedHeap::satisfy_failed_metadata_allocation (ZGC for example),
>> could still easily do so.
More information about the hotspot-gc-dev