8189360: JvmtiExport::weak_oops_do is called for all JNIHandleBlock instances
kim.barrett at oracle.com
Tue Oct 17 19:55:06 UTC 2017
> On Oct 16, 2017, at 11:40 AM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
> Hi all,
> Please review this patch to move the call of the static JvmtiExport::weak_oops_do out of the JNIHandleBlock::weak_oops_do member function into the new WeakProcessor.
> Today, this isn't causing any bugs because there's only one instance of JNIHandleBlock, the _weak_global_handles. However, in prototypes with more than one JNIHandleBlock, this results in multiple calls to JvmtiExport::weak_oops_do.
> This patch builds upon the patch in:
> Tested with JPRT.
Maybe remove #include “prims/jvmtiExport.hpp” ?
Otherwise looks good. I don’t need another webrev for that #include removal.
More information about the hotspot-dev