| Message ID | 20180115144159.25913-5-lionel.g.landwerlin@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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@intel.com> > --- > 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@intel.com> Regards, Tvrtko
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@intel.com> >> --- >> 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@intel.com> > > Regards, > > Tvrtko > > Tested, works fine.
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);
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@intel.com> --- drivers/gpu/drm/i915/i915_gpu_error.c | 9 +++++++++ 1 file changed, 9 insertions(+)