From: Eunki, Hong <eunkiki.hong@samsung.com>
Date: Wed, 18 Oct 2023 08:03:45 +0000 (+0900)
Subject: Revert "Remove unused program / shader caches"
X-Git-Tag: dali_2.2.49~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7c327cc945f101f1d50ad51670a6680b8f0e40ce;p=platform%2Fcore%2Fuifw%2Fdali-core.git

Revert "Remove unused program / shader caches"

This patch make some unusal bugs relative with shader.
Until resolve that issue, let we just revert this logic for now.

Change-Id: I17bea478ea4b401f009b1d01cfb90b5e67bcc793
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
---

diff --git a/dali/internal/render/common/render-manager.cpp b/dali/internal/render/common/render-manager.cpp
index b0ec223..1bc9be3 100644
--- a/dali/internal/render/common/render-manager.cpp
+++ b/dali/internal/render/common/render-manager.cpp
@@ -64,7 +64,8 @@ Debug::Filter* gLogFilter = Debug::Filter::New(Debug::NoLogging, false, "LOG_REN
 
 namespace
 {
-constexpr uint32_t CACHE_CLEAN_FRAME_COUNT = 600u; // 60fps * 10sec
+// TODO : Cache clean logic have some problem now. Just block it until bug resolved
+//constexpr uint32_t CACHE_CLEAN_FRAME_COUNT = 600u; // 60fps * 10sec
 
 inline Graphics::Rect2D RecalculateScissorArea(const Graphics::Rect2D& scissorArea, int orientation, const Rect<int32_t>& viewportRect)
 {
@@ -518,11 +519,14 @@ void RenderManager::PreRender(Integration::RenderStatus& status, bool forceClear
   mImpl->pipelineCache->PreRender();
 
   // Let we collect reference counts during CACHE_CLEAN_FRAME_COUNT frames.
+  // TODO : Cache clean logic have some problem now. Just block it until bug resolved
+  /*
   if(mImpl->frameCount % CACHE_CLEAN_FRAME_COUNT == 1)
   {
     mImpl->programController.ResetReferenceCount();
     mImpl->shaderCache.ResetReferenceCount();
   }
+  */
 
   mImpl->commandBufferSubmitted = false;
 }
@@ -1155,11 +1159,14 @@ void RenderManager::PostRender()
   }
 
   // Remove unused shader and programs during CACHE_CLEAN_FRAME_COUNT frames.
+  // TODO : Cache clean logic have some problem now. Just block it until bug resolved
+  /*
   if(mImpl->frameCount % CACHE_CLEAN_FRAME_COUNT == 0)
   {
     mImpl->programController.ClearUnusedCache();
     mImpl->shaderCache.ClearUnusedCache();
   }
+  */
 
   const bool haveInstructions = count > 0u;