RFR: 8266936: Add a finalization JFR event [v12]
mgronlun at openjdk.java.net
Mon Oct 18 16:51:54 UTC 2021
On Fri, 24 Sep 2021 22:31:18 GMT, Mandy Chung <mchung at openjdk.org> wrote:
>> Markus Grönlund has updated the pull request incrementally with one additional commit since the last revision:
>> no precompiled headers and mtServiceability nmt classification
> Hi Markus,
> It's a little surprised to see Finalizer.c to depend JMM interface which is used by `java.management` and `jdk.management` modules only. It's more appropriate for it to be a JVM_* entry point for Finalizer to report completion of the finalization instead. I understand that you want to make FinalizerService to be a conditional feature which is a good idea. Such JVM entry can be made as a nop if not INCLUDE_SERVICES.
Thank you for staying around with this protracted PR. Thanks @mlchung , @coleenp and @egahlin for your reviews!
More information about the hotspot-runtime-dev