From nobody Sat Feb 7 09:36:45 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 216.205.24.124 as permitted sender) client-ip=216.205.24.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 216.205.24.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1605721222; cv=none; d=zohomail.com; s=zohoarc; b=LEGHfE9X+tGhzPgdjwWMtZRQDOypwKoTimnfJEbGLBAR4Lb9BREqhVk0Ytg3Rif9z2UKHhmA8ODrqisZMtKqL+6S63NmwfOAO0zZNtg6HogOnYpAvBiGhSXf/fhAy/+qmOvBwaWQ5Yp6jM3iKCUp52t4PnIITPHx++Pdyce+B5c= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1605721222; h=Content-Type:Content-Transfer-Encoding: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=eCl1gpVp5sgIMk26Ypla3RXuifUik3bOAQDUUHbZ1WM=; b=nLSaqoeTc8OfWTeTDZGBqBKQ5kN/Lirl4nyLprmsxVABN7Y7WST1dQWu7DbQ5DPAoGiB9N9GgeHNMqmbPhD5SbgFE/m8c2pKI8tjmVBsDCcShC2azT8bmT6dykWlmE47qhiTi+u+sr/LU306BP5WjHQ4cOjY8NNj31saJsCZjdw= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 216.205.24.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) header.from= Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mx.zohomail.com with SMTPS id 1605721222868772.5235697540655; Wed, 18 Nov 2020 09:40:22 -0800 (PST) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-119-fgGzYxV4MpmkjS7VW-wpJQ-1; Wed, 18 Nov 2020 12:40:18 -0500 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 4A7AF1926DAD; Wed, 18 Nov 2020 17:40:12 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 1C7881964D; Wed, 18 Nov 2020 17:40:12 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id D69EA86BE9; Wed, 18 Nov 2020 17:40:11 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 0AIHeAch004408 for ; Wed, 18 Nov 2020 12:40:10 -0500 Received: by smtp.corp.redhat.com (Postfix) id DAD071A268; Wed, 18 Nov 2020 17:40:10 +0000 (UTC) Received: from localhost.localdomain.com (ovpn-115-41.ams2.redhat.com [10.36.115.41]) by smtp.corp.redhat.com (Postfix) with ESMTP id B80EF171CB; Wed, 18 Nov 2020 17:40:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1605721221; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=eCl1gpVp5sgIMk26Ypla3RXuifUik3bOAQDUUHbZ1WM=; b=V9a0YgrnFUDAWSzBOra6NJu2uYxT5w+7yldaRv1mkZfl647z3k9PEVGRdF1/sr4KJzGj7C 8YWnLEUocHB8YnmuEV1VbzZNLSwm9xpxT1u9t1d7rsaO5acEK7SYGYAM2tRrpGzY+dTugF athymwRmjTZjQsq6RFlyiG11cNHZa/A= X-MC-Unique: fgGzYxV4MpmkjS7VW-wpJQ-1 From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 04/10] conf: add support for ESP SCSI controller family Date: Wed, 18 Nov 2020 17:39:47 +0000 Message-Id: <20201118173953.576417-5-berrange@redhat.com> In-Reply-To: <20201118173953.576417-1-berrange@redhat.com> References: <20201118173953.576417-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) The NCR53C90 is the built-in SCSI controller on all sparc machine types, and some mips and m68k machine types. The DC390 and AM53C974 are PCI SCSI controllers that can be added to any PCI machine. These are only interesting for emulating obsolete hardware platforms. Signed-off-by: Daniel P. Berrang=C3=A9 --- docs/formatdomain.rst | 3 ++- docs/schemas/domaincommon.rng | 3 +++ src/conf/domain_conf.c | 8 ++++++++ src/conf/domain_conf.h | 3 +++ src/qemu/qemu_command.c | 3 +++ src/qemu/qemu_domain_address.c | 3 +++ src/qemu/qemu_validate.c | 6 ++++++ src/vbox/vbox_common.c | 3 +++ src/vmx/vmx.c | 3 +++ 9 files changed, 34 insertions(+), 1 deletion(-) diff --git a/docs/formatdomain.rst b/docs/formatdomain.rst index c2c23371b1..ff64996af2 100644 --- a/docs/formatdomain.rst +++ b/docs/formatdomain.rst @@ -3426,7 +3426,8 @@ specific features, such as: ``scsi`` A ``scsi`` controller has an optional attribute ``model``, which is one= of 'auto', 'buslogic', 'ibmvscsi', 'lsilogic', 'lsisas1068', 'lsisas1078', - 'virtio-scsi', 'vmpvscsi', 'virtio-transitional', 'virtio-non-transitio= nal'. + 'virtio-scsi', 'vmpvscsi', 'virtio-transitional', 'virtio-non-transitio= nal', + 'ncr53c90' (as builtin implicit controller only), 'am53c974', 'dc390'. See `Virtio transitional devices <#elementsVirtioTransitional>`__ for m= ore details. ``usb`` diff --git a/docs/schemas/domaincommon.rng b/docs/schemas/domaincommon.rng index f86a854863..1ccc9759ea 100644 --- a/docs/schemas/domaincommon.rng +++ b/docs/schemas/domaincommon.rng @@ -2381,6 +2381,9 @@ lsisas1078 virtio-transitional virtio-non-transitional + ncr53c90 + dc390 + am53c974 diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 498a8b6ef0..abfde5b161 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -445,6 +445,9 @@ VIR_ENUM_IMPL(virDomainControllerModelSCSI, "lsisas1078", "virtio-transitional", "virtio-non-transitional", + "ncr53c90", + "dc390", + "am53c974", ); =20 VIR_ENUM_IMPL(virDomainControllerModelISA, VIR_DOMAIN_CONTROLLER_MODEL_ISA= _LAST, @@ -4971,6 +4974,11 @@ virDomainSCSIDriveAddressIsUsed(const virDomainDef *= def, case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_BUSLOGIC: reserved =3D 7; break; + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_NCR53C90: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DC390: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AM53C974: + max =3D 6; + break; case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DEFAULT: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AUTO: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LAST: diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h index 3e3d4bd002..96e6c34553 100644 --- a/src/conf/domain_conf.h +++ b/src/conf/domain_conf.h @@ -649,6 +649,9 @@ typedef enum { VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LSISAS1078, VIR_DOMAIN_CONTROLLER_MODEL_SCSI_VIRTIO_TRANSITIONAL, VIR_DOMAIN_CONTROLLER_MODEL_SCSI_VIRTIO_NON_TRANSITIONAL, + VIR_DOMAIN_CONTROLLER_MODEL_SCSI_NCR53C90, + VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DC390, + VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AM53C974, =20 VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LAST } virDomainControllerModelSCSI; diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 34b5746c1a..d3de13c6ee 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_command.c @@ -2473,6 +2473,9 @@ qemuBuildControllerDevStr(const virDomainDef *domainD= ef, break; case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AUTO: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_BUSLOGIC: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_NCR53C90: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DC390: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AM53C974: virReportError(VIR_ERR_CONFIG_UNSUPPORTED, _("Unsupported controller model: %s"), virDomainControllerModelSCSITypeToString(def->m= odel)); diff --git a/src/qemu/qemu_domain_address.c b/src/qemu/qemu_domain_address.c index dd87915a97..f1fb532f39 100644 --- a/src/qemu/qemu_domain_address.c +++ b/src/qemu/qemu_domain_address.c @@ -637,6 +637,7 @@ qemuDomainDeviceCalculatePCIConnectFlags(virDomainDevic= eDefPtr dev, case VIR_DOMAIN_CONTROLLER_TYPE_SCSI: switch ((virDomainControllerModelSCSI) cont->model) { case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DEFAULT: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_NCR53C90: return 0; =20 case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_VIRTIO_SCSI: @@ -652,6 +653,8 @@ qemuDomainDeviceCalculatePCIConnectFlags(virDomainDevic= eDefPtr dev, case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_VMPVSCSI: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_IBMVSCSI: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LSISAS1078: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DC390: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AM53C974: return pciFlags; =20 case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LAST: diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c index 09cdcb20c2..2cd9ee8230 100644 --- a/src/qemu/qemu_validate.c +++ b/src/qemu/qemu_validate.c @@ -2998,6 +2998,9 @@ qemuValidateCheckSCSIControllerModel(virQEMUCapsPtr q= emuCaps, break; case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AUTO: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_BUSLOGIC: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_NCR53C90: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DC390: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AM53C974: virReportError(VIR_ERR_CONFIG_UNSUPPORTED, _("Unsupported controller model: %s"), virDomainControllerModelSCSITypeToString(model)); @@ -3116,6 +3119,9 @@ qemuValidateDomainDeviceDefControllerSCSI(const virDo= mainControllerDef *controll case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_VMPVSCSI: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_IBMVSCSI: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LSISAS1078: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_NCR53C90: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DC390: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AM53C974: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DEFAULT: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LAST: break; diff --git a/src/vbox/vbox_common.c b/src/vbox/vbox_common.c index fc897735b0..138403b034 100644 --- a/src/vbox/vbox_common.c +++ b/src/vbox/vbox_common.c @@ -433,6 +433,9 @@ vboxSetStorageController(virDomainControllerDefPtr cont= roller, case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_VIRTIO_TRANSITIONAL: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_VIRTIO_NON_TRANSITIONAL: case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LSISAS1078: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_NCR53C90: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_DC390: + case VIR_DOMAIN_CONTROLLER_MODEL_SCSI_AM53C974: virReportError(VIR_ERR_CONFIG_UNSUPPORTED, _("The vbox driver does not support %s SCSI " "controller model"), diff --git a/src/vmx/vmx.c b/src/vmx/vmx.c index 4001174380..0d79b8f0ec 100644 --- a/src/vmx/vmx.c +++ b/src/vmx/vmx.c @@ -560,6 +560,9 @@ VIR_ENUM_IMPL(virVMXControllerModelSCSI, "UNUSED lsisas1078", "UNUSED virtio-transitional", "UNUSED virtio-non-transitional", + "UNUSED ncr53c90", + "UNUSED dc390", + "UNUSED am53c974", ); =20 static int virVMXParseVNC(virConfPtr conf, virDomainGraphicsDefPtr *def); --=20 2.28.0