dawn-cmake/include/dawn/native/D3D12Backend.h

130 lines
4.8 KiB
C
Raw Permalink Normal View History

// Copyright 2018 The Dawn Authors
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
#ifndef INCLUDE_DAWN_NATIVE_D3D12BACKEND_H_
#define INCLUDE_DAWN_NATIVE_D3D12BACKEND_H_
#include <DXGI1_4.h>
#include <d3d12.h>
#include <windows.h>
#include <wrl/client.h>
#include <memory>
#include "dawn/dawn_wsi.h"
#include "dawn/native/DawnNative.h"
struct ID3D12Device;
struct ID3D12Resource;
namespace dawn::native::d3d12 {
class D3D11on12ResourceCache;
d3d12: Destroy ExternalImageDXGI resources on device destruction 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>
2022-07-13 11:33:51 +00:00
class Device;
class ExternalImageDXGIImpl;
DAWN_NATIVE_EXPORT Microsoft::WRL::ComPtr<ID3D12Device> GetD3D12Device(WGPUDevice device);
DAWN_NATIVE_EXPORT DawnSwapChainImplementation CreateNativeSwapChainImpl(WGPUDevice device,
HWND window);
DAWN_NATIVE_EXPORT WGPUTextureFormat
GetNativeSwapChainPreferredFormat(const DawnSwapChainImplementation* swapChain);
enum MemorySegment {
Local,
NonLocal,
};
DAWN_NATIVE_EXPORT uint64_t SetExternalMemoryReservation(WGPUDevice device,
uint64_t requestedReservationSize,
MemorySegment memorySegment);
struct DAWN_NATIVE_EXPORT ExternalImageDescriptorDXGISharedHandle : ExternalImageDescriptor {
public:
ExternalImageDescriptorDXGISharedHandle();
// Note: SharedHandle must be a handle to a texture object.
// TODO(dawn:576): Remove after changing Chromium code to set textureSharedHandle.
HANDLE sharedHandle = nullptr;
HANDLE textureSharedHandle = nullptr;
// Optional shared handle to a D3D11/12 fence which can be used to synchronize using wait/signal
// values specified in the access descriptor below. If null, the texture will be assumed to have
// an associated DXGI keyed mutex which will be used with a fixed key of 0 for synchronization.
HANDLE fenceSharedHandle = nullptr;
};
// Keyed mutex acquire/release uses a fixed key of 0 to match Chromium behavior.
constexpr UINT64 kDXGIKeyedMutexAcquireReleaseKey = 0;
struct DAWN_NATIVE_EXPORT ExternalImageAccessDescriptorDXGISharedHandle
: ExternalImageAccessDescriptor {
public:
// Value used for fence wait. A value of 0 is valid, but essentially a no-op since the fence
// lifetime starts with the 0 value signaled. A value of UINT64_MAX is ignored since it's also
// used by the D3D runtime to indicate that the device was removed.
uint64_t fenceWaitValue = 0;
// Value to signal the fence with after the texture is destroyed. A value of 0 means the fence
// will not be signaled.
uint64_t fenceSignalValue = 0;
// Whether the texture is for a WebGPU swap chain.
bool isSwapChainTexture = false;
};
// TODO(dawn:576): Remove after changing Chromium code to use the new struct name.
struct DAWN_NATIVE_EXPORT ExternalImageAccessDescriptorDXGIKeyedMutex
: ExternalImageAccessDescriptorDXGISharedHandle {
public:
// TODO(chromium:1241533): Remove deprecated keyed mutex params after removing associated
// code from Chromium - we use a fixed key of 0 for acquire and release everywhere now.
uint64_t acquireMutexKey;
uint64_t releaseMutexKey;
};
class DAWN_NATIVE_EXPORT ExternalImageDXGI {
public:
~ExternalImageDXGI();
static std::unique_ptr<ExternalImageDXGI> Create(
WGPUDevice device,
const ExternalImageDescriptorDXGISharedHandle* descriptor);
d3d12: Destroy ExternalImageDXGI resources on device destruction 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>
2022-07-13 11:33:51 +00:00
// Returns true if the external image resources are still valid, otherwise ProduceTexture() is
// guaranteed to fail e.g. after device destruction.
bool IsValid() const;
// TODO(sunnyps): |device| is ignored - remove after Chromium migrates to single parameter call.
WGPUTexture ProduceTexture(WGPUDevice device,
const ExternalImageAccessDescriptorDXGISharedHandle* descriptor);
d3d12: Destroy ExternalImageDXGI resources on device destruction 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>
2022-07-13 11:33:51 +00:00
WGPUTexture ProduceTexture(const ExternalImageAccessDescriptorDXGISharedHandle* descriptor);
private:
d3d12: Destroy ExternalImageDXGI resources on device destruction 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>
2022-07-13 11:33:51 +00:00
explicit ExternalImageDXGI(std::unique_ptr<ExternalImageDXGIImpl> impl);
std::unique_ptr<ExternalImageDXGIImpl> mImpl;
};
struct DAWN_NATIVE_EXPORT AdapterDiscoveryOptions : public AdapterDiscoveryOptionsBase {
AdapterDiscoveryOptions();
explicit AdapterDiscoveryOptions(Microsoft::WRL::ComPtr<IDXGIAdapter> adapter);
Microsoft::WRL::ComPtr<IDXGIAdapter> dxgiAdapter;
};
} // namespace dawn::native::d3d12
#endif // INCLUDE_DAWN_NATIVE_D3D12BACKEND_H_