On Fri, 17 Apr 2026 01:48:08 GMT, Leonid Mesnik <[email protected]> wrote:

>> I don't think it is important here. The check_hashmaps_for_heapwalk is 
>> called once during heapwalking operation which is extremely slow already.
>
> The another difference observed by @sspitsyn while discussing this issue.
> The remove_dead_entries_locked() is now called and work even if
> ` env()->is_enabled(JVMTI_EVENT_OBJECT_FREE)` 
> is false.
> It just cleans up tagmap table, but doesn't post events. Shouldn't change 
> behaviour observed by jvmti agent. 
> The performance impact is not significant comparing with heap walking.

One more difference that the `check_hashmaps_for_heapwalk()` is also called 
when `objects == nullptr`.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/30757#discussion_r3098303812

Reply via email to