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

Timestamp:

Feb 20, 2018 3:59:05 AM (7 years ago)

Author:

vboxsync

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

120927

Message:

VMM/HM: We no longer require NRIP to be present when decode-assist is present. The SVM R0 code explicitly checks
for presence of NRIP before using u64NextRIP field every where it uses it.

File:

• trunk/src/VBox/VMM/VMMR3/HM.cpp (modified) (1 diff)

trunk/src/VBox/VMM/VMMR3/HM.cpp

@@ -1744 +1744 @@
 
     /*
-     * SVM R0 code assumes if the decode-assist feature exists, NRIP feature exists too.
-     */
-    /** @todo This might not be true anymore, re-verify SVM R0 code for
-     *        assumption of this combination. */
-    AssertLogRelReturn(  !(pVM->hm.s.svm.u32Features & X86_CPUID_SVM_FEATURE_EDX_DECODE_ASSISTS)
-                       || (pVM->hm.s.svm.u32Features & X86_CPUID_SVM_FEATURE_EDX_NRIP_SAVE),
-                       VERR_HM_UNSUPPORTED_CPU_FEATURE_COMBO);
-
-    /*
      * Nested paging is determined in HMR3Init, verify the sanity of that.
      */