RFR: 8062206: Remove unusable G1RSLogCheckCardTable command line argument
Jesper Wilhelmsson
jesper.wilhelmsson at oracle.com
Mon Nov 3 18:48:46 UTC 2014
Looks good!
I can sponsor this.
/Jesper
Kim Barrett skrev 3/11/14 19:41:
> Please review this removal of conditional code that appears to have never
> worked and doesn't have an apparent repair.
>
> Webrev:
> http://cr.openjdk.java.net/~kbarrett/8062206/webrev/
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8062206
>
> Testing:
> jtreg hotspot/test/[closed/]{gc,runtime}, locally (linux x86_64)
> refworkload using G1, locally (linux x86_64)
> jprt -testset hotspot
>
> I'll also need a sponsor for this.
>
> Removed from G1RemSet::updateRS() the assertion that all cards are clean after
> remembered set updating. This "invariant" hasn't held during the entire life
> of G1 in the hotspot repository. Presently, cards for young generation are
> colored "young_card". And there are a variety of reasons why cards colored
> "dirty_card" may remain so (see G1RemSet::refine_card()), so changing the
> assertion to test for no such cards doesn't work either.
>
> Removed the G1RSLogCheckCardTable develop option that controlled the bogus
> assertion, since that's all that option does.
>
> Removed CountNonCleanMemRegionClosure class, since it is no longer used.
>
More information about the hotspot-gc-dev
mailing list