[Intel-gfx] [PATCH 1/2] drm/i915: Flush execution tasklets before checking request status

Chris Wilson chris at chris-wilson.co.uk
Wed Feb 5 15:36:42 UTC 2020


Quoting Mika Kuoppala (2020-02-05 14:42:16)
> Chris Wilson <chris at chris-wilson.co.uk> writes:
> 
> > Rather than flushing the submission tasklets just before we sleep, flush
> > before we check the request status. Ideally this gives us a moment to
> > process the tasklets after sleeping just before we timeout.
> 
> Makes sense to check the possibly most recent state.
> 
> Reviewed-by: Mika Kuoppala <mika.kuoppala at linux.intel.com>

I suppose I should mention a small counter argument is that by kicking
ksoftirqd before checking completed, we may add a small amount of
latency to the ready client.

A compromise is to kick before timeout?
-Chris


More information about the Intel-gfx mailing list