commit 73cdafe72f417c1c3987eff31619f7b7114263eb
author Lingfeng Yang <lfy@google.com> Fri Jan 22 20:51:03 2021 -0800
committer Lingfeng Yang <lfy@google.com> Sat Jan 23 02:13:32 2021 -0800
tree fc0cdc621d2dbee7b0b53598950658e82ec64b2b
parent a5589280ea5323e884f007d9ce13f9d9b83827b1

[8/n] RenderThread: Consume Vulkan first

Bug: 177241396
Change-Id: I6c8afdbc129e43c95fc4113cc0be5f23762e9b81

stream-servers/RenderThread.cpp

diff --git a/stream-servers/RenderThread.cpp b/stream-servers/RenderThread.cpp
index ac6cd7c..84040ef 100644
--- a/stream-servers/RenderThread.cpp
+++ b/stream-servers/RenderThread.cpp
@@ -437,10 +437,6 @@
                 seqnoPtr = FrameBuffer::getFB()->getProcessSequenceNumberPtr(tInfo.m_puid);
             }
 
-            if (mRunInLimitedMode) {
-                sThreadRunLimiter.lock();
-            }
-
             progress = false;
             size_t last;
 
@@ -448,6 +444,8 @@
             // try to process some of the command buffer using the
             // Vulkan decoder
             //
+            // Note: It's risky to limit Vulkan decoding to one thread,
+            // so we do it outside the limiter
             {
                 last = tInfo.m_vkDec.decode(readBuf.buf(), readBuf.validData(),
                                             ioStream);
@@ -458,7 +456,7 @@
             }
 
             if (mRunInLimitedMode) {
-                sThreadRunLimiter.unlock();
+                sThreadRunLimiter.lock();
             }
 
             // try to process some of the command buffer using the GLESv1
@@ -516,6 +514,10 @@
                 }
             }
 
+            if (mRunInLimitedMode) {
+                sThreadRunLimiter.unlock();
+            }
+
         } while (progress);
     }