RFR: 8273300: Check Mutex ranking during a safepoint [v6]
Coleen Phillimore
coleenp at openjdk.java.net
Wed Sep 15 16:32:24 UTC 2021
On Wed, 15 Sep 2021 16:02:18 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Remove safepoint conditional from other assert. The locks should now be in decreasing order.
>
> src/hotspot/share/runtime/mutex.cpp line 375:
>
>> 373: Mutex* locks_owned = thread->owned_locks();
>> 374:
>> 375: if (!SafepointSynchronize::is_at_safepoint()) {
>
> I think removing this conditional was okay in your previous change.
Thanks Patricio for noticing this omission. I've removed it and retesting.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5467
More information about the shenandoah-dev
mailing list