Skip to content

Commit

Permalink
8255706: The JDWP debug agent unecessarily checks for JVMTI_ERROR_INT…
Browse files Browse the repository at this point in the history
…ERRUPT after calling RawMonitorEnter

Reviewed-by: alanb, dholmes, sspitsyn
  • Loading branch information
plummercj committed Nov 5, 2020
1 parent e66fd6f commit e42c134
Showing 1 changed file with 3 additions and 10 deletions.
13 changes: 3 additions & 10 deletions src/jdk.jdwp.agent/share/native/libjdwp/util.c
Expand Up @@ -1008,16 +1008,9 @@ void
debugMonitorEnter(jrawMonitorID monitor)
{
jvmtiError error;
while (JNI_TRUE) {
error = JVMTI_FUNC_PTR(gdata->jvmti,RawMonitorEnter)
(gdata->jvmti, monitor);
error = ignore_vm_death(error);
if (error == JVMTI_ERROR_INTERRUPT) {
handleInterrupt();
} else {
break;
}
}
error = JVMTI_FUNC_PTR(gdata->jvmti,RawMonitorEnter)
(gdata->jvmti, monitor);
error = ignore_vm_death(error);
if (error != JVMTI_ERROR_NONE) {
EXIT_ERROR(error, "on raw monitor enter");
}
Expand Down

1 comment on commit e42c134

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.