From nobody Tue Feb 10 02:56:18 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass header.i=@eurecom.fr; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass(p=none dis=none) header.from=eurecom.fr ARC-Seal: i=1; a=rsa-sha256; t=1734884149; cv=none; d=zohomail.com; s=zohoarc; b=TB298w9Hg+fAO3IGxTAODUtVMZP7evJvrwVjQNIygEu51uOFaYIWbyFXcEQjwP6raitQ+Y6HEDppkyYAjzjzU2kG8PvcuaRr1+nhRI5eUCEwtm2Q1Esu6ahDyuHWbgIcJRaH/Ovc8NFWsllWqdZT1kZ+N0FopWW2ePuCMw8L+ac= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1734884149; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=QVDqXFRgD0m3ZJmnZ4zhQV/zN4jvrDC9qLv5sVf9fJ4=; b=K6lNsnGBYDwjuDnPx7Ic5LjoxJk2RzRPJN5EzbHmfE9//+c3MPWwuIe3Y8B4CJQ3XCrP1/z1UC9T/T/DGEiTKMsxDsulTNgLM92gp0HJ96WBo2k4meX1O7Y/Fg94qcjeoa3WN/zUoYk/NagCB6lGPlEyztgsyYxm+GLGAT40Wz8= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=@eurecom.fr; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 173488414975138.81686038094688; Sun, 22 Dec 2024 08:15:49 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.862711.1274293 (Exim 4.92) (envelope-from ) id 1tPObi-0000Be-T5; Sun, 22 Dec 2024 16:15:06 +0000 Received: by outflank-mailman (output) from mailman id 862711.1274293; Sun, 22 Dec 2024 16:15:06 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1tPObi-0000BX-QB; Sun, 22 Dec 2024 16:15:06 +0000 Received: by outflank-mailman (input) for mailman id 862711; Sun, 22 Dec 2024 16:15:05 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1tPObh-0008OR-MN for xen-devel@lists.xenproject.org; Sun, 22 Dec 2024 16:15:05 +0000 Received: from smtp.eurecom.fr (smtp.eurecom.fr [193.55.113.210]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id e77c0fe5-c07f-11ef-a0da-8be0dac302b0; Sun, 22 Dec 2024 17:15:04 +0100 (CET) Received: from waha.eurecom.fr (HELO smtps.eurecom.fr) ([10.3.2.236]) by drago1i.eurecom.fr with ESMTP; 22 Dec 2024 17:15:04 +0100 Received: from localhost.localdomain (88-183-119-157.subs.proxad.net [88.183.119.157]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtps.eurecom.fr (Postfix) with ESMTPSA id 877A225A7; Sun, 22 Dec 2024 17:15:04 +0100 (CET) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: e77c0fe5-c07f-11ef-a0da-8be0dac302b0 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=eurecom.fr; i=@eurecom.fr; q=dns/txt; s=default; t=1734884105; x=1766420105; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=eey0XVeri/nsLtW4COooiuqzSdIo6CGj8cotECliMN0=; b=d4qX6i5CkGAm633vmvfll/tXYJLLT23CJwhgMbJRuH+y6udOzrGJhZlu 4leH88OSWBwzg4Y/UuzheG3bm+E5rbKXB7y8D+2sp5lXGsk0E2T2QHTCj hygP1UA3PJ6gppqzUeUE/ft0y3h3WP0N1DG0jLiawz0wlOHaN/LAwix8Y 8=; X-CSE-ConnectionGUID: rRauINs7Q4OPKd5ErFeOGw== X-CSE-MsgGUID: EPAyPJxcQG+vCtQ8L3/hQQ== X-IronPort-AV: E=Sophos;i="6.12,255,1728943200"; d="scan'208";a="28291258" From: Ariel Otilibili To: xen-devel@lists.xenproject.org Cc: Ariel Otilibili , Jan Beulich , Anthony PERARD Subject: [PATCH v3 1/1] libacpi/Makefile: clear ASL warning about PCI0 Date: Sun, 22 Dec 2024 17:10:35 +0100 Message-ID: <20241222161444.1558599-2-Ariel.Otilibili-Anieli@eurecom.fr> X-Mailer: git-send-email 2.47.1 In-Reply-To: <20241222161444.1558599-1-Ariel.Otilibili-Anieli@eurecom.fr> References: <20241222161444.1558599-1-Ariel.Otilibili-Anieli@eurecom.fr> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @eurecom.fr) X-ZM-MESSAGEID: 1734884151924116600 Content-Type: text/plain; charset="utf-8" iasl complains _HID and _ADR cannot be used at the same time: ``` iasl -vs -p tools/firmware/hvmloader/dsdt_anycpu.tmp -tc tools/firmware/hvm= loader/dsdt_anycpu.asl 2>&1 | grep -B1 HID tools/firmware/hvmloader/dsdt_anycpu.asl 40: Device (PCI0) Warning 3073 - Multiple types ^ (Devic= e object requires either a _HID or _ADR, but not both) ``` The usage of both _HID and _ADR has changed [1,2]: From ACPI 2.0 (Jul. 27, 2000; Section 6.1, page 146): "A device object must contain either an _HID object or an _ADR object, but can contain both." [3] To ACPI 6.0 (April 2015; Section 6.1, page 278), "A device object must contain either an _HID object or an _ADR object, but should not contain both." [4] And from ACPI 6.0 to ACPI 6.5 (Aug. 2022): "A device object must contain either an _HID object or an _ADR object, but must not contain both." [5] Using its ID, the warning is now filtered. ``` $ iasl -vw3073 -vs -p ../firmware/hvmloader/dsdt_anycpu.tmp -tc ../firmware= /hvmloader/dsdt_anycpu.asl 2>&1 | grep HID; echo $? 1 ``` iasl has one ID per warning [6]; subsequent commits will address other ASL = warnings. ``` $ awk 'NR>533 && NR<556 {print NR ":" $0}' source/compiler/aslmethod.c 534: case PARSEOP_DEVICE: 535: 536: /* Check usage of _HID and _ADR objects */ 537: 538: HidExists =3D ApFindNameInDeviceTree (METHOD_NAME__HID, Op); 539: AdrExists =3D ApFindNameInDeviceTree (METHOD_NAME__ADR, Op); 540: 541: if (!HidExists && !AdrExists) 542: { 543: AslError (ASL_ERROR, ASL_MSG_MISSING_DEPENDENCY, Op, 544: "Device object requires a _HID or _ADR"); 545: } 546: else if (HidExists && AdrExists) 547: { 548: /* 549: * According to the ACPI spec, "A device object must contain 550: * either an _HID object or an _ADR object, but should not = contain 551: * both". 552: */ 553: AslError (ASL_WARNING, ASL_MSG_MULTIPLE_TYPES, Op, 554: "Device object requires either a _HID or _ADR, but not = both"); 555: } $ awk 'NR>188 && NR<206 || NR=3D=3D432 || /ASL_MSG_MULTIPLE_TYPES/ {print N= R ":" $0}' source/compiler/aslmessages.h 189:/* 190: * Values (message IDs) for all compiler messages. There are currently 191: * three distinct blocks of error messages (so that they can be expanded 192: * individually): 193: * Main ASL compiler 194: * Data Table compiler 195: * Preprocessor 196: * 197: * NOTE1: This list must match the tables of message strings in the file 198: * aslmessages.c exactly. 199: * 200: * NOTE2: With the introduction of the -vw option to disable specific 201: * messages, new messages should only be added to the end of these 202: * lists, so that values for existing messages are not disturbed. 203: */ 204:typedef enum 205:{ 280: ASL_MSG_MULTIPLE_TYPES, 432:} ASL_MESSAGE_IDS; $ git remote -v origin git@github.com:acpica/acpica.git (fetch) origin git@github.com:acpica/acpica.git (push) $ git log --pretty=3D'%h ("%s")' -n1 7dae72155 ("Logfile: Changes for version 20241212") ``` [1] https://uefi.org/acpi/specs [2] https://uefi.org/specifications [3] https://uefi.org/sites/default/files/resources/ACPI_2.pdf [4] https://uefi.org/sites/default/files/resources/ACPI_6.0.pdf [5] https://uefi.org/specs/ACPI/6.5/06_Device_Configuration.html?highlight= =3D_hid#device-identification-objects [6] https://github.com/acpica/acpica Fixes: 5a8b28bfd4 ("tools/libacpi: cleanup Makefile, don't check for iasl b= inary") Signed-off-by: Ariel Otilibili --- Cc: Jan Beulich Cc: Anthony PERARD --- tools/libacpi/Makefile | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tools/libacpi/Makefile b/tools/libacpi/Makefile index b21a64c6b4..4668ecb365 100644 --- a/tools/libacpi/Makefile +++ b/tools/libacpi/Makefile @@ -21,6 +21,8 @@ H_SRC +=3D $(addprefix $(ACPI_BUILD_DIR)/, ssdt_tpm.h ssd= t_tpm2.h ssdt_laptop_slat MKDSDT_CFLAGS-$(CONFIG_ARM_64) =3D -DCONFIG_ARM_64 MKDSDT_CFLAGS-$(CONFIG_X86) =3D -DCONFIG_X86 =20 +IASL_WARNS=3D3073 + # Suffix for temporary files. # # We will also use this suffix to workaround a bug in older iasl @@ -32,7 +34,7 @@ TMP_SUFFIX =3D tmp all: $(C_SRC) $(H_SRC) =20 $(H_SRC): $(ACPI_BUILD_DIR)/%.h: %.asl - $(IASL) -vs -p $(ACPI_BUILD_DIR)/$*.$(TMP_SUFFIX) -tc $< + $(IASL) $(IASL_WARNS:%=3D-vw%) -vs -p $(ACPI_BUILD_DIR)/$*.$(TMP_SUFFIX) = -tc $< sed -e 's/AmlCode/$*/g' -e 's/_aml_code//g' $(ACPI_BUILD_DIR)/$*.hex >$@ rm -f $(addprefix $(ACPI_BUILD_DIR)/, $*.aml $*.hex) =20 @@ -65,7 +67,7 @@ $(ACPI_BUILD_DIR)/dsdt_anycpu_arm.asl: $(MK_DSDT) mv -f $@.$(TMP_SUFFIX) $@ =20 $(C_SRC): $(ACPI_BUILD_DIR)/%.c: $(ACPI_BUILD_DIR)/%.asl - $(IASL) -vs -p $(ACPI_BUILD_DIR)/$*.$(TMP_SUFFIX) -tc $< + $(IASL) $(IASL_WARNS:%=3D-vw%) -vs -p $(ACPI_BUILD_DIR)/$*.$(TMP_SUFFIX) = -tc $< sed -e 's/AmlCode/$*/g' -e 's/_aml_code//g' $(ACPI_BUILD_DIR)/$*.hex > $@= .$(TMP_SUFFIX) echo "int $*_len=3Dsizeof($*);" >> $@.$(TMP_SUFFIX) mv -f $@.$(TMP_SUFFIX) $@ --=20 2.47.1