qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: John Levon <john.levon@nutanix.com>
To: "Cédric Le Goater" <clg@redhat.com>
Cc: qemu-devel@nongnu.org, "Zhao Liu" <zhao1.liu@intel.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"Thomas Huth" <thuth@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@linaro.org>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Thanos Makatos" <thanos.makatos@nutanix.com>
Subject: Re: [PATCH v3 0/3] vfio-user client functional test
Date: Tue, 2 Sep 2025 10:37:31 +0100	[thread overview]
Message-ID: <aLa6297F4kdr2qYH@lent> (raw)
In-Reply-To: <0d030541-e533-489f-bc0c-82ee2c7fcd70@redhat.com>

On Tue, Sep 02, 2025 at 11:25:46AM +0200, Cédric Le Goater wrote:

> On 8/28/25 12:05, John Levon wrote:
> > Add a basic functional test for the vfio-user client, along with a couple of
> > test framework extensions to support it.
> > 
> > John Levon (2):
> >    tests/functional: return output from cmd.py helpers
> >    tests/functional: add vm param to cmd.py helpers
> > 
> > Mark Cave-Ayland (1):
> >    tests/functional: add a vfio-user smoke test
> > 
> >   MAINTAINERS                                   |   1 +
> >   tests/functional/qemu_test/cmd.py             |  65 ++-
> >   tests/functional/x86_64/meson.build           |   1 +
> >   .../x86_64/test_vfio_user_client.py           | 407 ++++++++++++++++++
> >   4 files changed, 462 insertions(+), 12 deletions(-)
> >   create mode 100755 tests/functional/x86_64/test_vfio_user_client.py
> > 
> 
> The test is failing :

What is the parent commit you tried this with?

I've had to fix this from a rebasing before, but it seems it's more sensitive
than I expected, and we'll have to be looser in terms of what we expect to
see...

regards
john


  reply	other threads:[~2025-09-02  9:40 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-28 10:05 [PATCH v3 0/3] vfio-user client functional test John Levon
2025-08-28 10:05 ` [PATCH v3 1/3] tests/functional: return output from cmd.py helpers John Levon
2025-08-28 10:05 ` [PATCH v3 2/3] tests/functional: add vm param to " John Levon
2025-08-28 10:05 ` [PATCH v3 3/3] tests/functional: add a vfio-user smoke test John Levon
2025-09-02  9:25 ` [PATCH v3 0/3] vfio-user client functional test Cédric Le Goater
2025-09-02  9:37   ` John Levon [this message]
2025-09-02 13:03     ` 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=aLa6297F4kdr2qYH@lent \
    --to=john.levon@nutanix.com \
    --cc=berrange@redhat.com \
    --cc=clg@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=philmd@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=thanos.makatos@nutanix.com \
    --cc=thuth@redhat.com \
    --cc=zhao1.liu@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).