Changeset 32876 in vbox for trunk/src/VBox/Additions/WINNT/Graphics/Miniport/VBoxVideoHGSMI.cpp

Timestamp:

Oct 1, 2010 8:38:01 PM (14 years ago)

Author:

vboxsync

Message:

Additions/WINNT/VBoxVideo: start refactoring the HGSMI code

File:

• trunk/src/VBox/Additions/WINNT/Graphics/Miniport/VBoxVideoHGSMI.cpp (modified) (25 diffs)

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

@@ -35 +35 @@
 void HGSMINotifyHostCmdComplete (PDEVICE_EXTENSION PrimaryExtension, HGSMIOFFSET offt)
 {
-    VBoxHGSMIHostWrite(PrimaryExtension, offt);
+    VBoxHGSMIHostWrite(hgsmiFromDeviceExt(PrimaryExtension), offt);
 }
 
 void HGSMIClearIrq (PDEVICE_EXTENSION PrimaryExtension)
 {
-    VBoxHGSMIHostWrite(PrimaryExtension, HGSMIOFFSET_VOID);
+    VBoxHGSMIHostWrite(hgsmiFromDeviceExt(PrimaryExtension), HGSMIOFFSET_VOID);
 }
 
 static void HGSMIHostCmdComplete (PDEVICE_EXTENSION PrimaryExtension, void * pvMem)
 {
-    HGSMIOFFSET offMem = HGSMIPointerToOffset (&PrimaryExtension->u.primary.areaHostHeap, HGSMIBufferHeaderFromData (pvMem));
+    HGSMIOFFSET offMem = HGSMIPointerToOffset (&hgsmiFromDeviceExt(PrimaryExtension)->areaHostHeap, HGSMIBufferHeaderFromData (pvMem));
     Assert(offMem != HGSMIOFFSET_VOID);
     if(offMem != HGSMIOFFSET_VOID)
@@ -55 +55 @@
 static void hgsmiHostCmdProcess(PDEVICE_EXTENSION PrimaryExtension, HGSMIOFFSET offBuffer)
 {
-    int rc = HGSMIBufferProcess (&PrimaryExtension->u.primary.areaHostHeap,
-                                &PrimaryExtension->u.primary.channels,
+    int rc = HGSMIBufferProcess (&hgsmiFromDeviceExt(PrimaryExtension)->areaHostHeap,
+                                &hgsmiFromDeviceExt(PrimaryExtension)->channels,
                                 offBuffer);
     Assert(!RT_FAILURE(rc));
@@ -70 +70 @@
 static HGSMIOFFSET hgsmiGetHostBuffer (PDEVICE_EXTENSION PrimaryExtension)
 {
-    return VBoxHGSMIHostRead(PrimaryExtension);
+    return VBoxHGSMIHostRead(hgsmiFromDeviceExt(PrimaryExtension));
 }
 
@@ -190 +190 @@
     /* @todo: add synchronization */
 #endif
-    return HGSMIHeapAlloc (&PrimaryExtension->u.primary.hgsmiAdapterHeap, cbData, u8Ch, u16Op);
+    return HGSMIHeapAlloc (&hgsmiFromDeviceExt(PrimaryExtension)->hgsmiAdapterHeap, cbData, u8Ch, u16Op);
 }
 
@@ -198 +198 @@
     /* @todo: add synchronization */
 #endif
-    HGSMIHeapFree (&PrimaryExtension->u.primary.hgsmiAdapterHeap, pvBuffer);
+    HGSMIHeapFree (&hgsmiFromDeviceExt(PrimaryExtension)->hgsmiAdapterHeap, pvBuffer);
 }
 
@@ -204 +204 @@
 {
     /* Initialize the buffer and get the offset for port IO. */
-    HGSMIOFFSET offBuffer = HGSMIHeapBufferOffset (&PrimaryExtension->u.primary.hgsmiAdapterHeap, pvBuffer);
+    HGSMIOFFSET offBuffer = HGSMIHeapBufferOffset (&hgsmiFromDeviceExt(PrimaryExtension)->hgsmiAdapterHeap, pvBuffer);
 
     Assert(offBuffer != HGSMIOFFSET_VOID);
@@ -210 +210 @@
     {
         /* Submit the buffer to the host. */
-        VBoxHGSMIGuestWrite(PrimaryExtension, offBuffer);
+        VBoxHGSMIGuestWrite(hgsmiFromDeviceExt(PrimaryExtension), offBuffer);
         return VINF_SUCCESS;
     }
@@ -236 +236 @@
 #endif
 
-    void *p = HGSMIHeapAlloc (&PrimaryExtension->u.primary.hgsmiAdapterHeap, cbData, u8Ch, u16Op);
+    void *p = HGSMIHeapAlloc (&hgsmiFromDeviceExt(PrimaryExtension)->hgsmiAdapterHeap, cbData, u8Ch, u16Op);
 
     if (!p)
@@ -253 +253 @@
         {
             /* Initialize the buffer and get the offset for port IO. */
-            HGSMIOFFSET offBuffer = HGSMIHeapBufferOffset (&PrimaryExtension->u.primary.hgsmiAdapterHeap,
+            HGSMIOFFSET offBuffer = HGSMIHeapBufferOffset (&hgsmiFromDeviceExt(PrimaryExtension)->hgsmiAdapterHeap,
                                                            p);
 
             /* Submit the buffer to the host. */
-            VBoxHGSMIGuestWrite(PrimaryExtension, offBuffer);
+            VBoxHGSMIGuestWrite(hgsmiFromDeviceExt(PrimaryExtension), offBuffer);
 
             if (pfnFinalize)
@@ -271 +271 @@
 
         /* Free the IO buffer. */
-        HGSMIHeapFree (&PrimaryExtension->u.primary.hgsmiAdapterHeap, p);
+        HGSMIHeapFree (&hgsmiFromDeviceExt(PrimaryExtension)->hgsmiAdapterHeap, p);
     }
 
@@ -743 +743 @@
     PrimaryExtension->u.primary.pHostFlags               = NULL;
     PrimaryExtension->u.primary.ulMaxFrameBufferSize     = 0;
-    PrimaryExtension->u.primary.bHGSMI                   = VBoxHGSMIIsSupported (PrimaryExtension);
-    VBoxVideoCmnMemZero(&PrimaryExtension->u.primary.areaHostHeap, sizeof(HGSMIAREA));
+    hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI         = VBoxHGSMIIsSupported (PrimaryExtension);
+    VBoxVideoCmnMemZero(&hgsmiFromDeviceExt(PrimaryExtension)->areaHostHeap, sizeof(HGSMIAREA));
     VBoxVideoCmnMemZero(&PrimaryExtension->areaDisplay, sizeof(HGSMIAREA));
 
-    if (PrimaryExtension->u.primary.IOPortGuest == 0)
-    {
-        PrimaryExtension->u.primary.bHGSMI = false;
-    }
-
-    if (PrimaryExtension->u.primary.bHGSMI)
+    if (hgsmiFromDeviceExt(PrimaryExtension)->IOPortGuest == 0)
+    {
+        hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI = false;
+    }
+
+    if (hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI)
     {
         /* Map the adapter information. It will be needed for HGSMI IO. */
@@ -765 +765 @@
                      rc));
 
-            PrimaryExtension->u.primary.bHGSMI = FALSE;
+            hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI = FALSE;
         }
         else
         {
             /* Setup a HGSMI heap within the adapter information area. */
-            rc = HGSMIHeapSetup (&PrimaryExtension->u.primary.hgsmiAdapterHeap,
+            rc = HGSMIHeapSetup (&hgsmiFromDeviceExt(PrimaryExtension)->hgsmiAdapterHeap,
                                  PrimaryExtension->u.primary.pvAdapterInformation,
                                  VBVA_ADAPTER_INFORMATION_SIZE - sizeof(HGSMIHOSTFLAGS),
@@ -781 +781 @@
                          rc));
 
-                PrimaryExtension->u.primary.bHGSMI = FALSE;
+                hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI = FALSE;
             }
             else
@@ -792 +792 @@
 
     /* Setup the host heap and the adapter memory. */
-    if (PrimaryExtension->u.primary.bHGSMI)
+    if (hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI)
     {
         /* The miniport heap is used for the host buffers. */
@@ -836 +836 @@
                 PrimaryExtension->u.primary.pvMiniportHeap = NULL;
                 PrimaryExtension->u.primary.cbMiniportHeap = 0;
-                PrimaryExtension->u.primary.bHGSMI = FALSE;
+                hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI = FALSE;
             }
             else
@@ -845 +845 @@
 
                 /* Init the host hap area. Buffers from the host will be placed there. */
-                HGSMIAreaInitialize (&PrimaryExtension->u.primary.areaHostHeap,
+                HGSMIAreaInitialize (&hgsmiFromDeviceExt(PrimaryExtension)->areaHostHeap,
                                      PrimaryExtension->u.primary.pvMiniportHeap,
                                      PrimaryExtension->u.primary.cbMiniportHeap,
@@ -860 +860 @@
 
     /* Check whether the guest supports multimonitors. */
-    if (PrimaryExtension->u.primary.bHGSMI)
+    if (hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI)
     {
 #ifndef VBOX_WITH_WDDM
@@ -939 +939 @@
 #endif
 
-    if (PrimaryExtension->u.primary.bHGSMI)
+    if (hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI)
     {
         /* Setup the information for the host. */
@@ -946 +946 @@
         if (RT_FAILURE (rc))
         {
-            PrimaryExtension->u.primary.bHGSMI = FALSE;
+            hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI = FALSE;
         }
     }
 
 #ifdef VBOX_WITH_WDDM
-    if (PrimaryExtension->u.primary.bHGSMI)
+    if (hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI)
     {
         ULONG ulAvailable = PrimaryExtension->u.primary.cbVRAM
@@ -1040 +1040 @@
 
         if (RT_FAILURE(rc))
-            PrimaryExtension->u.primary.bHGSMI = FALSE;
-    }
-#endif
-
-    if (!PrimaryExtension->u.primary.bHGSMI)
+            hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI = FALSE;
+    }
+#endif
+
+    if (!hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI)
     {
         /* Unmap the memory if VBoxVideo is not supported. */
@@ -1050 +1050 @@
         VBoxUnmapAdapterInformation (PrimaryExtension);
 
-        HGSMIHeapDestroy (&PrimaryExtension->u.primary.hgsmiAdapterHeap);
-    }
-
-    if (PrimaryExtension->u.primary.bHGSMI)
+        HGSMIHeapDestroy (&hgsmiFromDeviceExt(PrimaryExtension)->hgsmiAdapterHeap);
+    }
+
+    if (hgsmiFromDeviceExt(PrimaryExtension)->bHGSMI)
     {
         VBoxVideoCmnSpinLockCreate(PrimaryExtension, &PrimaryExtension->u.primary.pSynchLock);
@@ -1099 +1099 @@
 */
             {
-                HGSMIHeapDestroy(&PrimaryExtension->u.primary.hgsmiAdapterHeap);
+                HGSMIHeapDestroy(&hgsmiFromDeviceExt(PrimaryExtension)->hgsmiAdapterHeap);
 
                 /* Map the adapter information. It will be needed for HGSMI IO. */
@@ -1369 +1369 @@
     VBoxVideoHGSMIDpc(PrimaryExtension, NULL);
 
-    HGSMICHANNEL * pChannel = HGSMIChannelFindById (&PrimaryExtension->u.primary.channels, u8Channel);
+    HGSMICHANNEL * pChannel = HGSMIChannelFindById (&hgsmiFromDeviceExt(PrimaryExtension)->channels, u8Channel);
     if(pChannel)
     {
@@ -1547 +1547 @@
 {
     VBVA_CHANNELCONTEXTS * pContexts;
-    HGSMICHANNEL * pChannel = HGSMIChannelFindById (&PrimaryExtension->u.primary.channels, u8Channel);
+    HGSMICHANNEL * pChannel = HGSMIChannelFindById (&hgsmiFromDeviceExt(PrimaryExtension)->channels, u8Channel);
     if(!pChannel)
     {
@@ -1584 +1584 @@
             if(!pChannel)
             {
-                rc = HGSMIChannelRegister (&PrimaryExtension->u.primary.channels,
+                rc = HGSMIChannelRegister (&hgsmiFromDeviceExt(PrimaryExtension)->channels,
                                            u8Channel,
                                            "VGA Miniport HGSMI channel",