RFR: 8205050: ZGC: Incorrect use of RootAccess in ZHeapIterator

Stefan Karlsson stefan.karlsson at oracle.com
Thu Jun 14 15:14:48 UTC 2018


Looks good.

StefanK

On 2018-06-14 16:44, Per Liden wrote:
> In ZHeapIteratorRootOopClosure::do_oop() we're using 
> RootAccess<>::oop_load() to load oops. However, as the comment right 
> above that line suggests, that's incorrect and we should have a load 
> barrier here. In fact, we used to have a load barrier here, but this 
> was for some mysterious reason changed. We should revert back to the 
> code that used to be there.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8205050
> Webrev: http://cr.openjdk.java.net/~pliden/8205050/webrev.0
>
> cheers,
> Per





More information about the hotspot-gc-dev mailing list