On ke, 2016-07-20 at 14:11 +0100, Chris Wilson wrote:
> The state stored in this struct is not only the information about the
> buffer object, but the ring used to communicate with the hardware. Using
> buffer here is overly specific and, for me at least, conflates with the
> notion of buffer objects themselves.
>
You should list all the renames here so the poor rebasers get to live.
With those listed;
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
Comment below.
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> ---
> drivers/gpu/drm/i915/i915_debugfs.c | 11 ++-
> drivers/gpu/drm/i915/i915_drv.h | 4 +-
> drivers/gpu/drm/i915/i915_gem.c | 16 ++--
> drivers/gpu/drm/i915/i915_gem_context.c | 6 +-
> drivers/gpu/drm/i915/i915_gem_execbuffer.c | 6 +-
> drivers/gpu/drm/i915/i915_gem_gtt.c | 6 +-
> drivers/gpu/drm/i915/i915_gem_request.c | 6 +-
> drivers/gpu/drm/i915/i915_gem_request.h | 2 +-
> drivers/gpu/drm/i915/i915_gpu_error.c | 8 +-
> drivers/gpu/drm/i915/i915_irq.c | 14 ++--
> drivers/gpu/drm/i915/intel_display.c | 10 +--
> drivers/gpu/drm/i915/intel_engine_cs.c | 2 +-
> drivers/gpu/drm/i915/intel_lrc.c | 34 ++++----
> drivers/gpu/drm/i915/intel_mocs.c | 4 +-
> drivers/gpu/drm/i915/intel_overlay.c | 8 +-
> drivers/gpu/drm/i915/intel_ringbuffer.c | 128 ++++++++++++++---------------
> drivers/gpu/drm/i915/intel_ringbuffer.h | 51 ++++++------
> 17 files changed, 157 insertions(+), 159 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index bde68741809b..dccc72d63dd0 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -1419,7 +1419,7 @@ static int i915_hangcheck_info(struct seq_file *m, void *unused)
> intel_runtime_pm_get(dev_priv);
>
> for_each_engine_id(engine, dev_priv, id) {
> - acthd[id] = intel_ring_get_active_head(engine);
> + acthd[id] = intel_engine_get_active_head(engine);
> seqno[id] = intel_engine_get_seqno(engine);
> }
>
> @@ -2017,12 +2017,11 @@ static int i915_gem_framebuffer_info(struct seq_file *m, void *data)
> return 0;
> }
>
> -static void describe_ctx_ringbuf(struct seq_file *m,
> - struct intel_ringbuffer *ringbuf)
> +static void describe_ctx_ring(struct seq_file *m, struct intel_ring *ring)
> {
> seq_printf(m, " (ringbuffer, space: %d, head: %u, tail: %u, last head: %d)",
> - ringbuf->space, ringbuf->head, ringbuf->tail,
> - ringbuf->last_retired_head);
> + ring->space, ring->head, ring->tail,
> + ring->last_retired_head);
> }
>
> static int i915_context_status(struct seq_file *m, void *unused)
> @@ -2067,7 +2066,7 @@ static int i915_context_status(struct seq_file *m, void *unused)
> if (ce->state)
> describe_obj(m, ce->state);
> if (ce->ring)
> - describe_ctx_ringbuf(m, ce->ring);
> + describe_ctx_ring(m, ce->ring);
> seq_putc(m, '\n');
> }
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index 87e06a6a797a..f32ec6db5bfa 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -518,7 +518,7 @@ struct drm_i915_error_state {
> bool waiting;
> int num_waiters;
> int hangcheck_score;
> - enum intel_ring_hangcheck_action hangcheck_action;
> + enum intel_engine_hangcheck_action hangcheck_action;
> int num_requests;
>
> /* our own tracking of ring head and tail */
> @@ -894,7 +894,7 @@ struct i915_gem_context {
>
> struct intel_context {
> struct drm_i915_gem_object *state;
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
> struct i915_vma *lrc_vma;
> uint32_t *lrc_reg_state;
> u64 lrc_desc;
> diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
> index 40047eb48826..95dbcfd94a80 100644
> --- a/drivers/gpu/drm/i915/i915_gem.c
> +++ b/drivers/gpu/drm/i915/i915_gem.c
> @@ -2486,7 +2486,7 @@ static void i915_gem_reset_engine_status(struct intel_engine_cs *engine)
>
> static void i915_gem_reset_engine_cleanup(struct intel_engine_cs *engine)
> {
> - struct intel_ringbuffer *buffer;
> + struct intel_ring *ring;
>
> while (!list_empty(&engine->active_list)) {
> struct drm_i915_gem_object *obj;
> @@ -2502,7 +2502,7 @@ static void i915_gem_reset_engine_cleanup(struct intel_engine_cs *engine)
> * (lockless) lookup doesn't try and wait upon the request as we
> * reset it.
> */
> - intel_ring_init_seqno(engine, engine->last_submitted_seqno);
> + intel_engine_init_seqno(engine, engine->last_submitted_seqno);
>
> /*
> * Clear the execlists queue up before freeing the requests, as those
> @@ -2541,9 +2541,9 @@ static void i915_gem_reset_engine_cleanup(struct intel_engine_cs *engine)
> * upon reset is less than when we start. Do one more pass over
> * all the ringbuffers to reset last_retired_head.
> */
> - list_for_each_entry(buffer, &engine->buffers, link) {
> - buffer->last_retired_head = buffer->tail;
> - intel_ring_update_space(buffer);
> + list_for_each_entry(ring, &engine->buffers, link) {
> + ring->last_retired_head = ring->tail;
> + intel_ring_update_space(ring);
> }
>
> engine->i915->gt.active_engines &= ~intel_engine_flag(engine);
> @@ -2867,7 +2867,7 @@ __i915_gem_object_sync(struct drm_i915_gem_object *obj,
>
> i915_gem_object_retire_request(obj, from_req);
> } else {
> - int idx = intel_ring_sync_index(from, to);
> + int idx = intel_engine_sync_index(from, to);
> u32 seqno = i915_gem_request_get_seqno(from_req);
>
> WARN_ON(!to_req);
> @@ -4567,8 +4567,8 @@ int i915_gem_init(struct drm_device *dev)
>
> if (!i915.enable_execlists) {
> dev_priv->gt.execbuf_submit = i915_gem_ringbuffer_submission;
> - dev_priv->gt.cleanup_engine = intel_cleanup_engine;
> - dev_priv->gt.stop_engine = intel_stop_engine;
> + dev_priv->gt.cleanup_engine = intel_engine_cleanup;
> + dev_priv->gt.stop_engine = intel_engine_stop;
I guess you added renaming gt.*_engine to your TODO already.
Regards, Joonas
> } else {
> dev_priv->gt.execbuf_submit = intel_execlists_submission;
> dev_priv->gt.cleanup_engine = intel_logical_ring_cleanup;
> diff --git a/drivers/gpu/drm/i915/i915_gem_context.c b/drivers/gpu/drm/i915/i915_gem_context.c
> index c8bf7b8e959f..d9b861b856dc 100644
> --- a/drivers/gpu/drm/i915/i915_gem_context.c
> +++ b/drivers/gpu/drm/i915/i915_gem_context.c
> @@ -174,7 +174,7 @@ void i915_gem_context_free(struct kref *ctx_ref)
>
> WARN_ON(ce->pin_count);
> if (ce->ring)
> - intel_ringbuffer_free(ce->ring);
> + intel_ring_free(ce->ring);
>
> i915_gem_object_put(ce->state);
> }
> @@ -552,7 +552,7 @@ static inline int
> mi_set_context(struct drm_i915_gem_request *req, u32 hw_flags)
> {
> struct drm_i915_private *dev_priv = req->i915;
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> u32 flags = hw_flags | MI_MM_SPACE_GTT;
> const int num_rings =
> /* Use an extended w/a on ivb+ if signalling from other rings */
> @@ -654,7 +654,7 @@ mi_set_context(struct drm_i915_gem_request *req, u32 hw_flags)
> static int remap_l3(struct drm_i915_gem_request *req, int slice)
> {
> u32 *remap_info = req->i915->l3_parity.remap_info[slice];
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int i, ret;
>
> if (!remap_info)
> diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> index 501a1751d432..12adfec2d6a9 100644
> --- a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> +++ b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
> @@ -1001,7 +1001,7 @@ i915_gem_execbuffer_move_to_gpu(struct drm_i915_gem_request *req,
> /* Unconditionally invalidate gpu caches and ensure that we do flush
> * any residual writes from the previous batch.
> */
> - return intel_ring_invalidate_all_caches(req);
> + return intel_engine_invalidate_all_caches(req);
> }
>
> static bool
> @@ -1173,7 +1173,7 @@ i915_gem_execbuffer_retire_commands(struct i915_execbuffer_params *params)
> static int
> i915_reset_gen7_sol_offsets(struct drm_i915_gem_request *req)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret, i;
>
> if (!IS_GEN7(req->i915) || req->engine->id != RCS) {
> @@ -1303,7 +1303,7 @@ i915_gem_ringbuffer_submission(struct i915_execbuffer_params *params,
>
> if (params->engine->id == RCS &&
> instp_mode != dev_priv->relative_constants_mode) {
> - struct intel_ringbuffer *ring = params->request->ring;
> + struct intel_ring *ring = params->request->ring;
>
> ret = intel_ring_begin(params->request, 4);
> if (ret)
> diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
> index a48329baf432..01b825169164 100644
> --- a/drivers/gpu/drm/i915/i915_gem_gtt.c
> +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
> @@ -669,7 +669,7 @@ static int gen8_write_pdp(struct drm_i915_gem_request *req,
> unsigned entry,
> dma_addr_t addr)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> BUG_ON(entry >= 4);
> @@ -1660,7 +1660,7 @@ static uint32_t get_pd_offset(struct i915_hw_ppgtt *ppgtt)
> static int hsw_mm_switch(struct i915_hw_ppgtt *ppgtt,
> struct drm_i915_gem_request *req)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> /* NB: TLBs must be flushed and invalidated before a switch */
> @@ -1688,7 +1688,7 @@ static int hsw_mm_switch(struct i915_hw_ppgtt *ppgtt,
> static int gen7_mm_switch(struct i915_hw_ppgtt *ppgtt,
> struct drm_i915_gem_request *req)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> /* NB: TLBs must be flushed and invalidated before a switch */
> diff --git a/drivers/gpu/drm/i915/i915_gem_request.c b/drivers/gpu/drm/i915/i915_gem_request.c
> index 0f415606a383..54b27369225a 100644
> --- a/drivers/gpu/drm/i915/i915_gem_request.c
> +++ b/drivers/gpu/drm/i915/i915_gem_request.c
> @@ -244,7 +244,7 @@ static int i915_gem_init_seqno(struct drm_i915_private *dev_priv, u32 seqno)
>
> /* Finally reset hw state */
> for_each_engine(engine, dev_priv)
> - intel_ring_init_seqno(engine, seqno);
> + intel_engine_init_seqno(engine, seqno);
>
> return 0;
> }
> @@ -425,7 +425,7 @@ void __i915_add_request(struct drm_i915_gem_request *request,
> bool flush_caches)
> {
> struct intel_engine_cs *engine;
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
> u32 request_start;
> u32 reserved_tail;
> int ret;
> @@ -456,7 +456,7 @@ void __i915_add_request(struct drm_i915_gem_request *request,
> if (i915.enable_execlists)
> ret = logical_ring_flush_all_caches(request);
> else
> - ret = intel_ring_flush_all_caches(request);
> + ret = intel_engine_flush_all_caches(request);
> /* Not allowed to fail! */
> WARN(ret, "*_ring_flush_all_caches failed: %d!\n", ret);
> }
> diff --git a/drivers/gpu/drm/i915/i915_gem_request.h b/drivers/gpu/drm/i915/i915_gem_request.h
> index 68868d825d9d..382ca5a163eb 100644
> --- a/drivers/gpu/drm/i915/i915_gem_request.h
> +++ b/drivers/gpu/drm/i915/i915_gem_request.h
> @@ -61,7 +61,7 @@ struct drm_i915_gem_request {
> */
> struct i915_gem_context *ctx;
> struct intel_engine_cs *engine;
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
> struct intel_signal_node signaling;
>
> /** GEM sequence number associated with the previous request,
> diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c b/drivers/gpu/drm/i915/i915_gpu_error.c
> index 09997c6adcd2..2fbe81d51af1 100644
> --- a/drivers/gpu/drm/i915/i915_gpu_error.c
> +++ b/drivers/gpu/drm/i915/i915_gpu_error.c
> @@ -221,7 +221,7 @@ static void print_error_buffers(struct drm_i915_error_state_buf *m,
> }
> }
>
> -static const char *hangcheck_action_to_str(enum intel_ring_hangcheck_action a)
> +static const char *hangcheck_action_to_str(enum intel_engine_hangcheck_action a)
> {
> switch (a) {
> case HANGCHECK_IDLE:
> @@ -882,7 +882,7 @@ static void gen8_record_semaphore_state(struct drm_i915_private *dev_priv,
> signal_offset = (GEN8_SIGNAL_OFFSET(engine, id) & (PAGE_SIZE - 1))
> / 4;
> tmp = error->semaphore_obj->pages[0];
> - idx = intel_ring_sync_index(engine, to);
> + idx = intel_engine_sync_index(engine, to);
>
> ering->semaphore_mboxes[idx] = tmp[signal_offset];
> ering->semaphore_seqno[idx] = engine->semaphore.sync_seqno[idx];
> @@ -983,7 +983,7 @@ static void i915_record_ring_state(struct drm_i915_private *dev_priv,
>
> ering->waiting = intel_engine_has_waiter(engine);
> ering->instpm = I915_READ(RING_INSTPM(engine->mmio_base));
> - ering->acthd = intel_ring_get_active_head(engine);
> + ering->acthd = intel_engine_get_active_head(engine);
> ering->seqno = intel_engine_get_seqno(engine);
> ering->last_seqno = engine->last_submitted_seqno;
> ering->start = I915_READ_START(engine);
> @@ -1091,7 +1091,7 @@ static void i915_gem_record_rings(struct drm_i915_private *dev_priv,
> request = i915_gem_find_active_request(engine);
> if (request) {
> struct i915_address_space *vm;
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
>
> vm = request->ctx->ppgtt ?
> &request->ctx->ppgtt->base : &ggtt->base;
> diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
> index 7104dc1463eb..5903111db718 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -2993,7 +2993,7 @@ static bool subunits_stuck(struct intel_engine_cs *engine)
> return stuck;
> }
>
> -static enum intel_ring_hangcheck_action
> +static enum intel_engine_hangcheck_action
> head_stuck(struct intel_engine_cs *engine, u64 acthd)
> {
> if (acthd != engine->hangcheck.acthd) {
> @@ -3011,11 +3011,11 @@ head_stuck(struct intel_engine_cs *engine, u64 acthd)
> return HANGCHECK_HUNG;
> }
>
> -static enum intel_ring_hangcheck_action
> -ring_stuck(struct intel_engine_cs *engine, u64 acthd)
> +static enum intel_engine_hangcheck_action
> +engine_stuck(struct intel_engine_cs *engine, u64 acthd)
> {
> struct drm_i915_private *dev_priv = engine->i915;
> - enum intel_ring_hangcheck_action ha;
> + enum intel_engine_hangcheck_action ha;
> u32 tmp;
>
> ha = head_stuck(engine, acthd);
> @@ -3124,7 +3124,7 @@ static void i915_hangcheck_elapsed(struct work_struct *work)
> if (engine->irq_seqno_barrier)
> engine->irq_seqno_barrier(engine);
>
> - acthd = intel_ring_get_active_head(engine);
> + acthd = intel_engine_get_active_head(engine);
> seqno = intel_engine_get_seqno(engine);
>
> /* Reset stuck interrupts between batch advances */
> @@ -3154,8 +3154,8 @@ static void i915_hangcheck_elapsed(struct work_struct *work)
> * being repeatedly kicked and so responsible
> * for stalling the machine.
> */
> - engine->hangcheck.action = ring_stuck(engine,
> - acthd);
> + engine->hangcheck.action =
> + engine_stuck(engine, acthd);
>
> switch (engine->hangcheck.action) {
> case HANGCHECK_IDLE:
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index d1932840a268..bff172c45ff7 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -11123,7 +11123,7 @@ static int intel_gen2_queue_flip(struct drm_device *dev,
> struct drm_i915_gem_request *req,
> uint32_t flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> u32 flip_mask;
> int ret;
> @@ -11157,7 +11157,7 @@ static int intel_gen3_queue_flip(struct drm_device *dev,
> struct drm_i915_gem_request *req,
> uint32_t flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> u32 flip_mask;
> int ret;
> @@ -11188,7 +11188,7 @@ static int intel_gen4_queue_flip(struct drm_device *dev,
> struct drm_i915_gem_request *req,
> uint32_t flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> struct drm_i915_private *dev_priv = to_i915(dev);
> struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> uint32_t pf, pipesrc;
> @@ -11226,7 +11226,7 @@ static int intel_gen6_queue_flip(struct drm_device *dev,
> struct drm_i915_gem_request *req,
> uint32_t flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> struct drm_i915_private *dev_priv = to_i915(dev);
> struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> uint32_t pf, pipesrc;
> @@ -11261,7 +11261,7 @@ static int intel_gen7_queue_flip(struct drm_device *dev,
> struct drm_i915_gem_request *req,
> uint32_t flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> uint32_t plane_bit = 0;
> int len, ret;
> diff --git a/drivers/gpu/drm/i915/intel_engine_cs.c b/drivers/gpu/drm/i915/intel_engine_cs.c
> index f4a35ec78481..f00bd55fe582 100644
> --- a/drivers/gpu/drm/i915/intel_engine_cs.c
> +++ b/drivers/gpu/drm/i915/intel_engine_cs.c
> @@ -154,7 +154,7 @@ cleanup:
> if (i915.enable_execlists)
> intel_logical_ring_cleanup(&dev_priv->engine[i]);
> else
> - intel_cleanup_engine(&dev_priv->engine[i]);
> + intel_engine_cleanup(&dev_priv->engine[i]);
> }
>
> return ret;
> diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
> index 7bc1d0c92799..5b9f98f6ed87 100644
> --- a/drivers/gpu/drm/i915/intel_lrc.c
> +++ b/drivers/gpu/drm/i915/intel_lrc.c
> @@ -767,7 +767,7 @@ err_unpin:
> static int
> intel_logical_ring_advance_and_submit(struct drm_i915_gem_request *request)
> {
> - struct intel_ringbuffer *ring = request->ring;
> + struct intel_ring *ring = request->ring;
> struct intel_engine_cs *engine = request->engine;
>
> intel_ring_advance(ring);
> @@ -818,7 +818,7 @@ int intel_execlists_submission(struct i915_execbuffer_params *params,
> struct drm_device *dev = params->dev;
> struct intel_engine_cs *engine = params->engine;
> struct drm_i915_private *dev_priv = to_i915(dev);
> - struct intel_ringbuffer *ring = params->request->ring;
> + struct intel_ring *ring = params->request->ring;
> u64 exec_start;
> int instp_mode;
> u32 instp_mask;
> @@ -973,7 +973,7 @@ static int intel_lr_context_pin(struct i915_gem_context *ctx,
>
> lrc_reg_state = vaddr + LRC_STATE_PN * PAGE_SIZE;
>
> - ret = intel_pin_and_map_ringbuffer_obj(dev_priv, ce->ring);
> + ret = intel_pin_and_map_ring(dev_priv, ce->ring);
> if (ret)
> goto unpin_map;
>
> @@ -1011,7 +1011,7 @@ void intel_lr_context_unpin(struct i915_gem_context *ctx,
> if (--ce->pin_count)
> return;
>
> - intel_unpin_ringbuffer_obj(ce->ring);
> + intel_unpin_ring(ce->ring);
>
> i915_gem_object_unpin_map(ce->state);
> i915_gem_object_ggtt_unpin(ce->state);
> @@ -1027,7 +1027,7 @@ static int intel_logical_ring_workarounds_emit(struct drm_i915_gem_request *req)
> {
> int ret, i;
> struct intel_engine_cs *engine = req->engine;
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> struct i915_workarounds *w = &req->i915->workarounds;
>
> if (w->count == 0)
> @@ -1543,7 +1543,7 @@ static int gen9_init_render_ring(struct intel_engine_cs *engine)
> static int intel_logical_ring_emit_pdps(struct drm_i915_gem_request *req)
> {
> struct i915_hw_ppgtt *ppgtt = req->ctx->ppgtt;
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> const int num_lri_cmds = GEN8_LEGACY_PDPES * 2;
> int i, ret;
>
> @@ -1570,7 +1570,7 @@ static int intel_logical_ring_emit_pdps(struct drm_i915_gem_request *req)
> static int gen8_emit_bb_start(struct drm_i915_gem_request *req,
> u64 offset, unsigned dispatch_flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> bool ppgtt = !(dispatch_flags & I915_DISPATCH_SECURE);
> int ret;
>
> @@ -1627,8 +1627,8 @@ static int gen8_emit_flush(struct drm_i915_gem_request *request,
> u32 invalidate_domains,
> u32 unused)
> {
> - struct intel_ringbuffer *ring = request->ring;
> - uint32_t cmd;
> + struct intel_ring *ring = request->ring;
> + u32 cmd;
> int ret;
>
> ret = intel_ring_begin(request, 4);
> @@ -1665,7 +1665,7 @@ static int gen8_emit_flush_render(struct drm_i915_gem_request *request,
> u32 invalidate_domains,
> u32 flush_domains)
> {
> - struct intel_ringbuffer *ring = request->ring;
> + struct intel_ring *ring = request->ring;
> struct intel_engine_cs *engine = request->engine;
> u32 scratch_addr = engine->scratch.gtt_offset + 2 * CACHELINE_BYTES;
> bool vf_flush_wa = false, dc_flush_wa = false;
> @@ -1779,7 +1779,7 @@ static void bxt_a_seqno_barrier(struct intel_engine_cs *engine)
>
> static int gen8_emit_request(struct drm_i915_gem_request *request)
> {
> - struct intel_ringbuffer *ring = request->ring;
> + struct intel_ring *ring = request->ring;
> int ret;
>
> ret = intel_ring_begin(request, 6 + WA_TAIL_DWORDS);
> @@ -1802,7 +1802,7 @@ static int gen8_emit_request(struct drm_i915_gem_request *request)
>
> static int gen8_emit_request_render(struct drm_i915_gem_request *request)
> {
> - struct intel_ringbuffer *ring = request->ring;
> + struct intel_ring *ring = request->ring;
> int ret;
>
> ret = intel_ring_begin(request, 8 + WA_TAIL_DWORDS);
> @@ -2154,7 +2154,7 @@ static int
> populate_lr_context(struct i915_gem_context *ctx,
> struct drm_i915_gem_object *ctx_obj,
> struct intel_engine_cs *engine,
> - struct intel_ringbuffer *ringbuf)
> + struct intel_ring *ring)
> {
> struct drm_i915_private *dev_priv = ctx->i915;
> struct i915_hw_ppgtt *ppgtt = ctx->ppgtt;
> @@ -2207,7 +2207,7 @@ populate_lr_context(struct i915_gem_context *ctx,
> RING_START(engine->mmio_base), 0);
> ASSIGN_CTX_REG(reg_state, CTX_RING_BUFFER_CONTROL,
> RING_CTL(engine->mmio_base),
> - ((ringbuf->size - PAGE_SIZE) & RING_NR_PAGES) | RING_VALID);
> + ((ring->size - PAGE_SIZE) & RING_NR_PAGES) | RING_VALID);
> ASSIGN_CTX_REG(reg_state, CTX_BB_HEAD_U,
> RING_BBADDR_UDW(engine->mmio_base), 0);
> ASSIGN_CTX_REG(reg_state, CTX_BB_HEAD_L,
> @@ -2335,7 +2335,7 @@ static int execlists_context_deferred_alloc(struct i915_gem_context *ctx,
> struct drm_i915_gem_object *ctx_obj;
> struct intel_context *ce = &ctx->engine[engine->id];
> uint32_t context_size;
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
> int ret;
>
> WARN_ON(ce->state);
> @@ -2351,7 +2351,7 @@ static int execlists_context_deferred_alloc(struct i915_gem_context *ctx,
> return PTR_ERR(ctx_obj);
> }
>
> - ring = intel_engine_create_ringbuffer(engine, ctx->ring_size);
> + ring = intel_engine_create_ring(engine, ctx->ring_size);
> if (IS_ERR(ring)) {
> ret = PTR_ERR(ring);
> goto error_deref_obj;
> @@ -2370,7 +2370,7 @@ static int execlists_context_deferred_alloc(struct i915_gem_context *ctx,
> return 0;
>
> error_ringbuf:
> - intel_ringbuffer_free(ring);
> + intel_ring_free(ring);
> error_deref_obj:
> i915_gem_object_put(ctx_obj);
> ce->ring = NULL;
> diff --git a/drivers/gpu/drm/i915/intel_mocs.c b/drivers/gpu/drm/i915/intel_mocs.c
> index fe63c7e79fb1..58db0e330ee6 100644
> --- a/drivers/gpu/drm/i915/intel_mocs.c
> +++ b/drivers/gpu/drm/i915/intel_mocs.c
> @@ -276,7 +276,7 @@ int intel_mocs_init_engine(struct intel_engine_cs *engine)
> static int emit_mocs_control_table(struct drm_i915_gem_request *req,
> const struct drm_i915_mocs_table *table)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> enum intel_engine_id engine = req->engine->id;
> unsigned int index;
> int ret;
> @@ -336,7 +336,7 @@ static inline u32 l3cc_combine(const struct drm_i915_mocs_table *table,
> static int emit_mocs_l3cc_table(struct drm_i915_gem_request *req,
> const struct drm_i915_mocs_table *table)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> unsigned int i;
> int ret;
>
> diff --git a/drivers/gpu/drm/i915/intel_overlay.c b/drivers/gpu/drm/i915/intel_overlay.c
> index 84b8f74bd13c..a5071e281088 100644
> --- a/drivers/gpu/drm/i915/intel_overlay.c
> +++ b/drivers/gpu/drm/i915/intel_overlay.c
> @@ -235,7 +235,7 @@ static int intel_overlay_on(struct intel_overlay *overlay)
> struct drm_i915_private *dev_priv = overlay->i915;
> struct intel_engine_cs *engine = &dev_priv->engine[RCS];
> struct drm_i915_gem_request *req;
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
> int ret;
>
> WARN_ON(overlay->active);
> @@ -270,7 +270,7 @@ static int intel_overlay_continue(struct intel_overlay *overlay,
> struct drm_i915_private *dev_priv = overlay->i915;
> struct intel_engine_cs *engine = &dev_priv->engine[RCS];
> struct drm_i915_gem_request *req;
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
> u32 flip_addr = overlay->flip_addr;
> u32 tmp;
> int ret;
> @@ -340,7 +340,7 @@ static int intel_overlay_off(struct intel_overlay *overlay)
> struct drm_i915_private *dev_priv = overlay->i915;
> struct intel_engine_cs *engine = &dev_priv->engine[RCS];
> struct drm_i915_gem_request *req;
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
> u32 flip_addr = overlay->flip_addr;
> int ret;
>
> @@ -426,7 +426,7 @@ static int intel_overlay_release_old_vid(struct intel_overlay *overlay)
> if (I915_READ(ISR) & I915_OVERLAY_PLANE_FLIP_PENDING_INTERRUPT) {
> /* synchronous slowpath */
> struct drm_i915_gem_request *req;
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
>
> req = i915_gem_request_alloc(engine, NULL);
> if (IS_ERR(req))
> diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c
> index 3cfbfe40f6e8..9aaf81ba66c8 100644
> --- a/drivers/gpu/drm/i915/intel_ringbuffer.c
> +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c
> @@ -47,7 +47,7 @@ int __intel_ring_space(int head, int tail, int size)
> return space - I915_RING_FREE_SPACE;
> }
>
> -void intel_ring_update_space(struct intel_ringbuffer *ringbuf)
> +void intel_ring_update_space(struct intel_ring *ringbuf)
> {
> if (ringbuf->last_retired_head != -1) {
> ringbuf->head = ringbuf->last_retired_head;
> @@ -60,9 +60,10 @@ void intel_ring_update_space(struct intel_ringbuffer *ringbuf)
>
> static void __intel_engine_submit(struct intel_engine_cs *engine)
> {
> - struct intel_ringbuffer *ringbuf = engine->buffer;
> - ringbuf->tail &= ringbuf->size - 1;
> - engine->write_tail(engine, ringbuf->tail);
> + struct intel_ring *ring = engine->buffer;
> +
> + ring->tail &= ring->size - 1;
> + engine->write_tail(engine, ring->tail);
> }
>
> static int
> @@ -70,7 +71,7 @@ gen2_render_ring_flush(struct drm_i915_gem_request *req,
> u32 invalidate_domains,
> u32 flush_domains)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> u32 cmd;
> int ret;
>
> @@ -97,7 +98,7 @@ gen4_render_ring_flush(struct drm_i915_gem_request *req,
> u32 invalidate_domains,
> u32 flush_domains)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> u32 cmd;
> int ret;
>
> @@ -187,7 +188,7 @@ gen4_render_ring_flush(struct drm_i915_gem_request *req,
> static int
> intel_emit_post_sync_nonzero_flush(struct drm_i915_gem_request *req)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> u32 scratch_addr =
> req->engine->scratch.gtt_offset + 2 * CACHELINE_BYTES;
> int ret;
> @@ -224,7 +225,7 @@ static int
> gen6_render_ring_flush(struct drm_i915_gem_request *req,
> u32 invalidate_domains, u32 flush_domains)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> u32 scratch_addr =
> req->engine->scratch.gtt_offset + 2 * CACHELINE_BYTES;
> u32 flags = 0;
> @@ -277,7 +278,7 @@ gen6_render_ring_flush(struct drm_i915_gem_request *req,
> static int
> gen7_render_ring_cs_stall_wa(struct drm_i915_gem_request *req)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> ret = intel_ring_begin(req, 4);
> @@ -299,7 +300,7 @@ static int
> gen7_render_ring_flush(struct drm_i915_gem_request *req,
> u32 invalidate_domains, u32 flush_domains)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> u32 scratch_addr =
> req->engine->scratch.gtt_offset + 2 * CACHELINE_BYTES;
> u32 flags = 0;
> @@ -364,7 +365,7 @@ static int
> gen8_emit_pipe_control(struct drm_i915_gem_request *req,
> u32 flags, u32 scratch_addr)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> ret = intel_ring_begin(req, 6);
> @@ -427,7 +428,7 @@ static void ring_write_tail(struct intel_engine_cs *engine,
> I915_WRITE_TAIL(engine, value);
> }
>
> -u64 intel_ring_get_active_head(struct intel_engine_cs *engine)
> +u64 intel_engine_get_active_head(struct intel_engine_cs *engine)
> {
> struct drm_i915_private *dev_priv = engine->i915;
> u64 acthd;
> @@ -553,8 +554,8 @@ static bool stop_ring(struct intel_engine_cs *engine)
> static int init_ring_common(struct intel_engine_cs *engine)
> {
> struct drm_i915_private *dev_priv = engine->i915;
> - struct intel_ringbuffer *ringbuf = engine->buffer;
> - struct drm_i915_gem_object *obj = ringbuf->obj;
> + struct intel_ring *ring = engine->buffer;
> + struct drm_i915_gem_object *obj = ring->obj;
> int ret = 0;
>
> intel_uncore_forcewake_get(dev_priv, FORCEWAKE_ALL);
> @@ -604,7 +605,7 @@ static int init_ring_common(struct intel_engine_cs *engine)
> (void)I915_READ_HEAD(engine);
>
> I915_WRITE_CTL(engine,
> - ((ringbuf->size - PAGE_SIZE) & RING_NR_PAGES)
> + ((ring->size - PAGE_SIZE) & RING_NR_PAGES)
> | RING_VALID);
>
> /* If the head is still not zero, the ring is dead */
> @@ -623,10 +624,10 @@ static int init_ring_common(struct intel_engine_cs *engine)
> goto out;
> }
>
> - ringbuf->last_retired_head = -1;
> - ringbuf->head = I915_READ_HEAD(engine);
> - ringbuf->tail = I915_READ_TAIL(engine) & TAIL_ADDR;
> - intel_ring_update_space(ringbuf);
> + ring->last_retired_head = -1;
> + ring->head = I915_READ_HEAD(engine);
> + ring->tail = I915_READ_TAIL(engine) & TAIL_ADDR;
> + intel_ring_update_space(ring);
>
> intel_engine_init_hangcheck(engine);
>
> @@ -680,7 +681,7 @@ err:
>
> static int intel_ring_workarounds_emit(struct drm_i915_gem_request *req)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> struct i915_workarounds *w = &req->i915->workarounds;
> int ret, i;
>
> @@ -688,7 +689,7 @@ static int intel_ring_workarounds_emit(struct drm_i915_gem_request *req)
> return 0;
>
> req->engine->gpu_caches_dirty = true;
> - ret = intel_ring_flush_all_caches(req);
> + ret = intel_engine_flush_all_caches(req);
> if (ret)
> return ret;
>
> @@ -706,7 +707,7 @@ static int intel_ring_workarounds_emit(struct drm_i915_gem_request *req)
> intel_ring_advance(ring);
>
> req->engine->gpu_caches_dirty = true;
> - ret = intel_ring_flush_all_caches(req);
> + ret = intel_engine_flush_all_caches(req);
> if (ret)
> return ret;
>
> @@ -1324,7 +1325,7 @@ static int gen8_rcs_signal(struct drm_i915_gem_request *signaller_req,
> unsigned int num_dwords)
> {
> #define MBOX_UPDATE_DWORDS 8
> - struct intel_ringbuffer *signaller = signaller_req->ring;
> + struct intel_ring *signaller = signaller_req->ring;
> struct drm_i915_private *dev_priv = signaller_req->i915;
> struct intel_engine_cs *waiter;
> enum intel_engine_id id;
> @@ -1366,7 +1367,7 @@ static int gen8_xcs_signal(struct drm_i915_gem_request *signaller_req,
> unsigned int num_dwords)
> {
> #define MBOX_UPDATE_DWORDS 6
> - struct intel_ringbuffer *signaller = signaller_req->ring;
> + struct intel_ring *signaller = signaller_req->ring;
> struct drm_i915_private *dev_priv = signaller_req->i915;
> struct intel_engine_cs *waiter;
> enum intel_engine_id id;
> @@ -1405,7 +1406,7 @@ static int gen8_xcs_signal(struct drm_i915_gem_request *signaller_req,
> static int gen6_signal(struct drm_i915_gem_request *signaller_req,
> unsigned int num_dwords)
> {
> - struct intel_ringbuffer *signaller = signaller_req->ring;
> + struct intel_ring *signaller = signaller_req->ring;
> struct drm_i915_private *dev_priv = signaller_req->i915;
> struct intel_engine_cs *useless;
> enum intel_engine_id id;
> @@ -1449,7 +1450,7 @@ static int gen6_signal(struct drm_i915_gem_request *signaller_req,
> static int
> gen6_add_request(struct drm_i915_gem_request *req)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> if (req->engine->semaphore.signal)
> @@ -1473,7 +1474,7 @@ static int
> gen8_render_add_request(struct drm_i915_gem_request *req)
> {
> struct intel_engine_cs *engine = req->engine;
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> if (engine->semaphore.signal)
> @@ -1518,7 +1519,7 @@ gen8_ring_sync(struct drm_i915_gem_request *waiter_req,
> struct intel_engine_cs *signaller,
> u32 seqno)
> {
> - struct intel_ringbuffer *waiter = waiter_req->ring;
> + struct intel_ring *waiter = waiter_req->ring;
> struct drm_i915_private *dev_priv = waiter_req->i915;
> u64 offset = GEN8_WAIT_OFFSET(waiter_req->engine, signaller->id);
> struct i915_hw_ppgtt *ppgtt;
> @@ -1552,7 +1553,7 @@ gen6_ring_sync(struct drm_i915_gem_request *waiter_req,
> struct intel_engine_cs *signaller,
> u32 seqno)
> {
> - struct intel_ringbuffer *waiter = waiter_req->ring;
> + struct intel_ring *waiter = waiter_req->ring;
> u32 dw1 = MI_SEMAPHORE_MBOX |
> MI_SEMAPHORE_COMPARE |
> MI_SEMAPHORE_REGISTER;
> @@ -1686,7 +1687,7 @@ bsd_ring_flush(struct drm_i915_gem_request *req,
> u32 invalidate_domains,
> u32 flush_domains)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> ret = intel_ring_begin(req, 2);
> @@ -1702,7 +1703,7 @@ bsd_ring_flush(struct drm_i915_gem_request *req,
> static int
> i9xx_add_request(struct drm_i915_gem_request *req)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> ret = intel_ring_begin(req, 4);
> @@ -1780,7 +1781,7 @@ i965_dispatch_execbuffer(struct drm_i915_gem_request *req,
> u64 offset, u32 length,
> unsigned dispatch_flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> ret = intel_ring_begin(req, 2);
> @@ -1807,7 +1808,7 @@ i830_dispatch_execbuffer(struct drm_i915_gem_request *req,
> u64 offset, u32 len,
> unsigned dispatch_flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> u32 cs_offset = req->engine->scratch.gtt_offset;
> int ret;
>
> @@ -1869,7 +1870,7 @@ i915_dispatch_execbuffer(struct drm_i915_gem_request *req,
> u64 offset, u32 len,
> unsigned dispatch_flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> ret = intel_ring_begin(req, 2);
> @@ -1977,7 +1978,7 @@ static int init_phys_status_page(struct intel_engine_cs *engine)
> return 0;
> }
>
> -void intel_unpin_ringbuffer_obj(struct intel_ringbuffer *ringbuf)
> +void intel_unpin_ring(struct intel_ring *ringbuf)
> {
> GEM_BUG_ON(!ringbuf->vma);
> GEM_BUG_ON(!ringbuf->vaddr);
> @@ -1992,8 +1993,8 @@ void intel_unpin_ringbuffer_obj(struct intel_ringbuffer *ringbuf)
> ringbuf->vma = NULL;
> }
>
> -int intel_pin_and_map_ringbuffer_obj(struct drm_i915_private *dev_priv,
> - struct intel_ringbuffer *ringbuf)
> +int intel_pin_and_map_ring(struct drm_i915_private *dev_priv,
> + struct intel_ring *ringbuf)
> {
> struct drm_i915_gem_object *obj = ringbuf->obj;
> /* Ring wraparound at offset 0 sometimes hangs. No idea why. */
> @@ -2045,14 +2046,14 @@ err_unpin:
> return ret;
> }
>
> -static void intel_destroy_ringbuffer_obj(struct intel_ringbuffer *ringbuf)
> +static void intel_destroy_ringbuffer_obj(struct intel_ring *ringbuf)
> {
> i915_gem_object_put(ringbuf->obj);
> ringbuf->obj = NULL;
> }
>
> static int intel_alloc_ringbuffer_obj(struct drm_device *dev,
> - struct intel_ringbuffer *ringbuf)
> + struct intel_ring *ringbuf)
> {
> struct drm_i915_gem_object *obj;
>
> @@ -2072,10 +2073,10 @@ static int intel_alloc_ringbuffer_obj(struct drm_device *dev,
> return 0;
> }
>
> -struct intel_ringbuffer *
> -intel_engine_create_ringbuffer(struct intel_engine_cs *engine, int size)
> +struct intel_ring *
> +intel_engine_create_ring(struct intel_engine_cs *engine, int size)
> {
> - struct intel_ringbuffer *ring;
> + struct intel_ring *ring;
> int ret;
>
> ring = kzalloc(sizeof(*ring), GFP_KERNEL);
> @@ -2113,7 +2114,7 @@ intel_engine_create_ringbuffer(struct intel_engine_cs *engine, int size)
> }
>
> void
> -intel_ringbuffer_free(struct intel_ringbuffer *ring)
> +intel_ring_free(struct intel_ring *ring)
> {
> intel_destroy_ringbuffer_obj(ring);
> list_del(&ring->link);
> @@ -2174,7 +2175,7 @@ static void intel_ring_context_unpin(struct i915_gem_context *ctx,
> static int intel_init_engine(struct intel_engine_cs *engine)
> {
> struct drm_i915_private *dev_priv = engine->i915;
> - struct intel_ringbuffer *ringbuf;
> + struct intel_ring *ringbuf;
> int ret;
>
> WARN_ON(engine->buffer);
> @@ -2199,7 +2200,7 @@ static int intel_init_engine(struct intel_engine_cs *engine)
> if (ret)
> goto error;
>
> - ringbuf = intel_engine_create_ringbuffer(engine, 32 * PAGE_SIZE);
> + ringbuf = intel_engine_create_ring(engine, 32 * PAGE_SIZE);
> if (IS_ERR(ringbuf)) {
> ret = PTR_ERR(ringbuf);
> goto error;
> @@ -2217,7 +2218,7 @@ static int intel_init_engine(struct intel_engine_cs *engine)
> goto error;
> }
>
> - ret = intel_pin_and_map_ringbuffer_obj(dev_priv, ringbuf);
> + ret = intel_pin_and_map_ring(dev_priv, ringbuf);
> if (ret) {
> DRM_ERROR("Failed to pin and map ringbuffer %s: %d\n",
> engine->name, ret);
> @@ -2228,11 +2229,11 @@ static int intel_init_engine(struct intel_engine_cs *engine)
> return 0;
>
> error:
> - intel_cleanup_engine(engine);
> + intel_engine_cleanup(engine);
> return ret;
> }
>
> -void intel_cleanup_engine(struct intel_engine_cs *engine)
> +void intel_engine_cleanup(struct intel_engine_cs *engine)
> {
> struct drm_i915_private *dev_priv;
>
> @@ -2242,11 +2243,11 @@ void intel_cleanup_engine(struct intel_engine_cs *engine)
> dev_priv = engine->i915;
>
> if (engine->buffer) {
> - intel_stop_engine(engine);
> + intel_engine_stop(engine);
> WARN_ON(!IS_GEN2(dev_priv) && (I915_READ_MODE(engine) & MODE_IDLE) == 0);
>
> - intel_unpin_ringbuffer_obj(engine->buffer);
> - intel_ringbuffer_free(engine->buffer);
> + intel_unpin_ring(engine->buffer);
> + intel_ring_free(engine->buffer);
> engine->buffer = NULL;
> }
>
> @@ -2309,7 +2310,7 @@ int intel_ring_alloc_request_extras(struct drm_i915_gem_request *request)
>
> static int wait_for_space(struct drm_i915_gem_request *req, int bytes)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> struct intel_engine_cs *engine = req->engine;
> struct drm_i915_gem_request *target;
>
> @@ -2354,7 +2355,7 @@ static int wait_for_space(struct drm_i915_gem_request *req, int bytes)
>
> int intel_ring_begin(struct drm_i915_gem_request *req, int num_dwords)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int remain_actual = ring->size - ring->tail;
> int remain_usable = ring->effective_size - ring->tail;
> int bytes = num_dwords * sizeof(u32);
> @@ -2411,7 +2412,7 @@ int intel_ring_begin(struct drm_i915_gem_request *req, int num_dwords)
> /* Align the ring tail to a cacheline boundary */
> int intel_ring_cacheline_align(struct drm_i915_gem_request *req)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int num_dwords =
> (ring->tail & (CACHELINE_BYTES - 1)) / sizeof(uint32_t);
> int ret;
> @@ -2432,7 +2433,7 @@ int intel_ring_cacheline_align(struct drm_i915_gem_request *req)
> return 0;
> }
>
> -void intel_ring_init_seqno(struct intel_engine_cs *engine, u32 seqno)
> +void intel_engine_init_seqno(struct intel_engine_cs *engine, u32 seqno)
> {
> struct drm_i915_private *dev_priv = engine->i915;
>
> @@ -2518,7 +2519,7 @@ static void gen6_bsd_ring_write_tail(struct intel_engine_cs *engine,
> static int gen6_bsd_ring_flush(struct drm_i915_gem_request *req,
> u32 invalidate, u32 flush)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> uint32_t cmd;
> int ret;
>
> @@ -2564,7 +2565,7 @@ gen8_ring_dispatch_execbuffer(struct drm_i915_gem_request *req,
> u64 offset, u32 len,
> unsigned dispatch_flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> bool ppgtt = USES_PPGTT(req->i915) &&
> !(dispatch_flags & I915_DISPATCH_SECURE);
> int ret;
> @@ -2590,7 +2591,7 @@ hsw_ring_dispatch_execbuffer(struct drm_i915_gem_request *req,
> u64 offset, u32 len,
> unsigned dispatch_flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> ret = intel_ring_begin(req, 2);
> @@ -2615,7 +2616,7 @@ gen6_ring_dispatch_execbuffer(struct drm_i915_gem_request *req,
> u64 offset, u32 len,
> unsigned dispatch_flags)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> int ret;
>
> ret = intel_ring_begin(req, 2);
> @@ -2638,7 +2639,7 @@ gen6_ring_dispatch_execbuffer(struct drm_i915_gem_request *req,
> static int gen6_ring_flush(struct drm_i915_gem_request *req,
> u32 invalidate, u32 flush)
> {
> - struct intel_ringbuffer *ring = req->ring;
> + struct intel_ring *ring = req->ring;
> uint32_t cmd;
> int ret;
>
> @@ -2955,7 +2956,7 @@ int intel_init_vebox_ring_buffer(struct intel_engine_cs *engine)
> }
>
> int
> -intel_ring_flush_all_caches(struct drm_i915_gem_request *req)
> +intel_engine_flush_all_caches(struct drm_i915_gem_request *req)
> {
> struct intel_engine_cs *engine = req->engine;
> int ret;
> @@ -2974,7 +2975,7 @@ intel_ring_flush_all_caches(struct drm_i915_gem_request *req)
> }
>
> int
> -intel_ring_invalidate_all_caches(struct drm_i915_gem_request *req)
> +intel_engine_invalidate_all_caches(struct drm_i915_gem_request *req)
> {
> struct intel_engine_cs *engine = req->engine;
> uint32_t flush_domains;
> @@ -2994,8 +2995,7 @@ intel_ring_invalidate_all_caches(struct drm_i915_gem_request *req)
> return 0;
> }
>
> -void
> -intel_stop_engine(struct intel_engine_cs *engine)
> +void intel_engine_stop(struct intel_engine_cs *engine)
> {
> int ret;
>
> diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.h b/drivers/gpu/drm/i915/intel_ringbuffer.h
> index 427fb19a7a2e..91d0aea695b2 100644
> --- a/drivers/gpu/drm/i915/intel_ringbuffer.h
> +++ b/drivers/gpu/drm/i915/intel_ringbuffer.h
> @@ -62,7 +62,7 @@ struct intel_hw_status_page {
> (i915_gem_obj_ggtt_offset(dev_priv->semaphore_obj) + \
> GEN8_SEMAPHORE_OFFSET(from, (__ring)->id))
>
> -enum intel_ring_hangcheck_action {
> +enum intel_engine_hangcheck_action {
> HANGCHECK_IDLE = 0,
> HANGCHECK_WAIT,
> HANGCHECK_ACTIVE,
> @@ -72,17 +72,17 @@ enum intel_ring_hangcheck_action {
>
> #define HANGCHECK_SCORE_RING_HUNG 31
>
> -struct intel_ring_hangcheck {
> +struct intel_engine_hangcheck {
> u64 acthd;
> unsigned long user_interrupts;
> u32 seqno;
> int score;
> - enum intel_ring_hangcheck_action action;
> + enum intel_engine_hangcheck_action action;
> int deadlock;
> u32 instdone[I915_NUM_INSTDONE_REG];
> };
>
> -struct intel_ringbuffer {
> +struct intel_ring {
> struct drm_i915_gem_object *obj;
> void *vaddr;
> struct i915_vma *vma;
> @@ -149,7 +149,7 @@ struct intel_engine_cs {
> u64 fence_context;
> u32 mmio_base;
> unsigned int irq_shift;
> - struct intel_ringbuffer *buffer;
> + struct intel_ring *buffer;
> struct list_head buffers;
>
> /* Rather than have every client wait upon all user interrupts,
> @@ -329,7 +329,7 @@ struct intel_engine_cs {
>
> struct i915_gem_context *last_context;
>
> - struct intel_ring_hangcheck hangcheck;
> + struct intel_engine_hangcheck hangcheck;
>
> struct {
> struct drm_i915_gem_object *obj;
> @@ -376,8 +376,8 @@ intel_engine_flag(const struct intel_engine_cs *engine)
> }
>
> static inline u32
> -intel_ring_sync_index(struct intel_engine_cs *engine,
> - struct intel_engine_cs *other)
> +intel_engine_sync_index(struct intel_engine_cs *engine,
> + struct intel_engine_cs *other)
> {
> int idx;
>
> @@ -439,45 +439,44 @@ intel_write_status_page(struct intel_engine_cs *engine,
> #define I915_GEM_HWS_SCRATCH_INDEX 0x40
> #define I915_GEM_HWS_SCRATCH_ADDR (I915_GEM_HWS_SCRATCH_INDEX << MI_STORE_DWORD_INDEX_SHIFT)
>
> -struct intel_ringbuffer *
> -intel_engine_create_ringbuffer(struct intel_engine_cs *engine, int size);
> -int intel_pin_and_map_ringbuffer_obj(struct drm_i915_private *dev_priv,
> - struct intel_ringbuffer *ringbuf);
> -void intel_unpin_ringbuffer_obj(struct intel_ringbuffer *ringbuf);
> -void intel_ringbuffer_free(struct intel_ringbuffer *ring);
> +struct intel_ring *
> +intel_engine_create_ring(struct intel_engine_cs *engine, int size);
> +int intel_pin_and_map_ring(struct drm_i915_private *dev_priv,
> + struct intel_ring *ring);
> +void intel_unpin_ring(struct intel_ring *ring);
> +void intel_ring_free(struct intel_ring *ring);
>
> -void intel_stop_engine(struct intel_engine_cs *engine);
> -void intel_cleanup_engine(struct intel_engine_cs *engine);
> +void intel_engine_stop(struct intel_engine_cs *engine);
> +void intel_engine_cleanup(struct intel_engine_cs *engine);
>
> int intel_ring_alloc_request_extras(struct drm_i915_gem_request *request);
>
> int __must_check intel_ring_begin(struct drm_i915_gem_request *req, int n);
> int __must_check intel_ring_cacheline_align(struct drm_i915_gem_request *req);
>
> -static inline void intel_ring_emit(struct intel_ringbuffer *ring, u32 data)
> +static inline void intel_ring_emit(struct intel_ring *ring, u32 data)
> {
> *(uint32_t *)(ring->vaddr + ring->tail) = data;
> ring->tail += 4;
> }
>
> -static inline void intel_ring_emit_reg(struct intel_ringbuffer *ring,
> - i915_reg_t reg)
> +static inline void intel_ring_emit_reg(struct intel_ring *ring, i915_reg_t reg)
> {
> intel_ring_emit(ring, i915_mmio_reg_offset(reg));
> }
>
> -static inline void intel_ring_advance(struct intel_ringbuffer *ring)
> +static inline void intel_ring_advance(struct intel_ring *ring)
> {
> ring->tail &= ring->size - 1;
> }
>
> int __intel_ring_space(int head, int tail, int size);
> -void intel_ring_update_space(struct intel_ringbuffer *ringbuf);
> +void intel_ring_update_space(struct intel_ring *ringbuf);
>
> int __must_check intel_engine_idle(struct intel_engine_cs *engine);
> -void intel_ring_init_seqno(struct intel_engine_cs *engine, u32 seqno);
> -int intel_ring_flush_all_caches(struct drm_i915_gem_request *req);
> -int intel_ring_invalidate_all_caches(struct drm_i915_gem_request *req);
> +void intel_engine_init_seqno(struct intel_engine_cs *engine, u32 seqno);
> +int intel_engine_flush_all_caches(struct drm_i915_gem_request *req);
> +int intel_engine_invalidate_all_caches(struct drm_i915_gem_request *req);
>
> int intel_init_pipe_control(struct intel_engine_cs *engine, int size);
> void intel_fini_pipe_control(struct intel_engine_cs *engine);
> @@ -491,7 +490,7 @@ int intel_init_bsd2_ring_buffer(struct intel_engine_cs *engine);
> int intel_init_blt_ring_buffer(struct intel_engine_cs *engine);
> int intel_init_vebox_ring_buffer(struct intel_engine_cs *engine);
>
> -u64 intel_ring_get_active_head(struct intel_engine_cs *engine);
> +u64 intel_engine_get_active_head(struct intel_engine_cs *engine);
> static inline u32 intel_engine_get_seqno(struct intel_engine_cs *engine)
> {
> return intel_read_status_page(engine, I915_GEM_HWS_INDEX);
> @@ -499,7 +498,7 @@ static inline u32 intel_engine_get_seqno(struct intel_engine_cs *engine)
>
> int init_workarounds_ring(struct intel_engine_cs *engine);
>
> -static inline u32 intel_ring_get_tail(struct intel_ringbuffer *ringbuf)
> +static inline u32 intel_ring_get_tail(struct intel_ring *ringbuf)
> {
> return ringbuf->tail;
> }
--
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
