Re: [Intel-gfx] [PATCH 3/4] drm/i915/execlists: Assert that we don't get mixed IDLE_ACTIVE | COMPLETE events

2017-11-20 Thread Chris Wilson
Quoting Joonas Lahtinen (2017-11-20 13:25:58) > On Mon, 2017-11-20 at 12:34 +, 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

Re: [Intel-gfx] [PATCH 3/4] drm/i915/execlists: Assert that we don't get mixed IDLE_ACTIVE | COMPLETE events

2017-11-20 Thread Joonas Lahtinen
On Mon, 2017-11-20 at 12:34 +, 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