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

Timestamp:

Jul 27, 2013 12:43:07 AM (12 years ago)

Author:

vboxsync

Message:

HMR0SVM.cpp: Added todo regarding I/O breakpoints.

File:

• trunk/src/VBox/VMM/VMMR0/HMSVMR0.cpp (modified) (1 diff)

trunk/src/VBox/VMM/VMMR0/HMSVMR0.cpp

@@ -4108 +4108 @@
         {
             /* If any IO breakpoints are armed, then we should check if a debug trap needs to be generated. */
+            /** @todo This is inefficient and wrong according to intel and amd specs
+             *        (regardless of which is correct).  See the same code in the VT-x case.
+             *        write testcase and refactor the code to use a mostly shared
+             *        implementation after the initial DR7/CR4 checks. */
             if (pCtx->dr[7] & X86_DR7_ENABLED_MASK)
             {