On Fri, Feb 24, 2017 at 09:13:29AM +0000, Chris Wilson wrote: > On Fri, Feb 24, 2017 at 09:13:05AM +0530, Kamble, Sagar A wrote: > > Reviewed-by: Sagar Arun Kamble [1]<sagar.a.kamble@xxxxxxxxx> > > > > On 2/24/2017 4:41 AM, Michel Thierry wrote: > > > > There was no way to check if the platform is running the latest firmware. > > > > Cc: Tvrtko Ursulin [2]<tvrtko.ursulin@xxxxxxxxx> > > Cc: Arkadiusz Hiler [3]<arkadiusz.hiler@xxxxxxxxx> > > Signed-off-by: Michel Thierry [4]<michel.thierry@xxxxxxxxx> > > --- > > drivers/gpu/drm/i915/i915_gpu_error.c | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c b/drivers/gpu/drm/i915/i915_gpu_error.c > > index 2b1d15668192..e022187916ee 100644 > > --- a/drivers/gpu/drm/i915/i915_gpu_error.c > > +++ b/drivers/gpu/drm/i915/i915_gpu_error.c > > @@ -632,6 +632,16 @@ int i915_error_state_to_str(struct drm_i915_error_state_buf *m, > > CSR_VERSION_MINOR(csr->version)); > > } > > > > + if (HAS_GUC_UCODE(dev_priv)) { > > + struct intel_uc_fw *guc_fw = &dev_priv->guc.fw; > > + > > + err_printf(m, "GuC loaded: %s\n", > > + yesno(guc_fw->load_status == > > + INTEL_UC_FIRMWARE_SUCCESS)); > > + err_printf(m, "GuC fw version: %d.%d\n", > > + guc_fw->major_ver_found, guc_fw->minor_ver_found); > > + } > > + > > Hmm. The firmware may change between the hang and cat > /sys/class/drm/card0/error (as it will be reloaded after the reset). Btw, maybe we should add counter that will be incremented on each fw reload and reported here ? -Michal _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx