support/testing/infra/emulator.py: add host load info in run log
Runtime tests running on test runners are subject to a high variability in term of performance and timing. Most or the runtime test commands are executed with a timeout, in pexpect. Slow or very loaded test runners can use the timeout_multiplier to globally increase those timeouts. Some runtime test commands sometimes needs to poll or query a state, rather than having purely sequential actions. It is sometimes hard to know, from the test writer point of view, the maximum timeout to set, or if a retry logic is needed. In order to help debugging runtime tests failing due very slow execution, this commit adds extra information on the host test runner about its load in the run log. Relevant information are: number of cpus, the load average at the moment the emulator is started and the current timeout_multiplier. Note: this change was discussed in: https://lists.buildroot.org/pipermail/buildroot/2024-July/759119.html Signed-off-by: Julien Olivain <ju.o@free.fr> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> (cherry picked from commit 7a6edbc7b9166c799b43cf9a9b78422c8e20ccc0) Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
This commit is contained in:
parent
c38cb8488e
commit
d46b2f5fea
@ -2,6 +2,8 @@ import pexpect
|
||||
|
||||
import infra
|
||||
|
||||
import os
|
||||
|
||||
|
||||
class Emulator(object):
|
||||
|
||||
@ -73,6 +75,11 @@ class Emulator(object):
|
||||
if kernel_cmdline:
|
||||
qemu_cmd += ["-append", " ".join(kernel_cmdline)]
|
||||
|
||||
self.logfile.write(f"> host cpu count: {os.cpu_count()}\n")
|
||||
ldavg = os.getloadavg()
|
||||
ldavg_str = f"{ldavg[0]:.2f}, {ldavg[1]:.2f}, {ldavg[2]:.2f}"
|
||||
self.logfile.write(f"> host loadavg: {ldavg_str}\n")
|
||||
self.logfile.write(f"> timeout multiplier: {self.timeout_multiplier}\n")
|
||||
self.logfile.write("> starting qemu with '%s'\n" % " ".join(qemu_cmd))
|
||||
self.qemu = pexpect.spawn(qemu_cmd[0], qemu_cmd[1:],
|
||||
timeout=5 * self.timeout_multiplier,
|
||||
|
Loading…
Reference in New Issue
Block a user