[PATCH v4 0/4] Replace seqno values with request structures

John.C.Harrison
2014-12-05T13:49:42Z

There is a general feeling that it is better to move away from using a simple
integer 'seqno' value to track batch buffer completion. Instead, the request
structure should be used. That provides for much more flexibility going
forwards. Especially which things like a GPU scheduler (which can re-order batch
buffers and hence seqnos after submission to the hardware), Android sync points
and other such features which potentially make seqno usage more and more
complex.

This patch set does the work of converting most of the driver to use request
structures in preference to seqno values. The only place left that still uses
seqnos is the semaphore code. It was decided to leave that alone for the time
being as the semaphores are hardware based and the hardware only understands
seqno values.

v2: Rebased to newer nightly tree which significantly changed some of the
display MMIO flip code (including making __wait_request public and renaming
it).

v3: Rebased to yet another nightly tree which included execlist pin/unpin
patches. NB: this is based on a tree including a pending update for an unpin
bug. Also includes a fix for a clash with the shrinker.

v4: Remaining patches rebased to remove caching of completion status patches.
This is now future work to be done in a different manner.

[Patches against drm-intel-nightly tree fetched 03/12/2014]

John Harrison (4):
drm/i915: Fix up seqno -> request merge issues
drm/i915: Zero fill the request structure
drm/i915: Add unique id to the request structure for debugging
drm/i915: Additional request structure tracing

drivers/gpu/drm/i915/i915_drv.h | 4 ++++
drivers/gpu/drm/i915/i915_irq.c | 2 +-
drivers/gpu/drm/i915/i915_trace.h | 22 ++++++++++++++++------
drivers/gpu/drm/i915/intel_display.c | 6 ++----
drivers/gpu/drm/i915/intel_lrc.c | 4 +++-
drivers/gpu/drm/i915/intel_ringbuffer.c | 4 +++-
6 files changed, 29 insertions(+), 13 deletions(-)