Reviewed-by: Alan Previn <alan.previn.teres.ale...@intel.com>
On Thu, 2023-02-02 at 17:10 -0800, Harrison, John C wrote:
> From: John Harrison <john.c.harri...@intel.com>
>
> Error captures are tagged with an 'ecode'. This is a pseduo-unique magic
> number that is meant to distinguish similar seeming bugs with
> different underlying signatures. It is a combination of two ring state
> registers. Unfortunately, the register state being used is only valid
> in execlist mode. In GuC mode, the register state exists in a separate
> list of arbitrary register address/value pairs rather than the named
> entry structure. So, search through that list to find the two exciting
> registers and copy them over to the structure's named members.
>
> v2: if else if instead of if if (Alan)
>
> Signed-off-by: John Harrison <john.c.harri...@intel.com>
> Fixes: a6f0f9cf330a ("drm/i915/guc: Plumb GuC-capture into gpu_coredump")
> Cc: Alan Previn <alan.previn.teres.ale...@intel.com>
> Cc: Umesh Nerlige Ramappa <umesh.nerlige.rama...@intel.com>
> Cc: Lucas De Marchi <lucas.demar...@intel.com>
> Cc: Jani Nikula <jani.nik...@linux.intel.com>
> Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
> Cc: Rodrigo Vivi <rodrigo.v...@intel.com>
> Cc: Tvrtko Ursulin <tvrtko.ursu...@linux.intel.com>
> Cc: Matt Roper <matthew.d.ro...@intel.com>
> Cc: Aravind Iddamsetty <aravind.iddamse...@intel.com>
> Cc: Michael Cheng <michael.ch...@intel.com>
> Cc: Matthew Brost <matthew.br...@intel.com>
> Cc: Bruce Chang <yu.bruce.ch...@intel.com>
> Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com>
> Cc: Matthew Auld <matthew.a...@intel.com>
> ---
>
alan:snip
