Integrated: 8274687: JDWP deadlocks if some Java thread reaches wait in blockOnDebuggerSuspend
Richard Reingruber
rrich at openjdk.java.net
Mon Nov 15 07:05:44 UTC 2021
On Thu, 7 Oct 2021 13:50:49 GMT, Richard Reingruber <rrich at openjdk.org> wrote:
> This change fixes deadlocks described in the JBS-bug by:
>
> * Releasing `handlerLock` before waiting on `threadLock` in `blockOnDebuggerSuspend()`
>
> * Notifying on `threadLock` in `threadControl_reset()`
>
> Also the actions in handleAppResumeBreakpoint() are moved/deferred until
> doPendingTasks() runs. This is necessary because an event handler must not
> release handlerLock first and foremost because handlers are called while
> iterating the handler chain for an event type which is protected by handlerLock
> (see https://github.com/openjdk/jdk/pull/5805)
>
> The first commit delays the cleanup actions after leaving the loop in
> `debugLoop_run()`. It allows to reproduce the deadlock running the dispose003
> test with the command
>
>
> make run-test TEST=test/hotspot/jtreg/vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose003
>
>
> The second commit adds a new test that reproduces the deadlock when calling
> threadControl_resumeThread() while a thread is waiting in
> blockOnDebuggerSuspend().
>
> The third commit contains the fix described above. With it the deadlocks
> cannot be reproduced anymore.
>
> The forth commit removes the diagnostic code introduced with the first commit again.
>
> The fix passed
>
> test/hotspot/jtreg/serviceability/jdwp
> test/jdk/com/sun/jdi
> test/hotspot/jtreg/vmTestbase/nsk/jdwp
> test/hotspot/jtreg/vmTestbase/nsk/jdi
This pull request has now been integrated.
Changeset: ca2efb73
Author: Richard Reingruber <rrich at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/ca2efb73f59112d9be2ec29db405deb4c58dd435
Stats: 333 lines in 2 files changed: 314 ins; 14 del; 5 mod
8274687: JDWP deadlocks if some Java thread reaches wait in blockOnDebuggerSuspend
Reviewed-by: cjplummer, sspitsyn, rschmelter
-------------
PR: https://git.openjdk.java.net/jdk/pull/5849
More information about the serviceability-dev
mailing list