]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.S
MdeModulePkg/UDF: Fix creation of UDF logical partition
[mirror_edk2.git] / MdeModulePkg / Universal / DebugSupportDxe / X64 / AsmFuncs.S
index 958c398a9579ac641171a177f7ea1fbb3ab7d8ec..fcc6838827a65df4ba0df3571e6179ecdb08d2b1 100644 (file)
@@ -1,24 +1,25 @@
-#/**@file\r
-# Low leve x64 specific debug support functions.\r
-#\r
-# Copyright (c) 2006 - 2009, Intel Corporation\r
-# All rights reserved. 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
-#\r
-# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
-# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
-#\r
-#**/    \r
+///**@file\r
+// Low leve x64 specific debug support functions.\r
+//\r
+// Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>\r
+// Portions copyright (c) 2008 - 2009, Apple Inc. 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
+//\r
+// THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
+// WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+//\r
+//**/\r
 \r
 ASM_GLOBAL ASM_PFX(OrigVector)\r
 ASM_GLOBAL ASM_PFX(InterruptEntryStub)\r
 ASM_GLOBAL ASM_PFX(StubSize)\r
 ASM_GLOBAL ASM_PFX(CommonIdtEntry)\r
 ASM_GLOBAL ASM_PFX(FxStorSupport)\r
-     \r
-.data \r
+\r
+.data\r
 \r
 ASM_PFX(StubSize):         .long   ASM_PFX(InterruptEntryStubEnd) - ASM_PFX(InterruptEntryStub)\r
 ASM_PFX(AppRsp):           .long   0x11111111 # ?\r
@@ -34,43 +35,41 @@ ASM_PFX(Rflags):           .long   0x55555555 # ?
 ASM_PFX(OrigVector):       .long   0x66666666 # ?\r
                            .long   0x66666666 # ?\r
 \r
-## The declarations below define the memory region that will be used for the debug stack.\r
-## The context record will be built by pushing register values onto this stack.\r
-## It is imparitive that alignment be carefully managed, since the FXSTOR and\r
-## FXRSTOR instructions will GP fault if their memory operand is not 16 byte aligned.\r
-##\r
-## The stub will switch stacks from the application stack to the debuger stack\r
-## and pushes the exception number.\r
-##\r
-## Then we building the context record on the stack. Since the stack grows down,\r
-## we push the fields of the context record from the back to the front.  There\r
-## are 336 bytes of stack used prior allocating the 512 bytes of stack to be\r
-## used as the memory buffer for the fxstor instruction. Therefore address of\r
-## the buffer used for the FXSTOR instruction is &Eax - 336 - 512, which\r
-## must be 16 byte aligned.\r
-##\r
-## We carefully locate the stack to make this happen.\r
-##\r
-## For reference, the context structure looks like this:\r
-##      struct {\r
-##        UINT64            ExceptionData;\r
-##        FX_SAVE_STATE_X64 FxSaveState;    // 512 bytes, must be 16 byte aligned\r
-##        UINT64            Dr0, Dr1, Dr2, Dr3, Dr6, Dr7;\r
-##        UINT64            Cr0, Cr1, Cr2, Cr3, Cr4, Cr8;\r
-##        UINT64            RFlags;\r
-##        UINT64            Ldtr, Tr;\r
-##        UINT64            Gdtr[2], Idtr[2];\r
-##        UINT64            Rip;\r
-##        UINT64            Gs, Fs, Es, Ds, Cs, Ss;\r
-##        UINT64            Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
-##        UINT64            R8, R9, R10, R11, R12, R13, R14, R15;\r
-##      } SYSTEM_CONTEXT_X64;  // 64 bit system context record\r
+// The declarations below define the memory region that will be used for the debug stack.\r
+// The context record will be built by pushing register values onto this stack.\r
+// It is imparitive that alignment be carefully managed, since the FXSTOR and\r
+// FXRSTOR instructions will GP fault if their memory operand is not 16 byte aligned.\r
+//\r
+// The stub will switch stacks from the application stack to the debuger stack\r
+// and pushes the exception number.\r
+//\r
+// Then we building the context record on the stack. Since the stack grows down,\r
+// we push the fields of the context record from the back to the front.  There\r
+// are 336 bytes of stack used prior allocating the 512 bytes of stack to be\r
+// used as the memory buffer for the fxstor instruction. Therefore address of\r
+// the buffer used for the FXSTOR instruction is &Eax - 336 - 512, which\r
+// must be 16 byte aligned.\r
+//\r
+// We carefully locate the stack to make this happen.\r
+//\r
+// For reference, the context structure looks like this:\r
+//      struct {\r
+//        UINT64            ExceptionData;\r
+//        FX_SAVE_STATE_X64 FxSaveState;    // 512 bytes, must be 16 byte aligned\r
+//        UINT64            Dr0, Dr1, Dr2, Dr3, Dr6, Dr7;\r
+//        UINT64            Cr0, Cr1, Cr2, Cr3, Cr4, Cr8;\r
+//        UINT64            RFlags;\r
+//        UINT64            Ldtr, Tr;\r
+//        UINT64            Gdtr[2], Idtr[2];\r
+//        UINT64            Rip;\r
+//        UINT64            Gs, Fs, Es, Ds, Cs, Ss;\r
+//        UINT64            Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
+//        UINT64            R8, R9, R10, R11, R12, R13, R14, R15;\r
+//      } SYSTEM_CONTEXT_X64;  // 64 bit system context record\r
 \r
 .p2align 4\r
 DebugStackEnd : .ascii     "DbgStkEnd >>>>>>"    # 16 byte long string - must be 16 bytes to preserve alignment\r
