On Fri, 24 Sep 2021 15:05:33 GMT, Lin Zang <lz...@openjdk.org> wrote:
>> yes
>> void Mutex::check_no_safepoint_state(Thread* thread) {
>> check_block_state(thread);
>> assert(!thread->is_active_Java_thread() || _safepoint_check_required
>> != _safepoint_check_always,
>> "This lock should always have a safepoint check for Java
>> threads: %s",
>> name());
>> }
>>
>> yes, we exclude the check for a non-java thread, which I thought was an odd
>> exclusion last time I looked. I pass the tests in
>> sun/tools/jmap/BasicJMapTest.java so maybe leave it for now?
>
> I agree, maybe a new issue could be created for tracking this.
👍
-------------
PR: https://git.openjdk.java.net/jdk/pull/5681
