Hi Nick,
The fix looks good to me.
Thank you for taking care about it!
Thanks,
Serguei
On 3/20/19 23:58, Nick Gasson wrote:
Hi,
Please review this small fix to a bug that causes the following tests
to fail when run with jtreg -timeoutFactor > 10 after the changes in
8207367:
vmTestbase/nsk/jdi/EventQueue/remove_l/remove_l004/TestDescription.java
vmTestbase/nsk/jdi/EventQueue/remove/remove004/TestDescription.java
Bug: https://bugs.openjdk.java.net/browse/JDK-8220451
Webrev: http://cr.openjdk.java.net/~ngasson/8220451/webrev.0
This test creates a debugee process that sleeps for 5 * timeoutFactor
* 10000 ms or until it is signalled to stop, and the parent sleeps for
5 * timeoutFactor * 1000 ms then signals the child and checks no
unexpected events were received in that time. However the jtreg
timeout factor is not passed from parent to debugee so the debugee
uses the default value 1.0. So if the jtreg timeout factor is 12 the
parent will sleep for 5 * 12 * 1000 = 60000 ms and the debugee will
sleep for 5 * 1 * 10000 = 50000 ms and exit before the parent wakes
up. The debugee exiting causes an unexpected event and the test fails.
Fix by passing the timeout factor system property to the debugee.
Tested with `make test TEST="vmTestbase/nsk/jdi/EventQueue"
JTREG="TIMEOUT_FACTOR=12"' and JTREG="TIMEOUT_FACTOR=4".
Thanks,
Nick
