Re: [PATCH] drm/i915: Include GuC fw version in error state

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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).
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux