RFR: 8255381: com/sun/jdi/EATests.java should not suspend graal threads [v3]
    Richard Reingruber 
    rrich at openjdk.java.net
       
    Tue Dec  8 14:00:25 UTC 2020
    
    
  
> 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.
-------------
Changes:
  - all: https://git.openjdk.java.net/jdk/pull/1625/files
  - new: https://git.openjdk.java.net/jdk/pull/1625/files/8e4b301f..ce12877f
Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1625&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1625&range=01-02
  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1625.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1625/head:pull/1625
PR: https://git.openjdk.java.net/jdk/pull/1625
    
    
More information about the serviceability-dev
mailing list