On Wed, 1 Jul 2026 15:49:54 GMT, Leonid Mesnik <[email protected]> wrote:

>> Thank you for the check. There is the line 3024 which needs the `ets`:
>> `     if (ets->is_enabled(JVMTI_EVENT_SAMPLED_OBJECT_ALLOC) &&`
>
> I wonder if it could be done through 'env' like
> `env->is_enabled(JVMTI_EVENT_VM_OBJECT_ALLOC)`
> ?

This event can be controlled at the thread level, so we need to check it is 
enabled at both thread and env levels.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2607#discussion_r3508133410

Reply via email to