RFR: 8253064: monitor list simplifications and getting rid of TSM [v5]

David Holmes david.holmes at oracle.com
Wed Nov 11 22:17:48 UTC 2020


On 11/11/2020 7:51 am, Daniel D.Daugherty wrote:
> On Tue, 10 Nov 2020 21:16:33 GMT, Robbin Ehn <rehn at openjdk.org> wrote:
> 
>>> So if I narrow the scope around the ThreadBlockInVM, then it would be fine?
>>>
>>> {
>>>    // Honor block request.
>>>    ThreadBlockInVM tbivm(self);
>>> }
>>>
>>> I can make that change before I integrate...
>>
>> Yes that avoids it!
> 
> Done. I also did the one in ObjectSynchronizer::request_deflate_idle_monitors().

Just to be crystal clear, the change in request_deflate_idle_monitors() 
was not needed as there is no logging code in the scope, and changing it 
was wrong as it put the sleep outside the scope of the TBIVM. Hence it 
was reverted and all is well.

Thanks,
David


More information about the hotspot-dev mailing list