[Intel-gfx] [PATCH v2] drm/i915: Pack params to engine->schedule() into a struct

Chris Wilson chris at chris-wilson.co.uk
Tue Apr 17 09:49:20 UTC 2018


Quoting Tvrtko Ursulin (2018-04-17 10:12:05)
> 
> On 12/04/2018 00:00, Chris Wilson wrote:
> > Today we only want to pass along the priority to engine->schedule(), but
> > in the future we want to have much more control over the various aspects
> > of the GPU during a context's execution, for example controlling the
> > frequency allowed. As we need an ever growing number of parameters for
> > scheduling, move those into a struct for convenience.
> 
> Sounds sensible in principle. Do you already have a tree implementing 
> ctx freq on top of this so I can get a better idea how exactly it will 
> be used?

It's because I want to pass in frequency adjustments from the caller and
percolate those through the dependencies, rather than pull them from the
context at that later timer. It also gets around the thorny issue of what
to do when the context is changed at runtime. What I'm playing around with is
https://cgit.freedesktop.org/~ickle/linux-2.6/commit/?h=prescheduler&id=e734f100cd74b340ff9675811268a30cbe9a7959
-Chris


More information about the Intel-gfx mailing list