On Wed, 11 Nov 2020 08:26:14 GMT, Robbin Ehn <r...@openjdk.org> wrote:
>> Daniel D. Daugherty has updated the pull request incrementally with one >> additional commit since the last revision: >> >> resolve more robehn and coleenp comments. > > src/hotspot/share/runtime/synchronizer.cpp line 1226: > >> 1224: ThreadBlockInVM tbivm(self->as_Java_thread()); >> 1225: } >> 1226: os::naked_short_sleep(999); // sleep for almost 1 second > > Hi Dan, you need to be blocked while sleeping, otherwise you are blocking > safepoints for "almost 1 second". > So previous code was correct. Good catch! Yes, I should not have written that change so late at night. I know better. I have reverted it. ------------- PR: https://git.openjdk.java.net/jdk/pull/642
