From nobody Sat May 4 00:28:35 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; 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 ARC-Seal: i=1; a=rsa-sha256; t=1616107373; cv=none; d=zohomail.com; s=zohoarc; b=l52q2J5nK5OG0tRFr92S4GVeivtppe630W5hyBma2qIi30UrWoeBw0qAx7Ej9bs51xH6VIwmPR+XThesYlTWl1GJ27KuDeX2c/yhLdhdfmXWvgdQIQnfE8DEQNWsbZuYQJ9HWfKn2AuTtfbv0IMDdxGv5/qimi/+Mtqx4bqacxw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616107373; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=hTc6y9LP3dmmT/6crdS1ftztmUCR8NNk0ZVCEQpKF98=; b=jXyoLqRr1weDwsXEDSHXiRzsOG6q9iKJqqE3nLlSLO+QMRDlb3BBLlrtEAyz6TW3LVn/sKNgXIkdOgBHEmzOBu9xG+b5J3fHI2NOM14BD+CrtXLryLMzZBvuSJlqsazbRIGkRYnXYEAi3M22IpUYSyrTcFq/5tYpRriOCqJ58YY= ARC-Authentication-Results: i=1; mx.zohomail.com; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1616107373877820.3549810339703; Thu, 18 Mar 2021 15:42:53 -0700 (PDT) Received: from localhost ([::1]:47668 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lN1Lw-0007y0-SZ for importer@patchew.org; Thu, 18 Mar 2021 18:42:52 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47524) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lN1Ia-0003yF-T1; Thu, 18 Mar 2021 18:39:24 -0400 Received: from mout.kundenserver.de ([212.227.126.134]:60215) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lN1IY-0003xs-VF; Thu, 18 Mar 2021 18:39:24 -0400 Received: from localhost.localdomain ([82.142.20.38]) by mrelayeu.kundenserver.de (mreue010 [212.227.15.167]) with ESMTPSA (Nemesis) id 1MnJQq-1m60dF0Axo-00jFct; Thu, 18 Mar 2021 23:39:17 +0100 From: Laurent Vivier To: qemu-devel@nongnu.org Subject: [PATCH 1/4] m68k: add the virtio devices aliases Date: Thu, 18 Mar 2021 23:39:04 +0100 Message-Id: <20210318223907.1344870-2-laurent@vivier.eu> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20210318223907.1344870-1-laurent@vivier.eu> References: <20210318223907.1344870-1-laurent@vivier.eu> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:Rk8+rPnec7ayTybXLWOB9lZ4DG1rKA3bLfWqZnwY5zvSctmUP1g 8GD/5I2EPsZHF//AB6qxTSYSYkrEnY9k9kuIfcqVUEes8GtIy1kayULxnn2uA6IqmAqD1nY F/8NEqqZZ+x0UZoxlResjASAjB9Soged8nDGVStrPMP6PrBfKH8wOb1gYlFJf4QTWUHXb/H bCqpIOrS043szYWZBQxrw== X-UI-Out-Filterresults: notjunk:1;V03:K0:MEfj3LRuyNI=:beAdjBklH0VLAo3tC0kARS 2xouMs4BZFpT7da08SzsTS7Wx6Jh2g7mtPtDSeulNdhU1plWBlPHZ4bPyTZTOqjW5F00Y8rUO W0P0riG+c8ppmiLp7t+zNrtuQkiHGzDQP6jGk7QS/rA/nSeac6fsfZYSB9aLjgwBzkHULib/D jTSaB83s+wiuEz07K3Y2zABR5VJWEQQIDrVFevvSDEmq3VKuvJvt1+LbWwhUs3mJW6v7YmuSO lyYfNslPxTAZGKEfW8Ervj6GsydE6haLCJ0dvmOYdeRvBz7GiMkr4QRJHsF9eH1/4HH6+QP40 mkRDnGHYMauZA3+vhME9kZRKa1eInHgLubtfAOmqgwJWeos/dF6/o4vu/g1CcGYxA8ARv1hLY D7WD3ctCKD9zBeHEszEG6yJxpoKE9siB5g/8Aimr/IQUtthzVsi1exAFuZQkaL50Ipp5y1vrH zUQ8Sdy04Q== 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: none client-ip=212.227.126.134; envelope-from=laurent@vivier.eu; helo=mout.kundenserver.de X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Kevin Wolf , Peter Maydell , Thomas Huth , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , Eduardo Habkost , qemu-block@nongnu.org, Cornelia Huck , Markus Armbruster , Max Reitz , qemu-s390x@nongnu.org, Paolo Bonzini , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Laurent Vivier Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Similarly to 5f629d943cb0 ("s390x: fix s390 virtio aliases"), define the virtio aliases. This allows to start machines with virtio devices without knowledge of the implementation type. For instance, we can use "-device virtio-scsi" on m68k, s390x or PC, and the device will be "virtio-scsi-device", "virtio-scsi-ccw" or "virtio-scsi-pci". This already exists for s390x and -ccw interfaces, adds them for m68k and MMIO (-device) interfaces. Signed-off-by: Laurent Vivier Reviewed-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: Thomas Huth --- softmmu/qdev-monitor.c | 46 +++++++++++++++++++++++++++--------------- 1 file changed, 30 insertions(+), 16 deletions(-) diff --git a/softmmu/qdev-monitor.c b/softmmu/qdev-monitor.c index 8dc656becca9..262d38b8c01e 100644 --- a/softmmu/qdev-monitor.c +++ b/softmmu/qdev-monitor.c @@ -42,6 +42,8 @@ #include "hw/qdev-properties.h" #include "hw/clock.h" =20 +#define QEMU_ARCH_NO_PCI (QEMU_ARCH_S390X | QEMU_ARCH_M68K) + /* * Aliases were a bad idea from the start. Let's keep them * from spreading further. @@ -60,34 +62,46 @@ static const QDevAlias qdev_alias_table[] =3D { { "ES1370", "es1370" }, /* -soundhw name */ { "ich9-ahci", "ahci" }, { "lsi53c895a", "lsi" }, + { "virtio-9p-device", "virtio-9p", QEMU_ARCH_M68K }, { "virtio-9p-ccw", "virtio-9p", QEMU_ARCH_S390X }, - { "virtio-9p-pci", "virtio-9p", QEMU_ARCH_ALL & ~QEMU_ARCH_S390X }, + { "virtio-9p-pci", "virtio-9p", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_PCI }, + { "virtio-balloon-device", "virtio-balloon", QEMU_ARCH_M68K }, { "virtio-balloon-ccw", "virtio-balloon", QEMU_ARCH_S390X }, - { "virtio-balloon-pci", "virtio-balloon", - QEMU_ARCH_ALL & ~QEMU_ARCH_S390X }, + { "virtio-balloon-pci", "virtio-balloon", QEMU_ARCH_ALL & + ~QEMU_ARCH_NO_PCI }, + { "virtio-blk-device", "virtio-blk", QEMU_ARCH_M68K }, { "virtio-blk-ccw", "virtio-blk", QEMU_ARCH_S390X }, - { "virtio-blk-pci", "virtio-blk", QEMU_ARCH_ALL & ~QEMU_ARCH_S390X }, + { "virtio-blk-pci", "virtio-blk", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_PCI }, { "virtio-gpu-ccw", "virtio-gpu", QEMU_ARCH_S390X }, - { "virtio-gpu-pci", "virtio-gpu", QEMU_ARCH_ALL & ~QEMU_ARCH_S390X }, + { "virtio-gpu-device", "virtio-gpu", QEMU_ARCH_M68K }, + { "virtio-gpu-pci", "virtio-gpu", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_PCI }, + { "virtio-input-host-device", "virtio-input-host", QEMU_ARCH_M68K }, { "virtio-input-host-ccw", "virtio-input-host", QEMU_ARCH_S390X }, - { "virtio-input-host-pci", "virtio-input-host", - QEMU_ARCH_ALL & ~QEMU_ARCH_S390X }, - { "virtio-iommu-pci", "virtio-iommu", QEMU_ARCH_ALL & ~QEMU_ARCH_S390X= }, + { "virtio-input-host-pci", "virtio-input-host", QEMU_ARCH_ALL & + ~QEMU_ARCH_NO_PCI }, + { "virtio-iommu-pci", "virtio-iommu", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_PC= I }, + { "virtio-keyboard-device", "virtio-keyboard", QEMU_ARCH_M68K }, { "virtio-keyboard-ccw", "virtio-keyboard", QEMU_ARCH_S390X }, - { "virtio-keyboard-pci", "virtio-keyboard", - QEMU_ARCH_ALL & ~QEMU_ARCH_S390X }, + { "virtio-keyboard-pci", "virtio-keyboard", QEMU_ARCH_ALL & + ~QEMU_ARCH_NO_PCI }, + { "virtio-mouse-device", "virtio-mouse", QEMU_ARCH_M68K }, { "virtio-mouse-ccw", "virtio-mouse", QEMU_ARCH_S390X }, - { "virtio-mouse-pci", "virtio-mouse", QEMU_ARCH_ALL & ~QEMU_ARCH_S390X= }, + { "virtio-mouse-pci", "virtio-mouse", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_PC= I }, + { "virtio-net-device", "virtio-net", QEMU_ARCH_M68K }, { "virtio-net-ccw", "virtio-net", QEMU_ARCH_S390X }, - { "virtio-net-pci", "virtio-net", QEMU_ARCH_ALL & ~QEMU_ARCH_S390X }, + { "virtio-net-pci", "virtio-net", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_PCI }, + { "virtio-rng-device", "virtio-rng", QEMU_ARCH_M68K }, { "virtio-rng-ccw", "virtio-rng", QEMU_ARCH_S390X }, - { "virtio-rng-pci", "virtio-rng", QEMU_ARCH_ALL & ~QEMU_ARCH_S390X }, + { "virtio-rng-pci", "virtio-rng", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_PCI }, + { "virtio-scsi-device", "virtio-scsi", QEMU_ARCH_M68K }, { "virtio-scsi-ccw", "virtio-scsi", QEMU_ARCH_S390X }, - { "virtio-scsi-pci", "virtio-scsi", QEMU_ARCH_ALL & ~QEMU_ARCH_S390X }, + { "virtio-scsi-pci", "virtio-scsi", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_PCI = }, + { "virtio-serial-device", "virtio-serial", QEMU_ARCH_M68K }, { "virtio-serial-ccw", "virtio-serial", QEMU_ARCH_S390X }, - { "virtio-serial-pci", "virtio-serial", QEMU_ARCH_ALL & ~QEMU_ARCH_S39= 0X }, + { "virtio-serial-pci", "virtio-serial", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_= PCI}, + { "virtio-tablet-device", "virtio-tablet", QEMU_ARCH_M68K }, { "virtio-tablet-ccw", "virtio-tablet", QEMU_ARCH_S390X }, - { "virtio-tablet-pci", "virtio-tablet", QEMU_ARCH_ALL & ~QEMU_ARCH_S39= 0X }, + { "virtio-tablet-pci", "virtio-tablet", QEMU_ARCH_ALL & ~QEMU_ARCH_NO_= PCI }, { } }; =20 --=20 2.30.2 From nobody Sat May 4 00:28:35 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; 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 ARC-Seal: i=1; a=rsa-sha256; t=1616107285; cv=none; d=zohomail.com; s=zohoarc; b=jGr6kx0A2t1JQm+Po1Vrx3tLVEKuxWwd0V4LurWugFIFKAGXhx6IrOCFjE7WOrCpKsmIvz7S6t8yMJXWmqLcUVcWRkhdI2CQUM/rb0i8pHs4w2Fkhe/YwO08w/OoiFvcsFqXLzw/TnXlcE9sbrRsV18br9nt6SQ15ED1/fEF4vU= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616107285; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=573GBmwXWvD9EQ1IWRDv91VgCvUl8sbVCcX3qNYHRSQ=; b=FHhpdFwzfykYoz6xQKK9Y5GoSNs479D/Z24Fp0sNplirWgIA60/mP7rZGX+QTljscwPBdLDyQYXBXsTXOPux6BXuQPM6fwBQkd3/rc1CqPWQMUdCeMpQjgHyIVw6Uqk1dj40gNOi0spMkr9JSCS96urlMdmHwlIrGPn9vWaMtLM= ARC-Authentication-Results: i=1; mx.zohomail.com; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1616107285466631.9625386883574; Thu, 18 Mar 2021 15:41:25 -0700 (PDT) Received: from localhost ([::1]:41946 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lN1KW-0005Yq-6x for importer@patchew.org; Thu, 18 Mar 2021 18:41:24 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47546) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lN1If-00040H-GZ; Thu, 18 Mar 2021 18:39:29 -0400 Received: from mout.kundenserver.de ([212.227.126.133]:40401) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lN1Ia-0003yo-Gv; Thu, 18 Mar 2021 18:39:29 -0400 Received: from localhost.localdomain ([82.142.20.38]) by mrelayeu.kundenserver.de (mreue010 [212.227.15.167]) with ESMTPSA (Nemesis) id 1MlfGs-1m4MTM0LgY-00ihvV; Thu, 18 Mar 2021 23:39:18 +0100 From: Laurent Vivier To: qemu-devel@nongnu.org Subject: [PATCH 2/4] iotests: Revert "iotests: use -ccw on s390x for 040, 139, and 182" Date: Thu, 18 Mar 2021 23:39:05 +0100 Message-Id: <20210318223907.1344870-3-laurent@vivier.eu> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20210318223907.1344870-1-laurent@vivier.eu> References: <20210318223907.1344870-1-laurent@vivier.eu> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:9bBFlCmvAjK24/aRFL4Yn2mKny4m8sEArd3dBUpCppYtI9FsG2o cWEEGmfz9gy43aTRzeGXOSPD0Z3eKDhN9Xv/FScQrlfIwbEut1A/xTeto1W8zJd4Ks9yqB6 EO/k0Oh6S8HC7qD3l3sJakCOfFsSxL9/uu82eb75hN2i4PC/IEZvM2/vyQiBJNEFWP+6HA9 BJ/P+r8mjjRTbVgahXVAg== X-UI-Out-Filterresults: notjunk:1;V03:K0:9CzYRzuEQBo=:XEpt6G+JZXAiEfBClMRnBB 4sEJR4sAPQwlKpNCEimNSwoqraQbAH1kNWH0QegzOWK9Nsp1dA5/0IhXpZLa6RFMGYjbk2RU1 YQ7S58lfY93yYI1yOZOPDnZFAI8cjqPbPDCChlWFdi/Hg23BgvpoeM2P/aOVk/OgAjlEM33nz 6Au4+DgmeCXvDi3byvzSqmG8+pFGACKanDIVLgVkOv4J0SlFwykjyH8Si8FifLazt9AhWiLQZ gi9OE7fsV0GEqQNVes/wAqus7ldGzB/WZBg40gL+dRU/uBYfgGp9ZV9UV5vQ9Pr/AgllEoYJG FZ2Acli3adbakFZWqcodYE/RGJbP2HlzGOJUTg6s30GmexFFAa2Nc+UiG3m1FVI3vODWCiBKQ 5igBBhX1Sj1P1n3g/GRhUNrNR1f0qdimwm5hW6KQ+NdGtF56fWpZWHwuIo/Jtu8Vkxxsk6eOF Gwh7+cYT7w== 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: none client-ip=212.227.126.133; envelope-from=laurent@vivier.eu; helo=mout.kundenserver.de X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Kevin Wolf , Peter Maydell , Thomas Huth , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , Eduardo Habkost , qemu-block@nongnu.org, Cornelia Huck , Markus Armbruster , Max Reitz , qemu-s390x@nongnu.org, Paolo Bonzini , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Laurent Vivier Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Commit f1d5516ab583 introduces a test in some iotests to check if the machine is a s390-ssw-virtio and to select virtio-*-ccw rather than virtio-*-pci. We don't need that because QEMU already provides aliases to use the correct virtio interface according to the machine type. This patch removes all virtio-*-pci and virtio-*-ccw to use virtio-* instead. Signed-off-by: Laurent Vivier cc: Cornelia Huck --- blockdev.c | 6 +----- tests/qemu-iotests/040 | 2 +- tests/qemu-iotests/051 | 12 +----------- tests/qemu-iotests/068 | 4 +--- tests/qemu-iotests/093 | 3 +-- tests/qemu-iotests/139 | 9 ++------- tests/qemu-iotests/182 | 13 ++----------- tests/qemu-iotests/238 | 4 +--- tests/qemu-iotests/240 | 10 +++++----- tests/qemu-iotests/257 | 4 ++-- tests/qemu-iotests/307 | 4 +--- tests/qemu-iotests/iotests.py | 5 ----- 12 files changed, 18 insertions(+), 58 deletions(-) diff --git a/blockdev.c b/blockdev.c index 5cc7c7effe9f..64da5350e3ad 100644 --- a/blockdev.c +++ b/blockdev.c @@ -969,11 +969,7 @@ DriveInfo *drive_new(QemuOpts *all_opts, BlockInterfac= eType block_default_type, QemuOpts *devopts; devopts =3D qemu_opts_create(qemu_find_opts("device"), NULL, 0, &error_abort); - if (arch_type =3D=3D QEMU_ARCH_S390X) { - qemu_opt_set(devopts, "driver", "virtio-blk-ccw", &error_abort= ); - } else { - qemu_opt_set(devopts, "driver", "virtio-blk-pci", &error_abort= ); - } + qemu_opt_set(devopts, "driver", "virtio-blk", &error_abort); qemu_opt_set(devopts, "drive", qdict_get_str(bs_opts, "id"), &error_abort); } diff --git a/tests/qemu-iotests/040 b/tests/qemu-iotests/040 index 336ff7c4f2ab..ba7cb34ce8cf 100755 --- a/tests/qemu-iotests/040 +++ b/tests/qemu-iotests/040 @@ -89,7 +89,7 @@ class TestSingleDrive(ImageCommitTestCase): qemu_io('-f', 'raw', '-c', 'write -P 0xab 0 524288', backing_img) qemu_io('-f', iotests.imgfmt, '-c', 'write -P 0xef 524288 524288',= mid_img) self.vm =3D iotests.VM().add_drive(test_img, "node-name=3Dtop,back= ing.node-name=3Dmid,backing.backing.node-name=3Dbase", interface=3D"none") - self.vm.add_device(iotests.get_virtio_scsi_device()) + self.vm.add_device('virtio-scsi') self.vm.add_device("scsi-hd,id=3Dscsi0,drive=3Ddrive0") self.vm.launch() self.has_quit =3D False diff --git a/tests/qemu-iotests/051 b/tests/qemu-iotests/051 index 7cbd1415ce7b..00382cc55e25 100755 --- a/tests/qemu-iotests/051 +++ b/tests/qemu-iotests/051 @@ -119,17 +119,7 @@ echo echo =3D=3D=3D Device without drive =3D=3D=3D echo =20 -case "$QEMU_DEFAULT_MACHINE" in - s390-ccw-virtio) - virtio_scsi=3Dvirtio-scsi-ccw - ;; - *) - virtio_scsi=3Dvirtio-scsi-pci - ;; -esac - -run_qemu -device $virtio_scsi -device scsi-hd | - sed -e "s/$virtio_scsi/VIRTIO_SCSI/" +run_qemu -device virtio-scsi -device scsi-hd =20 echo echo =3D=3D=3D Overriding backing file =3D=3D=3D diff --git a/tests/qemu-iotests/068 b/tests/qemu-iotests/068 index 03e03508a6e2..54e49c8ffab1 100755 --- a/tests/qemu-iotests/068 +++ b/tests/qemu-iotests/068 @@ -49,11 +49,9 @@ IMG_SIZE=3D128K case "$QEMU_DEFAULT_MACHINE" in s390-ccw-virtio) platform_parm=3D"-no-shutdown" - hba=3Dvirtio-scsi-ccw ;; *) platform_parm=3D"" - hba=3Dvirtio-scsi-pci ;; esac =20 @@ -61,7 +59,7 @@ _qemu() { $QEMU $platform_parm -nographic -monitor stdio -serial none \ -drive if=3Dnone,id=3Ddrive0,file=3D"$TEST_IMG",format=3D"$IMGFM= T" \ - -device $hba,id=3Dhba0 \ + -device virtio-scsi,id=3Dhba0 \ -device scsi-hd,drive=3Ddrive0 \ "$@" |\ _filter_qemu | _filter_hmp diff --git a/tests/qemu-iotests/093 b/tests/qemu-iotests/093 index 7745cb04b611..93274dc8cbde 100755 --- a/tests/qemu-iotests/093 +++ b/tests/qemu-iotests/093 @@ -371,8 +371,7 @@ class ThrottleTestGroupNames(iotests.QMPTestCase): class ThrottleTestRemovableMedia(iotests.QMPTestCase): def setUp(self): self.vm =3D iotests.VM() - self.vm.add_device("{},id=3Dvirtio-scsi".format( - iotests.get_virtio_scsi_device())) + self.vm.add_device("{},id=3Dvirtio-scsi".format('virtio-scsi')) self.vm.launch() =20 def tearDown(self): diff --git a/tests/qemu-iotests/139 b/tests/qemu-iotests/139 index e79b3c21fdce..178b1ee230ca 100755 --- a/tests/qemu-iotests/139 +++ b/tests/qemu-iotests/139 @@ -26,18 +26,13 @@ import time =20 base_img =3D os.path.join(iotests.test_dir, 'base.img') new_img =3D os.path.join(iotests.test_dir, 'new.img') -if iotests.qemu_default_machine =3D=3D 's390-ccw-virtio': - default_virtio_blk =3D 'virtio-blk-ccw' -else: - default_virtio_blk =3D 'virtio-blk-pci' =20 class TestBlockdevDel(iotests.QMPTestCase): =20 def setUp(self): iotests.qemu_img('create', '-f', iotests.imgfmt, base_img, '1M') self.vm =3D iotests.VM() - self.vm.add_device("{},id=3Dvirtio-scsi".format( - iotests.get_virtio_scsi_device())) + self.vm.add_device("{},id=3Dvirtio-scsi".format('virtio-scsi')) self.vm.launch() =20 def tearDown(self): @@ -93,7 +88,7 @@ class TestBlockdevDel(iotests.QMPTestCase): self.checkBlockDriverState(node, expect_error) =20 # Add a device model - def addDeviceModel(self, device, backend, driver =3D default_virtio_bl= k): + def addDeviceModel(self, device, backend, driver =3D 'virtio-blk'): result =3D self.vm.qmp('device_add', id =3D device, driver =3D driver, drive =3D backend) self.assert_qmp(result, 'return', {}) diff --git a/tests/qemu-iotests/182 b/tests/qemu-iotests/182 index 55a0384c0873..fcd1d796ebd0 100755 --- a/tests/qemu-iotests/182 +++ b/tests/qemu-iotests/182 @@ -46,26 +46,17 @@ _supported_proto file =20 size=3D32M =20 -case "$QEMU_DEFAULT_MACHINE" in - s390-ccw-virtio) - virtioblk=3Dvirtio-blk-ccw - ;; - *) - virtioblk=3Dvirtio-blk-pci - ;; -esac - _make_test_img $size =20 echo "Starting QEMU" _launch_qemu -drive file=3D$TEST_IMG,if=3Dnone,id=3Ddrive0,file.locking=3D= on \ - -device $virtioblk,drive=3Ddrive0 + -device virtio-blk,drive=3Ddrive0 =20 echo echo "Starting a second QEMU using the same image should fail" echo 'quit' | $QEMU -nographic -monitor stdio \ -drive file=3D$TEST_IMG,if=3Dnone,id=3Ddrive0,file.locking=3Don \ - -device $virtioblk,drive=3Ddrive0 2>&1 | _filter_testdir 2>&1 | + -device virtio-blk,drive=3Ddrive0 2>&1 | _filter_testdir 2>&1 | _filter_qemu | sed -e '/falling back to POSIX file/d' \ -e '/locks can be lost unexpectedly/d' diff --git a/tests/qemu-iotests/238 b/tests/qemu-iotests/238 index 8a10af57f17c..38bd3744e68c 100755 --- a/tests/qemu-iotests/238 +++ b/tests/qemu-iotests/238 @@ -26,14 +26,12 @@ from iotests import log =20 iotests.script_initialize() =20 -virtio_scsi_device =3D iotests.get_virtio_scsi_device() - vm =3D iotests.VM() vm.launch() =20 log(vm.qmp('blockdev-add', node_name=3D'hd0', driver=3D'null-co', read_zer= oes=3DTrue)) log(vm.qmp('object-add', qom_type=3D'iothread', id=3D'iothread0')) -log(vm.qmp('device_add', id=3D'scsi0', driver=3Dvirtio_scsi_device, iothre= ad=3D'iothread0')) +log(vm.qmp('device_add', id=3D'scsi0', driver=3D'virtio-scsi', iothread=3D= 'iothread0')) log(vm.qmp('device_add', id=3D'scsi-hd0', driver=3D'scsi-hd', drive=3D'hd0= ')) log(vm.qmp('block_set_io_throttle', id=3D'scsi-hd0', bps=3D0, bps_rd=3D0, = bps_wr=3D0, iops=3D1000, iops_rd=3D0, iops_wr=3D0, conv_keys=3DFalse)) diff --git a/tests/qemu-iotests/240 b/tests/qemu-iotests/240 index ab077f4ceb01..9b281e1dc036 100755 --- a/tests/qemu-iotests/240 +++ b/tests/qemu-iotests/240 @@ -42,7 +42,7 @@ class TestCase(iotests.QMPTestCase): iotests.log('=3D=3DUnplug a SCSI disk and then plug it again=3D=3D= ') self.vm.qmp_log('blockdev-add', driver=3D'null-co', read_zeroes=3D= True, node_name=3D'hd0') self.vm.qmp_log('object-add', qom_type=3D'iothread', id=3D"iothrea= d0") - self.vm.qmp_log('device_add', id=3D'scsi0', driver=3Diotests.get_v= irtio_scsi_device(), iothread=3D'iothread0', filters=3D[iotests.filter_qmp_= virtio_scsi]) + self.vm.qmp_log('device_add', id=3D'scsi0', driver=3D'virtio-scsi'= , iothread=3D'iothread0', filters=3D[iotests.filter_qmp_virtio_scsi]) self.vm.qmp_log('device_add', id=3D'scsi-hd0', driver=3D'scsi-hd',= drive=3D'hd0') self.vm.qmp_log('device_del', id=3D'scsi-hd0') self.vm.event_wait('DEVICE_DELETED') @@ -55,7 +55,7 @@ class TestCase(iotests.QMPTestCase): iotests.log('=3D=3DAttach two SCSI disks using the same block devi= ce and the same iothread=3D=3D') self.vm.qmp_log('blockdev-add', driver=3D'null-co', read_zeroes=3D= True, node_name=3D'hd0', read_only=3DTrue) self.vm.qmp_log('object-add', qom_type=3D'iothread', id=3D"iothrea= d0") - self.vm.qmp_log('device_add', id=3D'scsi0', driver=3Diotests.get_v= irtio_scsi_device(), iothread=3D'iothread0', filters=3D[iotests.filter_qmp_= virtio_scsi]) + self.vm.qmp_log('device_add', id=3D'scsi0', driver=3D'virtio-scsi'= , iothread=3D'iothread0', filters=3D[iotests.filter_qmp_virtio_scsi]) =20 self.vm.qmp_log('device_add', id=3D'scsi-hd0', driver=3D'scsi-hd',= drive=3D'hd0') self.vm.qmp_log('device_add', id=3D'scsi-hd1', driver=3D'scsi-hd',= drive=3D'hd0') @@ -73,8 +73,8 @@ class TestCase(iotests.QMPTestCase): self.vm.qmp_log('object-add', qom_type=3D'iothread', id=3D"iothrea= d0") self.vm.qmp_log('object-add', qom_type=3D'iothread', id=3D"iothrea= d1") =20 - self.vm.qmp_log('device_add', id=3D'scsi0', driver=3Diotests.get_v= irtio_scsi_device(), iothread=3D'iothread0', filters=3D[iotests.filter_qmp_= virtio_scsi]) - self.vm.qmp_log('device_add', id=3D'scsi1', driver=3Diotests.get_v= irtio_scsi_device(), iothread=3D'iothread1', filters=3D[iotests.filter_qmp_= virtio_scsi]) + self.vm.qmp_log('device_add', id=3D'scsi0', driver=3D'virtio-scsi'= , iothread=3D'iothread0', filters=3D[iotests.filter_qmp_virtio_scsi]) + self.vm.qmp_log('device_add', id=3D'scsi1', driver=3D'virtio-scsi'= , iothread=3D'iothread1', filters=3D[iotests.filter_qmp_virtio_scsi]) =20 self.vm.qmp_log('device_add', id=3D'scsi-hd0', driver=3D'scsi-hd',= drive=3D'hd0', bus=3D"scsi0.0") self.vm.qmp_log('device_add', id=3D'scsi-hd1', driver=3D'scsi-hd',= drive=3D'hd0', bus=3D"scsi1.0") @@ -99,7 +99,7 @@ class TestCase(iotests.QMPTestCase): self.vm.qmp_log('nbd-server-add', device=3D'hd0') =20 self.vm.qmp_log('object-add', qom_type=3D'iothread', id=3D"iothrea= d0") - self.vm.qmp_log('device_add', id=3D'scsi0', driver=3Diotests.get_v= irtio_scsi_device(), iothread=3D'iothread0', filters=3D[iotests.filter_qmp_= virtio_scsi]) + self.vm.qmp_log('device_add', id=3D'scsi0', driver=3D'virtio-scsi'= , iothread=3D'iothread0', filters=3D[iotests.filter_qmp_virtio_scsi]) self.vm.qmp_log('device_add', id=3D'scsi-hd0', driver=3D'scsi-hd',= drive=3D'hd0') =20 if __name__ =3D=3D '__main__': diff --git a/tests/qemu-iotests/257 b/tests/qemu-iotests/257 index 7cd25208296e..c72c82a171b4 100755 --- a/tests/qemu-iotests/257 +++ b/tests/qemu-iotests/257 @@ -292,7 +292,7 @@ def test_bitmap_sync(bsync_mode, msync_mode=3D'bitmap',= failure=3DNone): log('--- Preparing image & VM ---\n') drive0 =3D Drive(img_path, vm=3Dvm) drive0.img_create(iotests.imgfmt, SIZE) - vm.add_device("{},id=3Dscsi0".format(iotests.get_virtio_scsi_devic= e())) + vm.add_device("{},id=3Dscsi0".format('virtio-scsi')) vm.launch() =20 file_config =3D { @@ -449,7 +449,7 @@ def test_backup_api(): log('--- Preparing image & VM ---\n') drive0 =3D Drive(img_path, vm=3Dvm) drive0.img_create(iotests.imgfmt, SIZE) - vm.add_device("{},id=3Dscsi0".format(iotests.get_virtio_scsi_devic= e())) + vm.add_device("{},id=3Dscsi0".format('virtio-scsi')) vm.launch() =20 file_config =3D { diff --git a/tests/qemu-iotests/307 b/tests/qemu-iotests/307 index 9008974346b0..c7685347bc4c 100755 --- a/tests/qemu-iotests/307 +++ b/tests/qemu-iotests/307 @@ -40,13 +40,11 @@ with iotests.FilePath('image') as img, \ =20 iotests.log('=3D=3D=3D Launch VM =3D=3D=3D') =20 - virtio_scsi_device =3D iotests.get_virtio_scsi_device() - vm.add_object('iothread,id=3Diothread0') vm.add_blockdev(f'file,filename=3D{img},node-name=3Dfile') vm.add_blockdev(f'{iotests.imgfmt},file=3Dfile,node-name=3Dfmt') vm.add_blockdev('raw,file=3Dfile,node-name=3Dro,read-only=3Don') - vm.add_device(f'id=3Dscsi0,driver=3D{virtio_scsi_device},iothread=3Dio= thread0') + vm.add_device(f'id=3Dscsi0,driver=3Dvirtio-scsi,iothread=3Diothread0') vm.launch() =20 vm.qmp_log('nbd-server-start', diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py index 90d0b6252368..1e9e6a066e90 100644 --- a/tests/qemu-iotests/iotests.py +++ b/tests/qemu-iotests/iotests.py @@ -234,11 +234,6 @@ def qemu_io_silent_check(*args): stderr=3Dsubprocess.STDOUT) return exitcode =3D=3D 0 =20 -def get_virtio_scsi_device(): - if qemu_default_machine =3D=3D 's390-ccw-virtio': - return 'virtio-scsi-ccw' - return 'virtio-scsi-pci' - class QemuIoInteractive: def __init__(self, *args): self.args =3D qemu_io_args_no_fmt + list(args) --=20 2.30.2 From nobody Sat May 4 00:28:35 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; 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 ARC-Seal: i=1; a=rsa-sha256; t=1616107472; cv=none; d=zohomail.com; s=zohoarc; b=HLM52bpekALxjXVLhGi29JGh1PgjYual5oJMc2Yv5d5xjks7UiK5po2QKwJutObnkU/rObdpTzY7c64zybOewGc6KiU3myZrbKCwDYcdGDGPIvLirm5y/hwA9/HXlf70VPOYi4RP7PS82Yl0kITY8aF5W3k6XZzXa3p8+Pnl3WE= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616107472; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=3ea/gqFJofxEh8cyhAKO+3QOm/EnujRzyszyyr/TwhM=; b=cLXt+uhnDZEJCwAyCy3l8IWGT2OyOdV2HvgnBJs71NbZxU/5yy9AVZlDSJYoe0nBuqsfz3DaNfq3p1KIBbUnwEwYDZNyKxKCes2XD3v/z5uX+lFEqPkXS1Y3fNWn3oJ8eQ4Zfgt90TpA+Nay45+lHHQg4hvdU7pHlYxCmOt9SLQ= ARC-Authentication-Results: i=1; mx.zohomail.com; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1616107472726809.4932523297914; Thu, 18 Mar 2021 15:44:32 -0700 (PDT) Received: from localhost ([::1]:51398 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lN1NX-000142-Om for importer@patchew.org; Thu, 18 Mar 2021 18:44:31 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47556) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lN1Ih-00042F-CS; Thu, 18 Mar 2021 18:39:32 -0400 Received: from mout.kundenserver.de ([212.227.126.131]:52971) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lN1Ic-0003zp-1p; Thu, 18 Mar 2021 18:39:31 -0400 Received: from localhost.localdomain ([82.142.20.38]) by mrelayeu.kundenserver.de (mreue010 [212.227.15.167]) with ESMTPSA (Nemesis) id 1N4hj5-1lnW2x0igB-011kNg; Thu, 18 Mar 2021 23:39:20 +0100 From: Laurent Vivier To: qemu-devel@nongnu.org Subject: [PATCH 3/4] iotests: test m68k with the virt machine Date: Thu, 18 Mar 2021 23:39:06 +0100 Message-Id: <20210318223907.1344870-4-laurent@vivier.eu> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20210318223907.1344870-1-laurent@vivier.eu> References: <20210318223907.1344870-1-laurent@vivier.eu> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:lRjGTTDaLT7GgAORaeikSkAvQ1GfEYdI7+zcQtNk/13evhmTcCx 6LixES3+DgWqjgScuvkFI+h2WJATf9HWNhHvqlat+F5GLNSwQ+q4t3aOQH74BvklTtUgDs8 HbEGXLf2uBIsh4Yux4wTNhDbkoIjLJtoWayu5F7GLxEvguOtAbd+U+Om3/hqWv3biQ8Vy32 fPrh+XlCZmnypcCACG3Ng== X-UI-Out-Filterresults: notjunk:1;V03:K0:dU81XvDu+1o=:ocd9AJ6gy/dyxY420KzO6o xhQdwDYZkr181o5XnYoC9zqIsdwyg0e5i66wkMk8ZqK4TF5yJZGzdHkt0lLl+5Jc9f8ABz22M KEAD6NFVrWE/NA6fFUolL4kqBiJHBRtFW1K7ZiKJ9GKtNgL8sUsBNnu+7Qer/DzLDFnTSAmv/ P/Pms+TPpjAqz8DxL6c3A7CLV8xTKvX7fFBapcMgbMIMT6hZjPZm/nnsiqffLyTs9BYFZUEpR 9qmxmtOzGSaNKVCN8/5U46wVd/pDv4HOhtTZbvHwWLpj3SSeecTS2vDxjEEDxaOi6UQ38aRm7 1t2GTw/OsRjhHywT0gprSc4dxEm6XVr5sBhbbiVKLY+QheT1g3m3IMSHhq73A+Iz9txgNO8mv +2QYwHyumC+w2NowWWPaZQRSVvn1zOwVbN//J2V+eh0keEi2itWMBh03qdi6aY2Tqz3rWcjXL WI1J2sAOpA== 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: none client-ip=212.227.126.131; envelope-from=laurent@vivier.eu; helo=mout.kundenserver.de X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Kevin Wolf , Peter Maydell , Thomas Huth , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , Eduardo Habkost , qemu-block@nongnu.org, Cornelia Huck , Markus Armbruster , Max Reitz , qemu-s390x@nongnu.org, Paolo Bonzini , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Laurent Vivier Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" This allows to cover the virtio tests with a 32bit big-endian virtio-mmio machine. Signed-off-by: Laurent Vivier Reviewed-by: Philippe Mathieu-Daud=C3=A9 --- tests/qemu-iotests/testenv.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/qemu-iotests/testenv.py b/tests/qemu-iotests/testenv.py index 1fbec854c1f7..6d27712617a3 100644 --- a/tests/qemu-iotests/testenv.py +++ b/tests/qemu-iotests/testenv.py @@ -208,6 +208,7 @@ def __init__(self, imgfmt: str, imgproto: str, aiomode:= str, ('arm', 'virt'), ('aarch64', 'virt'), ('avr', 'mega2560'), + ('m68k', 'virt'), ('rx', 'gdbsim-r5f562n8'), ('tricore', 'tricore_testboard') ) --=20 2.30.2 From nobody Sat May 4 00:28:35 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; 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 ARC-Seal: i=1; a=rsa-sha256; t=1616107290; cv=none; d=zohomail.com; s=zohoarc; b=ln9wegvnDPlu31uP5c19gYdorTf2khNWk5MizP9gmWIDFYxjnVBSPihX8ijTPfrADFpTJRDCzBPL9JUYcy3/YCndXGugsIXrm/NF9vhqdIK6S960M4Tyc5JVAbLeinryN9QB0DMT4nKH2hAxBpj5wPW/r+CoFVmQ/bovKetg174= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616107290; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=mjNNMRrk72Sx40BQc6g1k+/vKg3Zuc3NzBkzbscdD28=; b=cmQc7zIh+oQa5kIMjIk2aHY5fEkRVksoXLqX1h7s3IurGy0afsLXWmJKKU8PaBMB8WndKGnq+17XAzMWKgRIt1wHNgkQ4U11/CWxxHXN4gWtmtWGx1CPk5gepvAvgK+ke0HXtpWzH/pWbqJ0X0qnwP8ZjkEqUHJY+vKjfNtVelE= ARC-Authentication-Results: i=1; mx.zohomail.com; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1616107290990204.27103601816168; Thu, 18 Mar 2021 15:41:30 -0700 (PDT) Received: from localhost ([::1]:42460 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lN1Kb-0005mk-UD for importer@patchew.org; Thu, 18 Mar 2021 18:41:29 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47558) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lN1Ij-00042U-2n; Thu, 18 Mar 2021 18:39:33 -0400 Received: from mout.kundenserver.de ([212.227.126.134]:45899) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lN1Ic-00040I-OI; Thu, 18 Mar 2021 18:39:32 -0400 Received: from localhost.localdomain ([82.142.20.38]) by mrelayeu.kundenserver.de (mreue010 [212.227.15.167]) with ESMTPSA (Nemesis) id 1M7sUE-1lI3DP14bK-004zRh; Thu, 18 Mar 2021 23:39:21 +0100 From: Laurent Vivier To: qemu-devel@nongnu.org Subject: [PATCH 4/4] iotests: iothreads need ioeventfd Date: Thu, 18 Mar 2021 23:39:07 +0100 Message-Id: <20210318223907.1344870-5-laurent@vivier.eu> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20210318223907.1344870-1-laurent@vivier.eu> References: <20210318223907.1344870-1-laurent@vivier.eu> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:bbzX22WuGK9O0KmZrBF4uHkALslmTyPsZ0L/3E1OKGJ6KXO+heg n0GBFmKD8ER2YD4392WSZy8mMEFypBlgJoKuL2RpBpXo+GYhP/TINHFeYKBZaAUqe0Hplsy 4vmxsJtL6L7XfJB5dNik1Mdpne3eKv0uLnFmpRJyWfRjITAWs/K4fK3/hudixNdDrelRnck zYOe6FLZFV2OsbVKLqNhQ== X-UI-Out-Filterresults: notjunk:1;V03:K0:ejs4NeCSYM8=:di9A+qBtT9OloN2rr0dS1+ rlLjgeC3taUpSmo6EYO89tH0jbBksvUoY4onB4nIQ3OTqmkCS/o6hzKN4lEswkQ3VMaTW1LEx JwAozKH6ty2L6p+S8LGxfd2W+mYg13aqSz6x8YtT8zWLfiMZApA6y1+YSIRI2Ono1+jgds2Zs 9lzppsrBGb9T2zeZgABo0kTO/wCepogNzygq+ITK0KfAqo+jrG6MPdNrO1eoxzqQOaXItcWN/ iQbzz/QSp5b3/8b1m1MGc21n/DYJ/Ax5F241pvIfxvGk/nBJAgyQmO3lm1F2bRMPLsJBmtY8o zJe/uErRZbObsrhWIrumq/AM4DrUS3AY3Y8xDN61Na+l/wI+GWsYSLFkoIf6Z54ot47ymHUjA bpYXH0Vjg1kcUdoML7vEYc4oF1QmTZbhQbyjKrSm5mBiuFDqWS0q9R8i+sstk4DLJQOqu3vRm 4uB4G7LduA== 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: none client-ip=212.227.126.134; envelope-from=laurent@vivier.eu; helo=mout.kundenserver.de X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Kevin Wolf , Peter Maydell , Thomas Huth , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , Eduardo Habkost , qemu-block@nongnu.org, Cornelia Huck , Markus Armbruster , Max Reitz , qemu-s390x@nongnu.org, Paolo Bonzini , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Laurent Vivier Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" And ioeventfd are only available with virtio-scsi-pci, so don't use the ali= as and add a rule to require virtio-scsi-pci for the tests that use iothreads. Signed-off-by: Laurent Vivier --- tests/qemu-iotests/127 | 4 ++-- tests/qemu-iotests/256 | 2 ++ tests/qemu-iotests/iotests.py | 5 +++++ 3 files changed, 9 insertions(+), 2 deletions(-) diff --git a/tests/qemu-iotests/127 b/tests/qemu-iotests/127 index 98e8e82a8210..a3693533685a 100755 --- a/tests/qemu-iotests/127 +++ b/tests/qemu-iotests/127 @@ -44,7 +44,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15 _supported_fmt qcow2 _supported_proto file fuse =20 -_require_devices virtio-scsi scsi-hd +_require_devices virtio-scsi-pci scsi-hd =20 IMG_SIZE=3D64K =20 @@ -62,7 +62,7 @@ $QEMU_IO -c 'write 0 42' "$TEST_IMG.overlay0" | _filter_q= emu_io _launch_qemu \ -object iothread,id=3Diothr \ -blockdev node-name=3Dsource,driver=3D$IMGFMT,file.driver=3Dfile,file.= filename=3D"$TEST_IMG.overlay0" \ - -device virtio-scsi,id=3Dscsi-bus,iothread=3Diothr \ + -device virtio-scsi-pci,id=3Dscsi-bus,iothread=3Diothr \ -device scsi-hd,bus=3Dscsi-bus.0,drive=3Dsource =20 _send_qemu_cmd $QEMU_HANDLE \ diff --git a/tests/qemu-iotests/256 b/tests/qemu-iotests/256 index 8d82a1dd865f..eb3af0dea80c 100755 --- a/tests/qemu-iotests/256 +++ b/tests/qemu-iotests/256 @@ -24,6 +24,8 @@ import os import iotests from iotests import log =20 +iotests._verify_virtio_scsi_pci() + iotests.script_initialize(supported_fmts=3D['qcow2']) size =3D 64 * 1024 * 1024 =20 diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py index 1e9e6a066e90..3404ed534bb5 100644 --- a/tests/qemu-iotests/iotests.py +++ b/tests/qemu-iotests/iotests.py @@ -1146,6 +1146,11 @@ def _verify_virtio_blk() -> None: if 'virtio-blk' not in out: notrun('Missing virtio-blk in QEMU binary') =20 +def _verify_virtio_scsi_pci() -> None: + out =3D qemu_pipe('-M', 'none', '-device', 'help') + if 'virtio-scsi-pci' not in out: + notrun('Missing virtio-scsi-pci in QEMU binary') + =20 def supports_quorum(): return 'quorum' in qemu_img_pipe('--help') --=20 2.30.2