From nobody Fri May 17 11:05:27 2024 Delivered-To: importer@patchew.org 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; 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; dmarc=fail(p=none dis=none) header.from=nutanix.com ARC-Seal: i=1; a=rsa-sha256; t=1587136984; cv=none; d=zohomail.com; s=zohoarc; b=c1hXXypg3dYcVv3y2ltinX4gsEFyuaglzcu8XUdNejw/PcA7PqxuwvOMUReFBSwI8K2btwNep98hXOKkueDFuDqOvGzkoD/yiuJExivpAJ94y+AclTiSZiaDYn3zIpBLFejVFiTVxdTMpo4Q3ofn1DN0yC+oEzqjxjxxW2g7CxU= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1587136984; h=Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:Message-ID:Sender:Subject:To; bh=5Hl+NLZ/708m6fPI6JtyilIM7UAPLd3Xj4kY/0rmInA=; b=VpoxRLT9wkwT24bKdBkN922qT0eipHG8S9qZaZYten1hmF+p/z3P/5yFYKp2o+VnamZx8wdgQyhvqeAdjAB0YGozbH846CCHlRQe/dgY2V18o1ebR3fG4wLarFVoS5Kbcm31n2exBdIkhC5d2xhYG2D+97AXlhwdsas8rJcTHoI= 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; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1587136984907143.90570399103945; Fri, 17 Apr 2020 08:23:04 -0700 (PDT) Received: from localhost ([::1]:48452 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jPSpa-00049b-PB for importer@patchew.org; Fri, 17 Apr 2020 11:23:02 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51334) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jPSof-00038B-B0 for qemu-devel@nongnu.org; Fri, 17 Apr 2020 11:22:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1jPSoe-00012f-0T for qemu-devel@nongnu.org; Fri, 17 Apr 2020 11:22:05 -0400 Received: from [192.146.154.243] (port=45985 helo=mcp01.nutanix.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1jPSod-00012J-PI for qemu-devel@nongnu.org; Fri, 17 Apr 2020 11:22:03 -0400 Received: from ani-sinha1.localdomain (ani-sinha1.dev.nutanix.com [10.41.71.199]) by mcp01.nutanix.com (Postfix) with ESMTP id 92D42102FFD8; Fri, 17 Apr 2020 15:13:44 +0000 (UTC) Received: by ani-sinha1.localdomain (Postfix, from userid 16671) id 8D90B47C89; Fri, 17 Apr 2020 15:13:44 +0000 (UTC) From: Ani Sinha To: qemu-devel@nongnu.org Subject: [PATCH] Add a new PIIX option to control PCI hot unplugging of devices on non-root buses Date: Fri, 17 Apr 2020 15:13:31 +0000 Message-Id: <1587136411-200885-1-git-send-email-ani.sinha@nutanix.com> X-Mailer: git-send-email 1.8.3.1 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 192.146.154.243 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: ani.sinha@nutanix.com, Eduardo Habkost , "Michael S. Tsirkin" , Paolo Bonzini , Marcel Apfelbaum , Igor Mammedov , Richard Henderson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" A new option "use_acpi_unplug" is introduced for PIIX which will selectively only disable hot unplugging of both hot plugged and cold plugged PCI devices on non-root PCI buses. This will prevent hot unplugging of devices from Windows based guests from system tray but will not prevent devices from being hot plugged into the guest. The patch is initial version and is a rough implementation. It has been tested on Windows guests. Signed-off-by: Ani Sinha --- hw/acpi/piix4.c | 3 +++ hw/i386/acpi-build.c | 40 ++++++++++++++++++++++++++-------------- 2 files changed, 29 insertions(+), 14 deletions(-) diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c index d706360..dad1bf4 100644 --- a/hw/acpi/piix4.c +++ b/hw/acpi/piix4.c @@ -82,6 +82,7 @@ typedef struct PIIX4PMState { =20 AcpiPciHpState acpi_pci_hotplug; bool use_acpi_pci_hotplug; + bool use_acpi_unplug; =20 uint8_t disable_s3; uint8_t disable_s4; @@ -676,6 +677,8 @@ static Property piix4_pm_properties[] =3D { DEFINE_PROP_UINT8(ACPI_PM_PROP_S4_VAL, PIIX4PMState, s4_val, 2), DEFINE_PROP_BOOL("acpi-pci-hotplug-with-bridge-support", PIIX4PMState, use_acpi_pci_hotplug, true), + DEFINE_PROP_BOOL("acpi-pci-hotunplug-enable-bridge", PIIX4PMState, + use_acpi_unplug, true), DEFINE_PROP_BOOL("memory-hotplug-support", PIIX4PMState, acpi_memory_hotplug.is_enabled, true), DEFINE_PROP_END_OF_LIST(), diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 2bc8117..526feb2 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -94,6 +94,7 @@ typedef struct AcpiPmInfo { bool s3_disabled; bool s4_disabled; bool pcihp_bridge_en; + bool pcihup_bridge_en; uint8_t s4_val; AcpiFadtData fadt; uint16_t cpu_hp_io_base; @@ -220,6 +221,9 @@ static void acpi_get_pm_info(AcpiPmInfo *pm) pm->pcihp_bridge_en =3D object_property_get_bool(obj, "acpi-pci-hotplug-with-bridge-suppor= t", NULL); + pm->pcihup_bridge_en =3D + object_property_get_bool(obj, "acpi-pci-hotunplug-enable-bridge", + NULL); } =20 static void acpi_get_misc_info(AcpiMiscInfo *info) @@ -430,7 +434,8 @@ static void build_append_pcihp_notify_entry(Aml *method= , int slot) } =20 static void build_append_pci_bus_devices(Aml *parent_scope, PCIBus *bus, - bool pcihp_bridge_en) + bool pcihp_bridge_en, + bool pcihup_bridge_en) { Aml *dev, *notify_method =3D NULL, *method; QObject *bsel; @@ -458,11 +463,14 @@ static void build_append_pci_bus_devices(Aml *parent_= scope, PCIBus *bus, dev =3D aml_device("S%.02X", PCI_DEVFN(slot, 0)); aml_append(dev, aml_name_decl("_SUN", aml_int(slot))); aml_append(dev, aml_name_decl("_ADR", aml_int(slot << 16))= ); - method =3D aml_method("_EJ0", 1, AML_NOTSERIALIZED); - aml_append(method, - aml_call2("PCEJ", aml_name("BSEL"), aml_name("_SUN")) - ); - aml_append(dev, method); + if (pcihup_bridge_en || pci_bus_is_root(bus)) { + method =3D aml_method("_EJ0", 1, AML_NOTSERIALIZED); + aml_append(method, + aml_call2("PCEJ", aml_name("BSEL"), + aml_name("_SUN")) + ); + aml_append(dev, method); + } aml_append(parent_scope, dev); =20 build_append_pcihp_notify_entry(notify_method, slot); @@ -516,12 +524,14 @@ static void build_append_pci_bus_devices(Aml *parent_= scope, PCIBus *bus, /* add _SUN/_EJ0 to make slot hotpluggable */ aml_append(dev, aml_name_decl("_SUN", aml_int(slot))); =20 - method =3D aml_method("_EJ0", 1, AML_NOTSERIALIZED); - aml_append(method, - aml_call2("PCEJ", aml_name("BSEL"), aml_name("_SUN")) - ); - aml_append(dev, method); - + if (pcihup_bridge_en || pci_bus_is_root(bus)) { + method =3D aml_method("_EJ0", 1, AML_NOTSERIALIZED); + aml_append(method, + aml_call2("PCEJ", aml_name("BSEL"), + aml_name("_SUN")) + ); + aml_append(dev, method); + } if (bsel) { build_append_pcihp_notify_entry(notify_method, slot); } @@ -532,7 +542,8 @@ static void build_append_pci_bus_devices(Aml *parent_sc= ope, PCIBus *bus, */ PCIBus *sec_bus =3D pci_bridge_get_sec_bus(PCI_BRIDGE(pdev)); =20 - build_append_pci_bus_devices(dev, sec_bus, pcihp_bridge_en); + build_append_pci_bus_devices(dev, sec_bus, pcihp_bridge_en, + pcihup_bridge_en); } /* slot descriptor has been composed, add it into parent context */ aml_append(parent_scope, dev); @@ -2133,7 +2144,8 @@ build_dsdt(GArray *table_data, BIOSLinker *linker, if (bus) { Aml *scope =3D aml_scope("PCI0"); /* Scan all PCI buses. Generate tables to support hotplug. */ - build_append_pci_bus_devices(scope, bus, pm->pcihp_bridge_en); + build_append_pci_bus_devices(scope, bus, pm->pcihp_bridge_en, + pm->pcihup_bridge_en); =20 if (TPM_IS_TIS(tpm_find())) { dev =3D aml_device("ISA.TPM"); --=20 1.9.4