[PATCH 0/2] drm/xe/guc: Handle crash notifications & drop default log verbosity

[Cavitt, Jonathan]

-----Original Message-----
From: Intel-xe <intel-xe-bounces at lists.freedesktop.org> On Behalf Of John.C.Harrison at Intel.com
Sent: Friday, November 8, 2024 1:28 PM
To: Intel-Xe at Lists.FreeDesktop.Org
Cc: Harrison, John C <john.c.harrison at intel.com>
Subject: [PATCH 0/2] drm/xe/guc: Handle crash notifications & drop default log verbosity
> 
> From: John Harrison <John.C.Harrison at Intel.com>
> 
> If an internal error occurs in the GuC firmware, it can send a crash
> notification message to the KMD. So add support for handling that by
> dumping state and triggering a reset.
> 
> Also, lower the default GuC log verbosity as max level is not
> generally required but is very verbose.
> 
> Signed-off-by: John Harrison <John.C.Harrison at Intel.com>

I just read through both patches in this series and they seem to be above board,
though I must admit I'm surprised we didn't have a G2H message processor for
crash dump and exception notifications before this.

Reviewed-by: Jonathan Cavitt <jonathan.cavitt at intel.com>
RB applies to all patches in series.
-Jonathan Cavitt

> 
> 
> John Harrison (2):
>   drm/xe/guc: Reduce default GuC log verbosity
>   drm/xe/guc: Support crash dump notification from GuC
> 
>  drivers/gpu/drm/xe/xe_guc_ct.c | 23 +++++++++++++++++++++++
>  drivers/gpu/drm/xe/xe_module.c |  2 +-
>  2 files changed, 24 insertions(+), 1 deletion(-)
> 
> -- 
> 2.47.0
> 
>