RFR: 8256265: G1: Improve parallelism in regions that failed evacuation [v12]

Hamlin Li mli at openjdk.java.net
Fri Mar 18 04:51:21 UTC 2022


On Tue, 15 Mar 2022 14:02:35 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Albert review
>
> src/hotspot/share/gc/g1/g1HeapRegionChunk.cpp line 30:
> 
>> 28: #include "gc/g1/g1ConcurrentMarkBitMap.inline.hpp"
>> 29: #include "gc/g1/g1HeapRegionChunk.hpp"
>> 30: #include "gc/g1/heapRegion.hpp"
> 
> I think including the .inline is better here as likely one or more of its methods are used in this code (I did not check), but there is a risk on the .inline contents being imported from somewhere else.

Seems no inline methods of HeapRegion are used here.

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

PR: https://git.openjdk.java.net/jdk/pull/7047



More information about the hotspot-gc-dev mailing list