[Buildroot] [PATCH 2/2] support/scripts/boot-qemu-image.py: increase the timeout by 10

[Yann E. MORIN]

Romain, All,

On 2022-01-15 15:33 +0100, Romain Naour spake thusly:
> As for the Buildroot testsuite, multiply every emulator timeout by 10
> to avoid sporadic failures in elastic runners.
> 
> qemu_arm_vexpress_tz_defconfig tested locally with sucess.
> 
> Fixes:
> https://gitlab.com/buildroot.org/buildroot/-/jobs/1970084046

Are you sure increasing the timeout will fix this failure? The
build-run.log contains another error:

    loaded file zImage from 40400000 to 408CAABF, 004CAAC0 bytes
    loaded file rootfs.cpio.uboot from 44000000 to 4438FA4E, 0038FA4F bytes
    Kernel image @ 0x40400000 [ 0x000000 - 0x4caac0 ]
    ## Loading init Ramdisk from Legacy Image at 44000000 ...
       Image Name:   
       Created:      2022-01-13  16:06:53 UTC
       Image Type:   ARM Linux RAMDisk Image (uncompressed)
       Data Size:    3734031 Bytes = 3.6 MiB
       Load Address: 00000000
       Entry Point:  00000000
       Verifying Checksum ... OK
    ## Flattened Device Tree blob at 40000000
       Booting using the fdt blob at 0x40000000
    ERROR: reserving fdt memory region failed (addr=7fe00000 size=200000)
       Using Device Tree in place at 40000000, end 400051f7

So, to me, this is not a timeout issue, but an actual boot issue.

Regards,
Yann E. MORIN.

> Signed-off-by: Romain Naour <romain.naour at gmail.com>
> ---
>  support/scripts/boot-qemu-image.py | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/support/scripts/boot-qemu-image.py b/support/scripts/boot-qemu-image.py
> index fba1533bb7..ba50ea69ee 100755
> --- a/support/scripts/boot-qemu-image.py
> +++ b/support/scripts/boot-qemu-image.py
> @@ -25,7 +25,7 @@ def main():
>      qemu_start = os.path.join(os.getcwd(), 'output/images/start-qemu.sh')
>  
>      child = pexpect.spawn(qemu_start, ['serial-only'],
> -                          timeout=5, encoding='utf-8',
> +                          timeout=50, encoding='utf-8',
>                            env={"QEMU_AUDIO_DRV": "none"})
>  
>      # We want only stdout into the log to avoid double echo
> @@ -36,7 +36,7 @@ def main():
>      time.sleep(1)
>  
>      try:
> -        child.expect(["buildroot login:"], timeout=60)
> +        child.expect(["buildroot login:"], timeout=600)
>      except pexpect.EOF as e:
>          # Some emulations require a fork of qemu-system, which may be
>          # missing on the system, and is not provided by Buildroot.
> @@ -58,7 +58,7 @@ def main():
>      child.sendline("root\r")
>  
>      try:
> -        child.expect(["# "], timeout=60)
> +        child.expect(["# "], timeout=600)
>      except pexpect.EOF:
>          print("Cannot connect to shell")
>          sys.exit(1)
> @@ -69,7 +69,7 @@ def main():
>      child.sendline("poweroff\r")
>  
>      try:
> -        child.expect(["System halted"], timeout=60)
> +        child.expect(["System halted"], timeout=600)
>          child.expect(pexpect.EOF)
>      except pexpect.EOF:
>          pass
> -- 
> 2.31.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot at buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'