From nobody Sat Apr 11 23:08:20 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=quarantine dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1772728841; cv=none; d=zohomail.com; s=zohoarc; b=ba/871UPAsYc2ZqF462VmOcK4V+yd/i5AN0R0ZHcHJ1U63b3gLeb3/pw4DxMxnntnp9BNLoXyTUB+UeH1mGkI37uHCZcg5aK4ZCXmC7Sx3fAF4aearDUvTaZrPQVkImziysXIKVm2Tofr3sa8EJw+ejV58FoQQeEc1qTeCyNFYU= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1772728841; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=uOd8J2I5KMMxkHVOYgfkIXRl2cZ0hslmMulWEOLM6Hs=; b=D8sMg0ZIqxNet3hDaM+2xjCj9PuDzzfQaHvaiJZL8G+JoiyFPVio222NcyaoaN9srL+W1ESPUJM9G/kmB2g6Ix4tfjIf9m8w4V2jrg7KEK+uL7wB2HODGDhxkvL0x/JifzrsyTHs1eVtblZZCcnRLW+WC69WUz+Ry6EhZp9CjZ0= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1772728841200288.30752253877677; Thu, 5 Mar 2026 08:40:41 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1vyBkQ-0006As-O7; Thu, 05 Mar 2026 11:40:26 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1vyBkG-0006AY-41 for qemu-devel@nongnu.org; Thu, 05 Mar 2026 11:40:19 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1vyBk9-0000mF-LN for qemu-devel@nongnu.org; Thu, 05 Mar 2026 11:40:11 -0500 Received: from mx-prod-mc-06.mail-002.prod.us-west-2.aws.redhat.com (ec2-35-165-154-97.us-west-2.compute.amazonaws.com [35.165.154.97]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-362-SbIkhCTaOCijS-_ocvTJlA-1; Thu, 05 Mar 2026 11:40:03 -0500 Received: from mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.12]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-06.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id D49D418005BD; Thu, 5 Mar 2026 16:40:01 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.44.34.122]) by mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 7D7751958DC5; Thu, 5 Mar 2026 16:39:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1772728807; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=uOd8J2I5KMMxkHVOYgfkIXRl2cZ0hslmMulWEOLM6Hs=; b=bRIVQ9VT7BpCF8Mlkkf36Gyol0nvIAjwuC1MyYXqHiQzRR6NsN7NzsvWM8nXElgYli2zBz SBVLINDbgWjYEJIbyKSbQXEpBHIeGsXrPyHqz0GEy+avLlrYJ3pygsj/Pr56vwYsiCW2g8 7BG3VaXruRzR5NZMLDaZHcaaMLI4x3Y= X-MC-Unique: SbIkhCTaOCijS-_ocvTJlA-1 X-Mimecast-MFC-AGG-ID: SbIkhCTaOCijS-_ocvTJlA_1772728802 From: =?UTF-8?q?Eugenio=20P=C3=A9rez?= To: qemu-devel@nongnu.org Cc: Maxime Coquelin , Lei Yang , Paolo Bonzini , "Michael S. Tsirkin" , Stefano Garzarella , Koushik Dutta , Fabiano Rosas , Jason Wang , Laurent Vivier Subject: [RFC PATCH 5/8] tests: vhost-vdpa: add descriptor chain tests Date: Thu, 5 Mar 2026 17:39:35 +0100 Message-ID: <20260305163938.3200787-6-eperezma@redhat.com> In-Reply-To: <20260305163938.3200787-1-eperezma@redhat.com> References: <20260305163938.3200787-1-eperezma@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.0 on 10.30.177.12 Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=170.10.133.124; envelope-from=eperezma@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -5 X-Spam_score: -0.6 X-Spam_bar: / X-Spam_report: (-0.6 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H5=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.892, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.622, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: qemu development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1772728843430154100 Extend TX tests to support multi-descriptor chains: - Add chain_len parameter to control chain length - Use GAsyncQueue to validate expected vs actual chain lengths - Test single descriptors, 2-descriptor chains, and out-of-order chains Chain descriptors all point to the same guest memory buffer to simplify allocation tracking while still testing chain traversal logic. Signed-off-by: Eugenio P=C3=A9rez --- tests/qtest/vhost-vdpa-test.c | 53 ++++++++++++++++++++++++++++++----- 1 file changed, 46 insertions(+), 7 deletions(-) diff --git a/tests/qtest/vhost-vdpa-test.c b/tests/qtest/vhost-vdpa-test.c index 8dde7d95b167..433e5d71ca7a 100644 --- a/tests/qtest/vhost-vdpa-test.c +++ b/tests/qtest/vhost-vdpa-test.c @@ -47,6 +47,9 @@ typedef struct VdpaThread { GMainLoop *loop; GMainContext *context; =20 + /* Expected elements queue to compare properties */ + GAsyncQueue *elem_queue; + /* Guest memory that must be free at the end of the test */ uint64_t qemu_mem_to_free; } VdpaThread; @@ -60,6 +63,7 @@ static void *vhost_vdpa_thread_function(void *data) =20 static void vhost_vdpa_thread_init(VdpaThread *t) { + t->elem_queue =3D g_async_queue_new(); t->context =3D g_main_context_new(); t->loop =3D g_main_loop_new(t->context, FALSE); t->thread =3D g_thread_new("vdpa-thread", vhost_vdpa_thread_function, = t->loop); @@ -76,6 +80,7 @@ static void vhost_vdpa_thread_cleanup(VdpaThread *t) =20 g_main_loop_unref(t->loop); g_main_context_unref(t->context); + g_async_queue_unref(t->elem_queue); } =20 static void vhost_vdpa_thread_add_source_fd(VdpaThread *t, int fd, @@ -109,29 +114,48 @@ static void vhost_vdpa_add_rx_pkts(QGuestAllocator *a= lloc, QVirtioNet *net, * @alloc: the guest allocator to allocate memory for the descriptors * @net: the virtio net device * @t: the vdpa thread to push the expected chain length if kick is true + * @chain_len: the number of descriptors in the chain to add * * Returns the kick_id you can use to kick the device in a later call to t= his * function. */ static uint32_t vhost_vdpa_add_tx_pkt_descs(QGuestAllocator *alloc, - QVirtioNet *net, VdpaThread *t) + QVirtioNet *net, VdpaThread *t, + uint32_t chain_len) { QTestState *qts =3D global_qtest; - uint32_t req_addr; + uint32_t req_addr, kick_id =3D UINT32_MAX; =20 + assert(chain_len > 0); /* TODO: Actually free this. RFC, is actually needed? */ req_addr =3D guest_alloc(alloc, 64); g_assert_cmpint(req_addr, >, 0); =20 - return qvirtqueue_add(qts, net->queues[1], req_addr, 64, /* write */ f= alse, - /* next */ false); + /* + * We set up the descriptors in a way that each of them points to the s= ame + * buffer. This simplifies guest's memory management while still exerci= sing + * chain traversal in SVQ. + */ + for (uint32_t i =3D 0; i < chain_len; i++) { + uint32_t head; + bool next =3D i !=3D chain_len - 1; + + head =3D qvirtqueue_add(qts, net->queues[1], req_addr, 64, + /* write */ false, next); + if (i =3D=3D 0) { + kick_id =3D head; + } + } + + return kick_id; } =20 static void vhost_vdpa_kick_tx_desc(VdpaThread *t, QVirtioNet *net, - uint32_t kick_id) + uint32_t kick_id, uint32_t chain_len) { QTestState *qts =3D global_qtest; =20 + g_async_queue_push(t->elem_queue, (void *)(intptr_t)chain_len); qvirtqueue_kick(qts, net->vdev, net->queues[1], kick_id); } =20 @@ -244,9 +268,12 @@ static gboolean vhost_vdpa_rxtx_handle_tx(int fd, GIOC= ondition condition, void *data) { VduseVirtq *vq =3D data; + VduseDev *dev =3D vduse_queue_get_dev(vq); + TestServer *s =3D vduse_dev_get_priv(dev); =20 eventfd_read(fd, (eventfd_t[]){0}); do { + intptr_t expected_elems; g_autofree VduseVirtqElement *elem =3D NULL; =20 elem =3D vduse_queue_pop(vq, sizeof(*elem)); @@ -254,7 +281,10 @@ static gboolean vhost_vdpa_rxtx_handle_tx(int fd, GIOC= ondition condition, break; } =20 + expected_elems =3D (intptr_t)g_async_queue_try_pop(s->vdpa_thread.= elem_queue); + g_assert_cmpint(expected_elems, >, 0); g_test_message("Got element with %d buffers", elem->out_num); + g_assert_cmpint(elem->out_num, =3D=3D, expected_elems); g_assert_cmpint(elem->in_num, =3D=3D, 0); =20 vduse_queue_push(vq, elem, 0); @@ -546,9 +576,18 @@ static void vhost_vdpa_tx_test(void *obj, void *arg, Q= GuestAllocator *alloc) /* Add some rx packets so SVQ must clean them at the end of QEMU run */ vhost_vdpa_add_rx_pkts(alloc, net, &server->vdpa_thread); =20 - free_head =3D vhost_vdpa_add_tx_pkt_descs(alloc, net, &server->vdpa_th= read); - vhost_vdpa_kick_tx_desc(&server->vdpa_thread, net, free_head); + /* Simple packet */ + free_head =3D vhost_vdpa_add_tx_pkt_descs(alloc, net, &server->vdpa_th= read, + 1); + vhost_vdpa_kick_tx_desc(&server->vdpa_thread, net, free_head, 1); + vhost_vdpa_get_tx_pkt(alloc, net, free_head, &server->vdpa_thread); + + /* Simple chain */ + free_head =3D vhost_vdpa_add_tx_pkt_descs(alloc, net, &server->vdpa_th= read, + 2); + vhost_vdpa_kick_tx_desc(&server->vdpa_thread, net, free_head, 2); vhost_vdpa_get_tx_pkt(alloc, net, free_head, &server->vdpa_thread); + } =20 static void register_vhost_vdpa_test(void) --=20 2.53.0