RFR: 8317371: GenShen: Needs improved support for humongous allocation [v14]

[William Kemper]

On Mon, 23 Oct 2023 20:37:14 GMT, Kelvin Nilsen <kdnilsen at openjdk.org> wrote:

>> src/hotspot/share/gc/shenandoah/shenandoahConcurrentGC.cpp line 174:
>> 
>>> 172: 
>>> 173:   {
>>> 174:     // TODO: Not sure there is value in logging free-set status right here
>> 
>> Doesn't early cleanup rebuild the freeset?
>
> I'm not sure.  It does recycle immediate trash.  But it appears to me that the rebuild of freeset following the recycling of trash happens somewhere else.  Of note, the last thing done at the end of each free set rebuild is to log the status, so even if we do rebuild inside entry_cleanup_early(), there would be no need to log the status again right here.
> 
> Of course, one reason I left this as a TODO comment instead of just yanking out the code is because it is a bit difficult to understand the control flows, so I thought this could deserve some additional attention at a later time.

Okay.

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/336#discussion_r1369329555