All of lore.kernel.org
 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 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.