Updates to ensure that changes to maxSlot changes are passed through …

…the ViewDependentState.
vsg-dev · Oct 19, 2023 · fc3d1e5 · fc3d1e5
1 parent 073f63d
commit fc3d1e5
Show file tree

Hide file tree

Showing 7 changed files with 38 additions and 5 deletions.
diff --git a/include/vsg/state/ViewDependentState.h b/include/vsg/state/ViewDependentState.h
@@ -124,6 +124,7 @@ namespace vsg
         std::vector<std::pair<dmat4, const SpotLight*>> spotLights;
 
         virtual void init(ResourceRequirements& requirements);
+        virtual void update(ResourceRequirements& requirements);
 
         virtual void clear();
         virtual void bindDescriptorSets(CommandBuffer& commandBuffer, VkPipelineBindPoint pipelineBindPoint, VkPipelineLayout layout, uint32_t firstSet);

diff --git a/src/vsg/app/CommandGraph.cpp b/src/vsg/app/CommandGraph.cpp
@@ -12,6 +12,8 @@ THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLI
 
 #include <vsg/app/CommandGraph.h>
 #include <vsg/app/RenderGraph.h>
+#include <vsg/app/View.h>
+#include <vsg/state/ViewDependentState.h>
 #include <vsg/io/DatabasePager.h>
 #include <vsg/ui/ApplicationEvent.h>
 #include <vsg/vk/State.h>

diff --git a/src/vsg/app/CompileManager.cpp b/src/vsg/app/CompileManager.cpp
@@ -16,6 +16,7 @@ THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLI
 #include <vsg/core/Exception.h>
 #include <vsg/io/Logger.h>
 #include <vsg/io/Options.h>
+#include <vsg/state/ViewDependentState.h>
 
 using namespace vsg;
 
@@ -175,14 +176,22 @@ CompileResult CompileManager::compile(ref_ptr<Object> object, ContextSelectionFu
             for (auto& context : compileTraversal->contexts)
             {
                 ref_ptr<View> view = context->view;
-                if (view && !viewDetailsStack.empty())
+
+                if (view)
                 {
-                    if (auto itr = result.views.find(view.get()); itr == result.views.end())
+                    if (view->viewDependentState)
                     {
-                        result.views[view] = viewDetailsStack.top();
+                        view->viewDependentState->update(requirements);
                     }
-                }
 
+                    if (!viewDetailsStack.empty())
+                    {
+                        if (auto itr = result.views.find(view.get()); itr == result.views.end())
+                        {
+                            result.views[view] = viewDetailsStack.top();
+                        }
+                    }
+                }
                 context->reserve(requirements);
             }
 

diff --git a/src/vsg/app/CompileTraversal.cpp b/src/vsg/app/CompileTraversal.cpp
@@ -275,6 +275,14 @@ void CompileTraversal::apply(CommandGraph& commandGraph)
         }
     };
 
+    for(auto& context : contexts)
+    {
+        if (context->resourceRequirements.maxSlot > commandGraph.maxSlot)
+        {
+            commandGraph.maxSlot = context->resourceRequirements.maxSlot;
+        }
+    }
+
     if (commandGraph.framebuffer)
     {
         traverseRenderedSubgraph(commandGraph.framebuffer->getRenderPass(), commandGraph.framebuffer->extent2D());

diff --git a/src/vsg/app/RecordAndSubmitTask.cpp b/src/vsg/app/RecordAndSubmitTask.cpp
@@ -202,7 +202,7 @@ VkResult RecordAndSubmitTask::finish(ref_ptr<RecordedCommandBuffers> recordedCom
 
 void vsg::updateTasks(RecordAndSubmitTasks& tasks, ref_ptr<CompileManager> compileManager, const CompileResult& compileResult)
 {
-    //info("vsg::updateTasks(RecordAndSubmitTasks& tasks..) ");
+    //info("vsg::updateTasks(RecordAndSubmitTasks& tasks..) compileResult.maxSlot = ", compileResult.maxSlot);
     if (compileResult.earlyDynamicData || compileResult.lateDynamicData)
     {
         for (auto& task : tasks)

diff --git a/src/vsg/state/ViewDependentState.cpp b/src/vsg/state/ViewDependentState.cpp
@@ -365,6 +365,14 @@ void ViewDependentState::init(ResourceRequirements& requirements)
     }
 }
 
+void ViewDependentState::update(ResourceRequirements& requirements)
+{
+    if (preRenderCommandGraph && requirements.maxSlot > preRenderCommandGraph->maxSlot)
+    {
+        preRenderCommandGraph->maxSlot = requirements.maxSlot;
+    }
+}
+
 void ViewDependentState::compile(Context& context)
 {
     descriptorSet->compile(context);

diff --git a/src/vsg/vk/Context.cpp b/src/vsg/vk/Context.cpp
@@ -205,6 +205,11 @@ ref_ptr<DescriptorSet::Implementation> Context::allocateDescriptorSet(Descriptor
 
 void Context::reserve(const ResourceRequirements& requirements)
 {
+    if (requirements.maxSlot > resourceRequirements.maxSlot)
+    {
+        resourceRequirements.maxSlot = requirements.maxSlot;
+    }
+
     auto maxSets = requirements.computeNumDescriptorSets();
     auto descriptorPoolSizes = requirements.computeDescriptorPoolSizes();