Quoting Chris Wilson (2018-02-26 14:28:36) > Quoting Michał Winiarski (2018-02-26 13:59:59) > > + for_each_engine(engine, dev_priv, id) { > struct intel_engine *ce = &client->owner->engine[id]; > > /* The preempt context must be pinned on each engine); > GEM_BUG_ON(!ce->pin_count); > > /* > * We rely on the context image *not* being saved after > * preemption. This ensures that the RING_HEAD / RING_TAIL > * do not advance and they remain pointing at this command > * sequence forever. > */ > GEM_BUG_ON(!(ce->reg_state[SR] & CTX_SR_SAVE_INHIBIT)); In fact, don't bug out, just set it here. Then it won't break again when icl enabling lands. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx