RFR: 8274282: Clarify special wait assert [v2]

Coleen Phillimore coleenp at openjdk.java.net
Mon Oct 4 13:16:34 UTC 2021


On Wed, 29 Sep 2021 04:26:32 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Coleen Phillimore has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR.
>
> src/hotspot/share/runtime/mutex.cpp line 389:
> 
>> 387:     // We enforce not holding locks of rank nosafepoint or lower while waiting for JavaThreads,
>> 388:     // because the held lock has a NoSafepointVerifier so waiting on a lock will block out safepoints.
>> 389:     // For NonJavaThreads, we enforce not waiting on the tty lock since that could block progress also.
> 
> You are checking this for all threads not just NJTs. Is the comment wrong or the code?

I could start the comment like, For JavaThreads, we enforce not holding ... and leave the For NonJavaThreads ...

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

PR: https://git.openjdk.java.net/jdk/pull/5684


More information about the hotspot-runtime-dev mailing list