commit 7317a448e5e30171659229b4797868ac89a5b4fc
author Austin Eng <enga@chromium.org> Mon Mar 18 22:59:24 2024 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Mar 18 22:59:24 2024 +0000
tree cdff50b5f97a3c216c5e4cd3d8098a87e91096c1
parent 46269650f56f65fd426141d5536d48e015a8c599

d3d11: Use passive command context to map pending buffers

CheckAndMapReadyBuffers always acquires the pending command context.
In "normal" mode, this always marked the context for needing submit,
resulting in necessary flushes, and the device to never idle.

Bug: dawn:2357
Change-Id: I7d310c916516992d9562928c946b2d6a623ee117
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/179840
Reviewed-by: Peng Huang <penghuang@chromium.org>
Commit-Queue: Austin Eng <enga@chromium.org>
Kokoro: Kokoro <noreply+kokoro@google.com>

src/dawn/native/d3d11/QueueD3D11.cpp

diff --git a/src/dawn/native/d3d11/QueueD3D11.cpp b/src/dawn/native/d3d11/QueueD3D11.cpp
index b39c17d..2689909 100644
--- a/src/dawn/native/d3d11/QueueD3D11.cpp
+++ b/src/dawn/native/d3d11/QueueD3D11.cpp
@@ -157,12 +157,11 @@
 }
 
 MaybeError Queue::CheckAndMapReadyBuffers(ExecutionSerial completedSerial) {
-    auto commandContext = GetScopedPendingCommandContext(QueueBase::SubmitMode::Normal);
+    auto commandContext = GetScopedPendingCommandContext(QueueBase::SubmitMode::Passive);
     for (auto buffer : mPendingMapBuffers.IterateUpTo(completedSerial)) {
         DAWN_TRY(buffer->FinalizeMap(&commandContext, completedSerial));
     }
     mPendingMapBuffers.ClearUpTo(completedSerial);
-
     return {};
 }