-                .rept  0x1ffc\r
-                  .long    0x00000000  \r
-                .endr\r
+                .fill 0x1ffc, 4, 0x00000000\r
                                                  # 32K should be enough stack\r
                                                  #   This allocation is coocked to insure\r
                                                  #   that the the buffer for the FXSTORE instruction\r
@@ -80,23 +79,23 @@ ASM_PFX(ExceptionNumber):  .long   0x77777777    # first entry will be the vecto
                            .long   0x77777777    # ?\r
 \r
 DebugStackBegin : .ascii    "<<<< DbgStkBegin"   # initial debug ESP == DebugStackBegin, set in stub\r
-           \r
+\r
 \r
 .text\r
 \r
-#------------------------------------------------------------------------------\r
-# BOOLEAN\r
-# FxStorSupport (\r
-#   void\r
-#   )\r
-#\r
-# Abstract: Returns TRUE if FxStor instructions are supported\r
-#\r
+//------------------------------------------------------------------------------\r
+// BOOLEAN\r
+// FxStorSupport (\r
+//   void\r
+//   )\r
+//\r
+// Abstract: Returns TRUE if FxStor instructions are supported\r
+//\r
 ASM_GLOBAL ASM_PFX(FxStorSupport)\r
-ASM_PFX(FxStorSupport):  \r
-#\r
-# cpuid corrupts rbx which must be preserved per the C calling convention\r
-#\r
+ASM_PFX(FxStorSupport):\r
+//\r
+// cpuid corrupts rbx which must be preserved per the C calling convention\r
+//\r
                 pushq   %rbx\r
                 movq    $1, %rax\r
                 cpuid\r
@@ -105,15 +104,15 @@ ASM_PFX(FxStorSupport):
                 shrq    $24, %rax\r
                 popq    %rbx\r
                 ret\r
-#------------------------------------------------------------------------------\r
-# void\r
-# Vect2Desc (\r
-#   IA32_IDT_GATE_DESCRIPTOR * DestDesc,  // rcx\r
-#   void (*Vector) (void)   // rdx\r
-#   )\r
-#\r
-# Abstract: Encodes an IDT descriptor with the given physical address\r
-#\r
+//------------------------------------------------------------------------------\r
+// void\r
+// Vect2Desc (\r
+//   IA32_IDT_GATE_DESCRIPTOR * DestDesc,  // rcx\r
+//   void (*Vector) (void)   // rdx\r
+//   )\r
+//\r
+// Abstract: Encodes an IDT descriptor with the given physical address\r
+//\r
 ASM_GLOBAL ASM_PFX(Vect2Desc)\r
 ASM_PFX(Vect2Desc):\r
                 movq    %rdx, %rax\r
