Quoting Chris Wilson (2018-03-29 02:01:37) > Quoting Francisco Jerez (2018-03-29 01:32:07) > > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > > > + else > > > + execlists_end(execlists); > > > } else { > > > port_set(port, port_pack(rq, count)); > > > } > > > > > > > Isn't there an "execlists_clear_active(execlists, > > EXECLISTS_ACTIVE_USER);" call in your tree a few lines below that is now > > redundant? > > This is only half the patch to show the gist. :) A more concrete sketch, https://patchwork.freedesktop.org/patch/213594/ -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx