From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 5CAFBAC1857 for ; Wed, 13 Mar 2024 17:33:12 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=hiyCMDc6PJ9bxImISEWC4rlFcV6rKX6DFdAWcaAjaVM=; c=relaxed/simple; d=groups.io; h=DKIM-Filter:Message-ID:Date:MIME-Version:User-Agent:Subject:From:To:Cc:Reply-To:References:In-Reply-To:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type:Content-Transfer-Encoding; s=20240206; t=1710351191; v=1; b=r5iyXg2Kc++1tp3nFKuJBPyK0xMKhyG+/nPUZLfM/2FsKCsFZsag0fO53TTrnTbHN+gVDKTq 3B+rkrxe7bUQzPE3C5KobhAVCg8mRhgHZej8APE6cCtBJHxUHkvMl7ZHc6MyX7rSH4MFnSOBs8Q AVvLAdK6cwDqxgkHW2i5GKlBLdX65tkI4TvYbE3ahcSK7afsbh8EzFvk4fB2njP7eU0KH9vAa/4 EQBLdIKGF9SpDab5A7MP/FUCrYX2/qfQEsHTHf4KwDAnyekJEscG3MmPn2ddbxRh+nPLP/ZJtaP VImYkQYVgRzEwA3v8VzalG0WQo35KiRcNtpfdKRMPuiNg== X-Received: by 127.0.0.2 with SMTP id QglcYY7687511xCcwq7oBz4D; Wed, 13 Mar 2024 10:33:11 -0700 X-Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mx.groups.io with SMTP id smtpd.web11.2020.1710351190171083398 for ; Wed, 13 Mar 2024 10:33:10 -0700 X-Received: from [10.137.194.171] (unknown [131.107.159.43]) by linux.microsoft.com (Postfix) with ESMTPSA id 979CA20B74C0; Wed, 13 Mar 2024 10:33:09 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 979CA20B74C0 Message-ID: <5155b48a-020f-43a7-a85c-bde5620e7ffb@linux.microsoft.com> Date: Wed, 13 Mar 2024 10:33:09 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [edk2-devel] [PATCH v2 0/3] MdeModulePkg: ImagePropertiesRecordLib Fixes From: "Oliver Smith-Denny" To: devel@edk2.groups.io, Liming Gao Cc: Leif Lindholm , Ard Biesheuvel , Sami Mujawar , Taylor Beebe Reply-To: devel@edk2.groups.io,osde@linux.microsoft.com References: <17BBD31426742776.6798@groups.io> In-Reply-To: <17BBD31426742776.6798@groups.io> Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Wed, 13 Mar 2024 10:33:10 -0700 List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: I5il8tQ2M1T2IHqaMOS1Les1x7686176AA= Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=r5iyXg2K; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=linux.microsoft.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io Hi Liming, Friendly ping, can you please review this patchset? Thanks, Oliver On 3/11/2024 2:29 PM, Oliver Smith-Denny wrote: > ImagePropertiesRecordLib is currently creating Image Records that > are not accurate. It is setting the CodeSegmentSize to be the size > of the raw data in the image file, however, when the image is > loaded into memory, the raw data size is aligned to the > section alignment. This caused the memory attributes table to > have incorrect entries for systems, like ARM64, where the section > alignment is not 4k for all modules. >=20 > In fixing this, I noticed that MemoryProtection.c is using its own > version of image record creation where this logic was actually > correct. ImagePropertiesRecordLib was created to consolidate the > logic around creating and managing image records, so this patchset > also updates MemoryProtection.c to use ImagePropertiesRecordsLib > after making a few small adjustments to ensure the same functionality > is present. >=20 > This patchset was tested on ArmVirtQemu to ensure that all image > records were the same before and after this, other than fixing > the CodeSegmentSize. >=20 > v2: > - Align VirtualSize instead of SizeOfRawData >=20 > Github PR: https://github.com/tianocore/edk2/pull/5402 >=20 > Cc: Liming Gao > Cc: Leif Lindholm > Cc: Ard Biesheuvel > Cc: Sami Mujawar > Cc: Taylor Beebe >=20 > Oliver Smith-Denny (3): > MdeModulePkg: ImagePropertiesRecordLib: Use SectionAlignment for > CodeSize > MdeModulePkg: ImagePropertiesRecordLib: Consolidate Usage > MdeModulePkg: MemoryProtection: Use ImageRecordPropertiesLib >=20 > MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c = | 241 +++----------------- > MdeModulePkg/Library/ImagePropertiesRecordLib/ImagePropertiesRecordLib.= c | 86 +++++-- > 2 files changed, 94 insertions(+), 233 deletions(-) >=20 -=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 (#116718): https://edk2.groups.io/g/devel/message/116718 Mute This Topic: https://groups.io/mt/104873191/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-