@@ -128,83 +127,83 @@ ASM_PFX(Vect2Desc):
 \r
                 ret\r
 \r
-#------------------------------------------------------------------------------\r
-# InterruptEntryStub\r
-#\r
-# Abstract: This code is not a function, but is a small piece of code that is\r
-#               copied and fixed up once for each IDT entry that is hooked.\r
-#\r
+//------------------------------------------------------------------------------\r
+// InterruptEntryStub\r
+//\r
+// Abstract: This code is not a function, but is a small piece of code that is\r
+//               copied and fixed up once for each IDT entry that is hooked.\r
+//\r
 ASM_GLOBAL ASM_PFX(InterruptEntryStub)\r
 ASM_PFX(InterruptEntryStub):\r
 \r
                 pushq  $0                       # push vector number - will be modified before installed\r
                 jmp     ASM_PFX(CommonIdtEntry)\r
-                \r
+\r
 ASM_GLOBAL ASM_PFX(InterruptEntryStubEnd)\r
 ASM_PFX(InterruptEntryStubEnd):\r
 \r
-#------------------------------------------------------------------------------\r
-# CommonIdtEntry\r
-#\r
-# Abstract: This code is not a function, but is the common part for all IDT\r
-#               vectors.\r
-#\r
+//------------------------------------------------------------------------------\r
+// CommonIdtEntry\r
+//\r
+// Abstract: This code is not a function, but is the common part for all IDT\r
+//               vectors.\r
+//\r
 ASM_GLOBAL ASM_PFX(CommonIdtEntry)\r
-##\r
-## At this point, the stub has saved the current application stack esp into AppRsp\r
-## and switched stacks to the debug stack, where it pushed the vector number\r
-##\r
-## The application stack looks like this:\r
-##\r
-##              ...\r
-##              (last application stack entry)\r
-##              [16 bytes alignment, do not care it]\r
-##              SS from interrupted task\r
-##              RSP from interrupted task\r
-##              rflags from interrupted task\r
-##              CS from interrupted task\r
-##              RIP from interrupted task\r
-##              Error code <-------------------- Only present for some exeption types\r
-##\r
-##              Vector Number <----------------- pushed in our IDT Entry\r
-##\r
-\r
-\r
-## The stub switched us to the debug stack and pushed the interrupt number.\r
-##\r
-## Next, construct the context record.  It will be build on the debug stack by\r
-## pushing the registers in the correct order so as to create the context structure\r
-## on the debug stack.  The context record must be built from the end back to the\r
-## beginning because the stack grows down...\r
-#\r
-## For reference, the context record looks like this:\r
-##\r
-## typedef\r
-## struct {\r
-##   UINT64            ExceptionData;\r
-##   FX_SAVE_STATE_X64 FxSaveState;\r
-##   UINT64            Dr0, Dr1, Dr2, Dr3, Dr6, Dr7;\r
-##   UINT64            Cr0, Cr2, Cr3, Cr4, Cr8;\r
-##   UINT64            RFlags;\r
-##   UINT64            Ldtr, Tr;\r
-##   UINT64            Gdtr[2], Idtr[2];\r
-##   UINT64            Rip;\r
-##   UINT64            Gs, Fs, Es, Ds, Cs, Ss;\r
-##   UINT64            Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
-##   UINT64            R8, R9, R10, R11, R12, R13, R14, R15;\r
-## } SYSTEM_CONTEXT_X64;  // 64\r
+//\r
+// At this point, the stub has saved the current application stack esp into AppRsp\r
+// and switched stacks to the debug stack, where it pushed the vector number\r
+//\r
+// The application stack looks like this:\r
+//\r
+//              ...\r
+//              (last application stack entry)\r
+//              [16 bytes alignment, do not care it]\r
+//              SS from interrupted task\r
+//              RSP from interrupted task\r
+//              rflags from interrupted task\r
+//              CS from interrupted task\r
+//              RIP from interrupted task\r
+//              Error code <-------------------- Only present for some exeption types\r
+//\r
+//              Vector Number <----------------- pushed in our IDT Entry\r
+//\r
+\r
+\r
+// The stub switched us to the debug stack and pushed the interrupt number.\r
+//\r
+// Next, construct the context record.  It will be build on the debug stack by\r
+// pushing the registers in the correct order so as to create the context structure\r
+// on the debug stack.  The context record must be built from the end back to the\r
+// beginning because the stack grows down...\r
+//\r
+// For reference, the context record looks like this:\r
+//\r
+// typedef\r
+// struct {\r
+//   UINT64            ExceptionData;\r
+//   FX_SAVE_STATE_X64 FxSaveState;\r
+//   UINT64            Dr0, Dr1, Dr2, Dr3, Dr6, Dr7;\r
+//   UINT64            Cr0, Cr2, Cr3, Cr4, Cr8;\r
+//   UINT64            RFlags;\r
+//   UINT64            Ldtr, Tr;\r
+//   UINT64            Gdtr[2], Idtr[2];\r
+//   UINT64            Rip;\r
+//   UINT64            Gs, Fs, Es, Ds, Cs, Ss;\r
+//   UINT64            Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
+//   UINT64            R8, R9, R10, R11, R12, R13, R14, R15;\r
+// } SYSTEM_CONTEXT_X64;  // 64\r
 ASM_PFX(CommonIdtEntry):\r
