From nobody Fri Apr 19 22:33:07 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+45317+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+45317+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1565446234; cv=none; d=zoho.com; s=zohoarc; b=Yhy0yTaQqTFNls7B2422Nu8/S2bQaS8j7iN45hdhpLq4NR3DN7DDQwCl4kwTR5gkrGNC6cLcb4mZGP0TrdRLC+F8U03zQiTgG24OsQsMVuzLLwADO48lUDqOIjj9Z7EctAuKBINjyLiDDQ4AxP4HapKxf5XnvLHqGydO141qcAQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1565446234; h=Cc:Date:From:List-Id:List-Unsubscribe:Message-ID:Reply-To:Sender:Subject:To:ARC-Authentication-Results; bh=L+G1NuzF68Qv89toDZiSZfMJL5M3gU8cQ0PMUAD33po=; b=NItnGFOvqL69E5Gj95no0gQjaHB2pDVDmWz5poF8gNaBBbX+hLHy2cvzMgA23I9jo7DDbPOzUmk2/lcboaNNj6KXJVDxxPsSFvhbZ5GzO1xzReRXSiucnFCSGnhwD5LvKbsaoRTWR9MtrHP+Ng0wmc5fr9386ha5uGk/HpmG0UU= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+45317+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1565446233862430.59694630988906; Sat, 10 Aug 2019 07:10:33 -0700 (PDT) Return-Path: X-Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by groups.io with SMTP; Sat, 10 Aug 2019 07:10:32 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 10 Aug 2019 07:10:32 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,369,1559545200"; d="scan'208";a="199668214" X-Received: from lgao4-mobl1.ccr.corp.intel.com ([10.249.168.23]) by fmsmga004.fm.intel.com with ESMTP; 10 Aug 2019 07:10:31 -0700 From: "Liming Gao" To: devel@edk2.groups.io Cc: Mike Turner , Jian J Wang , Hao A Wu , Dandan Bi Subject: [edk2-devel] [Patch] MdeModulePkg DxeCore: Fix for missing MAT update Date: Sat, 10 Aug 2019 22:10:22 +0800 Message-Id: <20190810141022.18228-1-liming.gao@intel.com> Precedence: Bulk List-Unsubscribe: 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,liming.gao@intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1565446233; bh=Kf3oxnf55THuTIsYx8FuiLNU/37gRazmJtN/0fKa0bw=; h=Cc:Date:From:Reply-To:Subject:To; b=YsGuR8K5KzghL5iFI/tRo639rUSP6pZfO7/ttX8QQ6N/AkLiIaJ5N7KQo6GTXDsIu8W U4wm+FsIJoXWkixX2O8MwitS66rds1B/8SYC6TPhGqyDYZaT63fWPC0foHXByb4HDy3/B cgRpAj2rEwAAUBdyUdC8bPQ7xz7KyQCuJWY= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Mike Turner The Fpdt driver (FirmwarePerformanceDxe) saves a memory address across reboots, and then does an AllocatePage for that memory address. If, on this boot, that memory comes from a Runtime memory bucket, the MAT table is not updated. This causes Windows to boot into Recovery. This patch blocks the memory manager from changing the page from a special bucket to a different memory type. Once the buckets are allocated, we freeze the memory ranges for the OS, and fragmenting the special buckets will cause errors resuming from hibernate. This patch is cherry pick from Project Mu: https://github.com/microsoft/mu_basecore/commit/a9be767d9be96af94016ebd391e= a6f340920735a With the minor change, 1. Update commit message format to keep the message in 80 characters one li= ne. 2. Remove // MU_CHANGE comments in source code. Cc: Jian J Wang Cc: Hao A Wu Cc: Dandan Bi Signed-off-by: Liming Gao Reviewed-by: Jian J Wang --- MdeModulePkg/Core/Dxe/Mem/Page.c | 43 ++++++++++++++++++++++++++++++++++--= ---- 1 file changed, 37 insertions(+), 6 deletions(-) diff --git a/MdeModulePkg/Core/Dxe/Mem/Page.c b/MdeModulePkg/Core/Dxe/Mem/P= age.c index bd9e116aa5..637518889d 100644 --- a/MdeModulePkg/Core/Dxe/Mem/Page.c +++ b/MdeModulePkg/Core/Dxe/Mem/Page.c @@ -1265,12 +1265,13 @@ CoreInternalAllocatePages ( IN BOOLEAN NeedGuard ) { - EFI_STATUS Status; - UINT64 Start; - UINT64 NumberOfBytes; - UINT64 End; - UINT64 MaxAddress; - UINTN Alignment; + EFI_STATUS Status; + UINT64 Start; + UINT64 NumberOfBytes; + UINT64 End; + UINT64 MaxAddress; + UINTN Alignment; + EFI_MEMORY_TYPE CheckType; =20 if ((UINT32)Type >=3D MaxAllocateType) { return EFI_INVALID_PARAMETER; @@ -1321,6 +1322,7 @@ CoreInternalAllocatePages ( // if (Start + NumberOfBytes) rolls over 0 or // if Start is above MAX_ALLOC_ADDRESS or // if End is above MAX_ALLOC_ADDRESS, + // if Start..End overlaps any tracked MemoryTypeStatistics range // return EFI_NOT_FOUND. // if (Type =3D=3D AllocateAddress) { @@ -1336,6 +1338,35 @@ CoreInternalAllocatePages ( (End > MaxAddress)) { return EFI_NOT_FOUND; } + + // Problem summary + + /* + A driver is allowed to call AllocatePages using an AllocateAddress typ= e. This type of + AllocatePage request the exact physical address if it is not used. Th= e existing code + will allow this request even in 'special' pages. The problem with thi= s is that the + reason to have 'special' pages for OS hibernate/resume is defeated as = memory is + fragmented. + */ + + for (CheckType =3D (EFI_MEMORY_TYPE) 0; CheckType < EfiMaxMemoryType; = CheckType++) { + if (MemoryType !=3D CheckType && + mMemoryTypeStatistics[CheckType].Special && + mMemoryTypeStatistics[CheckType].NumberOfPages > 0) { + if (Start >=3D mMemoryTypeStatistics[CheckType].BaseAddress && + Start <=3D mMemoryTypeStatistics[CheckType].MaximumAddress) { + return EFI_NOT_FOUND; + } + if (End >=3D mMemoryTypeStatistics[CheckType].BaseAddress && + End <=3D mMemoryTypeStatistics[CheckType].MaximumAddress) { + return EFI_NOT_FOUND; + } + if (Start < mMemoryTypeStatistics[CheckType].BaseAddress && + End > mMemoryTypeStatistics[CheckType].MaximumAddress) { + return EFI_NOT_FOUND; + } + } + } } =20 if (Type =3D=3D AllocateMaxAddress) { --=20 2.13.0.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 (#45317): https://edk2.groups.io/g/devel/message/45317 Mute This Topic: https://groups.io/mt/32821535/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-