[2/2] automation: increase tests maximum time from 10s to 30s
diff mbox series

Message ID 20191203103352.29728-3-roger.pau@citrix.com
State New
Headers show
Series
  • automation: improve tests
Related show

Commit Message

Roger Pau Monne Dec. 3, 2019, 10:33 a.m. UTC
10s is too low for the clang tests, this is the output from a clang
test:

(XEN) [    6.512748] ***************************************************
(XEN) [    6.513323] SELFTEST FAILURE: CORRECT BEHAVIOR CANNOT BE GUARANTEED
(XEN) [    6.513891] ***************************************************
(XEN) [    6.514469] 3... 2... 1...
(XEN) [    9.520011] *** Serial input to DOM0 (type 'CTRL-a' three times to switch input)
(XEN) [    9.544319] Freed 488kB init memory
--- Xen Test Framework ---
Environment: HVM 32bit (PAE 3 levels)
Hello World
Test result: SUCCESS
(XEN) [    9.610977] Hardware Dom0 halted: halting machine

As can be seen from the output above booting Xen and the XTF test
takes ~10s, without accounting for the time it takes for QEMU to
initialize.

Increase the timeout to 30s to be on the safe side.

Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
---
 automation/scripts/qemu-smoke-x86-64.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Wei Liu Dec. 3, 2019, 11:23 a.m. UTC | #1
On Tue, 3 Dec 2019 at 10:34, Roger Pau Monne <roger.pau@citrix.com> wrote:
>
> 10s is too low for the clang tests, this is the output from a clang
> test:
>
> (XEN) [    6.512748] ***************************************************
> (XEN) [    6.513323] SELFTEST FAILURE: CORRECT BEHAVIOR CANNOT BE GUARANTEED
> (XEN) [    6.513891] ***************************************************
> (XEN) [    6.514469] 3... 2... 1...
> (XEN) [    9.520011] *** Serial input to DOM0 (type 'CTRL-a' three times to switch input)
> (XEN) [    9.544319] Freed 488kB init memory
> --- Xen Test Framework ---
> Environment: HVM 32bit (PAE 3 levels)
> Hello World
> Test result: SUCCESS
> (XEN) [    9.610977] Hardware Dom0 halted: halting machine
>
> As can be seen from the output above booting Xen and the XTF test
> takes ~10s, without accounting for the time it takes for QEMU to
> initialize.
>
> Increase the timeout to 30s to be on the safe side.
>
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>

Acked-by: Wei Liu <wl@xen.org>

Patch
diff mbox series

diff --git a/automation/scripts/qemu-smoke-x86-64.sh b/automation/scripts/qemu-smoke-x86-64.sh
index f38eacfd9f..09152e3e9c 100755
--- a/automation/scripts/qemu-smoke-x86-64.sh
+++ b/automation/scripts/qemu-smoke-x86-64.sh
@@ -21,7 +21,7 @@  esac
 
 rm -f smoke.serial
 set +e
-timeout -k 1 10 \
+timeout -k 1 30 \
 qemu-system-x86_64 -nographic -kernel binaries/xen \
         -initrd xtf/tests/example/$k \
         -append "loglvl=all com1=115200,,8n1 console=com1 noreboot \