RFR: 8278891: G1: Call reset in G1RegionMarkStatsCache constructor [v2]

Hamlin Li mli at openjdk.java.net
Fri Dec 17 01:10:54 UTC 2021


> 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. )

Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:

  Albert review

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/6861/files
  - new: https://git.openjdk.java.net/jdk/pull/6861/files/4855018c..fc5f4ad5

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=6861&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=6861&range=00-01

  Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6861.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6861/head:pull/6861

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



More information about the hotspot-gc-dev mailing list