]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/EbcDxe/EbcExecute.c
MdeModulePkg: Clean up source files
[mirror_edk2.git] / MdeModulePkg / Universal / EbcDxe / EbcExecute.c
index b819c5f024360fa967a580ccdbd525f39fb85c0a..3418936c1012ae5dc7fb832a148faee836f87fc3 100644 (file)
@@ -1,8 +1,8 @@
 /** @file\r
   Contains code that implements the virtual machine.\r
 \r
-Copyright (c) 2006 - 2008, Intel Corporation. <BR>\r
-All rights reserved. This program and the accompanying materials\r
+Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.<BR>\r
+This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
 http://opensource.org/licenses/bsd-license.php\r
@@ -14,6 +14,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 \r
 #include "EbcInt.h"\r
 #include "EbcExecute.h"\r
+#include "EbcDebuggerHook.h"\r
 \r
 \r
 //\r
@@ -49,13 +50,13 @@ UINT64
     b14:12  - number of bits in this index assigned to natural units (=a)\r
     ba:11   - constant units = ConstUnits\r
     b0:a    - natural units = NaturalUnits\r
-  \r
+\r
   Given this info, the offset can be computed by:\r
     offset = sign_bit * (ConstUnits + NaturalUnits * sizeof(UINTN))\r
 \r
   Max offset is achieved with index = 0x7FFF giving an offset of\r
   0x27B (32-bit machine) or 0x477 (64-bit machine).\r
-  Min offset is achieved with index = \r
+  Min offset is achieved with index =\r
 \r
   @param  VmPtr             A pointer to VM context.\r
   @param  CodeOffset        Offset from IP of the location of the 16-bit index\r
@@ -108,7 +109,7 @@ VmReadIndex64 (
   @param  VmPtr             A pointer to VM context.\r
   @param  Addr              The memory address.\r
 \r
-  @return The 8-bit value from the memory adress.\r
+  @return The 8-bit value from the memory address.\r
 \r
 **/\r
 UINT8\r
