Skip to content

Commit 1ad3ff0

Browse files
authored
ArmPkg: Remove pragma pack from CpuDxe (#292)
## Description PageTableMemoryAllocation.c The packing was causing an MSVC compiler warning: `Compiler #4366: The result of the unary '&' operator may be unaligned.` Removing the packing also matches convention seen in https://github.com/microsoft/mu_basecore/blob/dev/202405/UefiCpuPkg/CpuDxe/CpuPageTable.c#L71 - [ ] Impacts functionality? - [ ] Impacts security? - [ ] Breaking change? - [ ] Includes tests? - [ ] Includes documentation? ## How This Was Tested Booted with QEMU SBSA ## Integration Instructions N/A Signed-off-by: Vivian Nowka-Keane <[email protected]>
1 parent 03dc848 commit 1ad3ff0

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

ArmPkg/Drivers/CpuDxe/PageTableMemoryAllocation.c

-4
Original file line numberDiff line numberDiff line change
@@ -13,17 +13,13 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
1313
#include <Library/UefiBootServicesTableLib.h>
1414
#include <Protocol/ArmPageTableMemoryAllocation.h>
1515

16-
#pragma pack(1)
17-
1816
typedef struct {
1917
UINT32 Signature;
2018
UINTN Offset;
2119
UINTN FreePages;
2220
LIST_ENTRY NextPool;
2321
} PAGE_TABLE_POOL;
2422

25-
#pragma pack()
26-
2723
#define MIN_PAGES_AVAILABLE 5
2824
#define PAGE_TABLE_POOL_SIGNATURE SIGNATURE_32 ('P','T','P','L')
2925

0 commit comments

Comments
 (0)