Changeset 54866 in vbox for trunk/src/VBox/Runtime/r0drv/linux/threadctxhooks-r0drv-linux.c

Timestamp:

Mar 20, 2015 1:00:25 PM (10 years ago)

Author:

vboxsync

Message:

threadctxhooks-r0drv-linux.c: Be a little bit more careful with EFLAGS in the context hook callouts.

File:

• trunk/src/VBox/Runtime/r0drv/linux/threadctxhooks-r0drv-linux.c (modified) (4 diffs)

trunk/src/VBox/Runtime/r0drv/linux/threadctxhooks-r0drv-linux.c

@@ -37 +37 @@
 #include <iprt/err.h>
 #include <iprt/asm.h>
+#if defined(RT_ARCH_AMD64) || defined(RT_ARCH_X86)
+# include <iprt/asm-amd64-x86.h>
+#endif
 #include "internal/thread.h"
 
@@ -85 +88 @@
 {
     PRTTHREADCTXINT pThis = RT_FROM_MEMBER(pPreemptNotifier, RTTHREADCTXINT, hPreemptNotifier);
+#if defined(RT_ARCH_AMD64) || defined(RT_ARCH_X86)
+    RTCCUINTREG fSavedEFlags = ASMGetFlags();
+    stac();
+#endif
+
     AssertPtr(pThis);
     AssertPtr(pThis->pfnThreadCtxHook);
@@ -90 +98 @@
     Assert(!RTThreadPreemptIsEnabled(NIL_RTTHREAD));
 
-    stac();
     pThis->pfnThreadCtxHook(RTTHREADCTXEVENT_PREEMPTING, pThis->pvUser);
+
+#if defined(RT_ARCH_AMD64) || defined(RT_ARCH_X86)
     clac();
+    ASMSetFlags(fSavedEFlags);
+#endif
 }
 
@@ -108 +119 @@
 {
     PRTTHREADCTXINT pThis = RT_FROM_MEMBER(pPreemptNotifier, RTTHREADCTXINT, hPreemptNotifier);
+#if defined(RT_ARCH_AMD64) || defined(RT_ARCH_X86)
+    RTCCUINTREG fSavedEFlags = ASMGetFlags();
+    stac();
+#endif
+
     AssertPtr(pThis);
     AssertPtr(pThis->pfnThreadCtxHook);
     Assert(pThis->fRegistered);
 
-    stac();
+
     pThis->pfnThreadCtxHook(RTTHREADCTXEVENT_RESUMED, pThis->pvUser);
+
+#if defined(RT_ARCH_AMD64) || defined(RT_ARCH_X86)
     clac();
+    ASMSetFlags(fSavedEFlags);
+#endif
 }