On Fri, 15 Oct 2021 11:52:09 GMT, Richard Reingruber <rr...@openjdk.org> wrote:
> > > Hm, I think this can be simplified by swaping blockOnDebuggerSuspend() and > trackAppResume(). Can't try it today but will on Monday. I've done that with commit https://github.com/openjdk/jdk/pull/5849/commits/0b0fef0e6670c20a0e1e34323847c5a622878469. The locking is clearer I would say. Since `resumeFrameDepth` is now set before resumee's suspendCount is 0 we must block debugger suspends only if `!handlingAppResume` because then we know resumee's suspendCount actually is 0. ------------- PR: https://git.openjdk.java.net/jdk/pull/5849