Quoting Tvrtko Ursulin (2020-06-18 08:46:18) > > On 17/06/2020 17:58, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2020-06-17 17:01:13) > >> From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > >> > >> Show average/min/max workload iteration and dropped period stats when 'p' > >> command is used. > >> > >> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > >> --- > >> benchmarks/gem_wsim.c | 19 +++++++++++++++---- > >> 1 file changed, 15 insertions(+), 4 deletions(-) > >> > >> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c > >> index 9e5bfe6a36d4..60982cb73ba7 100644 > >> --- a/benchmarks/gem_wsim.c > >> +++ b/benchmarks/gem_wsim.c > >> @@ -2101,7 +2101,8 @@ static void *run_workload(void *data) > >> struct w_step *w; > >> int throttle = -1; > >> int qd_throttle = -1; > >> - int count; > >> + int count, missed = 0; > >> + unsigned long time_tot = 0, time_min = ULONG_MAX, time_max = 0; > >> int i; > >> > >> clock_gettime(CLOCK_MONOTONIC, &t_start); > >> @@ -2121,12 +2122,19 @@ static void *run_workload(void *data) > >> do_sleep = w->delay; > >> } else if (w->type == PERIOD) { > >> struct timespec now; > >> + int elapsed; > >> > >> clock_gettime(CLOCK_MONOTONIC, &now); > >> - do_sleep = w->period - > >> - elapsed_us(&wrk->repeat_start, &now); > >> + elapsed = elapsed_us(&wrk->repeat_start, &now); > >> + do_sleep = w->period - elapsed; > >> + time_tot += elapsed; > >> + if (elapsed < time_min) > >> + time_min = elapsed; > >> + if (elapsed > time_max) > >> + time_max = elapsed; > > > > Keep the running average? > > Could do but why? I already have the count so adding up total elapsed > frame time sound easiest. Because I was blind and didn't see it in the printf. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx