Re: [PATCH v4 4/6] drm/i915: add rcs topology to error state

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 15/01/18 17:43, Tvrtko Ursulin wrote:

On 15/01/2018 14:41, Lionel Landwerlin wrote:
This might be useful information for developers looking at an error
state.

v2: Place topology towards the end of the error state (Chris)

v3: Reuse common printing code (Michal)

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
---
  drivers/gpu/drm/i915/i915_gpu_error.c | 9 +++++++++
  1 file changed, 9 insertions(+)

diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c b/drivers/gpu/drm/i915/i915_gpu_error.c
index 944059322daa..aaa34a2bebe1 100644
--- a/drivers/gpu/drm/i915/i915_gpu_error.c
+++ b/drivers/gpu/drm/i915/i915_gpu_error.c
@@ -605,6 +605,14 @@ static void err_print_uc(struct drm_i915_error_state_buf *m,
      print_error_obj(m, NULL, "GuC log buffer", error_uc->guc_log);
  }
  +static void err_print_rcs_topology(struct drm_i915_error_state_buf *m,
+                   const struct sseu_dev_info *sseu)
+{
+    struct drm_printer p = i915_error_printer(m);
+
+    intel_device_info_dump_topology(sseu, &p);
+}
+
  int i915_error_state_to_str(struct drm_i915_error_state_buf *m,
                  const struct i915_gpu_state *error)
  {
@@ -787,6 +795,7 @@ int i915_error_state_to_str(struct drm_i915_error_state_buf *m,
          intel_display_print_error_state(m, error->display);
        err_print_capabilities(m, &error->device_info);
+    err_print_rcs_topology(m, &INTEL_INFO(dev_priv)->sseu);
      err_print_params(m, &error->params);
      err_print_uc(m, &error->uc);


I trust you tested it works - code is trivial so:

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>

Regards,

Tvrtko


Tested, works fine.


_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux