From b1f27aae89a9da9fbf3cb15a47f1a401db5a7974 Mon Sep 17 00:00:00 2001 From: Derek Sollenberger Date: Mon, 2 Apr 2018 13:36:45 -0400 Subject: Free up all scratch resources when the app's UI is hidden Bug: 73808481 Test: hwui_unit_tests Change-Id: I9f191c776a936c1be40702ff0924c7ad054526d5 --- libs/hwui/renderthread/CacheManager.cpp | 12 ++++++++++-- libs/hwui/tests/unit/CacheManagerTests.cpp | 14 ++++++++++++-- 2 files changed, 22 insertions(+), 4 deletions(-) diff --git a/libs/hwui/renderthread/CacheManager.cpp b/libs/hwui/renderthread/CacheManager.cpp index 32d4c77c5fc3..3ca92953e5f7 100644 --- a/libs/hwui/renderthread/CacheManager.cpp +++ b/libs/hwui/renderthread/CacheManager.cpp @@ -151,7 +151,12 @@ void CacheManager::trimMemory(TrimMemoryMode mode) { mGrContext->freeGpuResources(); break; case TrimMemoryMode::UiHidden: - mGrContext->purgeUnlockedResources(mMaxResourceBytes - mBackgroundResourceBytes, true); + // Here we purge all the unlocked scratch resources and then toggle the resources cache + // limits between the background and max amounts. This causes the unlocked resources + // that have persistent data to be purged in LRU order. + mGrContext->purgeUnlockedResources(true); + mGrContext->setResourceCacheLimits(mMaxResources, mBackgroundResourceBytes); + mGrContext->setResourceCacheLimits(mMaxResources, mMaxResourceBytes); break; } } @@ -161,7 +166,10 @@ void CacheManager::trimStaleResources() { return; } mGrContext->flush(); - mGrContext->purgeResourcesNotUsedInMs(std::chrono::seconds(30)); + // Here we purge all the unlocked scratch resources (leaving those resources w/ persistent data) + // and then purge those w/ persistent data based on age. + mGrContext->purgeUnlockedResources(true); + mGrContext->purgeResourcesNotUsedInMs(std::chrono::seconds(10)); } sp CacheManager::acquireVectorDrawableAtlas() { diff --git a/libs/hwui/tests/unit/CacheManagerTests.cpp b/libs/hwui/tests/unit/CacheManagerTests.cpp index b1106f0d6a71..c235715073f5 100644 --- a/libs/hwui/tests/unit/CacheManagerTests.cpp +++ b/libs/hwui/tests/unit/CacheManagerTests.cpp @@ -20,6 +20,8 @@ #include "renderthread/EglManager.h" #include "tests/common/TestUtils.h" +#include + using namespace android; using namespace android::uirenderer; using namespace android::uirenderer::renderthread; @@ -49,7 +51,12 @@ RENDERTHREAD_SKIA_PIPELINE_TEST(CacheManager, trimMemory) { surfaces.push_back(surface); } - ASSERT_TRUE(1 < surfaces.size()); + // create an image and pin it so that we have something with a unique key in the cache + sk_sp bitmap = + Bitmap::allocateHeapBitmap(SkImageInfo::MakeA8(displayInfo.w, displayInfo.h)); + sk_sp filter; + sk_sp image = bitmap->makeImage(&filter); + ASSERT_TRUE(SkImage_pinAsTexture(image.get(), grContext)); // attempt to trim all memory while we still hold strong refs renderThread.cacheManager().trimMemory(CacheManager::TrimMemoryMode::Complete); @@ -61,11 +68,14 @@ RENDERTHREAD_SKIA_PIPELINE_TEST(CacheManager, trimMemory) { surfaces[i].reset(); } + // unpin the image which should add a unique purgeable key to the cache + SkImage_unpinAsTexture(image.get(), grContext); + // verify that we have enough purgeable bytes const size_t purgeableBytes = grContext->getResourceCachePurgeableBytes(); ASSERT_TRUE(renderThread.cacheManager().getBackgroundCacheSize() < purgeableBytes); - // UI hidden and make sure only some got purged + // UI hidden and make sure only some got purged (unique should remain) renderThread.cacheManager().trimMemory(CacheManager::TrimMemoryMode::UiHidden); ASSERT_TRUE(0 < grContext->getResourceCachePurgeableBytes()); ASSERT_TRUE(renderThread.cacheManager().getBackgroundCacheSize() > getCacheUsage(grContext)); -- cgit v1.2.3-59-g8ed1b