Changeset 96662 in vbox for trunk/src/VBox/Additions/WINNT/Graphics/Video

Timestamp:

Sep 8, 2022 9:22:45 PM (2 years ago)

Author:

vboxsync

Message:

Add/3D/mesa,WDDM: Don't release the presentation group reference at the end of NineDevice9_ctor, makes little sense to consume a reference for the pPresentationFactory parameter but not the pD3D9Ex one.

File:

• trunk/src/VBox/Additions/WINNT/Graphics/Video/disp/wddm/gallium/VBoxGaD3DDevice9Ex.cpp (modified) (1 diff)

trunk/src/VBox/Additions/WINNT/Graphics/Video/disp/wddm/gallium/VBoxGaD3DDevice9Ex.cpp

@@ -74 +74 @@
     if (SUCCEEDED(hr))
     {
+        /** @todo r=bird: The mpPresentationGroup parameter seems to always have been
+         * consumed by the NineDevice9_ctor() code, while for the mpD3D9Ex parameter
+         * it grabs a reference.  I've commented out the bogus looking
+         * ID3DPresentGroup_Release call from NineDevice9_ctor() rather than balancing
+         * reference to mpPresentionGroup here based on the hr value. See r153545.
+         * Please verify and rework the fix to your liking.
+         *
+         * (The release call in cleanup() would call into no man's land early during
+         * vlc.exe (v3.0.17.4) video startup on 32-bit w7 rtm.)
+         */
         hr = D3DAdapter9_CreateDeviceEx(mpD3D9Ex->GetAdapter9(),
                                         D3DADAPTER_DEFAULT, DeviceType,