<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-IN" link="#0563C1" vlink="#954F72" style="word-wrap:break-word"> <div class="WordSection1"> <div style="mso-element:para-border-div;border:solid #CCCCCC 1.0pt;padding:12.0pt 12.0pt 12.0pt 12.0pt;background:white"> diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c<o:p></o:p> index 3afca3dd9657..2c3ffa1db74e 100644<o:p></o:p> --- a/drivers/gpu/drm/xe/xe_gt.c<o:p></o:p> +++ b/drivers/gpu/drm/xe/xe_gt.c<o:p></o:p> @@ -8,6 +8,7 @@ <o:p></o:p> #include <linux/minmax.h><o:p></o:p> <o:p></o:p> #include <drm/drm_managed.h><o:p></o:p> +#include <drm/xe_drm.h><o:p></o:p> <o:p></o:p> #include "regs/xe_gt_regs.h"<o:p></o:p> #include "xe_bb.h"<o:p></o:p> @@ -590,6 +591,20 @@ static int do_gt_restart(struct xe_gt *gt)<o:p></o:p> return 0;<o:p></o:p> }<o:p></o:p> <o:p></o:p> +static void xe_uevent_gt_reset_failure(struct xe_device *xe, u8 id)<o:p></o:p> +{<o:p></o:p> + char *reset_event[5];<o:p></o:p> +<o:p></o:p> + reset_event[0] = XE_RESET_FAILED_UEVENT "=1";<o:p></o:p> + reset_event[1] = "RESET_ENABLED=1";<o:p></o:p> + reset_event[2] = "RESET_UNIT=gt";<o:p></o:p> + reset_event[3] = kasprintf(GFP_KERNEL, "RESET_ID=%d", id);<o:p></o:p> + reset_event[4] = NULL;<o:p></o:p> + kobject_uevent_env(&xe->drm.primary->kdev->kobj, KOBJ_CHANGE, reset_event);<o:p></o:p> +<o:p></o:p> + kfree(reset_event[3]);<o:p></o:p> +}<o:p></o:p> +<o:p></o:p> static int gt_reset(struct xe_gt *gt)<o:p></o:p> {<o:p></o:p> struct xe_device *xe = gt_to_xe(gt);<o:p></o:p> @@ -639,6 +654,9 @@ static int gt_reset(struct xe_gt *gt)<o:p></o:p> xe_device_mem_access_put(gt_to_xe(gt));<o:p></o:p> drm_err(&xe->drm, "GT reset failed, err=%d\n", err);<o:p></o:p> <o:p></o:p> The codebase here looks like moved on, in current code there is no drm_err logged it seems, <o:p></o:p> Though should not be this under if(err) ?<o:p></o:p> <o:p> </o:p> + /* Notify userspace about gt reset failure */<o:p></o:p> + xe_uevent_gt_reset_failure(xe, gt->info.id);<o:p></o:p> +<o:p></o:p> return err;<o:p></o:p> }<o:p></o:p> <o:p></o:p> diff --git a/drivers/gpu/drm/xe/xe_guc.h b/drivers/gpu/drm/xe/xe_guc.h<o:p></o:p> index 74a74051f354..845c57b95562 100644<o:p></o:p> --- a/drivers/gpu/drm/xe/xe_guc.h<o:p></o:p> +++ b/drivers/gpu/drm/xe/xe_guc.h<o:p></o:p> @@ -56,4 +56,22 @@ static inline u16 xe_engine_class_to_guc_class(enum xe_engine_class class)<o:p></o:p> }<o:p></o:p> }<o:p></o:p> <o:p></o:p> +static inline u16 xe_guc_class_to_engine_class(u8 guc_class)<o:p></o:p> +{<o:p></o:p> + switch (guc_class) {<o:p></o:p> + case GUC_RENDER_CLASS:<o:p></o:p> + return XE_ENGINE_CLASS_RENDER;<o:p></o:p> + case GUC_VIDEO_CLASS:<o:p></o:p> + return XE_ENGINE_CLASS_VIDEO_DECODE;<o:p></o:p> + case GUC_VIDEOENHANCE_CLASS:<o:p></o:p> + return XE_ENGINE_CLASS_VIDEO_ENHANCE;<o:p></o:p> + case GUC_BLITTER_CLASS:<o:p></o:p> + return XE_ENGINE_CLASS_COPY;<o:p></o:p> + case GUC_COMPUTE_CLASS:<o:p></o:p> + return XE_ENGINE_CLASS_COMPUTE;<o:p></o:p> <o:p> </o:p> Should you check GUC_GSC_OTHER_CLASS to return default here?<o:p></o:p> <o:p> </o:p> + default:<o:p></o:p> + XE_WARN_ON(guc_class);<o:p></o:p> + return -1;<o:p></o:p> + }<o:p></o:p> +}<o:p></o:p> #endif<o:p></o:p> diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c<o:p></o:p> index e857013070b9..663908573d5c 100644<o:p></o:p> --- a/drivers/gpu/drm/xe/xe_guc_submit.c<o:p></o:p> +++ b/drivers/gpu/drm/xe/xe_guc_submit.c<o:p></o:p> @@ -12,6 +12,7 @@ <o:p></o:p> #include <linux/dma-fence-array.h><o:p></o:p> <o:p></o:p> #include <drm/drm_managed.h><o:p></o:p> +#include <drm/xe_drm.h><o:p></o:p> <o:p></o:p> #include "regs/xe_lrc_layout.h"<o:p></o:p> #include "xe_device.h"<o:p></o:p> @@ -1589,9 +1590,25 @@ int xe_guc_engine_memory_cat_error_handler(struct xe_guc *guc, u32 *msg,<o:p></o:p> return 0;<o:p></o:p> }<o:p></o:p> <o:p></o:p> +static void xe_uevent_engine_reset_failure(struct xe_device *xe, const char *name)<o:p></o:p> +{<o:p></o:p> + char *reset_event[5];<o:p></o:p> +<o:p></o:p> + reset_event[0] = XE_RESET_FAILED_UEVENT "=1";<o:p></o:p> + reset_event[1] = "RESET_ENABLED=1";<o:p></o:p> + reset_event[2] = "RESET_UNIT=engine";<o:p></o:p> + reset_event[3] = kasprintf(GFP_KERNEL, "RESET_ID=%s", name);<o:p></o:p> + reset_event[4] = NULL;<o:p></o:p> + kobject_uevent_env(&xe->drm.primary->kdev->kobj, KOBJ_CHANGE, reset_event);<o:p></o:p> +<o:p></o:p> + kfree(reset_event[3]);<o:p></o:p> +}<o:p></o:p> +<o:p></o:p> int xe_guc_engine_reset_failure_handler(struct xe_guc *guc, u32 *msg, u32 len)<o:p></o:p> {<o:p></o:p> struct xe_device *xe = guc_to_xe(guc);<o:p></o:p> + struct xe_hw_engine *hwe;<o:p></o:p> + u16 engine_class;<o:p></o:p> u8 guc_class, instance;<o:p></o:p> u32 reason;<o:p></o:p> <o:p> </o:p> On upstream last time I got review comment from Andi saying this should be in Christmas tree order, something you might want to consider.<o:p></o:p> <o:p></o:p> @@ -1608,6 +1625,12 @@ int xe_guc_engine_reset_failure_handler(struct xe_guc *guc, u32 *msg, u32 len)<o:p></o:p> drm_err(&xe->drm, "GuC engine reset request failed on %d:%d because 0x%08X",<o:p></o:p> guc_class, instance, reason);<o:p></o:p> <o:p></o:p> + engine_class = xe_guc_class_to_engine_class(guc_class);<o:p></o:p> + hwe = xe_gt_hw_engine(guc_to_gt(guc), engine_class, instance, false);<o:p></o:p> +<o:p></o:p> + /* Notify userspace about engine reset failure */<o:p></o:p> + xe_uevent_engine_reset_failure(xe, hwe->name);<o:p></o:p> +<o:p></o:p> xe_gt_reset_async(guc_to_gt(guc));<o:p></o:p> <o:p></o:p> return 0;<o:p></o:p> diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h<o:p></o:p> index b0b80aae3ee8..79ef5947c172 100644<o:p></o:p> --- a/include/uapi/drm/xe_drm.h<o:p></o:p> +++ b/include/uapi/drm/xe_drm.h<o:p></o:p> @@ -36,6 +36,14 @@ extern "C" {<o:p></o:p> * subject to backwards-compatibility constraints.<o:p></o:p> */<o:p></o:p> <o:p></o:p> +/*<o:p></o:p> + * Uevents generated by xe on it's device node.<o:p></o:p> + *<o:p></o:p> + * XE_RESET_FAILED_UEVENT - Event is generated when attempt to reset engine<o:p></o:p> + * or gt fails. The value supplied with the event is always 1.<o:p></o:p> + */<o:p></o:p> +#define XE_RESET_FAILED_UEVENT "RESET_FAILED"<o:p></o:p> +<o:p></o:p> /**<o:p></o:p> * struct xe_user_extension - Base class for defining a chain of extensions<o:p></o:p> *<o:p></o:p> xe_uevent_engine_reset_failure/xe_uevent_gt_reset_failure can they be clubbed as one, up to you but most part looks same so though of this. You can ignore.<o:p></o:p> <o:p> </o:p> Please include me in cc next time, somehow I am not getting email for xe patches.<o:p></o:p> <o:p> </o:p> Thanks,<o:p></o:p> Tejas<o:p></o:p> </div> <o:p> </o:p> </div> </body> </html>