All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: Eric Auger <eric.auger@redhat.com>
Cc: eric.auger.pro@gmail.com, pbonzini@redhat.com,
	qemu-arm@nongnu.org, qemu-devel@nongnu.org, mst@redhat.com,
	david@gibson.dropbear.id.au, clg@kaod.org, eesposit@redhat.com,
	Coiby.Xu@gmail.com, stefanha@redhat.com,
	jean-philippe@linaro.org
Subject: Re: [PATCH v3 4/5] tests/qtest/vhost-user-blk-test: Temporary hack to get tests passing on aarch64
Date: Wed, 06 Apr 2022 18:34:15 +0100	[thread overview]
Message-ID: <871qyajetd.fsf@linaro.org> (raw)
In-Reply-To: <20220210145254.157790-5-eric.auger@redhat.com>


Eric Auger <eric.auger@redhat.com> writes:

> 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>

Hi Eric,

I sent a RFC patch which I think avoids the need for this hack:

  Subject: [RFC PATCH] tests/qtest: properly initialise the vring used idx
  Date: Wed,  6 Apr 2022 18:33:56 +0100
  Message-Id: <20220406173356.1891500-1-alex.bennee@linaro.org>

-- 
Alex Bennée

WARNING: multiple messages have this Message-ID (diff)
From: "Alex Bennée" <alex.bennee@linaro.org>
To: Eric Auger <eric.auger@redhat.com>
Cc: eesposit@redhat.com, jean-philippe@linaro.org, mst@redhat.com,
	qemu-devel@nongnu.org, Coiby.Xu@gmail.com, qemu-arm@nongnu.org,
	clg@kaod.org, stefanha@redhat.com, pbonzini@redhat.com,
	eric.auger.pro@gmail.com, david@gibson.dropbear.id.au
Subject: Re: [PATCH v3 4/5] tests/qtest/vhost-user-blk-test: Temporary hack to get tests passing on aarch64
Date: Wed, 06 Apr 2022 18:34:15 +0100	[thread overview]
Message-ID: <871qyajetd.fsf@linaro.org> (raw)
In-Reply-To: <20220210145254.157790-5-eric.auger@redhat.com>


Eric Auger <eric.auger@redhat.com> writes:

> 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>

Hi Eric,

I sent a RFC patch which I think avoids the need for this hack:

  Subject: [RFC PATCH] tests/qtest: properly initialise the vring used idx
  Date: Wed,  6 Apr 2022 18:33:56 +0100
  Message-Id: <20220406173356.1891500-1-alex.bennee@linaro.org>

-- 
Alex Bennée


  reply	other threads:[~2022-04-06 17:34 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-10 14:52 [PATCH v3 0/5] qtests/libqos: Allow PCI tests to be run with virt-machine Eric Auger
2022-02-10 14:52 ` [PATCH v3 1/5] tests/qtest/vhost-user-test.c: Use vhostforce=on Eric Auger
2022-02-10 14:52 ` [PATCH v3 2/5] tests/qtest/libqos/pci: Introduce pio_limit Eric Auger
2022-04-05 13:54   ` Alex Bennée
2022-04-05 13:54     ` Alex Bennée
2022-02-10 14:52 ` [PATCH v3 3/5] tests/qtest/libqos: Skip hotplug tests if pci root bus is not hotpluggable Eric Auger
2022-04-05 14:36   ` Alex Bennée
2022-04-05 14:36     ` Alex Bennée
2022-02-10 14:52 ` [PATCH v3 4/5] tests/qtest/vhost-user-blk-test: Temporary hack to get tests passing on aarch64 Eric Auger
2022-04-06 17:34   ` Alex Bennée [this message]
2022-04-06 17:34     ` Alex Bennée
2022-04-07  8:36     ` Eric Auger
2022-04-07  8:36       ` Eric Auger
2022-02-10 14:52 ` [PATCH v3 5/5] tests/qtest/libqos: Add generic pci host bridge in arm-virt machine Eric Auger
2022-03-06 10:53   ` Michael S. Tsirkin
2022-03-06 10:50 ` [PATCH v3 0/5] qtests/libqos: Allow PCI tests to be run with virt-machine Michael S. Tsirkin

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=871qyajetd.fsf@linaro.org \
    --to=alex.bennee@linaro.org \
    --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=mst@redhat.com \
    --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.