From nobody Mon Feb 9 19:06:09 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; 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=fail(p=none dis=none) header.from=redhat.com Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1626214515396393.5720530619658; Tue, 13 Jul 2021 15:15:15 -0700 (PDT) Received: from localhost ([::1]:48894 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1m3QgM-0000qx-Am for importer@patchew.org; Tue, 13 Jul 2021 18:15:14 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:55026) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1m3Qbl-0006ez-6N for qemu-devel@nongnu.org; Tue, 13 Jul 2021 18:10:29 -0400 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:43038) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1m3Qbj-0007hz-F8 for qemu-devel@nongnu.org; Tue, 13 Jul 2021 18:10:28 -0400 Received: from mail-wm1-f70.google.com (mail-wm1-f70.google.com [209.85.128.70]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-573-8kYCzsaNMue6nogLqOMP1w-1; Tue, 13 Jul 2021 18:10:25 -0400 Received: by mail-wm1-f70.google.com with SMTP id m31-20020a05600c3b1fb02902082e9b2132so971035wms.5 for ; Tue, 13 Jul 2021 15:10:25 -0700 (PDT) Received: from redhat.com ([2.55.15.23]) by smtp.gmail.com with ESMTPSA id t22sm283246wmi.22.2021.07.13.15.10.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 13 Jul 2021 15:10:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1626214226; 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: in-reply-to:in-reply-to:references:references; bh=VuLuulZoA+GBhWrc6iIgJsgWdXQ39YAw9fcUpNmdcKo=; b=SajJFllPValNOi6BrcnhCc1bi7CNfgMg8rlc3nJSLcH2BANLMgaAiapiK8GfdVtpSYo0Im NHsG2silbrQbzTRVCeFsiqrEZhLuoWV8IStH+qnH8kxxgWcSZ/bOPrWo+zJ/KC1/m41u36 xSHETX2IiTWkVl3hoWDyOZzKEZOqn64= X-MC-Unique: 8kYCzsaNMue6nogLqOMP1w-1 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=VuLuulZoA+GBhWrc6iIgJsgWdXQ39YAw9fcUpNmdcKo=; b=pz85x6eRk4sqG7K4+gNXmfKLCnP++YYv8dBolE0T+P1GbKQLwjGFihgFp4UaPtB0eN 09jmrK00/YfmS6k4NzeC/YJcRRCD4C1nF2wIJ2qHk362xr/Yfl5g6qUlAHXGBZCuwpPS g0MoJnD5rT765rTZs1ioTvFHjguk3NinK7eXjz3hzmHmzsnakKeYxDtTRObhThUm+Y2s nx0r3Od2ZyDy93cxGYr9sFJlodkuUo0EQH57txt7cySW0A08kgBvFCgZhvBo/gnnPxm1 kaveVVBU9C/xeq+vDRe4zXsL5seJ7rQ0SkfAlNpOuhAgECT2TJOD1Jc2C19pNve//6HO XQZA== X-Gm-Message-State: AOAM5326YoQsSq9pWrz1UB9N/CmFC0n4qAPOdQHQSavsDZHzcOjEZjCM SSG3p+Nr+bl9IPCygToKrhxoqEeuHABaRbX6gyNP1o8sKLCI1c8QKv2m13GFF3tHVhAkmsXErYv PO309TpEEQhG/hs25DaJpPT4aK3u8uFF6YDRfHA4fSjFuN6zIg2/uH87Eub1a X-Received: by 2002:a5d:4c8c:: with SMTP id z12mr8775572wrs.97.1626214223610; Tue, 13 Jul 2021 15:10:23 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx3er7tzuUYJbw3XpwF3jCBZpyvbRhVai343v77n/95LrZA9/5QsMiMTwLovw6utan8qdjXKw== X-Received: by 2002:a5d:4c8c:: with SMTP id z12mr8775543wrs.97.1626214223378; Tue, 13 Jul 2021 15:10:23 -0700 (PDT) Date: Tue, 13 Jul 2021 18:10:19 -0400 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Subject: [PULL 05/23] hw/acpi/ich9: Set ACPI PCI hot-plug as default on Q35 Message-ID: <20210713220946.212562-6-mst@redhat.com> References: <20210713220946.212562-1-mst@redhat.com> MIME-Version: 1.0 In-Reply-To: <20210713220946.212562-1-mst@redhat.com> X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=mst@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Disposition: inline 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=216.205.24.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -34 X-Spam_score: -3.5 X-Spam_bar: --- X-Spam_report: (-3.5 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.7, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-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: Peter Maydell , Eduardo Habkost , Richard Henderson , Julia Suvorova , Igor Mammedov , Paolo Bonzini , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZM-MESSAGEID: 1626214516589100001 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Julia Suvorova Q35 has three different types of PCI devices hot-plug: PCIe Native, SHPC Native and ACPI hot-plug. This patch changes the default choice for cold-plugged bridges from PCIe Native to ACPI Hot-plug with ability to use SHPC and PCIe Native for hot-plugged bridges. This is a list of the PCIe Native hot-plug issues that led to this change: * no racy behavior during boot (see 110c477c2ed) * no delay during deleting - after the actual power off software must wait at least 1 second before indicating about it. This case is quite important for users, it even has its own bug: https://bugzilla.redhat.com/show_bug.cgi?id=3D1594168 * no timer-based behavior - in addition to the previous example, the attention button has a 5-second waiting period, during which the operation can be canceled with a second press. While this looks fine for manual button control, automation will result in the need to queue or drop events, and the software receiving events in all sort of unspecified combinations of attention/power indicator states, which is racy and uppredictable. * fixes: * https://bugzilla.redhat.com/show_bug.cgi?id=3D1752465 * https://bugzilla.redhat.com/show_bug.cgi?id=3D1690256 To return to PCIe Native hot-plug: -global ICH9-LPC.acpi-pci-hotplug-with-bridge-support=3Doff Known issue: older linux guests need the following flag to allow hotplugged pci express devices to use io: -device pcie-root-port,io-reserve=3D4096. io is unusual for pci express so this seems minor. We'll fix this by a follow up patch. Signed-off-by: Julia Suvorova Reviewed-by: Igor Mammedov Message-Id: <20210713004205.775386-6-jusual@redhat.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: David Gibson --- hw/acpi/ich9.c | 2 +- hw/i386/pc.c | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c index 2f4eb453ac..778e27b659 100644 --- a/hw/acpi/ich9.c +++ b/hw/acpi/ich9.c @@ -427,7 +427,7 @@ void ich9_pm_add_properties(Object *obj, ICH9LPCPMRegs = *pm) pm->disable_s3 =3D 0; pm->disable_s4 =3D 0; pm->s4_val =3D 2; - pm->use_acpi_hotplug_bridge =3D false; + pm->use_acpi_hotplug_bridge =3D true; =20 object_property_add_uint32_ptr(obj, ACPI_PM_PROP_PM_IO_BASE, &pm->pm_io_base, OBJ_PROP_FLAG_READ); diff --git a/hw/i386/pc.c b/hw/i386/pc.c index 8e1220db72..7e03848792 100644 --- a/hw/i386/pc.c +++ b/hw/i386/pc.c @@ -98,6 +98,7 @@ GlobalProperty pc_compat_6_0[] =3D { { "qemu64" "-" TYPE_X86_CPU, "family", "6" }, { "qemu64" "-" TYPE_X86_CPU, "model", "6" }, { "qemu64" "-" TYPE_X86_CPU, "stepping", "3" }, + { "ICH9-LPC", "acpi-pci-hotplug-with-bridge-support", "off" }, }; const size_t pc_compat_6_0_len =3D G_N_ELEMENTS(pc_compat_6_0); =20 --=20 MST