-## NOTE: we save rsp here to prevent compiler put rip reference cause error AppRsp\r
+// NOTE: we save rsp here to prevent compiler put rip reference cause error AppRsp\r
                 pushq   %rax\r
                 movq    (8)(%rsp), %rax                  # save vector number\r
-                movq    %rax, ASM_PFX(ExceptionNumber)   # save vector number\r
+                movq    %rax, ASM_PFX(ExceptionNumber)(%rip)   # save vector number\r
                 popq    %rax\r
                 addq    $8, %rsp                         # pop vector number\r
-                movq    %rsp, ASM_PFX(AppRsp)            # save stack top\r
-                movq    DebugStackBegin, %rsp            # switch to debugger stack\r
+                movq    %rsp, ASM_PFX(AppRsp)(%rip)      # save stack top\r
+                movq    DebugStackBegin(%rip), %rsp      # switch to debugger stack\r
                 subq    $8, %rsp                         # leave space for vector number\r
-## UINT64  Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
-## UINT64  R8, R9, R10, R11, R12, R13, R14, R15;\r
+// UINT64  Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
+// UINT64  R8, R9, R10, R11, R12, R13, R14, R15;\r
                 pushq    %r15\r
                 pushq    %r14\r
                 pushq    %r13\r
@@ -221,77 +220,77 @@ ASM_PFX(CommonIdtEntry):
                 pushq    %rbp\r
                 pushq    %rsi\r
                 pushq    %rdi\r
-## Save interrupt state rflags register...\r
+// Save interrupt state rflags register...\r
                 pushfq\r
                 popq    %rax\r
-                movq    %rax, ASM_PFX(Rflags)\r
-## We need to determine if any extra data was pushed by the exception, and if so, save it\r
-## To do this, we check the exception number pushed by the stub, and cache the\r
-## result in a variable since we'll need this again.\r
-                cmpl    $0, ASM_PFX(ExceptionNumber)\r
+                movq    %rax, ASM_PFX(Rflags)(%rip)\r
+// We need to determine if any extra data was pushed by the exception, and if so, save it\r
+// To do this, we check the exception number pushed by the stub, and cache the\r
+// result in a variable since we'll need this again.\r
+                cmpl    $0, ASM_PFX(ExceptionNumber)(%rip)\r
                 jz      ExtraPushOne\r
-                cmpl    $10, ASM_PFX(ExceptionNumber)\r
+                cmpl    $10, ASM_PFX(ExceptionNumber)(%rip)\r
                 jz      ExtraPushOne\r
-                cmpl    $11, ASM_PFX(ExceptionNumber)\r
+                cmpl    $11, ASM_PFX(ExceptionNumber)(%rip)\r
                 jz      ExtraPushOne\r
