Hi Gary, Should you not also fail the test if the test fails at popping or resuming the thread?
Apart from that looks good to me (I would remark normally that I'm getting rid of the NSK_CPP_STUBs so you could just jump the gun and remove it directly but this is consistent with the current code base and my scripts will get to it eventually...). Thanks, Jc On Wed, Sep 26, 2018 at 7:54 AM Gary Adams <gary.ad...@oracle.com> wrote: > A race condition exists in hs203t003 between the main test thread and > the processing in callbackFieldAccess processing. The main thread > already includes a polling loop to wait for the redefine class operation > to be performed, but it does not wait for the following suspend thread > operation to be completed. > > This changeset adds an additional wait for the suspend thread to complete. > It also checks the error returns from popThreadFrame and resumeThread > to issue an additional error message if these native routines returned > an error. > > Webrev: http://cr.openjdk.java.net/~gadams/8210984/webrev.00/ > Issue: https://bugs.openjdk.java.net/browse/JDK-8210984 > -- Thanks, Jc
