From nobody Sat Apr 27 14:29:34 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+103734+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+103734+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1682650629; cv=none; d=zohomail.com; s=zohoarc; b=kqGLInEvbaKdWLVBsDaRUAZV16mwHXkqA7+bXNhYXjb2Dy1JbiuRnYfpWmaKoI3tQQx4ZmefLt1JfOvFajyKudi6ICDegnu+3w2rW9IAfElN8N0D5H9NgA+e4EBFPaZ10E/5/jIzSbVAl+6ZO3r/CkakBn6N09qEolMcyNnF9z0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1682650629; h=Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Sender:Subject:To; bh=c/1ER2yPhH4+CE6Mdtc09rIiB4n/flhda0TrcrCUFv4=; b=nYBLi3S6aebaBafzv2Fh0ch3LiUP/cJvmdS47PCUL/LGjpHLXAnzKnwL/XzZsUfJhAMH0PqQhbbAncjFJ0Yo0HxFeHJgvXqXO0JIushXbakPrnfu3QTnys6G3TZQK2rjqhg0bfi60Cw/G3MQBRZQ4gcQ4JfyGIQCkZvIWgDxcuM= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+103734+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1682650629436870.7106377529659; Thu, 27 Apr 2023 19:57:09 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id Ix0nYY1788612x2ufs2hTO8b; Thu, 27 Apr 2023 19:57:08 -0700 X-Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mx.groups.io with SMTP id smtpd.web11.10921.1682650627829945187 for ; Thu, 27 Apr 2023 19:57:07 -0700 X-IronPort-AV: E=McAfee;i="6600,9927,10693"; a="375615231" X-IronPort-AV: E=Sophos;i="5.99,233,1677571200"; d="scan'208";a="375615231" X-Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Apr 2023 19:57:07 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10693"; a="1024408822" X-IronPort-AV: E=Sophos;i="5.99,233,1677571200"; d="scan'208";a="1024408822" X-Received: from kxinfux-desk.gar.corp.intel.com ([10.227.107.32]) by fmsmga005.fm.intel.com with ESMTP; 27 Apr 2023 19:57:06 -0700 From: "VincentX Ke" To: devel@edk2.groups.io Cc: VincentX Ke , Chasel Chiu , Nate DeSimone , Isaac Oram , Liming Gao , Eric Dong Subject: [edk2-devel] [PATCH v3] MinPlatformPkg: Update HWSignature filed in FADT Date: Fri, 28 Apr 2023 10:56:59 +0800 Message-Id: <20230428025659.692-1-vincentx.ke@intel.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,vincentx.ke@intel.com X-Gm-Message-State: cRFlCI3oWtoIp8c5EGAJ7NxCx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1682650628; bh=PxNmijhqPOCoicaf1uItbrdRVfjVoMvc6pn7Du+kqYA=; h=Cc:Date:From:Reply-To:Subject:To; b=sbvpfmybstOvvCAydisO4LscAeGGw0uo/87g+4WCnCzP9AKsj2AJz5+yhT5t0YrW9Nc dPYu7CLaUhg48iXORR207JgOZtByUywskx9adRaJsf9SGEgRwctryhAJVolzBVYX5XfDJ 6BEGRjxq5S/umiCcnIgdQf10KwtODQHHZGo= X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1682650630183100001 Content-Type: text/plain; charset="utf-8" From: VincentX Ke REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4428 Calculating CRC based on checksum from all ACPI tables. Update HWSignature filed in FADT based on CRC while ACPI table changed. Signed-off-by: VincentX Ke Cc: Chasel Chiu Cc: Nate DeSimone Cc: Isaac Oram Cc: Liming Gao Cc: Eric Dong --- Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c | 110 +++++= ++++++++++++++- Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.inf | 1 + 2 files changed, 110 insertions(+), 1 deletion(-) diff --git a/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c b= /Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c index e967031a3b..d84c1d4f6d 100644 --- a/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c +++ b/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c @@ -1285,6 +1285,108 @@ IsHardwareChange ( FreePool (HWChange); } =20 +/** + This function calculates RCR based on Checksum from all ACPI tables. + It also calculates CRC and provides as HWSignature filed in FADT table. +**/ +VOID +IsAcpiTableChange ( + VOID + ) +{ + EFI_STATUS Status; + UINTN Index; + UINTN AcpiTableCount; + UINT32 Table; + UINT32 CRC; + UINT32 *AcpiTable; + EFI_ACPI_6_5_ROOT_SYSTEM_DESCRIPTION_POINTER *Rsdp; + EFI_ACPI_DESCRIPTION_HEADER *Rsdt; + EFI_ACPI_DESCRIPTION_HEADER *Xsdt; + EFI_ACPI_6_5_FIRMWARE_ACPI_CONTROL_STRUCTURE *FacsPtr; + EFI_ACPI_6_5_FIXED_ACPI_DESCRIPTION_TABLE *pFADT; + + AcpiTableCount =3D 0; + AcpiTable =3D NULL; + Rsdp =3D NULL; + Rsdt =3D NULL; + Xsdt =3D NULL; + FacsPtr =3D NULL; + pFADT =3D NULL; + + DEBUG ((DEBUG_INFO, "%a() - Start\n", __FUNCTION__)); + + Status =3D EfiGetSystemConfigurationTable (&gEfiAcpiTableGuid, (VOID **)= &Rsdp); + if (EFI_ERROR (Status) || (Rsdp =3D=3D NULL)) { + return; + } + + // + // ACPI table count starts with 2 as RSDT and XSDT are already located. + // Then add ACPI tables found by XSDT and FADT. + // + Rsdt =3D (EFI_ACPI_DESCRIPTION_HEADER *)(UINTN)Rsdp->RsdtAddre= ss; + Xsdt =3D (EFI_ACPI_DESCRIPTION_HEADER *)(UINTN)Rsdp->XsdtAddre= ss; + AcpiTableCount =3D AcpiTableCount + 2; + AcpiTableCount =3D AcpiTableCount + (Xsdt->Length - sizeof (EFI_ACPI_DES= CRIPTION_HEADER))/sizeof (UINT64); + + for (Index =3D sizeof (EFI_ACPI_DESCRIPTION_HEADER); Index < (Xsdt->Leng= th); Index =3D Index + sizeof (UINT64)) { + Table =3D *((UINT32 *)((UINT8 *)Xsdt + Index)); + if (((EFI_ACPI_DESCRIPTION_HEADER *)(UINTN)Table)->Signature =3D=3D EF= I_ACPI_6_5_FIXED_ACPI_DESCRIPTION_TABLE_SIGNATURE) { + pFADT =3D (EFI_ACPI_6_5_FIXED_ACPI_DESCRIPTION_TABLE *)(UINTN)Table; + if ((pFADT->XDsdt !=3D 0) || (pFADT->Dsdt !=3D 0)) { + AcpiTableCount =3D AcpiTableCount + 1; + } + } + } + + // + // Allocate memory for founded ACPI tables. + // + AcpiTable =3D AllocateZeroPool (sizeof (UINT32) * AcpiTableCount); + if (AcpiTable =3D=3D NULL) { + return; + } + + AcpiTableCount =3D 0; + AcpiTable[AcpiTableCount++] =3D Rsdt->Checksum; + AcpiTable[AcpiTableCount++] =3D Xsdt->Checksum; + + for (Index =3D sizeof (EFI_ACPI_DESCRIPTION_HEADER); Index < (Xsdt->Leng= th); Index =3D Index + sizeof (UINT64)) { + Table =3D *((UINT32 *)((UINT8 *)Xsdt + Index)); + AcpiTable[AcpiTableCount++] =3D ((EFI_ACPI_DESCRIPTION_HEADER *)(UINTN= )Table)->Checksum; + if (((EFI_ACPI_DESCRIPTION_HEADER *)(UINTN)Table)->Signature =3D=3D EF= I_ACPI_6_5_FIXED_ACPI_DESCRIPTION_TABLE_SIGNATURE) { + pFADT =3D (EFI_ACPI_6_5_FIXED_ACPI_DESCRIPTION_TABLE *)(UINTN)Table; + if (pFADT->XDsdt !=3D 0) { + AcpiTable[AcpiTableCount++] =3D ((EFI_ACPI_DESCRIPTION_HEADER *)(U= INTN)pFADT->XDsdt)->Checksum; + } else { + AcpiTable[AcpiTableCount++] =3D ((EFI_ACPI_DESCRIPTION_HEADER *)(U= INTN)pFADT->Dsdt)->Checksum; + } + } + } + + // + // pFADT table not found. + // + if (pFADT =3D=3D NULL) { + return; + } + + // + // Calculate CRC value. + // + Status =3D gBS->CalculateCrc32 (AcpiTable, AcpiTableCount, &CRC); + DEBUG ((DEBUG_INFO, "CRC =3D %x and Status =3D %r\n", CRC, Status)); + + // + // Set HardwareSignature value based on CRC value. + // + FacsPtr =3D (EFI_ACPI_6_5_FIRMWARE_ACPI_CONTROL_STRUC= TURE *)(UINTN)pFADT->FirmwareCtrl; + FacsPtr->HardwareSignature =3D CRC; + FreePool (AcpiTable); + DEBUG ((DEBUG_INFO, "%a() - End\n", __FUNCTION__)); +} + VOID UpdateLocalTable ( VOID @@ -1329,7 +1431,13 @@ AcpiEndOfDxeEvent ( // // Calculate Hardware Signature value based on current platform configur= ations // - IsHardwareChange (); + if ((PcdGet8 (PcdFadtMajorVersion) <=3D EFI_ACPI_6_3_FIXED_ACPI_DESCRIPT= ION_TABLE_REVISION) && + (PcdGet8 (PcdFadtMinorVersion) <=3D EFI_ACPI_6_3_FIXED_ACPI_DESCRIPT= ION_TABLE_MINOR_REVISION)) + { + IsHardwareChange (); + } else { + IsAcpiTableChange (); + } } =20 /** diff --git a/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.inf= b/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.inf index 694492112b..f47cc3908d 100644 --- a/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.inf +++ b/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.inf @@ -128,6 +128,7 @@ gEfiGlobalVariableGuid ## CONSUMES gEfiHobListGuid ## CONSUMES gEfiEndOfDxeEventGroupGuid ## CONSUMES + gEfiAcpiTableGuid ## CONSUMES =20 [Depex] gEfiAcpiTableProtocolGuid AND --=20 2.39.2.windows.1 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103734): https://edk2.groups.io/g/devel/message/103734 Mute This Topic: https://groups.io/mt/98551423/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-