Integrated: 8278891: G1: Call reset in G1RegionMarkStatsCache constructor

Hamlin Li mli at openjdk.java.net
Fri Dec 17 03:33:27 UTC 2021


On Thu, 16 Dec 2021 10:08:20 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Currently, G1RegionMarkStatsCache constructor does not call G1RegionMarkStatsCache::reset(), it depends on user to call it explicitly to make sure it works well and without crash in G1RegionMarkStatsCache::evict(uint idx).
> 
> In fact G1RegionMarkStatsCache constructor should call G1RegionMarkStatsCache::reset() to ensure its valid initial state.
> 
> ( I met this issue when I tried to reuse this class to implement some parallel cache when improve the parallelism of evacuation failure. )

This pull request has now been integrated.

Changeset: 8c73ec15
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/8c73ec155dd9324367d2a8e5f2e49da11683f8c8
Stats:     5 lines in 2 files changed: 1 ins; 4 del; 0 mod

8278891: G1: Call reset in G1RegionMarkStatsCache constructor

Reviewed-by: tschatzl, ayang

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

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



More information about the hotspot-gc-dev mailing list