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

Timestamp:

May 17, 2016 10:04:08 PM (9 years ago)

Author:

vboxsync

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

107276

Message:

PATMRCHandlerInt3PatchTrap: logging non-VINF_SUCCESS statuses, rearranged code for better karma.

File:

• trunk/src/VBox/VMM/VMMRC/PATMRC.cpp (modified) (1 diff)

trunk/src/VBox/VMM/VMMRC/PATMRC.cpp

@@ -532 +532 @@
             rcStrict = IEMExecOneBypassWithPrefetchedByPC(pVCpu, pCtxCore, pCtxCore->rip,
                                                           pRec->patch.aPrivInstr, pRec->patch.cbPrivInstr);
-            if (RT_FAILURE(rcStrict))
-            {
-                Log(("EMInterpretInstructionCPU failed with %Rrc\n", VBOXSTRICTRC_TODO(rcStrict)));
-                PATM_STAT_FAULT_INC(&pRec->patch);
-                pRec->patch.cTraps++;
-                return VINF_EM_RAW_EMULATE_INSTR;
-            }
-            return VBOXSTRICTRC_TODO(rcStrict);
+            if (RT_SUCCESS(rcStrict))
+            {
+                if (rcStrict != VINF_SUCCESS)
+                    Log(("PATMRCHandleInt3PatchTrap: returns %Rrc\n", VBOXSTRICTRC_TODO(rcStrict)));
+                return VBOXSTRICTRC_TODO(rcStrict);
+            }
+
+            Log(("IEMExecOneBypassWithPrefetchedByPC failed with %Rrc\n", VBOXSTRICTRC_TODO(rcStrict)));
+            PATM_STAT_FAULT_INC(&pRec->patch);
+            pRec->patch.cTraps++;
+            return VINF_EM_RAW_EMULATE_INSTR;
         }
     }