-                cmpl    $12, ASM_PFX(ExceptionNumber)\r
+                cmpl    $12, ASM_PFX(ExceptionNumber)(%rip)\r
                 jz      ExtraPushOne\r
-                cmpl    $13, ASM_PFX(ExceptionNumber)\r
+                cmpl    $13, ASM_PFX(ExceptionNumber)(%rip)\r
                 jz      ExtraPushOne\r
-                cmpl    $14, ASM_PFX(ExceptionNumber)\r
+                cmpl    $14, ASM_PFX(ExceptionNumber)(%rip)\r
                 jz      ExtraPushOne\r
-                cmpl    $17, ASM_PFX(ExceptionNumber)\r
+                cmpl    $17, ASM_PFX(ExceptionNumber)(%rip)\r
                 jz      ExtraPushOne\r
-                movl    $0, ASM_PFX(ExtraPush)\r
-                movl    $0, ASM_PFX(ExceptData)\r
+                movl    $0, ASM_PFX(ExtraPush)(%rip)\r
+                movl    $0, ASM_PFX(ExceptData)(%rip)\r
                 jmp     ExtraPushDone\r
 ExtraPushOne:\r
-                movl  $1, ASM_PFX(ExtraPush)\r
+                movl  $1, ASM_PFX(ExtraPush)(%rip)\r
 \r
-## If there's some extra data, save it also, and modify the saved AppRsp to effectively\r
-## pop this value off the application's stack.\r
-                movq    ASM_PFX(AppRsp), %rax \r
+// If there's some extra data, save it also, and modify the saved AppRsp to effectively\r
+// pop this value off the application's stack.\r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
                 movq    (%rax), %rbx\r
-                movq    %rbx, ASM_PFX(ExceptData)\r
+                movq    %rbx, ASM_PFX(ExceptData)(%rip)\r
                 addq    $8, %rax\r
-                movq    %rax, ASM_PFX(AppRsp)\r
+                movq    %rax, ASM_PFX(AppRsp)(%rip)\r
 \r
 ExtraPushDone:\r
 \r
-## The "push" above pushed the debug stack rsp.  Since what we're actually doing\r
-## is building the context record on the debug stack, we need to save the pushed\r
-## debug RSP, and replace it with the application's last stack entry...\r
+// The "push" above pushed the debug stack rsp.  Since what we're actually doing\r
+// is building the context record on the debug stack, we need to save the pushed\r
+// debug RSP, and replace it with the application's last stack entry...\r
                 movq    24(%rsp), %rax\r
-                movq    %rax, ASM_PFX(DebugRsp)\r
-                movq    ASM_PFX(AppRsp), %rax \r
-                addq    $40, %rax\r
+                movq    %rax, ASM_PFX(DebugRsp)(%rip)\r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
+                movq    24(%rax), %rax\r
                 # application stack has ss, rsp, rflags, cs, & rip, so\r
-                # last actual application stack entry is 40 bytes\r
-                # into the application stack.\r
+                # last actual application stack entry is saved at offset\r
+                # 24 bytes from stack top.\r
                 movq    %rax, 24(%rsp)\r
 \r
-## continue building context record\r
-## UINT64  Gs, Fs, Es, Ds, Cs, Ss;  insure high 16 bits of each is zero\r
-                movq    %ss, %rax\r
+// continue building context record\r
+// UINT64  Gs, Fs, Es, Ds, Cs, Ss;  insure high 16 bits of each is zero\r
+                mov     %ss, %rax\r
                 pushq   %rax\r
                 # CS from application is one entry back in application stack\r
-                movq    ASM_PFX(AppRsp), %rax\r
-                movzxw  8(%rax), %rax\r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
+                movzwq  8(%rax), %rax\r
                 pushq   %rax\r
 \r
-                movq    %ds, %rax\r
+                mov     %ds, %rax\r
                 pushq   %rax\r
-                movq    %es, %rax\r
+                mov     %es, %rax\r
                 pushq   %rax\r
-                movq    %fs, %rax\r
+                mov     %fs, %rax\r
                 pushq   %rax\r
-                movq    %gs, %rax\r
+                mov     %gs, %rax\r
                 pushq   %rax\r
