commit 29d712f9de99c646bbebd97cbe55ad5d81486612
author Natasha Lee <natlee@microsoft.com> Mon Mar 30 11:37:44 2020 +0000
committer Commit Bot service account <commit-bot@chromium.org> Mon Mar 30 11:37:44 2020 +0000
tree edb0258c0b964d94ca26ca00d7ee3eaffaf1b4f0
parent 81431f5034b4ce3a0cdc339709bd8ca8856fe427

D3D12 Immediately release pending commands

We want to release pending commands in Device::WaitForIdleForDestruction()
so that when we call TickImpl(), we can reset the command allocators.

Bug: dawn:269
Change-Id: Ibd8fdd685a4e9ed7cce8176ba097bc67687bcd97
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/18101
Reviewed-by: Austin Eng <enga@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>

src/dawn_native/d3d12/DeviceD3D12.cpp

diff --git a/src/dawn_native/d3d12/DeviceD3D12.cpp b/src/dawn_native/d3d12/DeviceD3D12.cpp
index e9fdf87..efa3837 100644
--- a/src/dawn_native/d3d12/DeviceD3D12.cpp
+++ b/src/dawn_native/d3d12/DeviceD3D12.cpp
@@ -416,6 +416,9 @@
     }
 
     MaybeError Device::WaitForIdleForDestruction() {
+        // Immediately forget about all pending commands
+        mPendingCommands.Release();
+
         DAWN_TRY(NextSerial());
         // Wait for all in-flight commands to finish executing
         DAWN_TRY(WaitForSerial(mLastSubmittedSerial));