Changeset 109165 in vbox for trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/wddm/gallium/VBoxMPGaWddm.cpp

Timestamp:

May 5, 2025 6:17:33 PM (4 days ago)

Author:

vboxsync

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

168723

Message:

WDDM: Miniport: unify handling of guest backing memory for surfaces, object tables, etc. bugref:10885

File:

• trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/wddm/gallium/VBoxMPGaWddm.cpp (modified) (6 diffs)

trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/wddm/gallium/VBoxMPGaWddm.cpp

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2016-2024 Oracle and/or its affiliates.
+ * Copyright (C) 2016-2025 Oracle and/or its affiliates.
  *
  * This file is part of VirtualBox base platform packages, as
@@ -183 +183 @@
     AssertReturn(pSvgaContext, STATUS_INSUFFICIENT_RESOURCES);
 
+    pSvgaContext->u32Cid = SVGA3D_INVALID_ID;
+    for (unsigned i = 0; i < RT_ELEMENTS(pSvgaContext->aCOT); ++i)
+        pSvgaContext->aCOT[i].mobid = SVGA3D_INVALID_ID;
     pSvgaContext->fDXContext = RT_BOOL(pInfo->u.vmsvga.u32Flags & VBOXWDDM_F_GA_CONTEXT_VGPU10);
 
@@ -236 +239 @@
         {
             PVMSVGACOT pCOT = &pSvgaContext->aCOT[i];
-            if (pCOT->pMob)
+            if (pCOT->mobid != SVGA3D_INVALID_ID)
             {
                 void *pvCmd = SvgaCmdBuf3dCmdReserve(pSvga, SVGA_3D_CMD_DX_SET_COTABLE, sizeof(SVGA3dCmdDXSetCOTable), SVGA3D_INVALID_ID);
@@ -257 +260 @@
 
                 uint32_t cbRequired = 0;
-                SvgaMobDestroy(pSvga, pCOT->pMob, NULL, 0, &cbRequired);
+                SvgaMobDestroy(pSvga, SVGA3D_INVALID_ID, NULL, 0, &cbRequired);
                 pvCmd = SvgaCmdBufReserve(pSvga, cbRequired, SVGA3D_INVALID_ID);
                 if (pvCmd)
                 {
-                    SvgaMobDestroy(pSvga, pCOT->pMob, pvCmd, cbRequired, &cbRequired);
+                    SvgaMobDestroy(pSvga, pCOT->mobid, pvCmd, cbRequired, &cbRequired);
                     SvgaCmdBufCommit(pSvga, cbRequired);
                 }
 
-                pCOT->pMob = NULL;
+                pCOT->mobid = SVGA3D_INVALID_ID;
             }
         }
@@ -1563 +1566 @@
 
     PVBOXWDDM_EXT_VMSVGA pSvga = (PVBOXWDDM_EXT_VMSVGA)Context;
-    if (pSvga->pMiniportMobData)
-    {
-        uint64_t const u64MobFence = ASMAtomicReadU64(&pSvga->pMiniportMobData->u64MobFence);
-
-        /* Move mobs which were deleted by the host to the local list under the lock. */
-        RTLISTANCHOR listDestroyedMobs;
-        RTListInit(&listDestroyedMobs);
-
-        KIRQL OldIrql;
-        SvgaHostObjectsLock(pSvga, &OldIrql);
-
-        PVMSVGAMOB pIter, pNext;
-        RTListForEachSafe(&pSvga->listMobDeferredDestruction, pIter, pNext, VMSVGAMOB, node)
-        {
-            if (gaFenceCmp64(pIter->u64MobFence, u64MobFence) <= 0)
-            {
-                RTListNodeRemove(&pIter->node);
-                RTListAppend(&listDestroyedMobs, &pIter->node);
-            }
-        }
-
-        SvgaHostObjectsUnlock(pSvga, OldIrql);
-
-        RTListForEachSafe(&listDestroyedMobs, pIter, pNext, VMSVGAMOB, node)
-        {
-            /* Delete the data. SvgaMobFree deallocates pIter. */
-            RTListNodeRemove(&pIter->node);
-            SvgaMobFree(pSvga, pIter);
-        }
-    }
+    SvgaDeferredMobDestruction(pSvga);
+
+    ASMAtomicDecS32(&pSvga->cQueuedWorkItems);
 }
 
@@ -1692 +1668 @@
         PIO_WORKITEM pWorkItem = IoAllocateWorkItem(pDevExt->pPDO);
         if (pWorkItem)
+        {
+            ASMAtomicIncS32(&pSvga->cQueuedWorkItems);
             IoQueueWorkItemEx(pWorkItem, dxDeferredMobDestruction, DelayedWorkQueue, pSvga);
+        }
     }
 }