On Mon, Feb 06, 2017 at 09:41:42AM +0100, Maarten Lankhorst wrote:
> Op 02-02-17 om 21:47 schreef Chris Wilson:
> > If we have any residual freed atomic state from earlier commits, flush
> > the freed list after performing the current modeset. This prevents the
> > freed list from ever-growing if userspace manages to starve the kernel
> > threads (i.e. we are never able to run our free state worker and
> > eventually the system may even oom).
> >
> > Fixes: eb955eee27d9 ("drm/i915: Move atomic state free from out of fence release")
> > Testcase: igt/kms_cursor/legacy/all-pipes-single-bo
> kms_cursor_legacy
> > Reported-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
> > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx>
> > ---
> >
> > Though the earlier patch is cc'ed for v4.10, I don't this fix to the fix
> > merits applying to v4.10 this late. The impact should be small, it takes
> > the deliberate abuse of RT tasks to starve the kernel so badly as that
> > the delayed free of the atomic state will cause system-wide consequences
> > (e.g. oom)
> Fixes the kms_cursor_legacy testcase, and the patch makes sense.
>
> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
Thanks for the catch, pushed.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
