On Tue, 8 Dec 2020 14:00:25 GMT, Richard Reingruber <rr...@openjdk.org> wrote:
>> This fixes a bug in the test test/jdk/com/sun/jdi/EATests.java that caused >> timeout failures when graal is enabled. >> >> The fix is to avoid suspending all threads when a breakpoint is reached and >> then resume >> just the main thread again. This pattern was used in the test case >> EAMaterializeLocalAtObjectPollReturnReturn. It caused timeouts because graal >> threads remained suspended and, running with -Xbatch, the main thread waited >> (with timeout) for completion of compile tasks. >> The fix was applied to all breakpoints in the test. All explicit suspend >> calls now apply only >> to the main test thread and all explicit resume calls apply to all java >> threads. >> >> Testing: duration of the test case >> EAMaterializeLocalAtObjectPollReturnReturn is >> reduced from 30s to 10s. > > Richard Reingruber has updated the pull request incrementally with one > additional commit since the last revision: > > Only main thread needs to be resumed in EARelockingObjectCurrentlyWaitingOn. Hi Richard, The fix looks okay to me. Thanks, Serguei ------------- Marked as reviewed by sspitsyn (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/1625