-## UINT64  Rip;\r
+// UINT64  Rip;\r
                 # Rip from application is on top of application stack\r
-                movq    ASM_PFX(AppRsp), %rax \r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
                 pushq   (%rax)\r
-## UINT64  Gdtr[2], Idtr[2];\r
+// UINT64  Gdtr[2], Idtr[2];\r
                 push    $0\r
                 push    $0\r
                 sidtq   (%rsp)\r
@@ -299,20 +298,20 @@ ExtraPushDone:
                 push    $0\r
                 sgdtq   (%rsp)\r
 \r
-## UINT64  Ldtr, Tr;\r
+// UINT64  Ldtr, Tr;\r
                 xorq    %rax, %rax\r
                 str     %ax\r
                 pushq   %rax\r
                 sldt    %ax\r
                 pushq   %rax\r
 \r
-## UINT64  RFlags;\r
-## Rflags from application is two entries back in application stack\r
-                movq    ASM_PFX(AppRsp), %rax \r
+// UINT64  RFlags;\r
+// Rflags from application is two entries back in application stack\r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
                 pushq   16(%rax)\r
-## UINT64  Cr0, Cr1, Cr2, Cr3, Cr4, Cr8;\r
-## insure FXSAVE/FXRSTOR is enabled in CR4...\r
-## ... while we're at it, make sure DE is also enabled...\r
+// UINT64  Cr0, Cr1, Cr2, Cr3, Cr4, Cr8;\r
+// insure FXSAVE/FXRSTOR is enabled in CR4...\r
+// ... while we're at it, make sure DE is also enabled...\r
                 movq    %cr8, %rax\r
                 pushq   %rax\r
                 movq    %cr4, %rax\r
@@ -326,16 +325,16 @@ ExtraPushDone:
                 push    $0\r
                 movq    %cr0, %rax\r
                 pushq   %rax\r
-## UINT64  Dr0, Dr1, Dr2, Dr3, Dr6, Dr7;\r
+// UINT64  Dr0, Dr1, Dr2, Dr3, Dr6, Dr7;\r
                 movq    %dr7, %rax\r
                 pushq   %rax\r
-## clear Dr7 while executing debugger itself\r
+// clear Dr7 while executing debugger itself\r
                 xorq    %rax, %rax\r
                 movq    %rax, %dr7\r
 \r
                 movq    %dr6, %rax\r
                 pushq   %rax\r
-## insure all status bits in dr6 are clear...\r
+// insure all status bits in dr6 are clear...\r
                 xorq    %rax, %rax\r
                 movq    %rax, %dr6\r
 \r
@@ -348,40 +347,43 @@ ExtraPushDone:
                 movq    %dr0, %rax\r
                 pushq   %rax\r
 \r
-## FX_SAVE_STATE_X64 FxSaveState;\r
+// FX_SAVE_STATE_X64 FxSaveState;\r
                 subq    $512, %rsp\r
                 movq    %rsp, %rdi\r
                 # IMPORTANT!! The debug stack has been carefully constructed to\r
                 # insure that rsp and rdi are 16 byte aligned when we get here.\r
                 # They MUST be.  If they are not, a GP fault will occur.\r
-          \r
+\r
                 #     FXSTOR_RDI\r
                 fxsave      (%rdi)\r
 \r
-## UINT64  ExceptionData;\r
-                movq    ASM_PFX(ExceptData),  %rax\r
+// UEFI calling convention for x64 requires that Direction flag in EFLAGs is clear\r
+                cld\r
+                \r
+// UINT64  ExceptionData;\r
+                movq    ASM_PFX(ExceptData)(%rip),  %rax\r
                 pushq   %rax\r
 \r
-# call to C code which will in turn call registered handler\r
-# pass in the vector number\r
+// call to C code which will in turn call registered handler\r
+// pass in the vector number\r
                 movq    %rsp, %rdx\r
-                movq    ASM_PFX(ExceptionNumber), %rcx \r
+                movq    ASM_PFX(ExceptionNumber)(%rip), %rcx\r
                 subq    $40, %rsp\r
                 call    ASM_PFX(InterruptDistrubutionHub)\r
                 addq    $40, %rsp\r
