On Thu, May 26, 2016 at 01:22:04PM +0300, Joonas Lahtinen wrote: > > @@ -2327,6 +2350,14 @@ void intel_cleanup_engine(struct intel_engine_cs *engine) > > > > i915_cmd_parser_fini_ring(engine); > > i915_gem_batch_pool_fini(&engine->batch_pool); > > + > > + kctx = dev_priv->kernel_context; > > + ce = &kctx->engine[engine->id]; > > I'd move these two upper in the function. We can't just yet, because it is silly and has a dev_priv == NULL check. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
