All of lore.kernel.org
 help / color / mirror / Atom feed
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




  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.