RFR: 8066827: Remove ReferenceProcessor::clean_up_discovered_references()

Kim Barrett kim.barrett at oracle.com
Thu Dec 18 03:21:38 UTC 2014


On Dec 17, 2014, at 9:51 PM, Srinivas Ramakrishna <ysr1729 at gmail.com> wrote:
> Looks good to me.
> 
> -- Ramki
> 
> ysr1729
> 
>> On Dec 11, 2014, at 23:28, Kim Barrett <kim.barrett at oracle.com> wrote:
>> 
>> Note: I will need a sponsor for this change.
>> 
>> Please review this change to replace the one call to
>> ReferenceProcessor::clean_up_discovered_references() with a call to
>> ReferenceProcessor::abandon_partial_discovery(), and then remove the
>> clean_up function and some follow-on cleanup.
>> 
>> […]
>> 
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8066827 
>> 
>> Webrev:
>> http://cr.openjdk.java.net/~kbarrett/8066827/webrev/
>> 
>> Testing: JPRT, local jtreg of gc tests
>> refworkload with CMS as the collector, with TraceCMSState enabled so I
>> could examine the logs and verify some of the tests passed through the
>> changed code; there were 113 occurrences with an iteration count of 1.
>> 
>> JPRT was after Bengt's recent changes to include gc jtreg tests.

Thanks for your review, Ramki.




More information about the hotspot-gc-dev mailing list