Changeset 32915 in vbox for trunk/src/VBox/Additions/WINNT/Graphics/Miniport

Timestamp:

Oct 5, 2010 1:32:57 PM (14 years ago)

Author:

vboxsync

Message:

Additions/WINNT/Graphics: more refactorings

Location:

trunk/src/VBox/Additions/WINNT/Graphics/Miniport

Files:

• VBoxVideo.cpp (modified) (1 diff)
• VBoxVideo.h (modified) (1 diff)
• VBoxVideoHGSMI.cpp (modified) (5 diffs)
• wddm/VBoxVideoWddm.cpp (modified) (1 diff)

trunk/src/VBox/Additions/WINNT/Graphics/Miniport/VBoxVideo.cpp

@@ -2082 +2082 @@
                 }
                 /* clear the IRQ */
-                HGSMIClearIrq (PrimaryExtension);
+                HGSMIClearIrq (hgsmiFromDeviceExt(PrimaryExtension));
                 return TRUE;
             }

trunk/src/VBox/Additions/WINNT/Graphics/Miniport/VBoxVideo.h

@@ -916 +916 @@
     );
 
-void HGSMIClearIrq (PDEVICE_EXTENSION PrimaryExtension);
+void HGSMIClearIrq (PHGSMI_GUEST_INFO pInfo);
 
 #endif /* VBOX_WITH_HGSMI */

trunk/src/VBox/Additions/WINNT/Graphics/Miniport/VBoxVideoHGSMI.cpp

@@ -33 +33 @@
 #define MEM_TAG 'HVBV'
 
-void HGSMINotifyHostCmdComplete (PDEVICE_EXTENSION PrimaryExtension, HGSMIOFFSET offt)
-{
-    VBoxHGSMIHostWrite(hgsmiFromDeviceExt(PrimaryExtension), offt);
-}
-
-void HGSMIClearIrq (PDEVICE_EXTENSION PrimaryExtension)
-{
-    VBoxHGSMIHostWrite(hgsmiFromDeviceExt(PrimaryExtension), HGSMIOFFSET_VOID);
-}
-
-static void HGSMIHostCmdComplete (PDEVICE_EXTENSION PrimaryExtension, void * pvMem)
-{
-    HGSMIOFFSET offMem = HGSMIPointerToOffset (&hgsmiFromDeviceExt(PrimaryExtension)->areaHostHeap, HGSMIBufferHeaderFromData (pvMem));
+void HGSMINotifyHostCmdComplete (PHGSMI_GUEST_INFO pInfo, HGSMIOFFSET offt)
+{
+    VBoxHGSMIHostWrite(pInfo, offt);
+}
+
+void HGSMIClearIrq (PHGSMI_GUEST_INFO pInfo)
+{
+    VBoxHGSMIHostWrite(pInfo, HGSMIOFFSET_VOID);
+}
+
+static void HGSMIHostCmdComplete (PHGSMI_GUEST_INFO pInfo, void * pvMem)
+{
+    HGSMIOFFSET offMem = HGSMIPointerToOffset (&pInfo->areaHostHeap, HGSMIBufferHeaderFromData (pvMem));
     Assert(offMem != HGSMIOFFSET_VOID);
     if(offMem != HGSMIOFFSET_VOID)
     {
-        HGSMINotifyHostCmdComplete (PrimaryExtension, offMem);
+        HGSMINotifyHostCmdComplete (pInfo, offMem);
     }
 }
@@ -63 +63 @@
         /* failure means the command was not submitted to the handler for some reason
          * it's our responsibility to notify its completion in this case */
-        HGSMINotifyHostCmdComplete(PrimaryExtension, offBuffer);
+        HGSMINotifyHostCmdComplete(hgsmiFromDeviceExt(PrimaryExtension), offBuffer);
     }
     /* if the cmd succeeded it's responsibility of the callback to complete it */
@@ -1353 +1353 @@
 {
     PDEVICE_EXTENSION PrimaryExtension = ((PDEVICE_EXTENSION)hHGSMI)->pPrimary;
-    HGSMIHostCmdComplete (PrimaryExtension, pCmd);
+    HGSMIHostCmdComplete (hgsmiFromDeviceExt(PrimaryExtension), pCmd);
 }
 
@@ -1467 +1467 @@
                             VBVAHOSTCMD * pNext = pCur->u.pNext;
                             pCur->u.pNext = NULL;
-                            HGSMIHostCmdComplete(pCallbacks->PrimaryExtension, pCur);
+                            HGSMIHostCmdComplete(hgsmiFromDeviceExt(pCallbacks->PrimaryExtension), pCur);
                             pCur = pNext;
                             Assert(!pCur);
@@ -1536 +1536 @@
     }
     /* no handlers were found, need to complete the command here */
-    HGSMIHostCmdComplete(pCallbacks->PrimaryExtension, pvBuffer);
+    HGSMIHostCmdComplete(hgsmiFromDeviceExt(pCallbacks->PrimaryExtension), pvBuffer);
     return VINF_SUCCESS;
 }

trunk/src/VBox/Additions/WINNT/Graphics/Miniport/wddm/VBoxVideoWddm.cpp

@@ -896 +896 @@
         if (bOur)
         {
-            HGSMIClearIrq (pDevExt);
+            HGSMIClearIrq (hgsmiFromDeviceExt(pDevExt));
 #ifdef DEBUG_misha
             /* this is not entirely correct since host may concurrently complete some commands and raise a new IRQ while we are here,