From c3b208cd76f4afc85a3079aeb41287ab8c679b73 Mon Sep 17 00:00:00 2001 From: Sebastien Hertz Date: Mon, 27 Mar 2017 11:35:54 +0200 Subject: [PATCH] Reduce timeout of JDWP tests This CL sets the timeout of JDWP tests to 10s (instead of the default timeout of 60s). This should save up time for continuous testing (~ 3min on host). Indeed, some tests do wait until the timeout to detect that no reply (or no event) has been received. Test: art/tools/run-jdwp-tests.sh --mode=host --variant=X64 Change-Id: Id03c8a1d629e6da551cf8eb405cc79754a363fe0 --- tools/run-jdwp-tests.sh | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/tools/run-jdwp-tests.sh b/tools/run-jdwp-tests.sh index 6d5c74b82..07c300e7a 100755 --- a/tools/run-jdwp-tests.sh +++ b/tools/run-jdwp-tests.sh @@ -51,6 +51,12 @@ host="no" # Use JIT compiling by default. use_jit=true variant_cmdline_parameter="--variant=X32" +# Timeout of JDWP test in ms. +# +# Note: some tests expect a timeout to check that *no* reply/event is received for a specific case. +# A lower timeout can save up several minutes when running the whole test suite, especially for +# continuous testing. This value can be adjusted to fit the configuration of the host machine(s). +jdwp_test_timeout=10000 while true; do if [[ "$1" == "--mode=host" ]]; then @@ -150,6 +156,8 @@ vogar $vm_command \ $image_compiler_option \ --timeout 800 \ --vm-arg -Djpda.settings.verbose=true \ + --vm-arg -Djpda.settings.timeout=$jdwp_test_timeout \ + --vm-arg -Djpda.settings.waitingTime=$jdwp_test_timeout \ --vm-arg -Djpda.settings.transportAddress=127.0.0.1:55107 \ --vm-arg -Djpda.settings.debuggeeJavaPath="$art_debugee $image $debuggee_args" \ --classpath $test_jack \ -- 2.11.0