diff options
author | Paul Lemire <paul.lemire@kdab.com> | 2018-02-27 11:16:14 +0100 |
---|---|---|
committer | Paul Lemire <paul.lemire@kdab.com> | 2018-02-28 12:15:42 +0000 |
commit | 461194b0aa6b35e7dd1c9ff411806b065f8b01f7 (patch) | |
tree | 33e425d14f2a5c53d2f749c3bd19049a90593234 /src/render/backend | |
parent | bf0b3bdf5ab7c15bf72eced860053a611eccda4a (diff) |
RenderThread: fix races with texture and buffer
Add dependencies to ensure we won't allow the RenderThread to proceed if we
haven't completed the execution of the texture and buffer gatherer jobs.
Change-Id: Iaad67be893738b948bf26ca903f3bff7f16a1b88
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Diffstat (limited to 'src/render/backend')
-rw-r--r-- | src/render/backend/renderer_p.h | 3 | ||||
-rw-r--r-- | src/render/backend/renderviewbuilder.cpp | 5 |
2 files changed, 8 insertions, 0 deletions
diff --git a/src/render/backend/renderer_p.h b/src/render/backend/renderer_p.h index 00f908d28..af4d5c9de 100644 --- a/src/render/backend/renderer_p.h +++ b/src/render/backend/renderer_p.h @@ -210,6 +210,9 @@ public: inline SynchronizerJobPtr textureLoadSyncJob() const { return m_syncTextureLoadingJob; } inline UpdateSkinningPaletteJobPtr updateSkinningPaletteJob() const { return m_updateSkinningPaletteJob; } inline Qt3DCore::QAspectJobPtr shaderGathererJob() const { return m_shaderGathererJob; } + inline Qt3DCore::QAspectJobPtr bufferGathererJob() const { return m_bufferGathererJob; } + inline Qt3DCore::QAspectJobPtr textureGathererJob() const { return m_textureGathererJob; } + Qt3DCore::QAbstractFrameAdvanceService *frameAdvanceService() const override; void registerEventFilter(Qt3DCore::QEventFilterService *service) override; diff --git a/src/render/backend/renderviewbuilder.cpp b/src/render/backend/renderviewbuilder.cpp index 68503ba7e..2fd8e4acf 100644 --- a/src/render/backend/renderviewbuilder.cpp +++ b/src/render/backend/renderviewbuilder.cpp @@ -556,7 +556,12 @@ QVector<Qt3DCore::QAspectJobPtr> RenderViewBuilder::buildJobHierachy() const m_syncRenderCommandBuildingJob->addDependency(m_filterProximityJob); m_syncRenderCommandBuildingJob->addDependency(m_lightGathererJob); m_syncRenderCommandBuildingJob->addDependency(m_frustumCullingJob); + + // Ensure the RenderThread won't be able to process dirtyResources + // before they have been completely gathered m_syncRenderCommandBuildingJob->addDependency(m_renderer->shaderGathererJob()); + m_syncRenderCommandBuildingJob->addDependency(m_renderer->bufferGathererJob()); + m_syncRenderCommandBuildingJob->addDependency(m_renderer->textureGathererJob()); for (const auto &renderViewCommandBuilder : qAsConst(m_renderViewBuilderJobs)) { renderViewCommandBuilder->addDependency(m_syncRenderCommandBuildingJob); |