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 <[email protected]> > > --- > > 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 [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
