Changeset 41640 in vbox for trunk/src/VBox/Additions/common

Timestamp:

Jun 11, 2012 9:17:49 AM (13 years ago)

Author:

vboxsync

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

78440

Message:

Additions/common/VBoxGuest: add an internal IOCtl based on the current Windows one for installing a kernel mouse event handler.

Location:

trunk/src/VBox/Additions/common/VBoxGuest

Files:

• VBoxGuest.cpp (modified) (5 diffs)
• VBoxGuestInternal.h (modified) (1 diff)

trunk/src/VBox/Additions/common/VBoxGuest/VBoxGuest.cpp

@@ -727 +727 @@
         pDevExt->acMouseFeatureUsage[i] = 0;
     pDevExt->fMouseStatus = 0;
+    pDevExt->MouseSetNotifyCallback.pfnNotify = NULL;
+    pDevExt->MouseSetNotifyCallback.pvNotify = NULL;
+    pDevExt->cISR = 0;
 
     /*
@@ -1225 +1228 @@
     return VINF_SUCCESS;
 }
+
+
+#ifndef RT_OS_WINDOWS
+/**
+ * Set the callback for the kernel mouse handler.
+ *
+ * returns IPRT status code.
+ * @param   pDevExt         The device extension.
+ * @param   pNotify         The new callback information.
+ * @note  This function takes the session spinlock to update the callback
+ *        information, but the interrupt handler will not do this.  To make
+ *        sure that the interrupt handler sees a consistent structure, we
+ *        set the function pointer to NULL before updating the data and only
+ *        set it to the correct value once the data is updated.  Since the
+ *        interrupt handler executes atomically this ensures that the data is
+ *        valid if the function pointer is non-NULL.
+ */
+int VBoxGuestCommonIOCtl_SetMouseNotifyCallback(PVBOXGUESTDEVEXT pDevExt, VBoxGuestMouseSetNotifyCallback *pNotify)
+{
+    Log(("VBoxGuestCommonIOCtl: SET_MOUSE_NOTIFY_CALLBACK\n"));
+    RTSpinlockAcquire(pDevExt->SessionSpinlock);
+    ASMAtomicWriteNullPtr(&pDevExt->MouseSetNotifyCallback.pfnNotify);
+    pDevExt->MouseSetNotifyCallback.pvNotify = pNotify->pvNotify;
+    ASMAtomicWritePtr(&pDevExt->MouseSetNotifyCallback.pfnNotify,
+                      pNotify->pfnNotify);
+    RTSpinlockReleaseNoInts(pDevExt->SessionSpinlock);
+    /* Make sure no one is referencing the old data - hacky but should be
+     * effective. */
+    while (pDevExt->cISR > 0);
+    return VINF_SUCCESS;
+}
+#endif
 
 
@@ -2504 +2539 @@
                 break;
 
+#ifndef RT_OS_WINDOWS  /* Windows has its own implementation of this. */
+            case VBOXGUEST_IOCTL_INTERNAL_SET_MOUSE_NOTIFY_CALLBACK:
+                CHECKRET_RING0("SET_MOUSE_NOTIFY_CALLBACK");
+                CHECKRET_SIZE("SET_MOUSE_NOTIFY_CALLBACK",
+                           sizeof(VBoxGuestMouseSetNotifyCallback));
+                rc = VBoxGuestCommonIOCtl_SetMouseNotifyCallback(pDevExt, (VBoxGuestMouseSetNotifyCallback *)pvData);
+                break;
+#endif
+
             case VBOXGUEST_IOCTL_WAITEVENT:
                 CHECKRET_MIN_SIZE("WAITEVENT", sizeof(VBoxGuestWaitEventInfo));
@@ -2607 +2651 @@
 
     /*
-     * Enter the spinlock and check if it's our IRQ or not.
+     * Enter the spinlock, increase the ISR count and check if it's our IRQ or
+     * not.
      */
     RTSpinlockAcquire(pDevExt->EventSpinlock);
+    ASMAtomicIncU32(&pDevExt->cISR);
     fOurIrq = pDevExt->pVMMDevMemory->V.V1_04.fHaveEvents;
     if (fOurIrq)
@@ -2713 +2759 @@
         ASMAtomicIncU32(&pDevExt->u32MousePosChangedSeq);
         VBoxGuestNativeISRMousePollEvent(pDevExt);
-    }
-
+#ifndef RT_OS_WINDOWS
+        if (pDevExt->MouseSetNotifyCallback.pfnNotify)
+            pDevExt->MouseSetNotifyCallback.pfnNotify(pDevExt->MouseSetNotifyCallback.pvNotify);
+#endif
+    }
+
+    ASMAtomicDecU32(&pDevExt->cISR);
     Assert(rc == 0);
     return fOurIrq;

trunk/src/VBox/Additions/common/VBoxGuest/VBoxGuestInternal.h

@@ -167 +167 @@
      * together inside the session spinlock. */
     uint32_t volatile           fMouseStatus;
+    /** Callback and user data for a kernel mouse handler. */
+    VBoxGuestMouseSetNotifyCallback volatile MouseSetNotifyCallback;
+    /** Counter of number of active ISRs.  Currently used for safely removing
+     * the mouse handler callback. */
+    uint32_t volatile           cISR;
 
     /** Windows part. */