Re: [PATCH] drm/i915: Bump priority of clean up work

[Mika Kuoppala <mika.kuoppala@linux.intel.com>]

Chris Wilson <chris@chris-wilson.co.uk> writes:

> Quoting Chris Wilson (2018-07-12 12:57:29)
>> We require that we keep the list of outstanding work short so that we do
>> not "leak" memory while pageflipping under stress. However that system
>> stress may delay kernel workers virtually indefinitely, which incurs the
>> pageflips stall and eventually hit a timeout waiting for the cleanup.
>> 
>> Try to combat CPU starvation of our short-lived cleanup workers by
>> switching to a high priority workqueue.
>> 
>> Testcase: igt/kms_cursor_legacy/all-pipes-torture-move
>> References: https://bugs.freedesktop.org/show_bug.cgi?id=107122
>> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
>> ---
>> Not sure if highpri is enough to combat our RT torture...
>
> CI thinks it is.

Not so familiar with the atomic commit. But it makes
sense and now there is evidence supporting it.

Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>

> -Chris
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx