Quoting Chris Wilson (2017-10-25 11:06:01)
> Instead of trying to create a timer with zero delay (i.e. with expires
> set to the current jiffies and not the future, an already expired
> timer), execute that request immediately.
>
> v2: Refactor list_del_init+signal into its own little function.
>
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Link:
> https://patchwork.freedesktop.org/patch/msgid/20171024220855.30155-1-ch...@chris-wilson.co.uk
> Reviewed-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Scratch that, the refactor was nice, because it made
> - request = first_request(engine);
> - if (request) {
> - list_del_init(&request->link);
> - mock_seqno_advance(&engine->base, request->base.global_seqno);
> + while ((request = first_request(engine))) {
> + advance(engine, request);
> + if (request->delay)
> + break;
this stand out as bogus.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
