Re: [PATCH 47/55] drm/i915: Disable waitboosting for a saturated engine

Chris Wilson
2016-07-26T13:11:24Z

On Tue, Jul 26, 2016 at 03:40:28PM +0300, Joonas Lahtinen wrote:
> On ma, 2016-07-25 at 18:32 +0100, Chris Wilson wrote:
> > If the user floods the GPU with so many requests that the engine stalls
> > waiting for free space, don't automatically promote the GPU to maximum
> > frequencies. If the GPU really is saturated with work, it will migrate
> > to high clocks by itself, otherwise it is merely a user flooding us with
> > busy-work.
> >
>
> Should this even come with Fixes: ?

A fixes patch would need dev_priv->rps.semaphores as the client. In
practice (outside of igt), I don't expect anyone to notice the difference
as you have both to fill the ring and keep the GPU (mostly) idle.
-Chris

--
Chris Wilson, Intel Open Source Technology Centre