-# restore context...\r
-## UINT64  ExceptionData;\r
+// restore context...\r
+// UINT64  ExceptionData;\r
                 addq    $8, %rsp\r
 \r
-## FX_SAVE_STATE_X64 FxSaveState;\r
+// FX_SAVE_STATE_X64 FxSaveState;\r
                 movq    %rsp, %rsi\r
-            \r
+\r
                 #    FXRSTOR_RSI\r
                 fxrstor     (%rsi)\r
 \r
                 addq    $512, %rsp\r
 \r
-## UINT64  Dr0, Dr1, Dr2, Dr3, Dr6, Dr7;\r
+// UINT64  Dr0, Dr1, Dr2, Dr3, Dr6, Dr7;\r
                 popq    %rax\r
                 movq    %rax, %dr0\r
                 popq    %rax\r
@@ -391,12 +393,12 @@ ExtraPushDone:
                 popq    %rax\r
                 movq    %rax, %dr3\r
 \r
-## skip restore of dr6.  We cleared dr6 during the context save.\r
+// skip restore of dr6.  We cleared dr6 during the context save.\r
                 addq    $8, %rsp\r
                 popq    %rax\r
                 movq    %rax, %dr7\r
 \r
-## UINT64  Cr0, Cr1, Cr2, Cr3, Cr4, Cr8;\r
+// UINT64  Cr0, Cr1, Cr2, Cr3, Cr4, Cr8;\r
                 popq    %rax\r
                 movq    %rax, %cr0\r
                 addq    $8, %rsp\r
@@ -408,34 +410,34 @@ ExtraPushDone:
                 movq    %rax, %cr4\r
                 popq    %rax\r
                 movq    %rax, %cr8\r
-## UINT64  RFlags;\r
-                movq    ASM_PFX(AppRsp), %rax \r
+// UINT64  RFlags;\r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
                 popq    16(%rax)\r
-## UINT64  Ldtr, Tr;\r
-## UINT64  Gdtr[2], Idtr[2];\r
-## Best not let anyone mess with these particular registers...\r
+// UINT64  Ldtr, Tr;\r
+// UINT64  Gdtr[2], Idtr[2];\r
+// Best not let anyone mess with these particular registers...\r
                 addq    $48, %rsp\r
-## UINT64  Rip;\r
+// UINT64  Rip;\r
                 popq    (%rax)\r
 \r
-## UINT64  Gs, Fs, Es, Ds, Cs, Ss;\r
-## NOTE - modified segment registers could hang the debugger...  We\r
-##        could attempt to insulate ourselves against this possibility,\r
-##        but that poses risks as well.\r
-##\r
+// UINT64  Gs, Fs, Es, Ds, Cs, Ss;\r
+// NOTE - modified segment registers could hang the debugger...  We\r
+//        could attempt to insulate ourselves against this possibility,\r
+//        but that poses risks as well.\r
+//\r
 \r
                 popq    %rax\r
-                # movq    %rax, %gs\r
+                # mov     %rax, %gs\r
                 popq    %rax\r
-                # movq    %rax, %fs\r
+                # mov     %rax, %fs\r
                 popq    %rax\r
-                movq    %rax, %es\r
+                mov     %rax, %es\r
                 popq    %rax\r
-                movq    %rax, %ds\r
-                movq    ASM_PFX(AppRsp), %rax \r
+                mov     %rax, %ds\r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
                 popq    8(%rax)\r
                 popq    %rax\r
-                movq    %rax, %ss\r
+                mov     %rax, %ss\r
 ## The next stuff to restore is the general purpose registers that were pushed\r
 ## using the "push" instruction.\r
 ##\r
@@ -445,12 +447,12 @@ ExtraPushDone:
 ## determine if we need to relocate the application stack.\r
 \r
                 movq    24(%rsp), %rbx  # move the potentially modified AppRsp into rbx\r
-                movq    ASM_PFX(AppRsp), %rax \r
-                addq    $40, %rax\r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
+                movq    24(%rax), %rax\r
                 cmpq    %rax, %rbx\r
                 je      NoAppStackMove\r
 \r
