Quoting Joonas Lahtinen (2017-11-20 13:25:58) > On Mon, 2017-11-20 at 12:34 +0000, Chris Wilson wrote: > > If IDLE_ACTIVE is set, then all other bits are invalid. For us, we can > > assert that if we see a COMPLETE | PREEMPTED event, then it should be > > impossible for it to also contain an IDLE_ACTIVE flag. > > > > Suggested-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx> > > Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > > Cc: Michal Winiarski <michal.winiarski@xxxxxxxxx> > > Cc: Michel Thierry <michel.thierry@xxxxxxxxx> > > Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Applied up to this point, as we are just tidying and adding more asserts. The next patch is the crutch. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx