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

Reply via email to