RFR: 8143847: Remove REF_CLEANER reference category

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Fri Jan 8 20:04:22 UTC 2016


Looks good!

Since you change vmStructs.cpp it's a good idea to run some SA test just to make 
sure there is no use of the deleted field in there. There's a bunch of jtreg 
tests in hotspot/test/serviceability/sa/. Last time I checked these were not run 
by jprt.

Slightly unrelated: Is it worth to make number_of_subclasses_of_ref() slightly 
more robust by adding a "REF_LAST" or similar to use in there? I guess we don't 
intend to add more reference types, but it is easy to forget to update this 
method if a new type is added last in the enum.

Thanks,
/Jesper

Den 8/1/16 kl. 20:46, skrev Kim Barrett:
> Please review this change to remove the REF_CLEANER reference
> category.  Now that JDK-8071507 has changed PhantomReferences
> (REF_PHANTOM) to also be cleared when notified, the distinction
> between REF_CLEANER and REF_PHANTOM is no longer needed.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8143847
>
> Webrev:
> http://cr.openjdk.java.net/~kbarrett/8143847/webrev.00/
>
> Testing:
> JPRT
>



More information about the hotspot-gc-dev mailing list