@@ -123,7 +124,7 @@ VmReadMem8 (
   @param  VmPtr             A pointer to VM context.\r
   @param  Addr              The memory address.\r
 \r
-  @return The 16-bit value from the memory adress.\r
+  @return The 16-bit value from the memory address.\r
 \r
 **/\r
 UINT16\r
@@ -138,7 +139,7 @@ VmReadMem16 (
   @param  VmPtr             A pointer to VM context.\r
   @param  Addr              The memory address.\r
 \r
-  @return The 32-bit value from the memory adress.\r
+  @return The 32-bit value from the memory address.\r
 \r
 **/\r
 UINT32\r
@@ -153,7 +154,7 @@ VmReadMem32 (
   @param  VmPtr             A pointer to VM context.\r
   @param  Addr              The memory address.\r
 \r
-  @return The 64-bit value from the memory adress.\r
+  @return The 64-bit value from the memory address.\r
 \r
 **/\r
 UINT64\r
@@ -179,7 +180,7 @@ VmReadMemN (
 \r
 /**\r
   Writes 8-bit data to memory address.\r
-   \r
+\r
   This routine is called by the EBC data\r
   movement instructions that write to memory. Since these writes\r
   may be to the stack, which looks like (high address on top) this,\r
@@ -189,14 +190,14 @@ VmReadMemN (
   [EBC stack]\r
 \r
   we need to detect all attempts to write to the EBC entry point argument\r
-  stack area and adjust the address (which will initially point into the \r
+  stack area and adjust the address (which will initially point into the\r
   VM stack) to point into the EBC entry point arguments.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
-  @param  Addr              Adddress to write to.\r
+  @param  Addr              Address to write to.\r
   @param  Data              Value to write to Addr.\r
 \r
-  @retval EFI_SUCCESS       The instruction is executed successfully.  \r
+  @retval EFI_SUCCESS       The instruction is executed successfully.\r
   @retval Other             Some error occurs when writing data to the address.\r
 \r
 **/\r
@@ -209,7 +210,7 @@ VmWriteMem8 (
 \r
 /**\r
   Writes 16-bit data to memory address.\r
-   \r
+\r
   This routine is called by the EBC data\r
   movement instructions that write to memory. Since these writes\r
   may be to the stack, which looks like (high address on top) this,\r
@@ -219,14 +220,14 @@ VmWriteMem8 (
   [EBC stack]\r
 \r
   we need to detect all attempts to write to the EBC entry point argument\r
-  stack area and adjust the address (which will initially point into the \r
+  stack area and adjust the address (which will initially point into the\r
   VM stack) to point into the EBC entry point arguments.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
-  @param  Addr              Adddress to write to.\r
+  @param  Addr              Address to write to.\r
   @param  Data              Value to write to Addr.\r
 \r
-  @retval EFI_SUCCESS       The instruction is executed successfully.  \r
+  @retval EFI_SUCCESS       The instruction is executed successfully.\r
   @retval Other             Some error occurs when writing data to the address.\r
 \r
 **/\r
@@ -239,7 +240,7 @@ VmWriteMem16 (
 \r
 /**\r
   Writes 32-bit data to memory address.\r
-   \r
+\r
   This routine is called by the EBC data\r
   movement instructions that write to memory. Since these writes\r
   may be to the stack, which looks like (high address on top) this,\r
@@ -249,14 +250,14 @@ VmWriteMem16 (
   [EBC stack]\r
 \r
   we need to detect all attempts to write to the EBC entry point argument\r
-  stack area and adjust the address (which will initially point into the \r
+  stack area and adjust the address (which will initially point into the\r
   VM stack) to point into the EBC entry point arguments.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
-  @param  Addr              Adddress to write to.\r
+  @param  Addr              Address to write to.\r
   @param  Data              Value to write to Addr.\r
 \r
-  @retval EFI_SUCCESS       The instruction is executed successfully.  \r
+  @retval EFI_SUCCESS       The instruction is executed successfully.\r
   @retval Other             Some error occurs when writing data to the address.\r
 \r
 **/\r
@@ -268,7 +269,7 @@ VmWriteMem32 (
   );\r
 \r
 /**\r
-  Reads 16-bit unsinged data from the code stream.\r
+  Reads 16-bit unsigned data from the code stream.\r
 \r
   This routine provides the ability to read raw unsigned data from the code\r
   stream.\r
@@ -286,7 +287,7 @@ VmReadCode16 (
   );\r
 \r
 /**\r
-  Reads 32-bit unsinged data from the code stream.\r
+  Reads 32-bit unsigned data from the code stream.\r
 \r
   This routine provides the ability to read raw unsigned data from the code\r
   stream.\r
@@ -304,7 +305,7 @@ VmReadCode32 (
   );\r
 \r
 /**\r
-  Reads 64-bit unsinged data from the code stream.\r
+  Reads 64-bit unsigned data from the code stream.\r
 \r
   This routine provides the ability to read raw unsigned data from the code\r
   stream.\r
@@ -440,7 +441,7 @@ ConvertStackAddr (
   @param  VmPtr             A pointer to VM context.\r
   @param  IsSignedOp        Indicates whether the operand is signed or not.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -472,7 +473,7 @@ ExecuteBREAK (
   Instruction syntax:\r
     JMP64{cs|cc} Immed64\r
     JMP32{cs|cc} {@}R1 {Immed32|Index32}\r
-  \r
+\r
   Encoding:\r
     b0.7 -  immediate data present\r
     b0.6 -  1 = 64 bit immediate data\r
@@ -557,7 +558,7 @@ ExecuteRET (
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -574,7 +575,7 @@ ExecuteCMP (
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -587,15 +588,15 @@ ExecuteCMPI (
   Execute the MOVxx instructions.\r
 \r
   Instruction format:\r
-    \r
+\r
     MOV[b|w|d|q|n]{w|d} {@}R1 {Index16|32}, {@}R2 {Index16|32}\r
     MOVqq {@}R1 {Index64}, {@}R2 {Index64}\r
-  \r
+\r
     Copies contents of [R2] -> [R1], zero extending where required.\r
-  \r
+\r
     First character indicates the size of the move.\r
     Second character indicates the size of the index(s).\r
-  \r
+\r
     Invalid to have R1 direct with index.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
@@ -613,20 +614,20 @@ ExecuteMOVxx (
   Execute the EBC MOVI.\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVI[b|w|d|q][w|d|q] {@}R1 {Index16}, ImmData16|32|64\r
-  \r
+\r
     First variable character specifies the move size\r
     Second variable character specifies size of the immediate data\r
-  \r
+\r
     Sign-extend the immediate data to the size of the operation, and zero-extend\r
     if storing to a register.\r
-  \r
+\r
     Operand1 direct with index/immed is invalid.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -640,12 +641,12 @@ ExecuteMOVI (
   index value into a register or memory location.\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVIn[w|d|q] {@}R1 {Index16}, Index16|32|64\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -659,12 +660,12 @@ ExecuteMOVIn (
   Dest <- Ip + ImmData\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVREL[w|d|q] {@}R1 {Index16}, ImmData16|32|64\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -750,7 +751,7 @@ ExecutePOP (
 \r
   @param  VmPtr             A pointer to VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -772,7 +773,7 @@ ExecuteSignedDataManip (
 \r
   @param  VmPtr             A pointer to VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -789,7 +790,7 @@ ExecuteUnsignedDataManip (
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -806,7 +807,7 @@ ExecuteLOADSP (
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -822,15 +823,15 @@ ExecuteSTORESP (
   is a register.\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVsnd {@}R1 {Indx32}, {@}R2 {Index32|Immed32}\r
-  \r
+\r
     0:7 1=>operand1 index present\r
     0:6 1=>operand2 index present\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -846,15 +847,15 @@ ExecuteMOVsnd (
   is a register.\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVsnw {@}R1 {Index16}, {@}R2 {Index16|Immed16}\r
-  \r
+\r
     0:7 1=>operand1 index present\r
     0:6 1=>operand2 index present\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -1250,7 +1251,7 @@ ExecuteEXTNDD (
 // Once we retrieve the operands for the data manipulation instructions,\r
 // call these functions to perform the operation.\r
 //\r
-STATIC CONST DATA_MANIP_EXEC_FUNCTION mDataManipDispatchTable[] = {\r
+CONST DATA_MANIP_EXEC_FUNCTION mDataManipDispatchTable[] = {\r
   ExecuteNOT,\r
   ExecuteNEG,\r
   ExecuteADD,\r
@@ -1272,7 +1273,7 @@ STATIC CONST DATA_MANIP_EXEC_FUNCTION mDataManipDispatchTable[] = {
   ExecuteEXTNDD,\r
 };\r
 \r
-STATIC CONST VM_TABLE_ENTRY           mVmOpcodeTable[] = {\r
+CONST VM_TABLE_ENTRY           mVmOpcodeTable[] = {\r
   { ExecuteBREAK },             // opcode 0x00\r
   { ExecuteJMP },               // opcode 0x01\r
   { ExecuteJMP8 },              // opcode 0x02\r
@@ -1330,19 +1331,19 @@ STATIC CONST VM_TABLE_ENTRY           mVmOpcodeTable[] = {
   { ExecutePOPn },              // opcode 0x36\r
   { ExecuteMOVI },              // opcode 0x37 - mov immediate data\r
   { ExecuteMOVIn },             // opcode 0x38 - mov immediate natural\r
-  { ExecuteMOVREL }             // opcode 0x39 - move data relative to PC\r
+  { ExecuteMOVREL },            // opcode 0x39 - move data relative to PC\r
+  { NULL },                     // opcode 0x3a\r
+  { NULL },                     // opcode 0x3b\r
+  { NULL },                     // opcode 0x3c\r
+  { NULL },                     // opcode 0x3d\r
+  { NULL },                     // opcode 0x3e\r
+  { NULL }                      // opcode 0x3f\r
 };\r
 \r
 //\r
 // Length of JMP instructions, depending on upper two bits of opcode.\r
 //\r
-STATIC CONST UINT8                    mJMPLen[] = { 2, 2, 6, 10 };\r
-\r
-//\r
-// Simple Debugger Protocol GUID\r
-//\r
-EFI_GUID mEbcSimpleDebuggerProtocolGuid = EFI_EBC_SIMPLE_DEBUGGER_PROTOCOL_GUID;\r
-\r
+CONST UINT8                    mJMPLen[] = { 2, 2, 6, 10 };\r
 \r
 /**\r
   Given a pointer to a new VM context, execute one or more instructions. This\r
@@ -1359,6 +1360,7 @@ EFI_GUID mEbcSimpleDebuggerProtocolGuid = EFI_EBC_SIMPLE_DEBUGGER_PROTOCOL_GUID;
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 EbcExecuteInstructions (\r
   IN EFI_EBC_VM_TEST_PROTOCOL *This,\r
   IN VM_CONTEXT               *VmPtr,\r
@@ -1387,12 +1389,12 @@ EbcExecuteInstructions (
   // call it if it's not null.\r
   //\r
   while (InstructionsLeft != 0) {\r
-    ExecFunc = (UINTN) mVmOpcodeTable[(*VmPtr->Ip & 0x3F)].ExecuteFunction;\r
+    ExecFunc = (UINTN) mVmOpcodeTable[(*VmPtr->Ip & OPCODE_M_OPCODE)].ExecuteFunction;\r
     if (ExecFunc == (UINTN) NULL) {\r
       EbcDebugSignalException (EXCEPT_EBC_INVALID_OPCODE, EXCEPTION_FLAG_FATAL, VmPtr);\r
       return EFI_UNSUPPORTED;\r
     } else {\r
-      mVmOpcodeTable[(*VmPtr->Ip & 0x3F)].ExecuteFunction (VmPtr);\r
+      mVmOpcodeTable[(*VmPtr->Ip & OPCODE_M_OPCODE)].ExecuteFunction (VmPtr);\r
       *InstructionCount = *InstructionCount + 1;\r
     }\r
 \r
@@ -1439,14 +1441,14 @@ EbcExecute (
     StackCorrupted = 1;\r
   }\r
 \r
-  VmPtr->FramePtr = (VOID *) ((UINT8 *) (UINTN) VmPtr->R[0] + 8);\r
+  VmPtr->FramePtr = (VOID *) ((UINT8 *) (UINTN) VmPtr->Gpr[0] + 8);\r
 \r
   //\r
   // Try to get the debug support for EBC\r
   //\r
   DEBUG_CODE_BEGIN ();\r
     Status = gBS->LocateProtocol (\r
-                    &mEbcSimpleDebuggerProtocolGuid,\r
+                    &gEfiEbcSimpleDebuggerProtocolGuid,\r
                     NULL,\r
                     (VOID **) &EbcSimpleDebugger\r
                     );\r
@@ -1477,14 +1479,6 @@ EbcExecute (
       }\r
     DEBUG_CODE_END ();\r
 \r
-    //\r
-    // Verify the opcode is in range. Otherwise generate an exception.\r
-    //\r
-    if ((*VmPtr->Ip & OPCODE_M_OPCODE) >= (sizeof (mVmOpcodeTable) / sizeof (mVmOpcodeTable[0]))) {\r
-      EbcDebugSignalException (EXCEPT_EBC_INVALID_OPCODE, EXCEPTION_FLAG_FATAL, VmPtr);\r
-      Status = EFI_UNSUPPORTED;\r
-      goto Done;\r
-    }\r
     //\r
     // Use the opcode bits to index into the opcode dispatch table. If the\r
     // function pointer is null then generate an exception.\r
@@ -1495,6 +1489,9 @@ EbcExecute (
       Status = EFI_UNSUPPORTED;\r
       goto Done;\r
     }\r
+\r
+    EbcDebuggerHookExecuteStart (VmPtr);\r
+\r
     //\r
     // The EBC VM is a strongly ordered processor, so perform a fence operation before\r
     // and after each instruction is executed.\r
@@ -1505,6 +1502,8 @@ EbcExecute (
 \r
     MemoryFence ();\r
 \r
+    EbcDebuggerHookExecuteEnd (VmPtr);\r
+\r
     //\r
     // If the step flag is set, signal an exception and continue. We don't\r
     // clear it here. Assuming the debugger is responsible for clearing it.\r
@@ -1519,7 +1518,7 @@ EbcExecute (
       EbcDebugSignalException (EXCEPT_EBC_STACK_FAULT, EXCEPTION_FLAG_FATAL, VmPtr);\r
       StackCorrupted = 1;\r
     }\r
-    if ((StackCorrupted == 0) && ((UINT64)VmPtr->R[0] <= (UINT64)(UINTN) VmPtr->StackTop)) {\r
+    if ((StackCorrupted == 0) && ((UINT64)VmPtr->Gpr[0] <= (UINT64)(UINTN) VmPtr->StackTop)) {\r
       EbcDebugSignalException (EXCEPT_EBC_STACK_FAULT, EXCEPTION_FLAG_FATAL, VmPtr);\r
       StackCorrupted = 1;\r
     }\r
@@ -1536,15 +1535,15 @@ Done:
   Execute the MOVxx instructions.\r
 \r
   Instruction format:\r
-    \r
+\r
     MOV[b|w|d|q|n]{w|d} {@}R1 {Index16|32}, {@}R2 {Index16|32}\r
     MOVqq {@}R1 {Index64}, {@}R2 {Index64}\r
-  \r
+\r
     Copies contents of [R2] -> [R1], zero extending where required.\r
-  \r
+\r
     First character indicates the size of the move.\r
     Second character indicates the size of the index(s).\r
-  \r
+\r
     Invalid to have R1 direct with index.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
@@ -1684,7 +1683,7 @@ ExecuteMOVxx (
     //\r
     // Indirect form @R2. Compute address of operand2\r
     //\r
-    Source = (UINTN) (VmPtr->R[OPERAND2_REGNUM (Operands)] + Index64Op2);\r
+    Source = (UINTN) (VmPtr->Gpr[OPERAND2_REGNUM (Operands)] + Index64Op2);\r
     //\r
     // Now get the data from the source. Always 0-extend and let the compiler\r
     // sign-extend where required.\r
@@ -1720,7 +1719,7 @@ ExecuteMOVxx (
     //\r
     // Not indirect source: MOVxx {@}Rx, Ry [Index]\r
     //\r
-    Data64 = VmPtr->R[OPERAND2_REGNUM (Operands)] + Index64Op2;\r
+    Data64 = (UINT64) (VmPtr->Gpr[OPERAND2_REGNUM (Operands)] + Index64Op2);\r
     //\r
     // Did Operand2 have an index? If so, treat as two signed values since\r
     // indexes are signed values.\r
@@ -1756,7 +1755,7 @@ ExecuteMOVxx (
     //\r
     // Reuse the Source variable to now be dest.\r
     //\r
-    Source = (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Index64Op1);\r
+    Source = (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index64Op1);\r
     //\r
     // Do the write based on the size\r
     //\r
@@ -1804,7 +1803,7 @@ ExecuteMOVxx (
     // Direct storage in register. Clear unused bits and store back to\r
     // register.\r
     //\r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = Data64 & DataMask;\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = Data64 & DataMask;\r
   }\r
   //\r
   // Advance the instruction pointer\r
@@ -1827,12 +1826,14 @@ ExecuteBREAK (
   IN VM_CONTEXT *VmPtr\r
   )\r
 {\r
+  EFI_STATUS  Status;\r
   UINT8       Operands;\r
   VOID        *EbcEntryPoint;\r
   VOID        *Thunk;\r
   UINT64      U64EbcEntryPoint;\r
   INT32       Offset;\r
 \r
+  Thunk = NULL;\r
   Operands = GETOPERANDS (VmPtr);\r
   switch (Operands) {\r
   //\r
@@ -1852,7 +1853,7 @@ ExecuteBREAK (
     //  16-8  = Major version\r
     //  7-0   = Minor version\r
     //\r
-    VmPtr->R[7] = GetVmVersion ();\r
+    VmPtr->Gpr[7] = GetVmVersion ();\r
     break;\r
 \r
   //\r
@@ -1882,26 +1883,29 @@ ExecuteBREAK (
   // After we're done, *(UINT64 *)R7 will be the address of the new thunk.\r
   //\r
   case 5:\r
-    Offset            = (INT32) VmReadMem32 (VmPtr, (UINTN) VmPtr->R[7]);\r
-    U64EbcEntryPoint  = (UINT64) (VmPtr->R[7] + Offset + 4);\r
+    Offset            = (INT32) VmReadMem32 (VmPtr, (UINTN) VmPtr->Gpr[7]);\r
+    U64EbcEntryPoint  = (UINT64) (VmPtr->Gpr[7] + Offset + 4);\r
     EbcEntryPoint     = (VOID *) (UINTN) U64EbcEntryPoint;\r
 \r
     //\r
     // Now create a new thunk\r
     //\r
-    EbcCreateThunks (VmPtr->ImageHandle, EbcEntryPoint, &Thunk, 0);\r
+    Status = EbcCreateThunks (VmPtr->ImageHandle, EbcEntryPoint, &Thunk, 0);\r
+    if (EFI_ERROR (Status)) {\r
+      return Status;\r
+    }\r
 \r
     //\r
     // Finally replace the EBC entry point memory with the thunk address\r
     //\r
-    VmWriteMem64 (VmPtr, (UINTN) VmPtr->R[7], (UINT64) (UINTN) Thunk);\r
+    VmWriteMem64 (VmPtr, (UINTN) VmPtr->Gpr[7], (UINT64) (UINTN) Thunk);\r
     break;\r
 \r
   //\r
   // Compiler setting version per value in R7\r
   //\r
   case 6:\r
-    VmPtr->CompilerVersion = (UINT32) VmPtr->R[7];\r
+    VmPtr->CompilerVersion = (UINT32) VmPtr->Gpr[7];\r
     //\r
     // Check compiler version against VM version?\r
     //\r
@@ -1928,7 +1932,7 @@ ExecuteBREAK (
   Instruction syntax:\r
     JMP64{cs|cc} Immed64\r
     JMP32{cs|cc} {@}R1 {Immed32|Index32}\r
-  \r
+\r
   Encoding:\r
     b0.7 -  immediate data present\r
     b0.6 -  1 = 64 bit immediate data\r
@@ -1978,7 +1982,9 @@ ExecuteJMP (
   ConditionFlag = (UINT8) VMFLAG_ISSET (VmPtr, VMFLAGS_CC);\r
   if ((Operand & CONDITION_M_CONDITIONAL) != 0) {\r
     if (CompareSet != ConditionFlag) {\r
+      EbcDebuggerHookJMPStart (VmPtr);\r
       VmPtr->Ip += Size;\r
+      EbcDebuggerHookJMPEnd (VmPtr);\r
       return EFI_SUCCESS;\r
     }\r
   }\r
@@ -2003,7 +2009,7 @@ ExecuteJMP (
     // 64-bit immediate data is full address. Read the immediate data,\r
     // check for alignment, and jump absolute.\r
     //\r
-    Data64 = VmReadImmed64 (VmPtr, 2);\r
+    Data64 = (UINT64) VmReadImmed64 (VmPtr, 2);\r
     if (!IS_ALIGNED ((UINTN) Data64, sizeof (UINT16))) {\r
       EbcDebugSignalException (\r
         EXCEPT_EBC_ALIGNMENT_CHECK,\r
@@ -2017,11 +2023,13 @@ ExecuteJMP (
     //\r
     // Take jump -- relative or absolute\r
     //\r
+    EbcDebuggerHookJMPStart (VmPtr);\r
     if ((Operand & JMP_M_RELATIVE) != 0) {\r
       VmPtr->Ip += (UINTN) Data64 + Size;\r
     } else {\r
       VmPtr->Ip = (VMIP) (UINTN) Data64;\r
     }\r
+    EbcDebuggerHookJMPEnd (VmPtr);\r
 \r
     return EFI_SUCCESS;\r
   }\r
@@ -2047,7 +2055,7 @@ ExecuteJMP (
   if (OPERAND1_REGNUM (Operand) == 0) {\r
     Data64 = 0;\r
   } else {\r
-    Data64 = OPERAND1_REGDATA (VmPtr, Operand);\r
+    Data64 = (UINT64) OPERAND1_REGDATA (VmPtr, Operand);\r
   }\r
   //\r
   // Decode the forms\r
@@ -2067,11 +2075,14 @@ ExecuteJMP (
       return EFI_UNSUPPORTED;\r
     }\r
 \r
+    EbcDebuggerHookJMPStart (VmPtr);\r
     if ((Operand & JMP_M_RELATIVE) != 0) {\r
       VmPtr->Ip += (UINTN) Addr + Size;\r
     } else {\r
       VmPtr->Ip = (VMIP) Addr;\r
     }\r
+    EbcDebuggerHookJMPEnd (VmPtr);\r
+\r
   } else {\r
     //\r
     // Form: JMP32 Rx {Immed32}\r
@@ -2087,11 +2098,14 @@ ExecuteJMP (
       return EFI_UNSUPPORTED;\r
     }\r
 \r
+    EbcDebuggerHookJMPStart (VmPtr);\r
     if ((Operand & JMP_M_RELATIVE) != 0) {\r
       VmPtr->Ip += (UINTN) Addr + Size;\r
     } else {\r
       VmPtr->Ip = (VMIP) Addr;\r
     }\r
+    EbcDebuggerHookJMPEnd (VmPtr);\r
+\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -2131,7 +2145,9 @@ ExecuteJMP8 (
   //\r
   if ((Opcode & CONDITION_M_CONDITIONAL) != 0) {\r
     if (CompareSet != ConditionFlag) {\r
+      EbcDebuggerHookJMP8Start (VmPtr);\r
       VmPtr->Ip += 2;\r
+      EbcDebuggerHookJMP8End (VmPtr);\r
       return EFI_SUCCESS;\r
     }\r
   }\r
@@ -2143,7 +2159,9 @@ ExecuteJMP8 (
   //\r
   // Want to check for offset == -2 and then raise an exception?\r
   //\r
+  EbcDebuggerHookJMP8Start (VmPtr);\r
   VmPtr->Ip += (Offset * 2) + 2;\r
+  EbcDebuggerHookJMP8End (VmPtr);\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -2152,20 +2170,20 @@ ExecuteJMP8 (
   Execute the EBC MOVI.\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVI[b|w|d|q][w|d|q] {@}R1 {Index16}, ImmData16|32|64\r
-  \r
+\r
     First variable character specifies the move size\r
     Second variable character specifies size of the immediate data\r
-  \r
+\r
     Sign-extend the immediate data to the size of the operation, and zero-extend\r
     if storing to a register.\r
-  \r
+\r
     Operand1 direct with index/immed is invalid.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -2249,12 +2267,12 @@ ExecuteMOVI (
       Mask64 = (UINT64)~0;\r
     }\r
 \r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = ImmData64 & Mask64;\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = ImmData64 & Mask64;\r
   } else {\r
     //\r
     // Get the address then write back based on size of the move\r
     //\r
-    Op1 = (UINT64) VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16;\r
+    Op1 = (UINT64) VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16;\r
     if ((Operands & MOVI_M_MOVEWIDTH) == MOVI_MOVEWIDTH8) {\r
       VmWriteMem8 (VmPtr, (UINTN) Op1, (UINT8) ImmData64);\r
     } else if ((Operands & MOVI_M_MOVEWIDTH) == MOVI_MOVEWIDTH16) {\r
@@ -2262,7 +2280,7 @@ ExecuteMOVI (
     } else if ((Operands & MOVI_M_MOVEWIDTH) == MOVI_MOVEWIDTH32) {\r
       VmWriteMem32 (VmPtr, (UINTN) Op1, (UINT32) ImmData64);\r
     } else {\r
-      VmWriteMem64 (VmPtr, (UINTN) Op1, ImmData64);\r
+      VmWriteMem64 (VmPtr, (UINTN) Op1, (UINT64) ImmData64);\r
     }\r
   }\r
   //\r
@@ -2278,12 +2296,12 @@ ExecuteMOVI (
   index value into a register or memory location.\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVIn[w|d|q] {@}R1 {Index16}, Index16|32|64\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -2359,13 +2377,13 @@ ExecuteMOVIn (
       return EFI_UNSUPPORTED;\r
     }\r
 \r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = ImmedIndex64;\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = ImmedIndex64;\r
   } else {\r
     //\r
     // Get the address\r
     //\r
-    Op1 = (UINT64) VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16;\r
-    VmWriteMemN (VmPtr, (UINTN) Op1, (INTN) ImmedIndex64);\r
+    Op1 = (UINT64) VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16;\r
+    VmWriteMemN (VmPtr, (UINTN) Op1, (UINTN)(INTN) ImmedIndex64);\r
   }\r
   //\r
   // Advance the instruction pointer\r
@@ -2380,12 +2398,12 @@ ExecuteMOVIn (
   Dest <- Ip + ImmData\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVREL[w|d|q] {@}R1 {Index16}, ImmData16|32|64\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -2458,14 +2476,14 @@ ExecuteMOVREL (
       return EFI_UNSUPPORTED;\r
     }\r
 \r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = (VM_REGISTER) Op2;\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = (VM_REGISTER) Op2;\r
   } else {\r
     //\r
     // Get the address = [Rx] + Index16\r
     // Write back the result. Always a natural size write, since\r
     // we're talking addresses here.\r
     //\r
-    Op1 = (UINT64) VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16;\r
+    Op1 = (UINT64) VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16;\r
     VmWriteMemN (VmPtr, (UINTN) Op1, (UINTN) Op2);\r
   }\r
   //\r
@@ -2483,15 +2501,15 @@ ExecuteMOVREL (
   is a register.\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVsnw {@}R1 {Index16}, {@}R2 {Index16|Immed16}\r
-  \r
+\r
     0:7 1=>operand1 index present\r
     0:6 1=>operand2 index present\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -2549,17 +2567,17 @@ ExecuteMOVsnw (
   //\r
   // Get the data from the source.\r
   //\r
-  Op2 = (INT64) ((INTN) (VmPtr->R[OPERAND2_REGNUM (Operands)] + Op2Index));\r
+  Op2 = (UINT64)(INT64)(INTN)(VmPtr->Gpr[OPERAND2_REGNUM (Operands)] + Op2Index);\r
   if (OPERAND2_INDIRECT (Operands)) {\r
-    Op2 = (INT64) (INTN) VmReadMemN (VmPtr, (UINTN) Op2);\r
+    Op2 = (UINT64)(INT64)(INTN)VmReadMemN (VmPtr, (UINTN) Op2);\r
   }\r
   //\r
   // Now write back the result.\r
   //\r
   if (!OPERAND1_INDIRECT (Operands)) {\r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = Op2;\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = Op2;\r
   } else {\r
-    VmWriteMemN (VmPtr, (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Op1Index), (UINTN) Op2);\r
+    VmWriteMemN (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Op1Index), (UINTN) Op2);\r
   }\r
   //\r
   // Advance the instruction pointer\r
@@ -2576,15 +2594,15 @@ ExecuteMOVsnw (
   is a register.\r
 \r
   Instruction syntax:\r
-  \r
+\r
     MOVsnd {@}R1 {Indx32}, {@}R2 {Index32|Immed32}\r
-  \r
+\r
     0:7 1=>operand1 index present\r
     0:6 1=>operand2 index present\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -2642,17 +2660,17 @@ ExecuteMOVsnd (
   //\r
   // Get the data from the source.\r
   //\r
-  Op2 = (INT64) ((INTN) (VmPtr->R[OPERAND2_REGNUM (Operands)] + Op2Index));\r
+  Op2 = (UINT64)(INT64)(INTN)(INT64)(VmPtr->Gpr[OPERAND2_REGNUM (Operands)] + Op2Index);\r
   if (OPERAND2_INDIRECT (Operands)) {\r
-    Op2 = (INT64) (INTN) VmReadMemN (VmPtr, (UINTN) Op2);\r
+    Op2 = (UINT64)(INT64)(INTN)(INT64)VmReadMemN (VmPtr, (UINTN) Op2);\r
   }\r
   //\r
   // Now write back the result.\r
   //\r
   if (!OPERAND1_INDIRECT (Operands)) {\r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = Op2;\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = Op2;\r
   } else {\r
-    VmWriteMemN (VmPtr, (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Op1Index), (UINTN) Op2);\r
+    VmWriteMemN (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Op1Index), (UINTN) Op2);\r
   }\r
   //\r
   // Advance the instruction pointer\r
@@ -2708,15 +2726,15 @@ ExecutePUSHn (
   // Get the data to push\r
   //\r
   if (OPERAND1_INDIRECT (Operands)) {\r
-    DataN = VmReadMemN (VmPtr, (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16));\r
+    DataN = VmReadMemN (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16));\r
   } else {\r
-    DataN = (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16);\r
+    DataN = (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16);\r
   }\r
   //\r
   // Adjust the stack down.\r
   //\r
-  VmPtr->R[0] -= sizeof (UINTN);\r
-  VmWriteMemN (VmPtr, (UINTN) VmPtr->R[0], DataN);\r
+  VmPtr->Gpr[0] -= sizeof (UINTN);\r
+  VmWriteMemN (VmPtr, (UINTN) VmPtr->Gpr[0], DataN);\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -2768,29 +2786,29 @@ ExecutePUSH (
   //\r
   if ((Opcode & PUSHPOP_M_64) != 0) {\r
     if (OPERAND1_INDIRECT (Operands)) {\r
-      Data64 = VmReadMem64 (VmPtr, (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16));\r
+      Data64 = VmReadMem64 (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16));\r
     } else {\r
-      Data64 = (UINT64) VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16;\r
+      Data64 = (UINT64) VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16;\r
     }\r
     //\r
     // Adjust the stack down, then write back the data\r
     //\r
-    VmPtr->R[0] -= sizeof (UINT64);\r
-    VmWriteMem64 (VmPtr, (UINTN) VmPtr->R[0], Data64);\r
+    VmPtr->Gpr[0] -= sizeof (UINT64);\r
+    VmWriteMem64 (VmPtr, (UINTN) VmPtr->Gpr[0], Data64);\r
   } else {\r
     //\r
     // 32-bit data\r
     //\r
     if (OPERAND1_INDIRECT (Operands)) {\r
-      Data32 = VmReadMem32 (VmPtr, (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16));\r
+      Data32 = VmReadMem32 (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16));\r
     } else {\r
-      Data32 = (UINT32) VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16;\r
+      Data32 = (UINT32) VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16;\r
     }\r
     //\r
     // Adjust the stack down and write the data\r
     //\r
-    VmPtr->R[0] -= sizeof (UINT32);\r
-    VmWriteMem32 (VmPtr, (UINTN) VmPtr->R[0], Data32);\r
+    VmPtr->Gpr[0] -= sizeof (UINT32);\r
+    VmWriteMem32 (VmPtr, (UINTN) VmPtr->Gpr[0], Data32);\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -2841,15 +2859,15 @@ ExecutePOPn (
   //\r
   // Read the data off the stack, then adjust the stack pointer\r
   //\r
-  DataN = VmReadMemN (VmPtr, (UINTN) VmPtr->R[0]);\r
-  VmPtr->R[0] += sizeof (UINTN);\r
+  DataN = VmReadMemN (VmPtr, (UINTN) VmPtr->Gpr[0]);\r
+  VmPtr->Gpr[0] += sizeof (UINTN);\r
   //\r
   // Do the write-back\r
   //\r
   if (OPERAND1_INDIRECT (Operands)) {\r
-    VmWriteMemN (VmPtr, (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16), DataN);\r
+    VmWriteMemN (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16), DataN);\r
   } else {\r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = (INT64) (UINT64) ((UINTN) DataN + Index16);\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = (INT64) (UINT64) (UINTN) (DataN + Index16);\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -2905,29 +2923,29 @@ ExecutePOP (
     //\r
     // Read the data off the stack, then adjust the stack pointer\r
     //\r
-    Data64 = VmReadMem64 (VmPtr, (UINTN) VmPtr->R[0]);\r
-    VmPtr->R[0] += sizeof (UINT64);\r
+    Data64 = VmReadMem64 (VmPtr, (UINTN) VmPtr->Gpr[0]);\r
+    VmPtr->Gpr[0] += sizeof (UINT64);\r
     //\r
     // Do the write-back\r
     //\r
     if (OPERAND1_INDIRECT (Operands)) {\r
-      VmWriteMem64 (VmPtr, (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16), Data64);\r
+      VmWriteMem64 (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16), Data64);\r
     } else {\r
-      VmPtr->R[OPERAND1_REGNUM (Operands)] = Data64 + Index16;\r
+      VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = Data64 + Index16;\r
     }\r
   } else {\r
     //\r
     // 32-bit pop. Read it off the stack and adjust the stack pointer\r
     //\r
-    Data32 = (INT32) VmReadMem32 (VmPtr, (UINTN) VmPtr->R[0]);\r
-    VmPtr->R[0] += sizeof (UINT32);\r
+    Data32 = (INT32) VmReadMem32 (VmPtr, (UINTN) VmPtr->Gpr[0]);\r
+    VmPtr->Gpr[0] += sizeof (UINT32);\r
     //\r
     // Do the write-back\r
     //\r
     if (OPERAND1_INDIRECT (Operands)) {\r
-      VmWriteMem32 (VmPtr, (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16), Data32);\r
+      VmWriteMem32 (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND1_REGNUM (Operands)] + Index16), Data32);\r
     } else {\r
-      VmPtr->R[OPERAND1_REGNUM (Operands)] = (INT64) Data32 + Index16;\r
+      VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = (INT64) Data32 + Index16;\r
     }\r
   }\r
 \r
@@ -2968,6 +2986,13 @@ ExecuteCALL (
   //\r
   Opcode    = GETOPCODE (VmPtr);\r
   Operands  = GETOPERANDS (VmPtr);\r
+\r
+  if ((Operands & OPERAND_M_NATIVE_CALL) != 0) {\r
+    EbcDebuggerHookCALLEXStart (VmPtr);\r
+  } else {\r
+    EbcDebuggerHookCALLStart (VmPtr);\r
+  }\r
+\r
   //\r
   // Assign these as well to avoid compiler warnings\r
   //\r
@@ -3002,11 +3027,11 @@ ExecuteCALL (
   // put our return address and frame pointer on the VM stack.\r
   //\r
   if ((Operands & OPERAND_M_NATIVE_CALL) == 0) {\r
-    VmPtr->R[0] -= 8;\r
-    VmWriteMemN (VmPtr, (UINTN) VmPtr->R[0], (UINTN) FramePtr);\r
-    VmPtr->FramePtr = (VOID *) (UINTN) VmPtr->R[0];\r
-    VmPtr->R[0] -= 8;\r
-    VmWriteMem64 (VmPtr, (UINTN) VmPtr->R[0], (UINT64) (UINTN) (VmPtr->Ip + Size));\r
+    VmPtr->Gpr[0] -= 8;\r
+    VmWriteMemN (VmPtr, (UINTN) VmPtr->Gpr[0], (UINTN) FramePtr);\r
+    VmPtr->FramePtr = (VOID *) (UINTN) VmPtr->Gpr[0];\r
+    VmPtr->Gpr[0] -= 8;\r
+    VmWriteMem64 (VmPtr, (UINTN) VmPtr->Gpr[0], (UINT64) (UINTN) (VmPtr->Ip + Size));\r
   }\r
   //\r
   // If 64-bit data, then absolute jump only\r
@@ -3021,7 +3046,7 @@ ExecuteCALL (
       //\r
       // Call external function, get the return value, and advance the IP\r
       //\r
-      EbcLLCALLEX (VmPtr, (UINTN) Immed64, (UINTN) VmPtr->R[0], FramePtr, Size);\r
+      EbcLLCALLEX (VmPtr, (UINTN) Immed64, (UINTN) VmPtr->Gpr[0], FramePtr, Size);\r
     }\r
   } else {\r
     //\r
@@ -3030,7 +3055,7 @@ ExecuteCALL (
     // Compiler should take care of upper bits if 32-bit machine.\r
     //\r
     if (OPERAND1_REGNUM (Operands) != 0) {\r
-      Immed64 = (UINT64) (UINTN) VmPtr->R[OPERAND1_REGNUM (Operands)];\r
+      Immed64 = (UINT64) (UINTN) VmPtr->Gpr[OPERAND1_REGNUM (Operands)];\r
     }\r
     //\r
     // Get final address\r
@@ -3058,17 +3083,23 @@ ExecuteCALL (
       // Native call. Relative or absolute?\r
       //\r
       if ((Operands & OPERAND_M_RELATIVE_ADDR) != 0) {\r
-        EbcLLCALLEX (VmPtr, (UINTN) (Immed64 + VmPtr->Ip + Size), (UINTN) VmPtr->R[0], FramePtr, Size);\r
+        EbcLLCALLEX (VmPtr, (UINTN) (Immed64 + VmPtr->Ip + Size), (UINTN) VmPtr->Gpr[0], FramePtr, Size);\r
       } else {\r
         if ((VmPtr->StopFlags & STOPFLAG_BREAK_ON_CALLEX) != 0) {\r
           CpuBreakpoint ();\r
         }\r
 \r
-        EbcLLCALLEX (VmPtr, (UINTN) Immed64, (UINTN) VmPtr->R[0], FramePtr, Size);\r
+        EbcLLCALLEX (VmPtr, (UINTN) Immed64, (UINTN) VmPtr->Gpr[0], FramePtr, Size);\r
       }\r
     }\r
   }\r
 \r
+  if ((Operands & OPERAND_M_NATIVE_CALL) != 0) {\r
+    EbcDebuggerHookCALLEXEnd (VmPtr);\r
+  } else {\r
+    EbcDebuggerHookCALLEnd (VmPtr);\r
+  }\r
+\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -3089,18 +3120,21 @@ ExecuteRET (
   IN VM_CONTEXT *VmPtr\r
   )\r
 {\r
+\r
+  EbcDebuggerHookRETStart (VmPtr);\r
+\r
   //\r
   // If we're at the top of the stack, then simply set the done\r
   // flag and return\r
   //\r
-  if (VmPtr->StackRetAddr == (UINT64) VmPtr->R[0]) {\r
+  if (VmPtr->StackRetAddr == (UINT64) VmPtr->Gpr[0]) {\r
     VmPtr->StopFlags |= STOPFLAG_APP_DONE;\r
   } else {\r
     //\r
     // Pull the return address off the VM app's stack and set the IP\r
     // to it\r
     //\r
-    if (!IS_ALIGNED ((UINTN) VmPtr->R[0], sizeof (UINT16))) {\r
+    if (!IS_ALIGNED ((UINTN) VmPtr->Gpr[0], sizeof (UINT16))) {\r
       EbcDebugSignalException (\r
         EXCEPT_EBC_ALIGNMENT_CHECK,\r
         EXCEPTION_FLAG_FATAL,\r
@@ -3110,12 +3144,15 @@ ExecuteRET (
     //\r
     // Restore the IP and frame pointer from the stack\r
     //\r
-    VmPtr->Ip = (VMIP) (UINTN) VmReadMem64 (VmPtr, (UINTN) VmPtr->R[0]);\r
-    VmPtr->R[0] += 8;\r
-    VmPtr->FramePtr = (VOID *) VmReadMemN (VmPtr, (UINTN) VmPtr->R[0]);\r
-    VmPtr->R[0] += 8;\r
+    VmPtr->Ip = (VMIP) (UINTN) VmReadMem64 (VmPtr, (UINTN) VmPtr->Gpr[0]);\r
+    VmPtr->Gpr[0] += 8;\r
+    VmPtr->FramePtr = (VOID *) VmReadMemN (VmPtr, (UINTN) VmPtr->Gpr[0]);\r
+    VmPtr->Gpr[0] += 8;\r
   }\r
 \r
+\r
+  EbcDebuggerHookRETEnd (VmPtr);\r
+\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -3128,7 +3165,7 @@ ExecuteRET (
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -3153,7 +3190,7 @@ ExecuteCMP (
   //\r
   // Get the register data we're going to compare to\r
   //\r
-  Op1 = VmPtr->R[OPERAND1_REGNUM (Operands)];\r
+  Op1 = VmPtr->Gpr[OPERAND1_REGNUM (Operands)];\r
   //\r
   // Get immediate data\r
   //\r
@@ -3174,15 +3211,15 @@ ExecuteCMP (
   //\r
   if (OPERAND2_INDIRECT (Operands)) {\r
     if ((Opcode & OPCODE_M_64BIT) != 0) {\r
-      Op2 = (INT64) VmReadMem64 (VmPtr, (UINTN) (VmPtr->R[OPERAND2_REGNUM (Operands)] + Index16));\r
+      Op2 = (INT64) VmReadMem64 (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND2_REGNUM (Operands)] + Index16));\r
     } else {\r
       //\r
       // 32-bit operations. 0-extend the values for all cases.\r
       //\r
-      Op2 = (INT64) (UINT64) ((UINT32) VmReadMem32 (VmPtr, (UINTN) (VmPtr->R[OPERAND2_REGNUM (Operands)] + Index16)));\r
+      Op2 = (INT64) (UINT64) ((UINT32) VmReadMem32 (VmPtr, (UINTN) (VmPtr->Gpr[OPERAND2_REGNUM (Operands)] + Index16)));\r
     }\r
   } else {\r
-    Op2 = VmPtr->R[OPERAND2_REGNUM (Operands)] + Index16;\r
+    Op2 = VmPtr->Gpr[OPERAND2_REGNUM (Operands)] + Index16;\r
   }\r
   //\r
   // Now do the compare\r
@@ -3271,7 +3308,7 @@ ExecuteCMP (
   if (Flag != 0) {\r
     VMFLAG_SET (VmPtr, VMFLAGS_CC);\r
   } else {\r
-    VMFLAG_CLEAR (VmPtr, VMFLAGS_CC);\r
+    VMFLAG_CLEAR (VmPtr, (UINT64)VMFLAGS_CC);\r
   }\r
   //\r
   // Advance the IP\r
@@ -3289,7 +3326,7 @@ ExecuteCMP (
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -3325,7 +3362,7 @@ ExecuteCMPI (
   //\r
   // Get operand1 data we're going to compare to\r
   //\r
-  Op1 = (INT64) VmPtr->R[OPERAND1_REGNUM (Operands)];\r
+  Op1 = (INT64) VmPtr->Gpr[OPERAND1_REGNUM (Operands)];\r
   if (OPERAND1_INDIRECT (Operands)) {\r
     //\r
     // Indirect operand1. Fetch 32 or 64-bit value based on compare size.\r
@@ -3450,7 +3487,7 @@ ExecuteCMPI (
   if (Flag != 0) {\r
     VMFLAG_SET (VmPtr, VMFLAGS_CC);\r
   } else {\r
-    VMFLAG_CLEAR (VmPtr, VMFLAGS_CC);\r
+    VMFLAG_CLEAR (VmPtr, (UINT64)VMFLAGS_CC);\r
   }\r
   //\r
   // Advance the IP\r
@@ -3555,7 +3592,7 @@ ExecuteSUB (
   if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
     return (UINT64) ((INT64) ((INT64) Op1 - (INT64) Op2));\r
   } else {\r
-    return (UINT64) ((INT64) ((INT32) Op1 - (INT32) Op2));\r
+    return (UINT64) ((INT64) ((INT32) ((INT32) Op1 - (INT32) Op2)));\r
   }\r
 }\r
 \r
@@ -3583,7 +3620,7 @@ ExecuteMUL (
   if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
     return MultS64x64 ((INT64)Op1, (INT64)Op2);\r
   } else {\r
-    return (UINT64) ((INT64) ((INT32) Op1 * (INT32) Op2));\r
+    return (UINT64) ((INT64) ((INT32) ((INT32) Op1 * (INT32) Op2)));\r
   }\r
 }\r
 \r
@@ -3611,7 +3648,7 @@ ExecuteMULU (
   if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
     return MultU64x64 (Op1, Op2);\r
   } else {\r
-    return (UINT64) ((UINT32) Op1 * (UINT32) Op2);\r
+    return (UINT64) ((UINT32) ((UINT32) Op1 * (UINT32) Op2));\r
   }\r
 }\r
 \r
@@ -3696,7 +3733,7 @@ ExecuteDIVU (
     // Get the destination register\r
     //\r
     if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
-      return (UINT64) (DivU64x64Remainder ((INT64)Op1, (INT64)Op2, &Remainder));\r
+      return (UINT64) (DivU64x64Remainder (Op1, Op2, &Remainder));\r
     } else {\r
       return (UINT64) ((UINT32) Op1 / (UINT32) Op2);\r
     }\r
@@ -4057,7 +4094,7 @@ ExecuteEXTNDD (
 \r
   @param  VmPtr             A pointer to VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -4087,7 +4124,7 @@ ExecuteSignedDataManip (
 \r
   @param  VmPtr             A pointer to VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -4118,7 +4155,7 @@ ExecuteUnsignedDataManip (
   @param  VmPtr             A pointer to VM context.\r
   @param  IsSignedOp        Indicates whether the operand is signed or not.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -4134,6 +4171,7 @@ ExecuteDataManip (
   UINT8   Size;\r
   UINT64  Op1;\r
   UINT64  Op2;\r
+  INTN    DataManipDispatchTableIndex;\r
 \r
   //\r
   // Get opcode and operands\r
@@ -4162,7 +4200,7 @@ ExecuteDataManip (
   //\r
   // Now get operand2 (source). It's of format {@}R2 {Index16|Immed16}\r
   //\r
-  Op2 = (UINT64) VmPtr->R[OPERAND2_REGNUM (Operands)] + Index16;\r
+  Op2 = (UINT64) VmPtr->Gpr[OPERAND2_REGNUM (Operands)] + Index16;\r
   if (OPERAND2_INDIRECT (Operands)) {\r
     //\r
     // Indirect form: @R2 Index16. Fetch as 32- or 64-bit data\r
@@ -4192,7 +4230,7 @@ ExecuteDataManip (
   // Get operand1 (destination and sometimes also an actual operand)\r
   // of form {@}R1\r
   //\r
-  Op1 = VmPtr->R[OPERAND1_REGNUM (Operands)];\r
+  Op1 = (UINT64) VmPtr->Gpr[OPERAND1_REGNUM (Operands)];\r
   if (OPERAND1_INDIRECT (Operands)) {\r
     if ((Opcode & DATAMANIP_M_64) != 0) {\r
       Op1 = VmReadMem64 (VmPtr, (UINTN) Op1);\r
@@ -4215,9 +4253,9 @@ ExecuteDataManip (
   //\r
   // Dispatch to the computation function\r
   //\r
-  if (((Opcode & OPCODE_M_OPCODE) - OPCODE_NOT) >=\r
-        (sizeof (mDataManipDispatchTable) / sizeof (mDataManipDispatchTable[0]))\r
-        ) {\r
+  DataManipDispatchTableIndex = (Opcode & OPCODE_M_OPCODE) - OPCODE_NOT;\r
+  if ((DataManipDispatchTableIndex < 0) ||\r
+      (DataManipDispatchTableIndex >= ARRAY_SIZE (mDataManipDispatchTable))) {\r
     EbcDebugSignalException (\r
       EXCEPT_EBC_INVALID_OPCODE,\r
       EXCEPTION_FLAG_ERROR,\r
@@ -4229,13 +4267,13 @@ ExecuteDataManip (
     VmPtr->Ip += Size;\r
     return EFI_UNSUPPORTED;\r
   } else {\r
-    Op2 = mDataManipDispatchTable[(Opcode & OPCODE_M_OPCODE) - OPCODE_NOT](VmPtr, Op1, Op2);\r
+    Op2 = mDataManipDispatchTable[DataManipDispatchTableIndex](VmPtr, Op1, Op2);\r
   }\r
   //\r
   // Write back the result.\r
   //\r
   if (OPERAND1_INDIRECT (Operands)) {\r
-    Op1 = VmPtr->R[OPERAND1_REGNUM (Operands)];\r
+    Op1 = (UINT64) VmPtr->Gpr[OPERAND1_REGNUM (Operands)];\r
     if ((Opcode & DATAMANIP_M_64) != 0) {\r
       VmWriteMem64 (VmPtr, (UINTN) Op1, Op2);\r
     } else {\r
@@ -4246,9 +4284,9 @@ ExecuteDataManip (
     // Storage back to a register. Write back, clearing upper bits (as per\r
     // the specification) if 32-bit operation.\r
     //\r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = Op2;\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = Op2;\r
     if ((Opcode & DATAMANIP_M_64) == 0) {\r
-      VmPtr->R[OPERAND1_REGNUM (Operands)] &= 0xFFFFFFFF;\r
+      VmPtr->Gpr[OPERAND1_REGNUM (Operands)] &= 0xFFFFFFFF;\r
     }\r
   }\r
   //\r
@@ -4267,7 +4305,7 @@ ExecuteDataManip (
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -4295,7 +4333,7 @@ ExecuteLOADSP (
     // Spec states that this instruction will not modify reserved bits in\r
     // the flags register.\r
     //\r
-    VmPtr->Flags = (VmPtr->Flags &~VMFLAGS_ALL_VALID) | (VmPtr->R[OPERAND2_REGNUM (Operands)] & VMFLAGS_ALL_VALID);\r
+    VmPtr->Flags = (VmPtr->Flags &~VMFLAGS_ALL_VALID) | (VmPtr->Gpr[OPERAND2_REGNUM (Operands)] & VMFLAGS_ALL_VALID);\r
     break;\r
 \r
   default:\r
@@ -4321,7 +4359,7 @@ ExecuteLOADSP (
 \r
   @param  VmPtr             A pointer to a VM context.\r
 \r
-  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.     \r
+  @retval EFI_UNSUPPORTED   The opcodes/operands is not supported.\r
   @retval EFI_SUCCESS       The instruction is executed successfully.\r
 \r
 **/\r
@@ -4348,14 +4386,14 @@ ExecuteSTORESP (
     //\r
     // Retrieve the value in the flags register, then clear reserved bits\r
     //\r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = (UINT64) (VmPtr->Flags & VMFLAGS_ALL_VALID);\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = (UINT64) (VmPtr->Flags & VMFLAGS_ALL_VALID);\r
     break;\r
 \r
   //\r
   // Get IP -- address of following instruction\r
   //\r
   case 1:\r
-    VmPtr->R[OPERAND1_REGNUM (Operands)] = (UINT64) (UINTN) VmPtr->Ip + 2;\r
+    VmPtr->Gpr[OPERAND1_REGNUM (Operands)] = (UINT64) (UINTN) VmPtr->Ip + 2;\r
     break;\r
 \r
   default:\r
@@ -4381,13 +4419,13 @@ ExecuteSTORESP (
     b14:12  - number of bits in this index assigned to natural units (=a)\r
     ba:11   - constant units = ConstUnits\r
     b0:a    - natural units = NaturalUnits\r
-  \r
+\r
   Given this info, the offset can be computed by:\r
     offset = sign_bit * (ConstUnits + NaturalUnits * sizeof(UINTN))\r
 \r
   Max offset is achieved with index = 0x7FFF giving an offset of\r
   0x27B (32-bit machine) or 0x477 (64-bit machine).\r
-  Min offset is achieved with index = \r
+  Min offset is achieved with index =\r
 \r
   @param  VmPtr             A pointer to VM context.\r
   @param  CodeOffset        Offset from IP of the location of the 16-bit index\r
@@ -4570,7 +4608,7 @@ VmReadIndex64 (
   //\r
   ConstUnits = ARShiftU64 (((Index &~0xF000000000000000ULL) & Mask), (UINTN)NBits);\r
 \r
-  Offset  = MultU64x64 (NaturalUnits, sizeof (UINTN)) + ConstUnits;\r
+  Offset  = MultU64x64 ((UINT64) NaturalUnits, sizeof (UINTN)) + ConstUnits;\r
 \r
   //\r
   // Now set the sign\r
@@ -4585,7 +4623,7 @@ VmReadIndex64 (
 \r
 /**\r
   Writes 8-bit data to memory address.\r
-   \r
+\r
   This routine is called by the EBC data\r
   movement instructions that write to memory. Since these writes\r
   may be to the stack, which looks like (high address on top) this,\r
@@ -4595,14 +4633,14 @@ VmReadIndex64 (
   [EBC stack]\r
 \r
   we need to detect all attempts to write to the EBC entry point argument\r
-  stack area and adjust the address (which will initially point into the \r
+  stack area and adjust the address (which will initially point into the\r
   VM stack) to point into the EBC entry point arguments.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
-  @param  Addr              Adddress to write to.\r
+  @param  Addr              Address to write to.\r
   @param  Data              Value to write to Addr.\r
 \r
-  @retval EFI_SUCCESS       The instruction is executed successfully.  \r
+  @retval EFI_SUCCESS       The instruction is executed successfully.\r
   @retval Other             Some error occurs when writing data to the address.\r
 \r
 **/\r
@@ -4623,7 +4661,7 @@ VmWriteMem8 (
 \r
 /**\r
   Writes 16-bit data to memory address.\r
-   \r
+\r
   This routine is called by the EBC data\r
   movement instructions that write to memory. Since these writes\r
   may be to the stack, which looks like (high address on top) this,\r
@@ -4633,14 +4671,14 @@ VmWriteMem8 (
   [EBC stack]\r
 \r
   we need to detect all attempts to write to the EBC entry point argument\r
-  stack area and adjust the address (which will initially point into the \r
+  stack area and adjust the address (which will initially point into the\r
   VM stack) to point into the EBC entry point arguments.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
-  @param  Addr              Adddress to write to.\r
+  @param  Addr              Address to write to.\r
   @param  Data              Value to write to Addr.\r
 \r
-  @retval EFI_SUCCESS       The instruction is executed successfully.  \r
+  @retval EFI_SUCCESS       The instruction is executed successfully.\r
   @retval Other             Some error occurs when writing data to the address.\r
 \r
 **/\r
@@ -4686,7 +4724,7 @@ VmWriteMem16 (
 \r
 /**\r
   Writes 32-bit data to memory address.\r
-   \r
+\r
   This routine is called by the EBC data\r
   movement instructions that write to memory. Since these writes\r
   may be to the stack, which looks like (high address on top) this,\r
@@ -4696,14 +4734,14 @@ VmWriteMem16 (
   [EBC stack]\r
 \r
   we need to detect all attempts to write to the EBC entry point argument\r
-  stack area and adjust the address (which will initially point into the \r
+  stack area and adjust the address (which will initially point into the\r
   VM stack) to point into the EBC entry point arguments.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
-  @param  Addr              Adddress to write to.\r
+  @param  Addr              Address to write to.\r
   @param  Data              Value to write to Addr.\r
 \r
-  @retval EFI_SUCCESS       The instruction is executed successfully.  \r
+  @retval EFI_SUCCESS       The instruction is executed successfully.\r
   @retval Other             Some error occurs when writing data to the address.\r
 \r
 **/\r
@@ -4749,7 +4787,7 @@ VmWriteMem32 (
 \r
 /**\r
   Writes 64-bit data to memory address.\r
-   \r
+\r
   This routine is called by the EBC data\r
   movement instructions that write to memory. Since these writes\r
   may be to the stack, which looks like (high address on top) this,\r
@@ -4759,14 +4797,14 @@ VmWriteMem32 (
   [EBC stack]\r
 \r
   we need to detect all attempts to write to the EBC entry point argument\r
-  stack area and adjust the address (which will initially point into the \r
+  stack area and adjust the address (which will initially point into the\r
   VM stack) to point into the EBC entry point arguments.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
-  @param  Addr              Adddress to write to.\r
+  @param  Addr              Address to write to.\r
   @param  Data              Value to write to Addr.\r
 \r
-  @retval EFI_SUCCESS       The instruction is executed successfully.  \r
+  @retval EFI_SUCCESS       The instruction is executed successfully.\r
   @retval Other             Some error occurs when writing data to the address.\r
 \r
 **/\r
@@ -4778,7 +4816,6 @@ VmWriteMem64 (
   )\r
 {\r
   EFI_STATUS  Status;\r
-  UINT32      Data32;\r
 \r
   //\r
   // Convert the address if it's in the stack gap\r
@@ -4800,8 +4837,7 @@ VmWriteMem64 (
     }\r
 \r
     MemoryFence ();\r
-    Data32 = (UINT32) (((UINT32 *) &Data)[1]);\r
-    if ((Status = VmWriteMem32 (VmPtr, Addr + sizeof (UINT32), Data32)) != EFI_SUCCESS) {\r
+    if ((Status = VmWriteMem32 (VmPtr, Addr + sizeof (UINT32), (UINT32) RShiftU64(Data, 32))) != EFI_SUCCESS) {\r
       return Status;\r
     }\r
 \r
@@ -4814,7 +4850,7 @@ VmWriteMem64 (
 \r
 /**\r
   Writes UINTN data to memory address.\r
-   \r
+\r
   This routine is called by the EBC data\r
   movement instructions that write to memory. Since these writes\r
   may be to the stack, which looks like (high address on top) this,\r
@@ -4824,14 +4860,14 @@ VmWriteMem64 (
   [EBC stack]\r
 \r
   we need to detect all attempts to write to the EBC entry point argument\r
-  stack area and adjust the address (which will initially point into the \r
+  stack area and adjust the address (which will initially point into the\r
   VM stack) to point into the EBC entry point arguments.\r
 \r
   @param  VmPtr             A pointer to a VM context.\r
-  @param  Addr              Adddress to write to.\r
+  @param  Addr              Address to write to.\r
   @param  Data              Value to write to Addr.\r
 \r
-  @retval EFI_SUCCESS       The instruction is executed successfully.  \r
+  @retval EFI_SUCCESS       The instruction is executed successfully.\r
   @retval Other             Some error occurs when writing data to the address.\r
 \r
 **/\r
@@ -4862,7 +4898,7 @@ VmWriteMemN (
       MemoryFence ();\r
       Status = VmWriteMem32 (VmPtr, Addr + Index * sizeof (UINT32), (UINT32) Data);\r
       MemoryFence ();\r
-      Data = (UINTN)RShiftU64 ((UINT64)Data, 32);\r
+      Data = (UINTN) RShiftU64 ((UINT64)Data, 32);\r
     }\r
   }\r
 \r
@@ -4969,8 +5005,8 @@ VmReadImmed32 (
   //\r
   // Return unaligned data\r
   //\r
-  Data = (UINT32) VmReadCode16 (VmPtr, Offset);\r
-  Data |= (UINT32) (VmReadCode16 (VmPtr, Offset + 2) << 16);\r
+  Data  = (UINT32) VmReadCode16 (VmPtr, Offset);\r
+  Data |= (UINT32)(VmReadCode16 (VmPtr, Offset + 2) << 16);\r
   return Data;\r
 }\r
 \r
@@ -5011,7 +5047,7 @@ VmReadImmed64 (
   Ptr             = (UINT8 *) &Data64;\r
   Data32          = VmReadCode32 (VmPtr, Offset);\r
   *(UINT32 *) Ptr = Data32;\r
-  Ptr += sizeof (Data32);\r
+  Ptr            += sizeof (Data32);\r
   Data32          = VmReadCode32 (VmPtr, Offset + sizeof (UINT32));\r
   *(UINT32 *) Ptr = Data32;\r
   return Data64;\r
@@ -5019,7 +5055,7 @@ VmReadImmed64 (
 \r
 \r
 /**\r
-  Reads 16-bit unsinged data from the code stream.\r
+  Reads 16-bit unsigned data from the code stream.\r
 \r
   This routine provides the ability to read raw unsigned data from the code\r
   stream.\r
@@ -5059,7 +5095,7 @@ VmReadCode16 (
 \r
 \r
 /**\r
-  Reads 32-bit unsinged data from the code stream.\r
+  Reads 32-bit unsigned data from the code stream.\r
 \r
   This routine provides the ability to read raw unsigned data from the code\r
   stream.\r
@@ -5093,7 +5129,7 @@ VmReadCode32 (
 \r
 \r
 /**\r
-  Reads 64-bit unsinged data from the code stream.\r
+  Reads 64-bit unsigned data from the code stream.\r
 \r
   This routine provides the ability to read raw unsigned data from the code\r
   stream.\r
@@ -5126,7 +5162,7 @@ VmReadCode64 (
   Ptr             = (UINT8 *) &Data64;\r
   Data32          = VmReadCode32 (VmPtr, Offset);\r
   *(UINT32 *) Ptr = Data32;\r
-  Ptr += sizeof (Data32);\r
+  Ptr            += sizeof (Data32);\r
   Data32          = VmReadCode32 (VmPtr, Offset + sizeof (UINT32));\r
   *(UINT32 *) Ptr = Data32;\r
   return Data64;\r
@@ -5139,7 +5175,7 @@ VmReadCode64 (
   @param  VmPtr             A pointer to VM context.\r
   @param  Addr              The memory address.\r
 \r
-  @return The 8-bit value from the memory adress.\r
+  @return The 8-bit value from the memory address.\r
 \r
 **/\r
 UINT8\r
@@ -5164,7 +5200,7 @@ VmReadMem8 (
   @param  VmPtr             A pointer to VM context.\r
   @param  Addr              The memory address.\r
 \r
-  @return The 16-bit value from the memory adress.\r
+  @return The 16-bit value from the memory address.\r
 \r
 **/\r
 UINT16\r
@@ -5195,7 +5231,7 @@ VmReadMem16 (
   @param  VmPtr             A pointer to VM context.\r
   @param  Addr              The memory address.\r
 \r
-  @return The 32-bit value from the memory adress.\r
+  @return The 32-bit value from the memory address.\r
 \r
 **/\r
 UINT32\r
@@ -5230,7 +5266,7 @@ VmReadMem32 (
   @param  VmPtr             A pointer to VM context.\r
   @param  Addr              The memory address.\r
 \r
-  @return The 64-bit value from the memory adress.\r
+  @return The 64-bit value from the memory address.\r
 \r
 **/\r
 UINT64\r
@@ -5256,9 +5292,9 @@ VmReadMem64 (
   //\r
   // Return unaligned data. Assume little endian.\r
   //\r
-  Data    = (UINT64) VmReadMem32 (VmPtr, Addr);\r
-  Data32  = VmReadMem32 (VmPtr, Addr + sizeof (UINT32));\r
-  *(UINT32 *) ((UINT32 *) &Data + 1) = Data32;\r
+  Data32 = VmReadMem32 (VmPtr, Addr);\r
+  Data  = (UINT64) VmReadMem32 (VmPtr, Addr + sizeof (UINT32));\r
+  Data  = LShiftU64 (Data, 32) | Data32;\r
   return Data;\r
 }\r
 \r
@@ -5340,7 +5376,7 @@ VmReadMemN (
 \r
 /**\r
   Returns the version of the EBC virtual machine.\r
-    \r
+\r
   @return The 64-bit version of EBC virtual machine.\r
 \r
 **/\r