From: Christian Schoenebeck <qemu_oss@crudebyte.com>
To: qemu-devel@nongnu.org,
Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
Cc: qemu-stable@nongnu.org, Greg Kurz <groug@kaod.org>,
Peter Maydell <peter.maydell@linaro.org>,
Feifan Qian <bea1e@proton.me>,
Stefano Stabellini <sstabellini@kernel.org>
Subject: Re: [PULL 00/23] 9p queue 2026-06-29
Date: Wed, 01 Jul 2026 18:20:07 +0200 [thread overview]
Message-ID: <8700308.T7Z3S40VBb@weasel> (raw)
In-Reply-To: <53159abf-ae84-465a-ac70-8ad71fa6c045@oss.qualcomm.com>
On Tuesday, 30 June 2026 09:28:48 CEST Pierrick Bouvier wrote:
> Hi Christian,
Hi Pierrick,
> This series brought a test regression:
> $ ./build/pyvenv/bin/meson test -C build \
> --setup thorough --print-errorlogs \
> qemu:qtest-aarch64/qos-test
> ...
> ERROR:../tests/qtest/libqos/virtio-9p-client.c:280:v9fs_req_recv:
> assertion failed (hdr.id == id): (7 == 121)
>
> It seems to come test added in patch 19
> "tests/9p: add 3 xattr FID limit test cases (synth fs driver)"
> This test runs only with slow setup, so I suspect it never worked and
> was never ran.
Of course I ran these slow tests.
I intentionally registered these tests as "slow" tests as they take a long
time to complete and QEMU tests being notorious on exceeding the overall CI
timeout limit. So these tests are exempted from running in the official CI
pipeline, but not from mine.
I just reran them with latest master head. However I am unable to reproduce
your reported test error so far.
You cropped the output too aggressively. I need to know:
- Which test exactly failed? (especially whether it's really a synth backend
or a local backend test that failed)
- What is the errno being printed *before* the "assertion failed" message?
("(hdr.id == id): (7 == 121)" just means 9p server responded with an error
response instead of the expected one)
- Were you running the tests in a Docker environment or something or just
plain locally?
- Does the underlying host file system support xattrs? (might matter if it's a
local backend test that's failing)
/Christian
next prev parent reply other threads:[~2026-07-01 16:21 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-29 13:28 [PULL 00/23] 9p queue 2026-06-29 Christian Schoenebeck
2026-06-29 13:28 ` [PULL 02/23] 9pfs/virtio: implement msize_limit callback Christian Schoenebeck
2026-06-29 13:28 ` [PULL 09/23] hw/9pfs: add xattr FID limit to prevent memory exhaustion Christian Schoenebeck
2026-06-29 13:28 ` [PULL 14/23] tests/9p: add Txattrcreate / Rxattrcreate test client functions Christian Schoenebeck
2026-06-29 13:28 ` [PULL 15/23] hw/9pfs: enable xattr (mockup) support for synth fs driver Christian Schoenebeck
2026-06-29 13:28 ` [PULL 05/23] hw/9pfs: add response_buffer_size transport callback Christian Schoenebeck
2026-06-29 13:28 ` [PULL 23/23] hw/9pfs/local: harden local_fid_fd() on FID types Christian Schoenebeck
2026-06-29 13:28 ` [PULL 19/23] tests/9p: add 3 xattr FID limit test cases (synth fs driver) Christian Schoenebeck
2026-06-29 13:28 ` [PULL 07/23] 9pfs/xen: implement response_buffer_size callback Christian Schoenebeck
2026-06-29 13:28 ` [PULL 11/23] qemu-options: document 9pfs max_xattr option Christian Schoenebeck
2026-06-29 13:28 ` [PULL 12/23] tests/9p: add Tread / Rread test client functions Christian Schoenebeck
2026-06-29 13:28 ` [PULL 22/23] hw/9pfs: fix invalid union access by v9fs_co_fstat() Christian Schoenebeck
2026-06-29 13:28 ` [PULL 13/23] tests/9p: add Tclunk / Rclunk test client functions Christian Schoenebeck
2026-06-29 13:28 ` [PULL 01/23] hw/9pfs: add msize_limit transport callback Christian Schoenebeck
2026-06-29 13:28 ` [PULL 17/23] tests/9p: increase P9_MAX_SIZE for test client Christian Schoenebeck
2026-06-29 13:28 ` [PULL 10/23] hw/9pfs: add max_xattr option Christian Schoenebeck
2026-06-29 13:28 ` [PULL 06/23] 9pfs/virtio: implement response_buffer_size callback Christian Schoenebeck
2026-06-29 13:28 ` [PULL 18/23] tests/9p: add virtio_9p_add_synth_driver_args() test client function Christian Schoenebeck
2026-06-29 13:28 ` [PULL 21/23] hw/9pfs: fix invalid union access by v9fs_co_fsync() Christian Schoenebeck
2026-06-29 13:28 ` [PULL 20/23] tests/9p: add 3 xattr FID limit test cases (local fs driver) Christian Schoenebeck
2026-06-29 13:28 ` [PULL 03/23] 9pfs/xen: implement msize_limit callback Christian Schoenebeck
2026-06-29 13:28 ` [PULL 16/23] hw/9pfs: add xattr count query interface to fs synth driver Christian Schoenebeck
2026-06-29 13:28 ` [PULL 04/23] hw/9pfs: cap negotiated msize to transport limit Christian Schoenebeck
2026-06-29 13:28 ` [PULL 08/23] hw/9pfs: cap Treaddir allocation (CVE-2026-9238) Christian Schoenebeck
2026-06-29 18:28 ` [PULL 00/23] 9p queue 2026-06-29 Stefan Hajnoczi
2026-06-30 7:28 ` Pierrick Bouvier
2026-07-01 16:20 ` Christian Schoenebeck [this message]
2026-07-02 18:21 ` Pierrick Bouvier
2026-07-03 14:36 ` Christian Schoenebeck
2026-07-03 17:20 ` Pierrick Bouvier
2026-07-01 10:15 ` Michael Tokarev
2026-07-01 17:21 ` Christian Schoenebeck
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=8700308.T7Z3S40VBb@weasel \
--to=qemu_oss@crudebyte.com \
--cc=bea1e@proton.me \
--cc=groug@kaod.org \
--cc=peter.maydell@linaro.org \
--cc=pierrick.bouvier@oss.qualcomm.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-stable@nongnu.org \
--cc=sstabellini@kernel.org \
/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.