Quoting Tvrtko Ursulin (2018-02-01 10:02:22)
>
> On 01/02/2018 08:29, Chris Wilson wrote:
> > Execlists is now enabled by default and included in the list of
> > capabilities printed out to dmesg and beyond. We do not need to mention
> > it again, every time we restart the engine, so kill the spam.
> >
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > ---
> > drivers/gpu/drm/i915/intel_lrc.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
> > index 40dbeaee9dfa..bb4e5db03edb 100644
> > --- a/drivers/gpu/drm/i915/intel_lrc.c
> > +++ b/drivers/gpu/drm/i915/intel_lrc.c
> > @@ -1477,7 +1477,6 @@ static int gen8_init_common_ring(struct intel_engine_cs *engine)
> > intel_engine_init_hangcheck(engine);
> >
> > enable_execlists(engine);
> > - DRM_DEBUG_DRIVER("Execlists enabled for %s\n", engine->name);
> >
> > GEM_BUG_ON(engine->id >= ARRAY_SIZE(gtiir));
> >
> >
>
> Hmmm, I always find it interesting to trim things, but in this case I
> think this part is so insignificant compared to all the rest that I
> don't see a benefit.
The logs I'm reading (for hangcheck) are 80% "Execlists enabled for %s".
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
