From: Daniel Henrique Barboza <danielhb413@gmail.com>
To: Thomas Huth <thuth@redhat.com>, qemu-devel@nongnu.org
Cc: "Alex Bennée" <alex.bennee@linaro.org>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
qemu-ppc@nongnu.org, "Beraldo Leal" <bleal@redhat.com>,
"Cleber Rosa" <crosa@redhat.com>,
"Kautuk Consul" <kconsul@linux.vnet.ibm.com>,
"Harsh Prateek Bora" <harshpb@linux.ibm.com>
Subject: Re: [PATCH] tests/avocado/tuxrun_baselines: Fix ppc64 tests for binaries without slirp
Date: Tue, 6 Jun 2023 17:16:30 -0300 [thread overview]
Message-ID: <e7f67494-c4ab-dcd5-6fbe-0a021288eff2@gmail.com> (raw)
In-Reply-To: <20230606192802.666000-1-thuth@redhat.com>
On 6/6/23 16:28, Thomas Huth wrote:
> The ppc64 tuxrun tests are currently failing if "slirp" has been
> disabled in the binary since they are using "-netdev user" now.
> We have to skip the test if this network backend is missing.
>
> Fixes: 6ee3624236 ("improve code coverage for ppc64")
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Thomas, let me know if you need me to queue this up. I'll send a PR in the
next few days.
Daniel
> tests/avocado/tuxrun_baselines.py | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tests/avocado/tuxrun_baselines.py b/tests/avocado/tuxrun_baselines.py
> index 3a46e7a745..e12250eabb 100644
> --- a/tests/avocado/tuxrun_baselines.py
> +++ b/tests/avocado/tuxrun_baselines.py
> @@ -184,6 +184,7 @@ def common_tuxrun(self,
>
> def ppc64_common_tuxrun(self, sums, prefix):
> # add device args to command line.
> + self.require_netdev('user')
> self.vm.add_args('-netdev', 'user,id=vnet,hostfwd=:127.0.0.1:0-:22',
> '-device', 'virtio-net,netdev=vnet')
> self.vm.add_args('-netdev', '{"type":"user","id":"hostnet0"}',
next prev parent reply other threads:[~2023-06-06 20:17 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-06 19:28 [PATCH] tests/avocado/tuxrun_baselines: Fix ppc64 tests for binaries without slirp Thomas Huth
2023-06-06 19:35 ` Philippe Mathieu-Daudé
2023-06-06 20:16 ` Daniel Henrique Barboza [this message]
2023-06-07 4:22 ` Joel Stanley
2023-06-07 7:40 ` Thomas Huth
2023-06-07 12:49 ` Daniel Henrique Barboza
2023-06-07 14:08 ` Alex Bennée
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=e7f67494-c4ab-dcd5-6fbe-0a021288eff2@gmail.com \
--to=danielhb413@gmail.com \
--cc=alex.bennee@linaro.org \
--cc=bleal@redhat.com \
--cc=crosa@redhat.com \
--cc=harshpb@linux.ibm.com \
--cc=kconsul@linux.vnet.ibm.com \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=thuth@redhat.com \
--cc=wainersm@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.