On 11/11/2020 7:51 am, Daniel D.Daugherty wrote:
On Tue, 10 Nov 2020 21:16:33 GMT, Robbin Ehn <r...@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
