On ma, 2016-09-19 at 12:32 +0100, Chris Wilson wrote:
> On Mon, Sep 19, 2016 at 01:47:30PM +0300, Joonas Lahtinen wrote:
> > On ke, 2016-09-14 at 07:52 +0100, Chris Wilson wrote:
> > > @@ -2677,8 +2681,21 @@ static void intel_ring_default_vfuncs(struct drm_i915_private *dev_priv,
> > > engine->reset_hw = reset_ring_common;
> > >
> > > engine->emit_request = i9xx_emit_request;
> > > - if (i915.semaphores)
> > > + engine->emit_request_sz = i9xx_emit_request_sz;
> > > + if (i915.semaphores) {
> > > + int num_rings;
> >
> > 'initialized_rings' to differentiate as suggested by Tvrtko too.
>
> Tvrtko wasn't referring to this. The point is that we don't know
> initialized_rings here, just the declared set of rings we think
> the hardware is supposed to have.
Can't we afford to calculate this to the info struct anyway?
Regards, Joonas
--
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
