RFR: Let nmethods be is_unloading() outside of safepoints
Erik Österlund
erik.osterlund at oracle.com
Mon Nov 12 22:44:33 UTC 2018
Hi,
All current GCs perform code cache unloading in safepoints. Therefore,
nmethods that are alive but is_unloading() are never observed outside of
safepoints. With concurrent class unloading, nmethods that are alive but
is_unloading() will become observable outside of safepoints. This must
be handled appropriately.
In this patch I changed the nmethod/compiled method iterators to accept
parameters describing whether they should filter out not is_alive() or
is_unloading() methods. Since there is no obvious default (all
combinations are used depending on call site), you have to explicitly
set what iteration mode you want.
Other than that, I make sure that the sweeper stays away from
is_unloading() nmethods that are not yet is_unloaded(). To make the
interactions between the sweeper and concurrent GC threads safe, I had
to move down the store that sets the state to unloaded, and use a
release_store there, to make sure no accesses float below it at
compile-time. Once that store is observed, nmethods may be deleted.
In the IC miss handler, I also need to lazily clean stale IC caches due
to calling is_unloading nmethods using nmethod entry barriers.
Webrev:
http://cr.openjdk.java.net/~eosterlund/8213755/webrev.00/
Bug:
https://bugs.openjdk.java.net/browse/JDK-8213755
Thanks,
/Erik
More information about the hotspot-dev
mailing list