drm/i915: Remove the impedance mismatch around intel_engine_enable_signaling
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 8 Mar 2018 14:07:32 +0000 (14:07 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Mon, 12 Mar 2018 22:06:19 +0000 (22:06 +0000)
commit6f9ec414ec47eea3f3e2c5ad4c67b4265bbff2a3
tree1369bf5ad7a11ae0822292652b0cb2fb6abd766c
parentcaa1fd660e6f6701255544a3586a498676527fa4
drm/i915: Remove the impedance mismatch around intel_engine_enable_signaling

There is some redundancy between dma_fence->ops->enable_signaling (via
i915_fence_enable_signaling) and our backend,
intel_engine_enable_signaling() in that both levels recheck the fence
status multiple times. If we convert intel_engine_enable_signaling() to
return the information desired by dma_fence->ops->enable_signaling, we
can reduce i915_fence_enable_signaling to a simple stub and avoid
trying to reinterpret the same information.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Mika Kuoppala <mika.kuoppala@intel.com>
Cc: Michal Winiarski <michal.winiarski@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180308140732.25090-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_request.c
drivers/gpu/drm/i915/intel_breadcrumbs.c
drivers/gpu/drm/i915/intel_ringbuffer.h