RFR: 8368080: G1: Unnecessary initialization of G1CMTask's mark stats table

Thomas Schatzl tschatzl at openjdk.org
Tue Sep 23 09:10:31 UTC 2025


On Mon, 22 Sep 2025 09:25:43 GMT, Ivan Walulya <iwalulya at openjdk.org> wrote:

>> Question: There is a use of `G1RegionMarkStatsCache` in `G1CMTask` (responsible for concurrent marking as well?) -- shouldn't the reset happen on that path as well? If so, and testing did not caught it, there should be some sort of assert somewhere?
>
>> Question: There is a use of `G1RegionMarkStatsCache` in `G1CMTask` (responsible for concurrent marking as well?) -- shouldn't the reset happen on that path as well? If so, and testing did not caught it, there should be some sort of assert somewhere?
> 
> That reset was already happening in `G1ConcurrentMark::reset()`.

Thanks @walulyai @shipilev for your reviews

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

PR Comment: https://git.openjdk.org/jdk/pull/27390#issuecomment-3323053535


More information about the hotspot-gc-dev mailing list