7ae5c41412
D3D12 objects can have implicit dependencies on device resources that are not captured by holding ComPtrs: "Direct3D 12 uses COM-style reference counting only for the lifetimes of interfaces (by using the weak reference model of Direct3D tied to the lifetime of the device). All resource and description memory lifetimes are the sole responsibly of the app to maintain for the proper duration, and are not reference counted. Direct3D 11 uses reference counting to manage the lifetimes of interface dependencies as well." Source: https://docs.microsoft.com/en-us/windows/win32/direct3d12/important-changes-from-directx-11-to-directx-12 ExternalImageDXGI can outlive the device it was created on e.g. the D3D shared image backing holds on to the ExternalImageDXGI for its lifetime. ExternalImageDXGI destructor can invoke code that depends on D3D12 resources that might have already been destroyed. In particular, this shows up as ComPtr::Release for ID3D12Fence crashing mysteriously, and is also speculated as the cause for a racy invalid function pointer dereference in crbug.com/1338470. This CL makes the D3D12 backend device destroy the ExternalImageDXGI's resources on device destruction making it effectively a weak pointer. This unblocks landing https://crrev.com/c/3700811 and hopefully fixes crbug.com/1338470 as well. This CL also deprecates unnecessary WGPUDevice param to ProduceTexture, and adds an IsValid() method so that the shared image can check it and decide to recreate the ExternalImageDXGI if needed. Bug: dawn:576, chromium:1338470 Change-Id: I2122cf807587cf3b1218ba29ea291263df0cf698 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/95860 Kokoro: Kokoro <noreply+kokoro@google.com> Auto-Submit: Sunny Sachanandani <sunnyps@chromium.org> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Corentin Wallez <cwallez@chromium.org> |
||
|---|---|---|
| .. | ||
| dawn | ||
| tint | ||
| webgpu | ||