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