[Intel-gfx] [PATCH v3 1/3] drm/i915: simplify allocation of driver-internal requests
Chris Wilson
chris at chris-wilson.co.uk
Tue Jan 12 06:27:32 PST 2016
On Tue, Jan 12, 2016 at 01:56:48PM +0000, Chris Wilson wrote:
> But we were removing the engine->default_context as it complicated the
> rest of the code. I strongly prefer keeping the contexts explicit as
> context separation should be first and foremost in the driver.
$ git grep kernel_context -- drivers/gpu/drm/i915/ | grep request_alloc
drivers/gpu/drm/i915/i915_gem_evict.c: req = i915_gem_request_alloc(ring, dev_priv->kernel_context);
drivers/gpu/drm/i915/intel_overlay.c: return i915_gem_request_alloc(ring, dev_priv->kernel_context);
Changing those *two* callsites to pass NULL seems on the odd side, and
at least for the eviction case discards important information.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list