Skip to content

Commit

Permalink
Add the ability to configure the memory type used for the FD region (#…
Browse files Browse the repository at this point in the history
…140)

## Description

Add the ability to configure the memory type used for the FD region on
the non-secure UEFI side.

- [x] Impacts functionality?
- [ ] Impacts security?
- [ ] Breaking change?
- [ ] Includes tests?
- [ ] Includes documentation?

## How This Was Tested

Tested on internal platform build.

## Integration Instructions

N/A
  • Loading branch information
cfernald authored and ProjectMuBot committed Nov 22, 2024
1 parent f1daef5 commit ba1fff5
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 1 deletion.
4 changes: 4 additions & 0 deletions ArmPkg/ArmPkg.dec
Original file line number Diff line number Diff line change
Expand Up @@ -223,6 +223,10 @@
gArmTokenSpaceGuid.PcdFdSize|0|UINT32|0x0000002C
gArmTokenSpaceGuid.PcdFvSize|0|UINT32|0x0000002E


# MU_CHANGE: PCD To configure memory type used for FD region allocation. Defaults to EfiBootServicesData
gArmTokenSpaceGuid.PcdFdMemoryType|4|UINT32|0x1000046 # MU_CHANGE

#
# Value to add to a host address to obtain a device address, using
# unsigned 64-bit integer arithmetic on both ARM and AArch64. This
Expand Down
2 changes: 1 addition & 1 deletion ArmPlatformPkg/MemoryInitPei/MemoryInitPeiLib.c
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ MemoryPeim (
BuildMemoryAllocationHob (
PcdGet64 (PcdFdBaseAddress),
PcdGet32 (PcdFdSize),
EfiBootServicesData
PcdGet32 (PcdFdMemoryType) // MU_CHANGE
);

Found = TRUE;
Expand Down
1 change: 1 addition & 0 deletions ArmPlatformPkg/MemoryInitPei/MemoryInitPeiLib.inf
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
[FixedPcd]
gArmTokenSpaceGuid.PcdFdBaseAddress
gArmTokenSpaceGuid.PcdFdSize
gArmTokenSpaceGuid.PcdFdMemoryType # MU_CHANGE

gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize

Expand Down

0 comments on commit ba1fff5

Please sign in to comment.