From: "Cédric Le Goater" <clg@redhat.com>
To: qemu-devel@nongnu.org, qemu-arm@nongnu.org
Cc: "Thomas Huth" <thuth@redhat.com>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Steven Lee" <steven_lee@aspeedtech.com>,
"Troy Lee" <leetroy@gmail.com>,
"Jamin Lin" <jamin_lin@aspeedtech.com>,
"Andrew Jeffery" <andrew@codeconstruct.com.au>,
"Joel Stanley" <joel@jms.id.au>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Cédric Le Goater" <clg@redhat.com>
Subject: [PATCH v2 0/4] tests/functional: Finish conversion of Aspeed tests
Date: Tue, 12 Nov 2024 14:02:42 +0100 [thread overview]
Message-ID: <20241112130246.970281-1-clg@redhat.com> (raw)
Hello,
The first patch is a proposal to fix a long-standing issue when
capturing the console output. In some cases, the expected string does
not match in the output, causing the test to fail with a timeout. The
change introduces a _console_read() routine that processes the console
output character by character as a possible fix.
Last patches complete the conversion of the Aspeed tests and remove
the console workarounds.
Thanks,
C.
Changes in v2:
- Redirected all console output in console.log file
- Added a "found" prefix when an expected string is captured
- Used a user mode network backend in SDK tests
- Changed the expected string to "login:" in buildroot tests
Cédric Le Goater (4):
tests/functional: Introduce _console_read()
tests/functional: Convert Aspeed aarch64 SDK tests
tests/functional: Convert Aspeed arm SDK tests
tests/functional: Remove sleep workarounds from Aspeed tests
tests/avocado/machine_aspeed.py | 202 ------------------------
tests/functional/meson.build | 2 +
tests/functional/qemu_test/cmd.py | 22 ++-
tests/functional/test_aarch64_aspeed.py | 97 ++++++++++++
tests/functional/test_arm_aspeed.py | 79 ++++++++-
5 files changed, 191 insertions(+), 211 deletions(-)
delete mode 100644 tests/avocado/machine_aspeed.py
create mode 100644 tests/functional/test_aarch64_aspeed.py
--
2.47.0
next reply other threads:[~2024-11-12 13:03 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-11-12 13:02 Cédric Le Goater [this message]
2024-11-12 13:02 ` [PATCH v2 1/4] tests/functional: Introduce _console_read() Cédric Le Goater
2024-11-12 13:45 ` Thomas Huth
2024-11-12 13:52 ` Cédric Le Goater
2024-11-12 13:59 ` Thomas Huth
2024-11-12 14:02 ` Daniel P. Berrangé
2024-11-12 13:02 ` [PATCH v2 2/4] tests/functional: Convert Aspeed aarch64 SDK tests Cédric Le Goater
2024-11-12 13:02 ` [PATCH v2 3/4] tests/functional: Convert Aspeed arm " Cédric Le Goater
2024-11-12 13:58 ` Thomas Huth
2024-11-12 13:02 ` [PATCH v2 4/4] tests/functional: Remove sleep workarounds from Aspeed tests Cédric Le Goater
2024-11-12 13:36 ` Thomas Huth
2024-11-12 13:46 ` Cédric Le Goater
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241112130246.970281-1-clg@redhat.com \
--to=clg@redhat.com \
--cc=andrew@codeconstruct.com.au \
--cc=jamin_lin@aspeedtech.com \
--cc=joel@jms.id.au \
--cc=leetroy@gmail.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=steven_lee@aspeedtech.com \
--cc=thuth@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.