On Fri, 19 May 2023 03:02:33 GMT, David Holmes <[email protected]> wrote:
>> Paul Hohensee has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8304074: Need acquire/release in incr_exited_allocated_bytes
>
> src/hotspot/share/services/threadService.hpp line 109:
>
>> 107: static jlong get_daemon_thread_count() { return
>> _atomic_daemon_threads_count; }
>> 108:
>> 109: static jlong exited_allocated_bytes() { return
>> _exited_allocated_bytes; }
>
> This needs to be an atomic::load too .
Done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13814#discussion_r1199651489
