From 65eb4fba7e22bc5e5420290a4ef31e59054609e5 Mon Sep 17 00:00:00 2001 From: Robert Phillips Date: Thu, 31 May 2018 13:27:52 -0400 Subject: Add DDL skpbench bot for just DDL recording time Change-Id: If75799bfd719ccc47c2064571723e6852c07bfaa Reviewed-on: https://skia-review.googlesource.com/131144 Commit-Queue: Robert Phillips Reviewed-by: Eric Boren Reviewed-by: Kevin Lubick --- tools/skpbench/skpbench.cpp | 9 ++++++--- tools/skpbench/skpbench.py | 4 ++++ 2 files changed, 10 insertions(+), 3 deletions(-) (limited to 'tools/skpbench') diff --git a/tools/skpbench/skpbench.cpp b/tools/skpbench/skpbench.cpp index f8a6e9b1a8..a4583ce0f6 100644 --- a/tools/skpbench/skpbench.cpp +++ b/tools/skpbench/skpbench.cpp @@ -51,6 +51,7 @@ DEFINE_bool(ddl, false, "record the skp into DDLs before rendering"); DEFINE_int32(ddlNumAdditionalThreads, 0, "number of DDL recording threads in addition to main one"); DEFINE_int32(ddlTilingWidthHeight, 0, "number of tiles along one edge when in DDL mode"); +DEFINE_bool(ddlRecordTime, false, "report just the cpu time spent recording DDLs"); DEFINE_int32(duration, 5000, "number of milliseconds to run the benchmark"); DEFINE_int32(sampleMs, 50, "minimum duration of a sample"); @@ -117,9 +118,11 @@ static void ddl_sample(GrContext* context, DDLTileHelper* tiles, GpuSync* gpuSyn tiles->createDDLsInParallel(); - tiles->drawAllTilesAndFlush(context, true); - if (gpuSync) { - gpuSync->syncToPreviousFrame(); + if (!FLAGS_ddlRecordTime) { + tiles->drawAllTilesAndFlush(context, true); + if (gpuSync) { + gpuSync->syncToPreviousFrame(); + } } *startStopTime = clock::now(); diff --git a/tools/skpbench/skpbench.py b/tools/skpbench/skpbench.py index bb88a76a1d..a9ec618ab2 100755 --- a/tools/skpbench/skpbench.py +++ b/tools/skpbench/skpbench.py @@ -73,6 +73,8 @@ __argparse.add_argument('--ddlNumAdditionalThreads', help="number of DDL recording threads in addition to main one") __argparse.add_argument('--ddlTilingWidthHeight', type=int, default=0, help="number of tiles along one edge when in DDL mode") +__argparse.add_argument('--ddlRecordTime', + action='store_true', help="report just the cpu time spent recording DDLs") __argparse.add_argument('skps', nargs='+', help=".skp files or directories to expand for .skp files") @@ -136,6 +138,8 @@ class SKPBench: str(FLAGS.ddlNumAdditionalThreads)]) if FLAGS.ddlTilingWidthHeight: ARGV.extend(['--ddlTilingWidthHeight', str(FLAGS.ddlTilingWidthHeight)]) + if FLAGS.ddlRecordTime: + ARGV.extend(['--ddlRecordTime', 'true']) if FLAGS.adb: if FLAGS.device_serial is None: -- cgit v1.2.3