From mboxrd@z Thu Jan 1 00:00:00 1970 From: John.C.Harrison@Intel.com Subject: [RFC 36/44] drm/i915: Added debug state dump facilities to scheduler Date: Thu, 26 Jun 2014 18:24:27 +0100 Message-ID: <1403803475-16337-37-git-send-email-John.C.Harrison@Intel.com> References: <1403803475-16337-1-git-send-email-John.C.Harrison@Intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by gabe.freedesktop.org (Postfix) with ESMTP id 97B3F6E275 for ; Thu, 26 Jun 2014 10:26:00 -0700 (PDT) In-Reply-To: <1403803475-16337-1-git-send-email-John.C.Harrison@Intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Intel-GFX@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org From: John Harrison When debugging batch buffer submission issues, it is useful to be able to see what the current state of the scheduler is. This change adds functions for decoding the internal scheduler state and reporting it. --- drivers/gpu/drm/i915/i915_scheduler.c | 255 +++++++++++++++++++++++++++++++++ drivers/gpu/drm/i915/i915_scheduler.h | 17 +++ 2 files changed, 272 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_scheduler.c b/drivers/gpu/drm/i915/i915_scheduler.c index 6782249..7c03fb7 100644 --- a/drivers/gpu/drm/i915/i915_scheduler.c +++ b/drivers/gpu/drm/i915/i915_scheduler.c @@ -37,6 +37,101 @@ bool i915_scheduler_is_enabled(struct drm_device *dev) #ifdef CONFIG_DRM_I915_SCHEDULER +const char *i915_qe_state_str(struct i915_scheduler_queue_entry *node) +{ + static char str[50]; + char *ptr = str; + + *(ptr++) = node->bumped ? 'B' : '-', + + *ptr = 0; + + return str; +} + +char i915_scheduler_queue_status_chr(enum i915_scheduler_queue_status status) +{ + switch (status) { + case i915_sqs_none: + return 'N'; + + case i915_sqs_queued: + return 'Q'; + + case i915_sqs_flying: + return 'F'; + + case i915_sqs_complete: + return 'C'; + + default: + break; + } + + return '?'; +} + +const char *i915_scheduler_queue_status_str( + enum i915_scheduler_queue_status status) +{ + static char str[50]; + + switch (status) { + case i915_sqs_none: + return "None"; + + case i915_sqs_queued: + return "Queued"; + + case i915_sqs_flying: + return "Flying"; + + case i915_sqs_complete: + return "Complete"; + + default: + break; + } + + sprintf(str, "[Unknown_%d!]", status); + return str; +} + +const char *i915_scheduler_flag_str(uint32_t flags) +{ + static char str[100]; + char *ptr = str; + + *ptr = 0; + +#define TEST_FLAG(flag, msg) \ + if (flags & (flag)) { \ + strcpy(ptr, msg); \ + ptr += strlen(ptr); \ + flags &= ~(flag); \ + } + + TEST_FLAG(i915_sf_interrupts_enabled, "IntOn|"); + TEST_FLAG(i915_sf_submitting, "Submitting|"); + TEST_FLAG(i915_sf_dump_force, "DumpForce|"); + TEST_FLAG(i915_sf_dump_details, "DumpDetails|"); + TEST_FLAG(i915_sf_dump_dependencies, "DumpDeps|"); + +#undef TEST_FLAG + + if (flags) { + sprintf(ptr, "Unknown_0x%X!", flags); + ptr += strlen(ptr); + } + + if (ptr == str) + strcpy(str, "-"); + else + ptr[-1] = 0; + + return str; +}; + int i915_scheduler_init(struct drm_device *dev) { struct drm_i915_private *dev_priv = dev->dev_private; @@ -589,6 +684,166 @@ int i915_scheduler_remove(struct intel_engine_cs *ring) return ret; } +int i915_scheduler_dump_all(struct drm_device *dev, const char *msg) +{ + struct drm_i915_private *dev_priv = dev->dev_private; + struct i915_scheduler *scheduler = dev_priv->scheduler; + unsigned long flags; + int ret; + + spin_lock_irqsave(&scheduler->lock, flags); + ret = i915_scheduler_dump_all_locked(dev, msg); + spin_unlock_irqrestore(&scheduler->lock, flags); + + return ret; +} + +int i915_scheduler_dump_all_locked(struct drm_device *dev, const char *msg) +{ + struct drm_i915_private *dev_priv = dev->dev_private; + struct i915_scheduler *scheduler = dev_priv->scheduler; + struct intel_engine_cs *ring; + int i, r, ret = 0; + + for_each_ring(ring, dev_priv, i) { + scheduler->flags[ring->id] |= i915_sf_dump_force | + i915_sf_dump_details | + i915_sf_dump_dependencies; + r = i915_scheduler_dump_locked(ring, msg); + if (ret == 0) + ret = r; + } + + return ret; +} + +int i915_scheduler_dump(struct intel_engine_cs *ring, const char *msg) +{ + struct drm_i915_private *dev_priv = ring->dev->dev_private; + struct i915_scheduler *scheduler = dev_priv->scheduler; + unsigned long flags; + int ret; + + spin_lock_irqsave(&scheduler->lock, flags); + ret = i915_scheduler_dump_locked(ring, msg); + spin_unlock_irqrestore(&scheduler->lock, flags); + + return ret; +} + +int i915_scheduler_dump_locked(struct intel_engine_cs *ring, const char *msg) +{ + struct drm_i915_private *dev_priv = ring->dev->dev_private; + struct i915_scheduler *scheduler = dev_priv->scheduler; + struct i915_scheduler_queue_entry *node; + int flying = 0, queued = 0, complete = 0, other = 0; + static int old_flying = -1, old_queued = -1, old_complete = -1; + bool b_dumped = false, b_dump; + char brkt[2] = { '<', '>' }; + + if (!ring) + return -EINVAL; + + list_for_each_entry(node, &scheduler->node_queue[ring->id], link) { + if (I915_SQS_IS_QUEUED(node)) + queued++; + else if (I915_SQS_IS_FLYING(node)) + flying++; + else if (I915_SQS_IS_COMPLETE(node)) + complete++; + else + other++; + } + + b_dump = (flying != old_flying) || + (queued != old_queued) || + (complete != old_complete); + if (scheduler->flags[ring->id] & i915_sf_dump_force) { + if (!b_dump) { + b_dump = true; + brkt[0] = '{'; + brkt[1] = '}'; + } + + scheduler->flags[ring->id] &= ~i915_sf_dump_force; + } + + if (b_dump) { + old_flying = flying; + old_queued = queued; + old_complete = complete; + DRM_DEBUG_SCHED("<%s> Q:%02d, F:%02d, C:%02d, O:%02d, " \ + "Flags = %s, OLR = %d %c%s%c\n", + ring->name, queued, flying, complete, other, + i915_scheduler_flag_str(scheduler->flags[ring->id]), + ring->outstanding_lazy_seqno, + brkt[0], msg, brkt[1]); + b_dumped = true; + } /* else + DRM_DEBUG_SCHED("<%s> Q:%02d, F:%02d, C:%02d, O:%02d" \ + ", Flags = %s, OLR = %d [%s]\n", + ring->name, + queued, flying, complete, other, + i915_scheduler_flag_str(scheduler->flags[ring->id]), + ring->outstanding_lazy_seqno, msg); */ + + if (b_dumped && (scheduler->flags[ring->id] & i915_sf_dump_details)) { + uint32_t seqno; + int i, deps; + uint32_t count, counts[i915_sqs_MAX]; + + memset(counts, 0x00, sizeof(counts)); + + seqno = ring->get_seqno(ring, true); + list_for_each_entry(node, &scheduler->node_queue[ring->id], link) { + if (node->status < i915_sqs_MAX) { + count = counts[node->status]++; + } else { + DRM_DEBUG_SCHED("<%s> Unknown status: %d!\n", + ring->name, node->status); + count = -1; + } + + deps = 0; + for (i = 0; i < node->num_deps; i++) + if (i915_scheduler_is_dependency_valid(node, i)) + deps++; + + DRM_DEBUG_SCHED("<%s> %c:%02d> index = %d, seqno" \ + " = %d/%s, deps = %d / %d, %s [pri = " \ + "%4d]\n", ring->name, + i915_scheduler_queue_status_chr(node->status), + count, + node->params.scheduler_index, + node->params.seqno, + node->params.ring->name, + deps, node->num_deps, + i915_qe_state_str(node), + node->priority); + + if ((scheduler->flags[ring->id] & i915_sf_dump_dependencies) + == 0) + continue; + + for (i = 0; i < node->num_deps; i++) + if (node->dep_list[i]) + DRM_DEBUG_SCHED("<%s> |-%c:" \ + "%02d%c seqno = %d/%s, %s [pri = %4d]\n", + ring->name, + i915_scheduler_queue_status_chr(node->dep_list[i]->status), + i, + i915_scheduler_is_dependency_valid(node, i) + ? '>' : '#', + node->dep_list[i]->params.seqno, + node->dep_list[i]->params.ring->name, + i915_qe_state_str(node->dep_list[i]), + node->dep_list[i]->priority); + } + } + + return 0; +} + int i915_scheduler_flush_seqno(struct intel_engine_cs *ring, bool is_locked, uint32_t seqno) { diff --git a/drivers/gpu/drm/i915/i915_scheduler.h b/drivers/gpu/drm/i915/i915_scheduler.h index 78a92c9..bbfd13c 100644 --- a/drivers/gpu/drm/i915/i915_scheduler.h +++ b/drivers/gpu/drm/i915/i915_scheduler.h @@ -59,6 +59,9 @@ enum i915_scheduler_queue_status { /* Limit value for use with arrays/loops */ i915_sqs_MAX }; +char i915_scheduler_queue_status_chr(enum i915_scheduler_queue_status status); +const char *i915_scheduler_queue_status_str( + enum i915_scheduler_queue_status status); #define I915_SQS_IS_QUEUED(node) (((node)->status == i915_sqs_queued)) #define I915_SQS_IS_FLYING(node) (((node)->status == i915_sqs_flying)) @@ -80,6 +83,7 @@ struct i915_scheduler_queue_entry { struct timespec stamp; struct list_head link; }; +const char *i915_qe_state_str(struct i915_scheduler_queue_entry *node); #ifdef CONFIG_DRM_I915_SCHEDULER # define I915_SCHEDULER_FLUSH_ALL(ring, locked) \ @@ -117,9 +121,16 @@ struct i915_scheduler { /* Flag bits for i915_scheduler::flags */ enum { + /* Internal state */ i915_sf_interrupts_enabled = (1 << 0), i915_sf_submitting = (1 << 1), + + /* Dump/debug flags */ + i915_sf_dump_force = (1 << 8), + i915_sf_dump_details = (1 << 9), + i915_sf_dump_dependencies = (1 << 10), }; +const char *i915_scheduler_flag_str(uint32_t flags); /* Options for 'scheduler_override' module parameter: */ enum { @@ -142,6 +153,12 @@ int i915_scheduler_submit_max_priority(struct intel_engine_cs *ring, bool is_locked); uint32_t i915_scheduler_count_flying(struct i915_scheduler *scheduler, struct intel_engine_cs *ring); +int i915_scheduler_dump(struct intel_engine_cs *ring, + const char *msg); +int i915_scheduler_dump_locked(struct intel_engine_cs *ring, + const char *msg); +int i915_scheduler_dump_all(struct drm_device *dev, const char *msg); +int i915_scheduler_dump_all_locked(struct drm_device *dev, const char *msg); void i915_scheduler_priority_bump_clear(struct i915_scheduler *scheduler, struct intel_engine_cs *ring); int i915_scheduler_priority_bump(struct i915_scheduler *scheduler, -- 1.7.9.5