Quoting john.c.harri...@intel.com (2019-11-21 00:31:42)
> From: John Harrison <john.c.harri...@intel.com>
> 
> Added gen & GT info to the error capture.
> 
> Signed-off-by: John Harrison <john.c.harri...@intel.com>
> CC: Matthew Brost <matthew.br...@intel.com>
> ---
>  drivers/gpu/drm/i915/i915_gpu_error.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c 
> b/drivers/gpu/drm/i915/i915_gpu_error.c
> index 7118ecb7f144..4c92892d05fb 100644
> --- a/drivers/gpu/drm/i915/i915_gpu_error.c
> +++ b/drivers/gpu/drm/i915/i915_gpu_error.c
> @@ -693,6 +693,8 @@ static void __err_print_to_sgl(struct 
> drm_i915_error_state_buf *m,
>         err_printf(m, "Subplatform: 0x%x\n",
>                    intel_subplatform(&error->runtime_info,
>                                      error->device_info.platform));
> +       err_printf(m, "Gen: %d\n", error->device_info.gen);

Gen is in the very first line. But repetition is not terrible.

> +       err_printf(m, "GT: %d\n", error->device_info.gt);

device_info is dumped later, but is missing from the printers. It should
be added there first.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to