[edk2-devel] [Patch V3] UefiCpuPkg/CpuDxe: Eliminate the unused variable.

[Yuanhao Xie]

This change does not introduce any functional modifications.

Remove the unused mValidMtrrAddressMask and mValidMtrrBitsMask.

Reviewed-by: Ray Ni <ray.ni at intel.com>
Signed-off-by: Yuanhao Xie <yuanhao.xie at intel.com>
Cc: Eric Dong <eric.dong at intel.com>
Cc: Rahul Kumar <rahul1.kumar at intel.com>
Cc: Gerd Hoffmann <kraxel at redhat.com>
Cc: Ray Ni <ray.ni at intel.com>
---
 UefiCpuPkg/CpuDxe/CpuDxe.c | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/UefiCpuPkg/CpuDxe/CpuDxe.c b/UefiCpuPkg/CpuDxe/CpuDxe.c
index c5ac624f9e..bf03978710 100644
--- a/UefiCpuPkg/CpuDxe/CpuDxe.c
+++ b/UefiCpuPkg/CpuDxe/CpuDxe.c
@@ -15,15 +15,13 @@
 //
 // Global Variables
 //
-BOOLEAN                InterruptState = FALSE;
-EFI_HANDLE             mCpuHandle     = NULL;
-BOOLEAN                mIsFlushingGCD;
-BOOLEAN                mIsAllocatingPageTable = FALSE;
-UINT64                 mValidMtrrAddressMask;
-UINT64                 mValidMtrrBitsMask;
-UINT64                 mTimerPeriod     = 0;
-UINT32                 mCpuTargetCState = 0;
-EFI_CPU_ARCH_PROTOCOL  gCpu             = {
+BOOLEAN     InterruptState = FALSE;
+EFI_HANDLE  mCpuHandle     = NULL;
+BOOLEAN     mIsFlushingGCD;
+BOOLEAN     mIsAllocatingPageTable = FALSE;
+UINT64      mTimerPeriod           = 0;
+
+EFI_CPU_ARCH_PROTOCOL  gCpu = {
   CpuFlushCpuDataCache,
   CpuEnableInterrupt,
   CpuDisableInterrupt,
-- 
2.36.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#109536): https://edk2.groups.io/g/devel/message/109536
Mute This Topic: https://groups.io/mt/101911140/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-