commit fd37d2a41512aa23135069d48b9811d0843fd4ac
author Leon Scroggins III <scroggo@google.com> Wed Nov 03 15:11:48 2021 -0400
committer Leon Scroggins III <scroggo@google.com> Mon Nov 08 09:30:53 2021 -0500
tree 668489fade7b1a925acd2b337d9eeca429fa9b20
parent b76f17d5f71e03f86a0f9ee05cc4614bb4461dce

sf-latency: Explicitly time SKIA_GL_THREADED

Bug: 193240340
Test: This

My original intent for this benchmark was to time drawLayers, without
worrying about threading overhead. But using SKIA_GL means that we do
not execute mapExternalTextureBuffer ahead of time, so we have to do
this work each frame. mapExternalTextureBuffer should be tracked by a
separate benchmark.

Change-Id: I0d58fdf67a6322f0bbc04835ab1cdb08b73bcf7b

libs/renderengine/benchmark/RenderEngineBench.cpp

diff --git a/libs/renderengine/benchmark/RenderEngineBench.cpp b/libs/renderengine/benchmark/RenderEngineBench.cpp
index 9ee7e4d..6c8f8e8 100644
--- a/libs/renderengine/benchmark/RenderEngineBench.cpp
+++ b/libs/renderengine/benchmark/RenderEngineBench.cpp
@@ -46,8 +46,8 @@
 }
 
 /**
- * Passed (indirectly - see RunSkiaGL) to Benchmark::Apply to create a Benchmark
- * which specifies which RenderEngineType it uses.
+ * Passed (indirectly - see RunSkiaGLThreaded) to Benchmark::Apply to create a
+ * Benchmark which specifies which RenderEngineType it uses.
  *
  * This simplifies calling ->Arg(type)->Arg(type) and provides strings to make
  * it obvious which version is being run.
@@ -62,10 +62,10 @@
 }
 
 /**
- * Run a benchmark once using SKIA_GL.
+ * Run a benchmark once using SKIA_GL_THREADED.
  */
-static void RunSkiaGL(benchmark::internal::Benchmark* b) {
-    AddRenderEngineType(b, RenderEngine::RenderEngineType::SKIA_GL);
+static void RunSkiaGLThreaded(benchmark::internal::Benchmark* b) {
+    AddRenderEngineType(b, RenderEngine::RenderEngineType::SKIA_GL_THREADED);
 }
 
 ///////////////////////////////////////////////////////////////////////////////
@@ -256,4 +256,4 @@
     benchDrawLayers(*re, layers, benchState, "blurred");
 }
 
-BENCHMARK(BM_blur)->Apply(RunSkiaGL);
+BENCHMARK(BM_blur)->Apply(RunSkiaGLThreaded);