Changeset 58697 in vbox for trunk/src/VBox/Runtime/r0drv

Timestamp:

Nov 16, 2015 2:42:34 PM (9 years ago)

Author:

vboxsync

Message:

Solaris/r0drv,SUPDrv: Don't drop log messages on bad-preemption context as we miss errors.
Added a warning instead and during normal VBox operation we should eliminate messages with the warning prefix.

File:

• trunk/src/VBox/Runtime/r0drv/solaris/RTLogWriteDebugger-r0drv-solaris.c (modified) (2 diffs)

trunk/src/VBox/Runtime/r0drv/solaris/RTLogWriteDebugger-r0drv-solaris.c

@@ -47 +47 @@
         AssertBreakpoint();
 
-    /* cmn_err() acquires adaptive mutexes. Not preemption safe, see @bugref{6657}. */
-    if (!RTThreadPreemptIsEnabled(NIL_RTTHREAD))
-        return;
-
     if (    !g_frtSolSplSetsEIF
 #if defined(RT_ARCH_AMD64) || defined(RT_ARCH_X86)
@@ -59 +55 @@
         )
     {
-        cmn_err(CE_CONT, pch);
+        /* cmn_err() acquires adaptive mutexes. Not preemption safe, see @bugref{6657}. */
+        if (RTThreadPreemptIsEnabled(NIL_RTTHREAD))
+            cmn_err(CE_CONT, pch);
+        else
+            cmn_err(CE_CONT, "[!BadPreemptCtx!] %s", pch);
     }