Quoting Tvrtko Ursulin (2021-02-02 16:44:26) > > On 02/02/2021 15:14, Chris Wilson wrote: > > + err = 0; > > + count = 0; > > + for_each_uabi_engine(engine, i915) { > > + if (!intel_engine_has_scheduler(engine)) > > + continue; > > + > > + rq = __write_timestamp(engine, obj, count, rq); > > + if (IS_ERR(rq)) { > > + err = PTR_ERR(rq); > > + break; > > + } > > + > > + count++; > > + } > > ^^^^ vvvv - two of the same by copy&paste error or couldn't be bothered > with outer loop?
It was just my thought process at the time, I wanted the A->Z; A->Z pair so that it clear that it was cyclic and just didn't think of putting it inside another loop. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx