On 6/21/19 1:24 AM, serguei.spit...@oracle.com wrote:
Please, review the test bug fix for:
https://bugs.openjdk.java.net/browse/JDK-8224555
Webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8224555-mon-events2-test.1/
test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/contention/TC02/tc02t001.java
L99: for (int j = 0; j < 1000; j++) {
Why a literal '1000' this time? You could use "timeout / 20"
based on:
L120: timeout = argHandler.getWaitTime() * 60 * 1000;
That seems to be what all the other timeout logic is based on.
test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/contention/TC02/tc02t001/tc02t001.cpp
No comments.
Thumbs up. I don't need a new webrev if you decide to make the
minor change above.
Dan
Summary:
The test sleeps for 1 sec in hope to get a contention on the monitor
tc02t001Thread.M.
It seems, this is not enough when the JFR is enabled.
The fix uses a better approach to ensure events are really happen.
This approach is similar to the on in the fix of 8223736 (just
reviewed).
Also, the class line number sensitive tc02t001Thread is moved to the
beginning of
the file to make it independent from the rest of the file.
Testing:
A mach5 submission is in progress.
Thanks,
Serguei