-                movq    ASM_PFX(AppRsp), %rax \r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
                 movq    (%rax), %rcx     # RIP\r
                 movq    %rcx, (%rbx)\r
 \r
@@ -467,40 +469,40 @@ ExtraPushDone:
                 movq    %rcx, 32(%rbx)\r
 \r
                 movq    %rbx, %rax       # modify the saved AppRsp to the new AppRsp\r
-                movq    %rax, ASM_PFX(AppRsp)\r
+                movq    %rax, ASM_PFX(AppRsp)(%rip)\r
 NoAppStackMove:\r
-                movq    ASM_PFX(DebugRsp), %rax   # restore the DebugRsp on the debug stack\r
+                movq    ASM_PFX(DebugRsp)(%rip), %rax   # restore the DebugRsp on the debug stack\r
                                          # so our "pop" will not cause a stack switch\r
                 movq    %rax, 24(%rsp)\r
 \r
-                cmpl    $0x068, ASM_PFX(ExceptionNumber)\r
+                cmpl    $0x068, ASM_PFX(ExceptionNumber)(%rip)\r
                 jne     NoChain\r
 \r
 Chain:\r
 \r
-## Restore rflags so when we chain, the flags will be exactly as if we were never here.\r
-## We gin up the stack to do an iretq so we can get ALL the flags.\r
-                movq    ASM_PFX(AppRsp), %rax \r
+// Restore rflags so when we chain, the flags will be exactly as if we were never here.\r
+// We gin up the stack to do an iretq so we can get ALL the flags.\r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
                 movq    40(%rax), %rbx\r
                 pushq   %rbx\r
-                movq    %ss, %rax\r
+                mov     %ss, %rax\r
                 pushq   %rax\r
                 movq    %rsp, %rax\r
                 addq    $16, %rax\r
                 pushq   %rax\r
-                movq    ASM_PFX(AppRsp), %rax \r
+                movq    ASM_PFX(AppRsp)(%rip), %rax\r
                 movq    16(%rax), %rbx\r
                 andq    $0xfffffffffffffcff, %rbx  # special handling for IF and TF\r
                 pushq   %rbx\r
-                movq    %cs, %rax\r
+                mov     %cs, %rax\r
                 pushq   %rax\r
-                movq    PhonyIretq, %rax \r
+                movq    PhonyIretq(%rip), %rax\r
                 pushq   %rax\r
                 iretq\r
 PhonyIretq:\r
 \r
-## UINT64  Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
-## UINT64  R8, R9, R10, R11, R12, R13, R14, R15;\r
+// UINT64  Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
+// UINT64  R8, R9, R10, R11, R12, R13, R14, R15;\r
                 popq    %rdi\r
                 popq    %rsi\r
                 popq    %rbp\r
@@ -518,13 +520,13 @@ PhonyIretq:
                 popq    %r14\r
                 popq    %r15\r
 \r
-## Switch back to application stack\r
-                movq    ASM_PFX(AppRsp), %rsp \r
-## Jump to original handler\r
+// Switch back to application stack\r
+                movq    ASM_PFX(AppRsp)(%rip), %rsp\r
+// Jump to original handler\r
                 jmp     ASM_PFX(OrigVector)\r
 NoChain:\r
-## UINT64  Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
-## UINT64  R8, R9, R10, R11, R12, R13, R14, R15;\r
+// UINT64  Rdi, Rsi, Rbp, Rsp, Rbx, Rdx, Rcx, Rax;\r
+// UINT64  R8, R9, R10, R11, R12, R13, R14, R15;\r
                 popq    %rdi\r
                 popq    %rsi\r
                 popq    %rbp\r
@@ -542,8 +544,8 @@ NoChain:
                 popq    %r14\r
                 popq    %r15\r
 \r
-## Switch back to application stack\r
-                movq    ASM_PFX(AppRsp), %rsp \r
+// Switch back to application stack\r
+                movq    ASM_PFX(AppRsp)(%rip), %rsp\r
 \r
-## We're outa here...\r
+// We're outa here...\r
                 iret\r