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.lahti...@linux.intel.com> > > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> > > Cc: Mika Kuoppala <mika.kuopp...@linux.intel.com> > > Cc: Tvrtko Ursulin <tvrtko.ursu...@intel.com> > > Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> > > Cc: Michal Winiarski <michal.winiar...@intel.com> > > Cc: Michel Thierry <michel.thie...@intel.com> > > Reviewed-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
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@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
