Changeset 99998 in vbox for trunk/src/VBox/Additions/WINNT

Timestamp:

May 29, 2023 12:30:01 PM (23 months ago)

Author:

vboxsync

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

157663

Message:

WDDM: take into account that DXGK aligns allocation size. bugref:9845

File:

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

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

@@ -342 +342 @@
     pAllocation->enmType = VBOXWDDM_ALLOC_TYPE_D3D;
     pAllocation->dx.desc = *(PVBOXDXALLOCATIONDESC)pAllocationInfo->pPrivateDriverData;
-    pAllocation->dx.desc.cbAllocation = RT_ALIGN_32(pAllocation->dx.desc.cbAllocation, PAGE_SIZE); /* DXGK expects it to be page aligned. */
+    pAllocation->dx.desc.cbAllocation = pAllocation->dx.desc.cbAllocation;
     pAllocation->dx.sid = SVGA3D_INVALID_ID;
     pAllocation->dx.mobid = SVGA3D_INVALID_ID;
@@ -653 +653 @@
     AssertReturn(pAllocation, STATUS_INVALID_PARAMETER);
 
-    SIZE_T cbAllocation = svgaGetAllocationSize(pAllocation);
+    /* "The size value is expanded to a multiple of the native host page size (for example, 4 KB on the x86 architecture)."
+     * I.e. TransferOffset and TransferSize are within the aligned size.
+     */
+    SIZE_T const cbAllocation = RT_ALIGN_32(svgaGetAllocationSize(pAllocation), PAGE_SIZE);
     AssertReturn(   pBuildPagingBuffer->Transfer.TransferOffset <= cbAllocation
                  && pBuildPagingBuffer->Transfer.TransferSize <= cbAllocation - pBuildPagingBuffer->Transfer.TransferOffset,