Changeset 72129 in vbox for trunk/src/VBox/VMM

Timestamp:

May 4, 2018 10:43:03 PM (7 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

122525

Message:

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

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. */
+    /** @todo This isn't correct. BPs work without setting LE and GE under AMD-V.  They are also documented as unsupported by P6+. */
     if (!(uGstDr7 & (X86_DR7_LE | X86_DR7_GE)))
         uGstDr7 = 0;