Fix build break when doing 32-bit build with some certain C compiler option combinations.
authorrsun3 <rsun3@6f19259b-4bc3-4df7-8a09-765794883524>
Thu, 28 Oct 2010 02:01:43 +0000 (02:01 +0000)
committerrsun3 <rsun3@6f19259b-4bc3-4df7-8a09-765794883524>
Thu, 28 Oct 2010 02:01:43 +0000 (02:01 +0000)
Use the library functions for shift operations in BaseLib for a 64-bit integer where the code is shared for 32-bit and 64-bit.
Defining bitfields in structures with > 32 bits will cause these types of issues on IA32 builds. So the largest bitfield should be type UINT32 with a max size of :32.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@10983 6f19259b-4bc3-4df7-8a09-765794883524

UefiCpuPkg/Include/Register/LocalApic.h
UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c

index e22900d..9f8f1bb 100644 (file)
 \r
 typedef union {\r
   struct {\r
-    UINT64  Reserved0:8;   ///< Reserved.\r
-    UINT64  Bsp:1;         ///< Processor is BSP.\r
-    UINT64  Reserved1:1;   ///< Reserved.\r
-    UINT64  Extd:1;        ///< Enable x2APIC mode.\r
-    UINT64  En:1;          ///< xAPIC global enable/disable.\r
-    UINT64  ApicBase:52;   ///< APIC Base physical address. The actual field width depends on physical address width.\r
+    UINT32  Reserved0:8;     ///< Reserved.\r
+    UINT32  Bsp:1;           ///< Processor is BSP.\r
+    UINT32  Reserved1:1;     ///< Reserved.\r
+    UINT32  Extd:1;          ///< Enable x2APIC mode.\r
+    UINT32  En:1;            ///< xAPIC global enable/disable.\r
+    UINT32  ApicBaseLow:20;  ///< APIC Base physical address. The actual field width depends on physical address width.\r
+    UINT32  ApicBaseHigh:32;\r
   } Bits;\r
   UINT64    Uint64;\r
 } MSR_IA32_APIC_BASE;\r
index 1cba34c..9f08f5d 100644 (file)
@@ -151,7 +151,7 @@ SendIpi (
     // For x2APIC, A single MSR write to the Interrupt Command Register is required for dispatching an \r
     // interrupt in x2APIC mode.\r
     //\r
-    MsrValue = (((UINT64)ApicId) << 32) | IcrLow;\r
+    MsrValue = LShiftU64 ((UINT64) ApicId, 32) | IcrLow;\r
     AsmWriteMsr64 (X2APIC_MSR_ICR_ADDRESS, MsrValue);\r
   }\r
 }\r