blob: 165827323f3428a971b04b2d4e7fc2b7f0f3b21e [file] [log] [blame]
Robert Phillips5af44de2017-07-18 14:49:38 -04001/*
2 * Copyright 2017 Google Inc.
3 *
4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file.
6 */
7
Robert Phillipse19babf2020-04-06 13:57:30 -04008#include "include/core/SkCanvas.h"
9#include "include/core/SkSurface.h"
Robert Phillips6d344c32020-07-06 10:56:46 -040010#include "include/gpu/GrDirectContext.h"
Mike Kleinc0bd9f92019-04-23 12:05:21 -050011#include "src/gpu/GrContextPriv.h"
Mike Kleinc0bd9f92019-04-23 12:05:21 -050012#include "src/gpu/GrGpu.h"
13#include "src/gpu/GrProxyProvider.h"
14#include "src/gpu/GrResourceAllocator.h"
15#include "src/gpu/GrResourceProvider.h"
16#include "src/gpu/GrSurfaceProxyPriv.h"
Greg Daniel456f9b52020-03-05 19:14:18 +000017#include "src/gpu/GrTexture.h"
Greg Danielf91aeb22019-06-18 09:58:02 -040018#include "src/gpu/GrTextureProxy.h"
Robert Phillipse19babf2020-04-06 13:57:30 -040019#include "tests/Test.h"
Greg Danielc1ad77c2020-05-06 11:40:03 -040020#include "tests/TestUtils.h"
Robert Phillips1734dd32018-08-21 13:52:09 -040021
Robert Phillips57aa3672017-07-21 11:38:13 -040022struct ProxyParams {
23 int fSize;
Brian Salomonf2c2ba92019-07-17 09:59:59 -040024 GrRenderable fRenderable;
Robert Phillipsc80b0e92019-07-23 10:27:09 -040025 GrColorType fColorType;
Robert Phillips57aa3672017-07-21 11:38:13 -040026 SkBackingFit fFit;
27 int fSampleCnt;
Robert Phillipsc476e5d2019-03-26 14:50:08 -040028 SkBudgeted fBudgeted;
Robert Phillips57aa3672017-07-21 11:38:13 -040029 // TODO: do we care about mipmapping
30};
31
Robert Phillips3d4cac52019-06-11 08:08:08 -040032static sk_sp<GrSurfaceProxy> make_deferred(GrProxyProvider* proxyProvider, const GrCaps* caps,
33 const ProxyParams& p) {
Robert Phillips0a15cc62019-07-30 12:49:10 -040034 const GrBackendFormat format = caps->getDefaultBackendFormat(p.fColorType, p.fRenderable);
Brian Salomondf1bd6d2020-03-26 20:37:01 -040035 return proxyProvider->createProxy(format, {p.fSize, p.fSize}, p.fRenderable, p.fSampleCnt,
36 GrMipMapped::kNo, p.fFit, p.fBudgeted, GrProtected::kNo);
Robert Phillips57aa3672017-07-21 11:38:13 -040037}
38
Robert Phillips3d4cac52019-06-11 08:08:08 -040039static sk_sp<GrSurfaceProxy> make_backend(GrContext* context, const ProxyParams& p,
40 GrBackendTexture* backendTex) {
Robert Phillips9da87e02019-02-04 13:26:26 -050041 GrProxyProvider* proxyProvider = context->priv().proxyProvider();
Robert Phillips0bd24dc2018-01-16 08:06:32 -050042
Robert Phillipsc80b0e92019-07-23 10:27:09 -040043 SkColorType skColorType = GrColorTypeToSkColorType(p.fColorType);
44 SkASSERT(SkColorType::kUnknown_SkColorType != skColorType);
45
Greg Danielc1ad77c2020-05-06 11:40:03 -040046 CreateBackendTexture(context, backendTex, p.fSize, p.fSize, skColorType,
47 SkColors::kTransparent, GrMipMapped::kNo, GrRenderable::kNo);
48
Robert Phillips646f6372018-09-25 09:31:10 -040049 if (!backendTex->isValid()) {
50 return nullptr;
51 }
Robert Phillips57aa3672017-07-21 11:38:13 -040052
Brian Salomon8a78e9c2020-03-27 10:42:15 -040053 return proxyProvider->wrapBackendTexture(*backendTex, kBorrow_GrWrapOwnership,
Greg Daniel3a365112020-02-14 10:47:18 -050054 GrWrapCacheable::kNo, kRead_GrIOType);
Robert Phillips57aa3672017-07-21 11:38:13 -040055}
56
Brian Salomon26102cb2018-03-09 09:33:19 -050057static void cleanup_backend(GrContext* context, const GrBackendTexture& backendTex) {
Robert Phillips5c7a25b2019-05-20 08:38:07 -040058 context->deleteBackendTexture(backendTex);
Robert Phillips57aa3672017-07-21 11:38:13 -040059}
60
Robert Phillips5af44de2017-07-18 14:49:38 -040061// Basic test that two proxies with overlapping intervals and compatible descriptors are
62// assigned different GrSurfaces.
Robert Phillips57aa3672017-07-21 11:38:13 -040063static void overlap_test(skiatest::Reporter* reporter, GrResourceProvider* resourceProvider,
Robert Phillips3d4cac52019-06-11 08:08:08 -040064 sk_sp<GrSurfaceProxy> p1, sk_sp<GrSurfaceProxy> p2,
Brian Salomon2c791fc2019-04-02 11:52:03 -040065 bool expectedResult) {
Brian Salomonbeb7f522019-08-30 16:19:42 -040066 GrResourceAllocator alloc(resourceProvider SkDEBUGCODE(, 1));
Robert Phillips5af44de2017-07-18 14:49:38 -040067
Robert Phillips3d4cac52019-06-11 08:08:08 -040068 alloc.addInterval(p1.get(), 0, 4, GrResourceAllocator::ActualUse::kYes);
Robert Phillipsc476e5d2019-03-26 14:50:08 -040069 alloc.incOps();
Robert Phillips3d4cac52019-06-11 08:08:08 -040070 alloc.addInterval(p2.get(), 1, 2, GrResourceAllocator::ActualUse::kYes);
Robert Phillipsc476e5d2019-03-26 14:50:08 -040071 alloc.incOps();
Greg Danielf41b2bd2019-08-22 16:19:24 -040072 alloc.markEndOfOpsTask(0);
Robert Phillips5af44de2017-07-18 14:49:38 -040073
Robert Phillipsc73666f2019-04-24 08:49:48 -040074 alloc.determineRecyclability();
75
Robert Phillipseafd48a2017-11-16 07:52:08 -050076 int startIndex, stopIndex;
Greg Danielaa3dfbe2018-01-29 10:34:25 -050077 GrResourceAllocator::AssignError error;
Brian Salomon577aa0f2018-11-30 13:32:23 -050078 alloc.assign(&startIndex, &stopIndex, &error);
Greg Danielaa3dfbe2018-01-29 10:34:25 -050079 REPORTER_ASSERT(reporter, GrResourceAllocator::AssignError::kNoError == error);
Robert Phillips5af44de2017-07-18 14:49:38 -040080
Brian Salomonfd98c2c2018-07-31 17:25:29 -040081 REPORTER_ASSERT(reporter, p1->peekSurface());
82 REPORTER_ASSERT(reporter, p2->peekSurface());
Robert Phillips57aa3672017-07-21 11:38:13 -040083 bool doTheBackingStoresMatch = p1->underlyingUniqueID() == p2->underlyingUniqueID();
84 REPORTER_ASSERT(reporter, expectedResult == doTheBackingStoresMatch);
Robert Phillips5af44de2017-07-18 14:49:38 -040085}
86
Robert Phillips57aa3672017-07-21 11:38:13 -040087// Test various cases when two proxies do not have overlapping intervals.
88// This mainly acts as a test of the ResourceAllocator's free pool.
89static void non_overlap_test(skiatest::Reporter* reporter, GrResourceProvider* resourceProvider,
Robert Phillips3d4cac52019-06-11 08:08:08 -040090 sk_sp<GrSurfaceProxy> p1, sk_sp<GrSurfaceProxy> p2,
Robert Phillips57aa3672017-07-21 11:38:13 -040091 bool expectedResult) {
Brian Salomonbeb7f522019-08-30 16:19:42 -040092 GrResourceAllocator alloc(resourceProvider SkDEBUGCODE(, 1));
Robert Phillipsc476e5d2019-03-26 14:50:08 -040093
94 alloc.incOps();
95 alloc.incOps();
96 alloc.incOps();
97 alloc.incOps();
98 alloc.incOps();
99 alloc.incOps();
Robert Phillips57aa3672017-07-21 11:38:13 -0400100
Robert Phillips3d4cac52019-06-11 08:08:08 -0400101 alloc.addInterval(p1.get(), 0, 2, GrResourceAllocator::ActualUse::kYes);
102 alloc.addInterval(p2.get(), 3, 5, GrResourceAllocator::ActualUse::kYes);
Greg Danielf41b2bd2019-08-22 16:19:24 -0400103 alloc.markEndOfOpsTask(0);
Robert Phillips57aa3672017-07-21 11:38:13 -0400104
Robert Phillipsc73666f2019-04-24 08:49:48 -0400105 alloc.determineRecyclability();
106
Robert Phillipseafd48a2017-11-16 07:52:08 -0500107 int startIndex, stopIndex;
Greg Danielaa3dfbe2018-01-29 10:34:25 -0500108 GrResourceAllocator::AssignError error;
Brian Salomon577aa0f2018-11-30 13:32:23 -0500109 alloc.assign(&startIndex, &stopIndex, &error);
Greg Danielaa3dfbe2018-01-29 10:34:25 -0500110 REPORTER_ASSERT(reporter, GrResourceAllocator::AssignError::kNoError == error);
Robert Phillips57aa3672017-07-21 11:38:13 -0400111
Brian Salomonfd98c2c2018-07-31 17:25:29 -0400112 REPORTER_ASSERT(reporter, p1->peekSurface());
113 REPORTER_ASSERT(reporter, p2->peekSurface());
Robert Phillips57aa3672017-07-21 11:38:13 -0400114 bool doTheBackingStoresMatch = p1->underlyingUniqueID() == p2->underlyingUniqueID();
115 REPORTER_ASSERT(reporter, expectedResult == doTheBackingStoresMatch);
116}
117
118DEF_GPUTEST_FOR_RENDERING_CONTEXTS(ResourceAllocatorTest, reporter, ctxInfo) {
Robert Phillips6d344c32020-07-06 10:56:46 -0400119 auto direct = ctxInfo.directContext();
120 const GrCaps* caps = direct->priv().caps();
121 GrProxyProvider* proxyProvider = direct->priv().proxyProvider();
122 GrResourceProvider* resourceProvider = direct->priv().resourceProvider();
Robert Phillips5af44de2017-07-18 14:49:38 -0400123
Robert Phillips57aa3672017-07-21 11:38:13 -0400124 struct TestCase {
125 ProxyParams fP1;
126 ProxyParams fP2;
127 bool fExpectation;
128 };
129
Brian Salomonf2c2ba92019-07-17 09:59:59 -0400130 constexpr GrRenderable kRT = GrRenderable::kYes;
131 constexpr GrRenderable kNotRT = GrRenderable::kNo;
Robert Phillips57aa3672017-07-21 11:38:13 -0400132
133 constexpr bool kShare = true;
134 constexpr bool kDontShare = false;
135 // Non-RT GrSurfaces are never recycled on some platforms.
136 bool kConditionallyShare = resourceProvider->caps()->reuseScratchTextures();
137
Robert Phillipsc80b0e92019-07-23 10:27:09 -0400138 const GrColorType kRGBA = GrColorType::kRGBA_8888;
Greg Danield51fa2f2020-01-22 16:53:38 -0500139 const GrColorType kAlpha = GrColorType::kAlpha_8;
Robert Phillips57aa3672017-07-21 11:38:13 -0400140
141 const SkBackingFit kE = SkBackingFit::kExact;
142 const SkBackingFit kA = SkBackingFit::kApprox;
143
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400144 const SkBudgeted kNotB = SkBudgeted::kNo;
145
Robert Phillips57aa3672017-07-21 11:38:13 -0400146 //--------------------------------------------------------------------------------------------
147 TestCase gOverlappingTests[] = {
Greg Daniel3a365112020-02-14 10:47:18 -0500148 //----------------------------------------------------------------------------------------
149 // Two proxies with overlapping intervals and compatible descriptors should never share
150 // RT version
151 {{64, kRT, kRGBA, kA, 1, kNotB}, {64, kRT, kRGBA, kA, 1, kNotB}, kDontShare},
152 // non-RT version
153 {{64, kNotRT, kRGBA, kA, 1, kNotB}, {64, kNotRT, kRGBA, kA, 1, kNotB}, kDontShare},
Robert Phillips57aa3672017-07-21 11:38:13 -0400154 };
155
156 for (auto test : gOverlappingTests) {
Robert Phillips3d4cac52019-06-11 08:08:08 -0400157 sk_sp<GrSurfaceProxy> p1 = make_deferred(proxyProvider, caps, test.fP1);
158 sk_sp<GrSurfaceProxy> p2 = make_deferred(proxyProvider, caps, test.fP2);
Robert Phillipse5f73282019-06-18 17:15:04 -0400159 overlap_test(reporter, resourceProvider, std::move(p1), std::move(p2), test.fExpectation);
Robert Phillips57aa3672017-07-21 11:38:13 -0400160 }
161
Greg Daniel6fa62e22019-08-07 15:52:37 -0400162 auto beFormat = caps->getDefaultBackendFormat(GrColorType::kRGBA_8888, GrRenderable::kYes);
Robert Phillips6d344c32020-07-06 10:56:46 -0400163 int k2 = direct->priv().caps()->getRenderTargetSampleCount(2, beFormat);
164 int k4 = direct->priv().caps()->getRenderTargetSampleCount(4, beFormat);
Robert Phillips57aa3672017-07-21 11:38:13 -0400165
166 //--------------------------------------------------------------------------------------------
167 TestCase gNonOverlappingTests[] = {
Greg Daniel3a365112020-02-14 10:47:18 -0500168 //----------------------------------------------------------------------------------------
169 // Two non-overlapping intervals w/ compatible proxies should share
170 // both same size & approx
171 {{64, kRT, kRGBA, kA, 1, kNotB}, {64, kRT, kRGBA, kA, 1, kNotB}, kShare},
172 {{64, kNotRT, kRGBA, kA, 1, kNotB},
173 {64, kNotRT, kRGBA, kA, 1, kNotB},
174 kConditionallyShare},
175 // diffs sizes but still approx
176 {{64, kRT, kRGBA, kA, 1, kNotB}, {50, kRT, kRGBA, kA, 1, kNotB}, kShare},
177 {{64, kNotRT, kRGBA, kA, 1, kNotB},
178 {50, kNotRT, kRGBA, kA, 1, kNotB},
179 kConditionallyShare},
180 // sames sizes but exact
181 {{64, kRT, kRGBA, kE, 1, kNotB}, {64, kRT, kRGBA, kE, 1, kNotB}, kShare},
182 {{64, kNotRT, kRGBA, kE, 1, kNotB},
183 {64, kNotRT, kRGBA, kE, 1, kNotB},
184 kConditionallyShare},
185 //----------------------------------------------------------------------------------------
186 // Two non-overlapping intervals w/ different exact sizes should not share
187 {{56, kRT, kRGBA, kE, 1, kNotB}, {54, kRT, kRGBA, kE, 1, kNotB}, kDontShare},
188 // Two non-overlapping intervals w/ _very different_ approx sizes should not share
189 {{255, kRT, kRGBA, kA, 1, kNotB}, {127, kRT, kRGBA, kA, 1, kNotB}, kDontShare},
190 // Two non-overlapping intervals w/ different MSAA sample counts should not share
191 {{64, kRT, kRGBA, kA, k2, kNotB}, {64, kRT, kRGBA, kA, k4, kNotB}, k2 == k4},
192 // Two non-overlapping intervals w/ different configs should not share
193 {{64, kRT, kRGBA, kA, 1, kNotB}, {64, kRT, kAlpha, kA, 1, kNotB}, kDontShare},
194 // Two non-overlapping intervals w/ different RT classifications should never share
195 {{64, kRT, kRGBA, kA, 1, kNotB}, {64, kNotRT, kRGBA, kA, 1, kNotB}, kDontShare},
196 {{64, kNotRT, kRGBA, kA, 1, kNotB}, {64, kRT, kRGBA, kA, 1, kNotB}, kDontShare},
197 // Two non-overlapping intervals w/ different origins should share
198 {{64, kRT, kRGBA, kA, 1, kNotB}, {64, kRT, kRGBA, kA, 1, kNotB}, kShare},
Robert Phillips57aa3672017-07-21 11:38:13 -0400199 };
200
201 for (auto test : gNonOverlappingTests) {
Robert Phillips3d4cac52019-06-11 08:08:08 -0400202 sk_sp<GrSurfaceProxy> p1 = make_deferred(proxyProvider, caps, test.fP1);
203 sk_sp<GrSurfaceProxy> p2 = make_deferred(proxyProvider, caps, test.fP2);
Robert Phillips715d08c2018-07-18 13:56:48 -0400204
Robert Phillips57aa3672017-07-21 11:38:13 -0400205 if (!p1 || !p2) {
206 continue; // creation can fail (i.e., for msaa4 on iOS)
207 }
Robert Phillips715d08c2018-07-18 13:56:48 -0400208
Robert Phillipse5f73282019-06-18 17:15:04 -0400209 non_overlap_test(reporter, resourceProvider, std::move(p1), std::move(p2),
210 test.fExpectation);
Robert Phillips57aa3672017-07-21 11:38:13 -0400211 }
212
213 {
214 // Wrapped backend textures should never be reused
215 TestCase t[1] = {
Greg Daniel3a365112020-02-14 10:47:18 -0500216 {{64, kNotRT, kRGBA, kE, 1, kNotB}, {64, kNotRT, kRGBA, kE, 1, kNotB}, kDontShare}};
Robert Phillips57aa3672017-07-21 11:38:13 -0400217
Robert Phillipsd21b2a52017-12-12 13:01:25 -0500218 GrBackendTexture backEndTex;
Robert Phillips6d344c32020-07-06 10:56:46 -0400219 sk_sp<GrSurfaceProxy> p1 = make_backend(direct, t[0].fP1, &backEndTex);
Robert Phillips3d4cac52019-06-11 08:08:08 -0400220 sk_sp<GrSurfaceProxy> p2 = make_deferred(proxyProvider, caps, t[0].fP2);
Robert Phillips715d08c2018-07-18 13:56:48 -0400221
Robert Phillipse5f73282019-06-18 17:15:04 -0400222 non_overlap_test(reporter, resourceProvider, std::move(p1), std::move(p2),
223 t[0].fExpectation);
Robert Phillips715d08c2018-07-18 13:56:48 -0400224
Robert Phillips6d344c32020-07-06 10:56:46 -0400225 cleanup_backend(direct, backEndTex);
Robert Phillips57aa3672017-07-21 11:38:13 -0400226 }
Robert Phillips5af44de2017-07-18 14:49:38 -0400227}
Robert Phillips1734dd32018-08-21 13:52:09 -0400228
229static void draw(GrContext* context) {
230 SkImageInfo ii = SkImageInfo::Make(1024, 1024, kRGBA_8888_SkColorType, kPremul_SkAlphaType);
231
232 sk_sp<SkSurface> s = SkSurface::MakeRenderTarget(context, SkBudgeted::kYes,
233 ii, 1, kTopLeft_GrSurfaceOrigin, nullptr);
234
235 SkCanvas* c = s->getCanvas();
236
237 c->clear(SK_ColorBLACK);
238}
239
240
241DEF_GPUTEST_FOR_RENDERING_CONTEXTS(ResourceAllocatorStressTest, reporter, ctxInfo) {
Robert Phillips6d344c32020-07-06 10:56:46 -0400242 auto context = ctxInfo.directContext();
Robert Phillips1734dd32018-08-21 13:52:09 -0400243
Robert Phillipscf39f372019-09-03 10:29:20 -0400244 size_t maxBytes = context->getResourceCacheLimit();
Robert Phillips1734dd32018-08-21 13:52:09 -0400245
Robert Phillipscf39f372019-09-03 10:29:20 -0400246 context->setResourceCacheLimit(0); // We'll always be overbudget
Robert Phillips1734dd32018-08-21 13:52:09 -0400247
248 draw(context);
249 draw(context);
250 draw(context);
251 draw(context);
Greg Daniel0a2464f2020-05-14 15:45:44 -0400252 context->flushAndSubmit();
Robert Phillips1734dd32018-08-21 13:52:09 -0400253
Robert Phillipscf39f372019-09-03 10:29:20 -0400254 context->setResourceCacheLimit(maxBytes);
Robert Phillips1734dd32018-08-21 13:52:09 -0400255}
Brian Salomon577aa0f2018-11-30 13:32:23 -0500256
257sk_sp<GrSurfaceProxy> make_lazy(GrProxyProvider* proxyProvider, const GrCaps* caps,
Brian Salomonbeb7f522019-08-30 16:19:42 -0400258 const ProxyParams& p) {
Brian Salomon4eb38b72019-08-05 12:58:39 -0400259 const auto format = caps->getDefaultBackendFormat(p.fColorType, p.fRenderable);
Brian Salomon577aa0f2018-11-30 13:32:23 -0500260
Brian Salomon63410e92020-03-23 18:32:50 -0400261 auto callback = [](GrResourceProvider* resourceProvider,
262 const GrSurfaceProxy::LazySurfaceDesc& desc) {
Brian Salomonb6a3a3b2019-04-01 12:29:34 -0400263 sk_sp<GrTexture> texture;
Brian Salomon63410e92020-03-23 18:32:50 -0400264 if (desc.fFit == SkBackingFit::kApprox) {
265 texture = resourceProvider->createApproxTexture(desc.fDimensions, desc.fFormat,
266 desc.fRenderable, desc.fSampleCnt,
267 desc.fProtected);
Brian Salomon577aa0f2018-11-30 13:32:23 -0500268 } else {
Brian Salomon63410e92020-03-23 18:32:50 -0400269 texture = resourceProvider->createTexture(
270 desc.fDimensions, desc.fFormat, desc.fRenderable, desc.fSampleCnt,
271 desc.fMipMapped, desc.fBudgeted, desc.fProtected);
Brian Salomon577aa0f2018-11-30 13:32:23 -0500272 }
Brian Salomonbeb7f522019-08-30 16:19:42 -0400273 return GrSurfaceProxy::LazyCallbackResult(std::move(texture));
Brian Salomon577aa0f2018-11-30 13:32:23 -0500274 };
Brian Salomon577aa0f2018-11-30 13:32:23 -0500275 GrInternalSurfaceFlags flags = GrInternalSurfaceFlags::kNone;
Brian Salomon63410e92020-03-23 18:32:50 -0400276 SkISize dims = {p.fSize, p.fSize};
Brian Salomondf1bd6d2020-03-26 20:37:01 -0400277 return proxyProvider->createLazyProxy(callback, format, dims, p.fRenderable, p.fSampleCnt,
278 GrMipMapped::kNo, GrMipMapsStatus::kNotAllocated, flags,
279 p.fFit, p.fBudgeted, GrProtected::kNo,
280 GrSurfaceProxy::UseAllocator::kYes);
Brian Salomon577aa0f2018-11-30 13:32:23 -0500281}
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400282
Greg Danielf41b2bd2019-08-22 16:19:24 -0400283// Set up so there are two opsTasks that need to be flushed but the resource allocator thinks
284// it is over budget. The two opsTasks should be flushed separately and the opsTask indices
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400285// returned from assign should be correct.
286DEF_GPUTEST_FOR_RENDERING_CONTEXTS(ResourceAllocatorOverBudgetTest, reporter, ctxInfo) {
Robert Phillips6d344c32020-07-06 10:56:46 -0400287 auto context = ctxInfo.directContext();
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400288 const GrCaps* caps = context->priv().caps();
289 GrProxyProvider* proxyProvider = context->priv().proxyProvider();
290 GrResourceProvider* resourceProvider = context->priv().resourceProvider();
291
Robert Phillipscf39f372019-09-03 10:29:20 -0400292 size_t origMaxBytes = context->getResourceCacheLimit();
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400293
294 // Force the resource allocator to always believe it is over budget
Robert Phillipscf39f372019-09-03 10:29:20 -0400295 context->setResourceCacheLimit(0);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400296
Greg Daniel3a365112020-02-14 10:47:18 -0500297 const ProxyParams params = {
298 64, GrRenderable::kNo, GrColorType::kRGBA_8888, SkBackingFit::kExact,
299 1, SkBudgeted::kYes};
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400300
301 {
Robert Phillips3d4cac52019-06-11 08:08:08 -0400302 sk_sp<GrSurfaceProxy> p1 = make_deferred(proxyProvider, caps, params);
303 sk_sp<GrSurfaceProxy> p2 = make_deferred(proxyProvider, caps, params);
304 sk_sp<GrSurfaceProxy> p3 = make_deferred(proxyProvider, caps, params);
305 sk_sp<GrSurfaceProxy> p4 = make_deferred(proxyProvider, caps, params);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400306
Brian Salomonbeb7f522019-08-30 16:19:42 -0400307 GrResourceAllocator alloc(resourceProvider SkDEBUGCODE(, 2));
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400308
Robert Phillips3d4cac52019-06-11 08:08:08 -0400309 alloc.addInterval(p1.get(), 0, 0, GrResourceAllocator::ActualUse::kYes);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400310 alloc.incOps();
Robert Phillips3d4cac52019-06-11 08:08:08 -0400311 alloc.addInterval(p2.get(), 1, 1, GrResourceAllocator::ActualUse::kYes);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400312 alloc.incOps();
Greg Danielf41b2bd2019-08-22 16:19:24 -0400313 alloc.markEndOfOpsTask(0);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400314
Robert Phillips3d4cac52019-06-11 08:08:08 -0400315 alloc.addInterval(p3.get(), 2, 2, GrResourceAllocator::ActualUse::kYes);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400316 alloc.incOps();
Robert Phillips3d4cac52019-06-11 08:08:08 -0400317 alloc.addInterval(p4.get(), 3, 3, GrResourceAllocator::ActualUse::kYes);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400318 alloc.incOps();
Greg Danielf41b2bd2019-08-22 16:19:24 -0400319 alloc.markEndOfOpsTask(1);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400320
321 int startIndex, stopIndex;
322 GrResourceAllocator::AssignError error;
323
Robert Phillipsc73666f2019-04-24 08:49:48 -0400324 alloc.determineRecyclability();
325
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400326 alloc.assign(&startIndex, &stopIndex, &error);
327 REPORTER_ASSERT(reporter, GrResourceAllocator::AssignError::kNoError == error);
328 REPORTER_ASSERT(reporter, 0 == startIndex && 1 == stopIndex);
329
330 alloc.assign(&startIndex, &stopIndex, &error);
331 REPORTER_ASSERT(reporter, GrResourceAllocator::AssignError::kNoError == error);
332 REPORTER_ASSERT(reporter, 1 == startIndex && 2 == stopIndex);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400333 }
334
Robert Phillipscf39f372019-09-03 10:29:20 -0400335 context->setResourceCacheLimit(origMaxBytes);
Robert Phillipsc476e5d2019-03-26 14:50:08 -0400336}
Greg Danield72dd4d2019-08-29 14:37:46 -0400337
338// This test is used to make sure we are tracking the current task index during the assign call in
339// the GrResourceAllocator. Specifically we can fall behind if we have intervals that don't
340// use the allocator. In this case we need to possibly increment the fCurOpsTaskIndex multiple times
341// to get in back in sync. We had a bug where we'd only every increment the index by one,
342// http://crbug.com/996610.
343DEF_GPUTEST_FOR_RENDERING_CONTEXTS(ResourceAllocatorCurOpsTaskIndexTest,
344 reporter, ctxInfo) {
Robert Phillips6d344c32020-07-06 10:56:46 -0400345 auto context = ctxInfo.directContext();
Greg Danield72dd4d2019-08-29 14:37:46 -0400346 const GrCaps* caps = context->priv().caps();
347 GrProxyProvider* proxyProvider = context->priv().proxyProvider();
348 GrResourceProvider* resourceProvider = context->priv().resourceProvider();
349
Robert Phillipscf39f372019-09-03 10:29:20 -0400350 size_t origMaxBytes = context->getResourceCacheLimit();
Greg Danield72dd4d2019-08-29 14:37:46 -0400351
352 // Force the resource allocator to always believe it is over budget
Robert Phillipscf39f372019-09-03 10:29:20 -0400353 context->setResourceCacheLimit(0);
Greg Danield72dd4d2019-08-29 14:37:46 -0400354
355 ProxyParams params;
356 params.fFit = SkBackingFit::kExact;
Greg Danield72dd4d2019-08-29 14:37:46 -0400357 params.fColorType = GrColorType::kRGBA_8888;
358 params.fRenderable = GrRenderable::kYes;
359 params.fSampleCnt = 1;
360 params.fSize = 100;
361 params.fBudgeted = SkBudgeted::kYes;
362
363 sk_sp<GrSurfaceProxy> proxy1 = make_deferred(proxyProvider, caps, params);
364 if (!proxy1) {
365 return;
366 }
367 sk_sp<GrSurfaceProxy> proxy2 = make_deferred(proxyProvider, caps, params);
368 if (!proxy2) {
369 return;
370 }
371
372 // Wrapped proxy that will be ignored by the resourceAllocator. We use this to try and get the
373 // resource allocator fCurOpsTaskIndex to fall behind what it really should be.
374 GrBackendTexture backEndTex;
Robert Phillips6d344c32020-07-06 10:56:46 -0400375 sk_sp<GrSurfaceProxy> proxyWrapped = make_backend(context, params, &backEndTex);
Greg Danield72dd4d2019-08-29 14:37:46 -0400376 if (!proxyWrapped) {
377 return;
378 }
Greg Danield72dd4d2019-08-29 14:37:46 -0400379
380 // Same as above, but we actually need to have at least two intervals that don't go through the
381 // resource allocator to expose the index bug.
382 GrBackendTexture backEndTex2;
Robert Phillips6d344c32020-07-06 10:56:46 -0400383 sk_sp<GrSurfaceProxy> proxyWrapped2 = make_backend(context, params, &backEndTex2);
Greg Danield72dd4d2019-08-29 14:37:46 -0400384 if (!proxyWrapped2) {
Robert Phillips6d344c32020-07-06 10:56:46 -0400385 cleanup_backend(context, backEndTex);
Greg Danield72dd4d2019-08-29 14:37:46 -0400386 return;
387 }
Greg Danield72dd4d2019-08-29 14:37:46 -0400388
Brian Salomonbeb7f522019-08-30 16:19:42 -0400389 GrResourceAllocator alloc(resourceProvider SkDEBUGCODE(, 4));
Greg Danield72dd4d2019-08-29 14:37:46 -0400390
391 alloc.addInterval(proxyWrapped.get(), 0, 0, GrResourceAllocator::ActualUse::kYes);
392 alloc.incOps();
393 alloc.markEndOfOpsTask(0);
394
395 alloc.addInterval(proxyWrapped2.get(), 1, 1, GrResourceAllocator::ActualUse::kYes);
396 alloc.incOps();
397 alloc.markEndOfOpsTask(1);
398
399 alloc.addInterval(proxy1.get(), 2, 2, GrResourceAllocator::ActualUse::kYes);
400 alloc.incOps();
401 alloc.markEndOfOpsTask(2);
402
403 // We want to force the resource allocator to do a intermediateFlush for the previous interval.
404 // But if it is the resource allocator is at the of its list of intervals it skips the
405 // intermediate flush call, so we add another interval here so it is not skipped.
406 alloc.addInterval(proxy2.get(), 3, 3, GrResourceAllocator::ActualUse::kYes);
407 alloc.incOps();
408 alloc.markEndOfOpsTask(3);
409
410 int startIndex, stopIndex;
411 GrResourceAllocator::AssignError error;
412
413 alloc.determineRecyclability();
414
415 alloc.assign(&startIndex, &stopIndex, &error);
416 REPORTER_ASSERT(reporter, GrResourceAllocator::AssignError::kNoError == error);
417 // The original bug in the allocator here would return a stopIndex of 2 since it would have only
418 // incremented its fCurOpsTaskIndex once instead of the needed two times to skip the first two
419 // unused intervals.
420 REPORTER_ASSERT(reporter, 0 == startIndex && 3 == stopIndex);
421
422 alloc.assign(&startIndex, &stopIndex, &error);
423 REPORTER_ASSERT(reporter, GrResourceAllocator::AssignError::kNoError == error);
424 REPORTER_ASSERT(reporter, 3 == startIndex && 4 == stopIndex);
425
Robert Phillips6d344c32020-07-06 10:56:46 -0400426 cleanup_backend(context, backEndTex);
427 cleanup_backend(context, backEndTex2);
Greg Danield72dd4d2019-08-29 14:37:46 -0400428
Robert Phillipscf39f372019-09-03 10:29:20 -0400429 context->setResourceCacheLimit(origMaxBytes);
Greg Danield72dd4d2019-08-29 14:37:46 -0400430}
431