commit 69ba5fbca101d4a4797aa4040e0edab5041bb276
author Kaiyi Li <kaiyili@google.com> Thu Oct 20 17:05:23 2022 -0700
committer Kaiyi Li <kaiyili@google.com> Thu Oct 20 18:39:07 2022 -0700
tree d89346d4f818746bf31f31990d5f4456993fa849
parent a5a825053287c54421ed182e0b72e5240a203dab

vulkan: don't use seqno if ProcessResources is not found

On guest thread initialization, RenderThread may receive packets before
PUID is set. e.g. feature querying commands. And that's valid.

Change-Id: I068dba9355799ed06ccdbc09f06748c50c63d4d3

stream-servers/RenderThread.cpp

diff --git a/stream-servers/RenderThread.cpp b/stream-servers/RenderThread.cpp
index bc3090a..94d9c2d 100644
--- a/stream-servers/RenderThread.cpp
+++ b/stream-servers/RenderThread.cpp
@@ -457,10 +457,17 @@
                     .healthMonitor = &FrameBuffer::getFB()->getHealthMonitor(),
                     .metricsLogger = &metricsLogger,
                 };
+                uint32_t* seqno = nullptr;
+                if (processResources) {
+                    seqno = processResources->getSequenceNumberPtr();
+                }
                 last = tInfo.m_vkInfo->m_vkDec.decode(readBuf.buf(), readBuf.validData(), ioStream,
-                                                      processResources->getSequenceNumberPtr(),
-                                                      context);
+                                                      seqno, context);
                 if (last > 0) {
+                    if (!processResources) {
+                        ERR("Processed some Vulkan packets without process resources created. "
+                            "That's problematic.");
+                    }
                     readBuf.consume(last);
                     progress = true;
                 }