[v2] UefiCpuPkg/CpuDxe: Eliminate the unused variable.

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

Posted by Yuanhao Xie 6 months, 3 weeks ago

This change does not introduce any functional modifications.

Remove the unused mValidMtrrAddressMask and mValidMtrrBitsMask.

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

diff --git a/UefiCpuPkg/CpuDxe/CpuDxe.c b/UefiCpuPkg/CpuDxe/CpuDxe.c
index c5ac624f9e..bf8d0eb5e5 100644
--- a/UefiCpuPkg/CpuDxe/CpuDxe.c
+++ b/UefiCpuPkg/CpuDxe/CpuDxe.c
@@ -15,15 +15,14 @@
 //
 // 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;
+UINT32      mCpuTargetCState       = 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 (#109521): https://edk2.groups.io/g/devel/message/109521
Mute This Topic: https://groups.io/mt/101893039/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

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

Posted by Ni, Ray 6 months, 3 weeks ago

Reviewed-by: Ray Ni <ray.ni@intel.com>

Thanks,
Ray
________________________________
From: Xie, Yuanhao <yuanhao.xie@intel.com>
Sent: Wednesday, October 11, 2023 5:08 PM
To: devel@edk2.groups.io <devel@edk2.groups.io>
Cc: Xie, Yuanhao <yuanhao.xie@intel.com>; Dong, Eric <eric.dong@intel.com>; Kumar, Rahul R <rahul.r.kumar@intel.com>; Gerd Hoffmann <kraxel@redhat.com>; Ni, Ray <ray.ni@intel.com>
Subject: [Patch V2] UefiCpuPkg/CpuDxe: Eliminate the unused variable.

This change does not introduce any functional modifications.

Remove the unused mValidMtrrAddressMask and mValidMtrrBitsMask.

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

diff --git a/UefiCpuPkg/CpuDxe/CpuDxe.c b/UefiCpuPkg/CpuDxe/CpuDxe.c
index c5ac624f9e..bf8d0eb5e5 100644
--- a/UefiCpuPkg/CpuDxe/CpuDxe.c
+++ b/UefiCpuPkg/CpuDxe/CpuDxe.c
@@ -15,15 +15,14 @@
 //
 // 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;
+UINT32      mCpuTargetCState       = 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 (#109522): https://edk2.groups.io/g/devel/message/109522
Mute This Topic: https://groups.io/mt/101893039/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3901457/1787277/102458076/xyzzy [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-