Changeset 72299 in vbox for trunk/src

Timestamp:

May 23, 2018 1:50:39 PM (7 years ago)

Author:

vboxsync

Message:

VMM/HMVMXR0: Comment update regarding ICEBP/INT1.

File:

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

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

@@ -13501 +13501 @@
          *
          * It is important to reflect what the VM-exit gave us (preserving the interruption-type) rather than use
-         * hmR0VmxSetPendingXcptDB() as the #DB could've been raised while executing ICEBP and not the 'normal' #DB.
-         * Thus it -may- trigger different handling in the CPU (like skipped DPL checks). See @bugref{6398}.
+         * hmR0VmxSetPendingXcptDB() as the #DB could've been raised while executing ICEBP (INT1) and not the
+         * regular #DB. Thus it -may- trigger different handling in the CPU (like skipped DPL checks), see @bugref{6398}.
          *
-         * Since ICEBP isn't documented on Intel, see AMD spec. 15.20 "Event Injection".
+         * Intel re-documented ICEBP/INT1 on May 2018 previously documented as part of Intel 386,
+         * see Intel spec. 24.8.3 "VM-Entry Controls for Event Injection".
          */
         rc  = hmR0VmxReadExitIntInfoVmcs(pVmxTransient);