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