Changeset 72128 in vbox

Timestamp:

May 4, 2018 10:15:36 PM (7 years ago)

Author:

vboxsync

Message:

CPUMAllRegs.cpp/CPUMRecalcHyperDRx: Added comment regarding misconception of the LE and GE flags.

File:

• trunk/src/VBox/VMM/VMMAll/CPUMAllRegs.cpp (modified) (1 diff)

trunk/src/VBox/VMM/VMMAll/CPUMAllRegs.cpp

@@ -1557 +1557 @@
      */
     RTGCUINTREG uGstDr7 = CPUMGetGuestDR7(pVCpu);
+    /** @todo This isn't correct. BPs work without setting LE and GE under AMD-V.  They are also documented as unsupported by P6+.
+     * The LE|GE flags does have an effect in real mode on AMD-V with instruction fetch breakpoints, they seem to work a little like traps rather than faults. */
     if (!(uGstDr7 & (X86_DR7_LE | X86_DR7_GE)))
         uGstDr7 = 0;