Quoting Dmitry Rogozhkin (2017-11-20 11:49:46) > Code on loop() function shadowed function parameter which led > to inability to try out different engines in the test: we always > loaded RCS0. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103804 > Signed-off-by: Dmitry Rogozhkin <dmitry.v.rogozhkin@xxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > --- > benchmarks/gem_exec_nop.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/benchmarks/gem_exec_nop.c b/benchmarks/gem_exec_nop.c > index 03a0354..8eb88c1 100644 > --- a/benchmarks/gem_exec_nop.c > +++ b/benchmarks/gem_exec_nop.c > @@ -106,11 +106,11 @@ static int loop(unsigned ring, int reps, int ncpus, unsigned flags) > return 77; > > all_nengine = 0; > - for (ring = 1; ring < 16; ring++) { > + for (unsigned r = 1; r < 16; r++) { > execbuf.flags &= ~ENGINE_FLAGS; > - execbuf.flags |= ring; > + execbuf.flags |= r; > if (__gem_execbuf(fd, &execbuf) == 0) > - all_engines[all_nengine++] = ring; > + all_engines[all_nengine++] = r; > } Exactly what I did on Sat and forgot to send on. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
