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