RFR: 8265753: Remove manual JavaThread transitions to blocked [v6]
Robbin Ehn
rehn at openjdk.java.net
Tue May 25 06:35:11 UTC 2021
On Mon, 24 May 2021 16:28:15 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:
>> Robbin Ehn has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Final fixes: last famous words
>
> src/hotspot/share/prims/jvmtiRawMonitor.cpp line 405:
>
>> 403:
>> 404: guarantee(self == _owner, "invariant");
>> 405: guarantee(save == _recursions, "invariant");
>
> Since you just restored the `_recursions` field on L402, why do
> you need this guarantee() 2 code lines later?
I left it since if we do changes here those two should be true even after any changes.
I can remove it.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3875
More information about the serviceability-dev
mailing list