From nobody Sat Apr 27 15:32:08 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; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1616533114; cv=none; d=zohomail.com; s=zohoarc; b=VaMNtWIWEMgPo7QAq9MIA0G3PE5nAdd651wxTXdBt5+bHvD3Da+eSYJErzeEWZVKz99GKYA9jEVIbiUqFCbKoT31rkdwNTzDFENYNyaAZkYKSyWsNHkq8GOI7XR3dLgPaKBaqvEPSzlBNGz81pigohmI2wtrUELersZvkhrTcJ8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616533114; 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=/cO7qmCytfYOY8UO/GW54GTp2LbumSs1vac8K8BZWus=; b=E+xrQn+puHSurriIB402OJGPwT0wFTrICJwiskcemZKXIZBPX+W98ZQQAnHk4fDfgqSnC2yCAaxX6xToRBMVsT8cZCM/Mnd0Sv/74JUcc2z4SsGQufZN5MBnmq60MiWMFwRlgQvdJN94hygz+7rWBUCvfVl46j3d1BOrgiGERZk= 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 161653311451896.3534942202017; Tue, 23 Mar 2021 13:58:34 -0700 (PDT) Received: from localhost ([::1]:41802 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lOo6j-00082F-Fk for importer@patchew.org; Tue, 23 Mar 2021 16:58:33 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60440) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lOo2z-00027C-Py for qemu-devel@nongnu.org; Tue, 23 Mar 2021 16:54:41 -0400 Received: from mga11.intel.com ([192.55.52.93]:35910) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lOo2v-0000Ds-38 for qemu-devel@nongnu.org; Tue, 23 Mar 2021 16:54:41 -0400 Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2021 13:54:28 -0700 Received: from ls.sc.intel.com (HELO localhost) ([143.183.96.54]) by fmsmga005-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2021 13:54:26 -0700 IronPort-SDR: Wpn9gFRQVVCI1OzaOghxov/Www/PlZAylDTmzZRDBMWUxdxT4J2eKORoEXjSvxq7GZX4srz2if bUlUng4yMHkg== X-IronPort-AV: E=McAfee;i="6000,8403,9932"; a="187248555" X-IronPort-AV: E=Sophos;i="5.81,272,1610438400"; d="scan'208";a="187248555" IronPort-SDR: KqThw6/KJA5a9Kc8xjdI/MW997bNdrhuZnq1Rb2wsR4hPdf7dVO6FI/ptbDRQ8V3hK5onUS0xE 1yJRw6e4Wg2Q== X-IronPort-AV: E=Sophos;i="5.81,272,1610438400"; d="scan'208";a="607869568" From: Isaku Yamahata To: qemu-devel@nongnu.org, mst@redhat.com, peter.maydell@linaro.org, imammedo@redhat.com, f4bug@amsat.org Subject: [PATCH v3 1/4] acpi/piix4: reinitialize acpi PM device on reset Date: Tue, 23 Mar 2021 13:52:24 -0700 Message-Id: <8a5bbd19727045ec863523830078dd4ca63f6a9a.1616532563.git.isaku.yamahata@intel.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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=192.55.52.93; envelope-from=isaku.yamahata@intel.com; helo=mga11.intel.com X-Spam_score_int: -68 X-Spam_score: -6.9 X-Spam_bar: ------ X-Spam_report: (-6.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5, 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: isaku.yamahata@intel.com, berrange@redhat.com, Reinoud Zandijk , isaku.yamahata@gmail.com, pbonzini@redhat.com, aurelien@aurel32.net Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Commit 6be8cf56bc8b made sure that SCI is enabled in PM1.CNT on reset in acpi_only mode by modifying acpi_pm1_cnt_reset() and that worked for q35 as expected. The function was introduced by commit eaba51c573a (acpi, acpi_piix, vt82c686: factor out PM1_CNT logic) that forgot to actually call it at piix4 reset time and as result SCI_EN wasn't set as was expected by 6be8cf56bc8b in acpi_only mode. So Windows crashes when it notices that SCI_EN is not set and FADT is not providing information about how to enable it anymore. Reproducer: qemu-system-x86_64 -enable-kvm -M pc-i440fx-6.0,smm=3Doff -cdrom any_win= dows_10x64.iso Fix it by calling acpi_pm1_cnt_reset() at piix4 reset time. Occasionally this patch adds reset acpi PM related registers on piix4 reset time and de-assert sci. piix4_pm_realize() initializes acpi pm tmr, evt, cnt and gpe. Reset them on device reset. pm_reset() in ich9.c correctly calls corresponding reset functions. Fixes: 6be8cf56bc8b (acpi/core: always set SCI_EN when SMM isn't supported) Reported-by: Reinoud Zandijk Co-developed-by: Igor Mammedov Signed-off-by: Igor Mammedov Signed-off-by: Isaku Yamahata --- CC: imammedo@redhat.com CC: isaku.yamahata@intel.com CC: mst@redhat.com CC: reinoud@NetBSD.org CC: isaku.yamahata@gmail.com CC: berrange@redhat.com CC: pbonzini@redhat.com CC: f4bug@amsat.org CC: aurelien@aurel32.net --- hw/acpi/piix4.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c index 6056d51667..8f8b0e95e5 100644 --- a/hw/acpi/piix4.c +++ b/hw/acpi/piix4.c @@ -326,6 +326,13 @@ static void piix4_pm_reset(DeviceState *dev) /* Mark SMM as already inited (until KVM supports SMM). */ pci_conf[0x5B] =3D 0x02; } + + acpi_pm1_evt_reset(&s->ar); + acpi_pm1_cnt_reset(&s->ar); + acpi_pm_tmr_reset(&s->ar); + acpi_gpe_reset(&s->ar); + acpi_update_sci(&s->ar, s->irq); + pm_io_space_update(s); acpi_pcihp_reset(&s->acpi_pci_hotplug, !s->use_acpi_root_pci_hotplug); } --=20 2.25.1 From nobody Sat Apr 27 15:32:08 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; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1616533073; cv=none; d=zohomail.com; s=zohoarc; b=J52eDKAQ0odwPvykWfyUoVzywymvWUK+engDxNFgycU+cqSNHrDlkgZInHDsWMmQMc74NrHQGt47QyZlwaBLjU+/CadrAx+C7ZJR1XjlX3h8Sp2gyleaZ3jeopzxzOmljuYjf9lDo9FHsO4yWMic6ipjccJjPsvkqNQL5x85ZqE= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616533073; h=Content-Type: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=vKMhtF7i2XfExnuKhVzVUNlCRvZCzAUtBDGPLRw2nFY=; b=N6GPwzZYwktX8S+rMnuOR14aORNW364lAS1iHCTQxDtAaw/1NO361YsCokJyZqFUj/afrqu+T6JGAr7I8RWhWfJ4nunWeNLbP7mzHgmeZzwpnh+W7ZUFosJjRJgXLI3Ukk4seYVta8+Ol7V0c2Eu90NnvcUzIlvyruhq3L/b2RU= 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 1616533073906154.88072988681836; Tue, 23 Mar 2021 13:57:53 -0700 (PDT) Received: from localhost ([::1]:39450 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lOo64-0006mV-Qw for importer@patchew.org; Tue, 23 Mar 2021 16:57:52 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60402) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lOo2w-0001xG-EK for qemu-devel@nongnu.org; Tue, 23 Mar 2021 16:54:38 -0400 Received: from mga11.intel.com ([192.55.52.93]:35899) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lOo2u-00008L-Ej for qemu-devel@nongnu.org; Tue, 23 Mar 2021 16:54:38 -0400 Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2021 13:54:28 -0700 Received: from ls.sc.intel.com (HELO localhost) ([143.183.96.54]) by fmsmga005-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2021 13:54:26 -0700 IronPort-SDR: +Dky8TxVKM5isR/GXfUvxF57PfnjpQjTg1n1KWgwHEzXx7S0oW8aPHzmTMKsu62Yf7wgSkGsoK I4uwMaZ7l07w== X-IronPort-AV: E=McAfee;i="6000,8403,9932"; a="187248556" X-IronPort-AV: E=Sophos;i="5.81,272,1610438400"; d="scan'208";a="187248556" IronPort-SDR: NpZ6QtKd4oIxPT6gIMler5CXZ5bweMMxxxjhOFvohhIuxeCkq4L9ZSEGqHrKMy56UyeRNkmEFM IDcZAC2CZX9w== X-IronPort-AV: E=Sophos;i="5.81,272,1610438400"; d="scan'208";a="607869572" From: Isaku Yamahata To: qemu-devel@nongnu.org, mst@redhat.com, peter.maydell@linaro.org, imammedo@redhat.com, f4bug@amsat.org Subject: [PATCH v3 2/4] vt82c686.c: don't raise SCI when PCI_INTERRUPT_PIN isn't setup Date: Tue, 23 Mar 2021 13:52:25 -0700 Message-Id: <62a5fc69e453fb848bfd4794bae1852a75af73c5.1616532563.git.isaku.yamahata@intel.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable 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=192.55.52.93; envelope-from=isaku.yamahata@intel.com; helo=mga11.intel.com X-Spam_score_int: -68 X-Spam_score: -6.9 X-Spam_bar: ------ X-Spam_report: (-6.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5, 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: isaku.yamahata@intel.com, Huacai Chen , isaku.yamahata@gmail.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Without this patch, the following patch will triger clan runtime sanitizer warnings as follows. This patch proactively works around it. I leave a correct fix to v582c686.c maintainerfix as I'm not sure about fuloong2e device model. > MALLOC_PERTURB_=3D${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))} > QTEST_QEMU_IMG=3D./qemu-img > G_TEST_DBUS_DAEMON=3D/home/petmay01/linaro/qemu-for-merges/tests/dbus-vms= tate-daemon.sh > QTEST_QEMU_BINARY=3D./qemu-system-mips64el tests/qtest/qom-test --tap -k > PASS 1 qtest-mips64el/qom-test /mips64el/qom/loongson3-virt > PASS 2 qtest-mips64el/qom-test /mips64el/qom/none > PASS 3 qtest-mips64el/qom-test /mips64el/qom/magnum > PASS 4 qtest-mips64el/qom-test /mips64el/qom/mipssim > PASS 5 qtest-mips64el/qom-test /mips64el/qom/malta > ../../hw/pci/pci.c:252:30: runtime error: shift exponent -1 is negative > PASS 6 qtest-mips64el/qom-test /mips64el/qom/fuloong2e > PASS 7 qtest-mips64el/qom-test /mips64el/qom/boston > PASS 8 qtest-mips64el/qom-test /mips64el/qom/pica61 > > and similarly for eg > > MALLOC_PERTURB_=3D${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))} > QTEST_QEMU_IMG=3D./qemu-img > G_TEST_DBUS_DAEMON=3D/home/petmay01/linaro/qemu-for-merges/tests/dbus-vms= tate-daemon.sh > QTEST_QEMU_BINARY=3D./qemu-system-mips64el tests/qtest/endianness-test > --tap -k > ../../hw/pci/pci.c:252:30: runtime error: shift exponent -1 is negative > PASS 1 qtest-mips64el/endianness-test /mips64el/endianness/fuloong2e > ../../hw/pci/pci.c:252:30: runtime error: shift exponent -1 is negative > PASS 2 qtest-mips64el/endianness-test /mips64el/endianness/split/fuloong2e > ../../hw/pci/pci.c:252:30: runtime error: shift exponent -1 is negative > PASS 3 qtest-mips64el/endianness-test /mips64el/endianness/combine/fuloon= g2e Cc: BALATON Zoltan Cc: Huacai Chen Cc: "Philippe Mathieu-Daud=C3=A9" Cc: Jiaxun Yang Reported-by: Peter Maydell Signed-off-by: Isaku Yamahata --- hw/isa/vt82c686.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/hw/isa/vt82c686.c b/hw/isa/vt82c686.c index 05d084f698..f0fb309f12 100644 --- a/hw/isa/vt82c686.c +++ b/hw/isa/vt82c686.c @@ -144,7 +144,18 @@ static void pm_update_sci(ViaPMState *s) ACPI_BITMASK_POWER_BUTTON_ENABLE | ACPI_BITMASK_GLOBAL_LOCK_ENABLE | ACPI_BITMASK_TIMER_ENABLE)) !=3D 0); - pci_set_irq(&s->dev, sci_level); + if (pci_get_byte(s->dev.config + PCI_INTERRUPT_PIN)) { + /* + * FIXME: + * Fix device model that realizes this PM device and remove + * this work around. + * The device model should wire SCI and setup + * PCI_INTERRUPT_PIN properly. + * If PIN# =3D 0(interrupt pin isn't used), don't raise SCI as + * work around. + */ + pci_set_irq(&s->dev, sci_level); + } /* schedule a timer interruption if needed */ acpi_pm_tmr_update(&s->ar, (s->ar.pm1.evt.en & ACPI_BITMASK_TIMER_ENAB= LE) && !(pmsts & ACPI_BITMASK_TIMER_STATUS)); --=20 2.25.1 From nobody Sat Apr 27 15:32:08 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; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1616533180; cv=none; d=zohomail.com; s=zohoarc; b=SPCM6V/n76kCQJaZ8pu268MC40KAcMojOc/jbm/9odcxh3hswds3fcmFuq0PdpZWIouoNOVff1pX0rgde+UHLlQoNrtieLyjVx0Cu0iXO+HGLfw4GrvzEROXO1SjPa2hEZO6ZfNTHjX7mQ71HweFvGxhGscbK1EDQPLKvEnlA6Y= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616533180; h=Content-Type: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=PClMDAYiv+yW1sS2j1LiP820TZ1Ter/euZ913TO4R44=; b=lbY5LqXdWV6lm147hoeJsDuIM8HEwVsH/tqDKW7/8MvOBwpyYWcUtKnVUuv07JKaIWujAUDr1O3X0DM+Lug598jd8Job0jma48ieyCfym/K/G4l/gKCDI/SkIIpe1XqoGtopzEQU8qwQSe9NmQgVDH9KK5tXQx/TcOMG8upJ9z0= 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 1616533180157754.5978146794976; Tue, 23 Mar 2021 13:59:40 -0700 (PDT) Received: from localhost ([::1]:45250 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lOo7n-0001oO-0l for importer@patchew.org; Tue, 23 Mar 2021 16:59:39 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60438) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lOo2y-00024I-Id for qemu-devel@nongnu.org; Tue, 23 Mar 2021 16:54:40 -0400 Received: from mga11.intel.com ([192.55.52.93]:35899) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lOo2w-00008L-R3 for qemu-devel@nongnu.org; Tue, 23 Mar 2021 16:54:40 -0400 Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2021 13:54:28 -0700 Received: from ls.sc.intel.com (HELO localhost) ([143.183.96.54]) by fmsmga005-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2021 13:54:26 -0700 IronPort-SDR: LHyjM2WyfY7F5YZDCJgVxYdouFpnqgG0dnLyQUpApK7OlpKi3wjVZbBZo359RosXR51S0lNYxg NQ7dSuXd+lRQ== X-IronPort-AV: E=McAfee;i="6000,8403,9932"; a="187248557" X-IronPort-AV: E=Sophos;i="5.81,272,1610438400"; d="scan'208";a="187248557" IronPort-SDR: Kwld6cM6bqvSOx1gEjgGsaqpBU+YVakRrGJWOhK2NUT0nqYi++93uhyb54P7Ka+wFx34Nl75AY cNKbU6oLxm0A== X-IronPort-AV: E=Sophos;i="5.81,272,1610438400"; d="scan'208";a="607869576" From: Isaku Yamahata To: qemu-devel@nongnu.org, mst@redhat.com, peter.maydell@linaro.org, imammedo@redhat.com, f4bug@amsat.org Subject: [PATCH v3 3/4] isa/v582c686: Reinitialize ACPI PM device on reset Date: Tue, 23 Mar 2021 13:52:26 -0700 Message-Id: <0a3fe998525552860919a690ce83dab8f663ab99.1616532563.git.isaku.yamahata@intel.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable 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=192.55.52.93; envelope-from=isaku.yamahata@intel.com; helo=mga11.intel.com X-Spam_score_int: -68 X-Spam_score: -6.9 X-Spam_bar: ------ X-Spam_report: (-6.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5, 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: isaku.yamahata@intel.com, Huacai Chen , isaku.yamahata@gmail.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Commit 6be8cf56bc8b made sure that SCI is enabled in PM1.CNT on reset in acpi_only mode by modifying acpi_pm1_cnt_reset() and that worked for q35 as expected. This patch adds reset ACPI PM related registers on vt82c686 reset time and de-assert sci. via_pm_realize() initializes acpi pm tmr, evt, cnt and gpe. Reset them on device reset. Cc: BALATON Zoltan Cc: Huacai Chen Cc: "Philippe Mathieu-Daud=C3=A9" Cc: Jiaxun Yang Signed-off-by: Isaku Yamahata --- hw/isa/vt82c686.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/hw/isa/vt82c686.c b/hw/isa/vt82c686.c index f0fb309f12..98325bb32b 100644 --- a/hw/isa/vt82c686.c +++ b/hw/isa/vt82c686.c @@ -178,6 +178,11 @@ static void via_pm_reset(DeviceState *d) /* SMBus IO base */ pci_set_long(s->dev.config + 0x90, 1); =20 + acpi_pm1_evt_reset(&s->ar); + acpi_pm1_cnt_reset(&s->ar); + acpi_pm_tmr_reset(&s->ar); + pm_update_sci(s); + pm_io_space_update(s); smb_io_space_update(s); } --=20 2.25.1 From nobody Sat Apr 27 15:32:08 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; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1616532975; cv=none; d=zohomail.com; s=zohoarc; b=WVCG/LkHNUbM8jkzuZ5038Mlc9cjQCq8Wua94DL472vLGGcd4NfSsIfzqgzYYaeWbONEWyAY66MZncUGTakOujfij2Y+RGjAhuUJy2/Ue6hEkz5YU8AlSlioxhRY7gx4Gr0q6hCdtNiC3qj/l6HR1MJei0RfBl5hIgNb8Yd4Wzo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616532975; 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=P7Iq8Y7P4DG213XGyvB7X47zFQBl2JY21qo6H56CmgY=; b=lKq+Lz9Dfq0dWKv30978uPpuAP5KrhyRaaPIMd3aH/C/xBt8rCtO5kP+8v7+dlJI62xnLlVOBKwsRawVo5gipNkLlFEmKeAmFfnuJbro/fwr80i1eY3+GE0eAPm+ILSp7ngXsmIGRjBRve6IoZ1asm/zG1xlMzOIL00fTE/G21k= 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 1616532975750178.35512835821498; Tue, 23 Mar 2021 13:56:15 -0700 (PDT) Received: from localhost ([::1]:35108 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lOo4U-0003rk-2p for importer@patchew.org; Tue, 23 Mar 2021 16:56:14 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60372) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lOo2u-0001r5-4X for qemu-devel@nongnu.org; Tue, 23 Mar 2021 16:54:36 -0400 Received: from mga11.intel.com ([192.55.52.93]:35899) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lOo2r-00008L-4g for qemu-devel@nongnu.org; Tue, 23 Mar 2021 16:54:35 -0400 Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2021 13:54:28 -0700 Received: from ls.sc.intel.com (HELO localhost) ([143.183.96.54]) by fmsmga005-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Mar 2021 13:54:26 -0700 IronPort-SDR: 6FSYfM6oUMyIlW28JrBKot7kkHZL1jymMLyhB60fZ9TVA2GS3CxqawsseRHSy/cLnXgHHbC/+5 LLI435H4kERA== X-IronPort-AV: E=McAfee;i="6000,8403,9932"; a="187248558" X-IronPort-AV: E=Sophos;i="5.81,272,1610438400"; d="scan'208";a="187248558" IronPort-SDR: 7vwSwlhAhBl3Yim/7PoyJQ7JA2wfIZtmIQUJgifzBjNdSzQNuU9Tzn0cAQN47XRondGXGus9iM lT1uiI7HNO5g== X-IronPort-AV: E=Sophos;i="5.81,272,1610438400"; d="scan'208";a="607869579" From: Isaku Yamahata To: qemu-devel@nongnu.org, mst@redhat.com, peter.maydell@linaro.org, imammedo@redhat.com, f4bug@amsat.org Subject: [PATCH v3 4/4] pci: sprinkle assert in PCI pin number Date: Tue, 23 Mar 2021 13:52:27 -0700 Message-Id: <9cf8ac3b17e162daac0971d7be32deb6a33ae6ec.1616532563.git.isaku.yamahata@intel.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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=192.55.52.93; envelope-from=isaku.yamahata@intel.com; helo=mga11.intel.com X-Spam_score_int: -68 X-Spam_score: -6.9 X-Spam_bar: ------ X-Spam_report: (-6.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5, 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: isaku.yamahata@intel.com, isaku.yamahata@gmail.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" If a device model (a) doesn't set the value to a correct interrupt number and then (b) triggers an interrupt for itself, it's device model bug. Add assert on interrupt pin number to catch this kind of bug more obviously. Suggested-by: Peter Maydell Signed-off-by: Isaku Yamahata --- hw/pci/pci.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/hw/pci/pci.c b/hw/pci/pci.c index ac9a24889c..8f35e13a0c 100644 --- a/hw/pci/pci.c +++ b/hw/pci/pci.c @@ -1450,6 +1450,8 @@ static void pci_irq_handler(void *opaque, int irq_num= , int level) PCIDevice *pci_dev =3D opaque; int change; =20 + assert(0 <=3D irq_num && irq_num < PCI_NUM_PINS); + assert(level =3D=3D 0 || level =3D=3D 1); change =3D level - pci_irq_state(pci_dev, irq_num); if (!change) return; @@ -1469,6 +1471,7 @@ static inline int pci_intx(PCIDevice *pci_dev) qemu_irq pci_allocate_irq(PCIDevice *pci_dev) { int intx =3D pci_intx(pci_dev); + assert(0 <=3D intx && intx < PCI_NUM_PINS); =20 return qemu_allocate_irq(pci_irq_handler, pci_dev, intx); } --=20 2.25.1