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