Quoting Tvrtko Ursulin (2019-04-15 12:10:17) > > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c > > index 58956b49f392..6c6bd50d87c9 100644 > > --- a/drivers/gpu/drm/i915/i915_debugfs.c > > +++ b/drivers/gpu/drm/i915/i915_debugfs.c > > @@ -34,6 +34,7 @@ > > > > #include "gt/intel_reset.h" > > > > +#include "i915_gem_context.h" > > #include "intel_dp.h" > > #include "intel_drv.h" > > #include "intel_fbc.h" > > @@ -396,9 +397,11 @@ static void print_context_stats(struct seq_file *m, > > struct i915_gem_context *ctx; > > > > list_for_each_entry(ctx, &i915->contexts.list, link) { > > - struct intel_context *ce; > > + struct i915_gem_engines_iter it; > > + > > + for_each_gem_engine(it, ctx) { > > Why no i915_gem_context_lock_engines in debugfs? Because this version of for_each_gem_engine() provided the locking. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx