All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Eric Auger <eric.auger@redhat.com>
Cc: eesposit@redhat.com, jean-philippe@linaro.org,
	qemu-devel@nongnu.org, Coiby.Xu@gmail.com, qemu-arm@nongnu.org,
	clg@kaod.org, Stefan Hajnoczi <stefanha@redhat.com>,
	pbonzini@redhat.com, eric.auger.pro@gmail.com,
	david@gibson.dropbear.id.au
Subject: Re: [PATCH v4 3/4] tests/qtest/vhost-user-blk-test: Temporary hack to get tests passing on aarch64
Date: Tue, 8 Mar 2022 06:09:13 -0500	[thread overview]
Message-ID: <20220308060823-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <d85bd084-2856-87d6-3b98-148b2a7fa03a@redhat.com>

On Tue, Mar 08, 2022 at 11:09:13AM +0100, Eric Auger wrote:
> Hi Stefan,
> 
> On 3/8/22 11:02 AM, Stefan Hajnoczi wrote:
> > On Mon, Mar 07, 2022 at 10:14:38PM +0100, Eric Auger wrote:
> >> When run on ARM, basic and indirect tests currently fail with the
> >> following error:
> >>
> >> ERROR:../tests/qtest/libqos/virtio.c:224:qvirtio_wait_used_elem:
> >> assertion failed (got_desc_idx == desc_idx): (50331648 == 0)
> >> Bail out! ERROR:../tests/qtest/libqos/virtio.c:224: qvirtio_wait_used_elem:
> >> assertion failed (got_desc_idx == desc_idx): (50331648 == 0)
> >>
> >> I noticed it worked when I set up MSI and I further reduced the
> >> code to a simple guest_alloc() that removes the error. At the moment
> >> I am not able to identify where ths issue is and this blocks the
> >> whole pci/aarch64 enablement.
> >>
> >> Signed-off-by: Eric Auger <eric.auger@redhat.com>
> >> ---
> >>  tests/qtest/vhost-user-blk-test.c | 6 ++++++
> >>  1 file changed, 6 insertions(+)
> > What is the plan to identify the root cause?
> To be honest, I have no precise plan yet. However I commit to work on it
> within the next weeks.
> 
> If maintainers prefer to delay the introduction of qtest libqos
> pci/aarch64 tests until this gets fixed, I would understand though.
> 
> Eric

Given soft freeze sounds like a good plan.

> >
> > Acked-by: Stefan Hajnoczi <stefanha@redhat.com>


  reply	other threads:[~2022-03-08 11:09 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-07 21:14 [PATCH v4 0/4] qtests/libqos: Allow PCI tests to be run with virt-machine Eric Auger
2022-03-07 21:14 ` [PATCH v4 1/4] tests/qtest/libqos/pci: Introduce pio_limit Eric Auger
2022-03-07 21:14 ` [PATCH v4 2/4] tests/qtest/libqos: Skip hotplug tests if pci root bus is not hotpluggable Eric Auger
2022-03-07 21:14 ` [PATCH v4 3/4] tests/qtest/vhost-user-blk-test: Temporary hack to get tests passing on aarch64 Eric Auger
2022-03-08 10:02   ` Stefan Hajnoczi
2022-03-08 10:09     ` Eric Auger
2022-03-08 11:09       ` Michael S. Tsirkin [this message]
2022-03-07 21:14 ` [PATCH v4 4/4] tests/qtest/libqos: Add generic pci host bridge in arm-virt machine Eric Auger

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=20220308060823-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=Coiby.Xu@gmail.com \
    --cc=clg@kaod.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=eesposit@redhat.com \
    --cc=eric.auger.pro@gmail.com \
    --cc=eric.auger@redhat.com \
    --cc=jean-philippe@linaro.org \
    --cc=pbonzini@redhat.com \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@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.