<div><br></div><div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Dec 15, 2023 at 5:13 AM Aravind Iddamsetty <<a href="mailto:aravind.iddamsetty@linux.intel.com">aravind.iddamsetty@linux.intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
On 12/15/23 15:13, Francois Dugast wrote:<br>
> From: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com" target="_blank">rodrigo.vivi@intel.com</a>><br>
><br>
> This kernel uevent is getting removed for now. It will come<br>
> back later with a better future proof name.<br>
><br>
> v2: Rebase (Francois Dugast)<br>
as mentioned in IGT series we need to drop this commit as well<br>
<br>
14a663d83da2 drm/xe: Introduce fault injection for gt reset.<br>
</blockquote><div dir="auto"><br></div><div dir="auto">No need for dropping that. We are going to restore the uevent soon. But with the proper final name.</div><div dir="auto">The problem is only with the uapi because we won’t be able to rename or change when we are in-tree.</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Thanks,<br>
Aravind.<br>
<br>
><br>
> Cc: Himal Prasad Ghimiray <<a href="mailto:himal.prasad.ghimiray@intel.com" target="_blank">himal.prasad.ghimiray@intel.com</a>><br>
> Cc: Lucas De Marchi <<a href="mailto:lucas.demarchi@intel.com" target="_blank">lucas.demarchi@intel.com</a>><br>
> Cc: Francois Dugast <<a href="mailto:francois.dugast@intel.com" target="_blank">francois.dugast@intel.com</a>><br>
> Cc: Aravind Iddamsetty <<a href="mailto:aravind.iddamsetty@linux.intel.com" target="_blank">aravind.iddamsetty@linux.intel.com</a>><br>
> Signed-off-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com" target="_blank">rodrigo.vivi@intel.com</a>><br>
> Signed-off-by: Francois Dugast <<a href="mailto:francois.dugast@intel.com" target="_blank">francois.dugast@intel.com</a>><br>
> ---<br>
>  drivers/gpu/drm/xe/xe_gt.c | 18 ------------------<br>
>  include/uapi/drm/xe_drm.h  | 11 -----------<br>
>  2 files changed, 29 deletions(-)<br>
><br>
> diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c<br>
> index f5d18e98f8b6..3af2adec1295 100644<br>
> --- a/drivers/gpu/drm/xe/xe_gt.c<br>
> +++ b/drivers/gpu/drm/xe/xe_gt.c<br>
> @@ -589,20 +589,6 @@ static int do_gt_restart(struct xe_gt *gt)<br>
>       return 0;<br>
>  }<br>
>  <br>
> -static void xe_uevent_gt_reset_failure(struct pci_dev *pdev, u8 tile_id, u8 gt_id)<br>
> -{<br>
> -     char *reset_event[4];<br>
> -<br>
> -     reset_event[0] = DRM_XE_RESET_FAILED_UEVENT "=NEEDS_RESET";<br>
> -     reset_event[1] = kasprintf(GFP_KERNEL, "TILE_ID=%d", tile_id);<br>
> -     reset_event[2] = kasprintf(GFP_KERNEL, "GT_ID=%d", gt_id);<br>
> -     reset_event[3] = NULL;<br>
> -     kobject_uevent_env(&pdev->dev.kobj, KOBJ_CHANGE, reset_event);<br>
> -<br>
> -     kfree(reset_event[1]);<br>
> -     kfree(reset_event[2]);<br>
> -}<br>
> -<br>
>  static int gt_reset(struct xe_gt *gt)<br>
>  {<br>
>       int err;<br>
> @@ -659,10 +645,6 @@ static int gt_reset(struct xe_gt *gt)<br>
>  err_fail:<br>
>       xe_gt_err(gt, "reset failed (%pe)\n", ERR_PTR(err));<br>
>  <br>
> -     /* Notify userspace about gt reset failure */<br>
> -     xe_uevent_gt_reset_failure(to_pci_dev(gt_to_xe(gt)-><a href="http://drm.dev" rel="noreferrer" target="_blank">drm.dev</a>),<br>
> -                                gt_to_tile(gt)->id, gt-><a href="http://info.id" rel="noreferrer" target="_blank">info.id</a>);<br>
> -<br>
>       gt_to_xe(gt)->needs_flr_on_fini = true;<br>
>  <br>
>       return err;<br>
> diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h<br>
> index b905ab16aa40..8c35ec43faef 100644<br>
> --- a/include/uapi/drm/xe_drm.h<br>
> +++ b/include/uapi/drm/xe_drm.h<br>
> @@ -20,7 +20,6 @@ extern "C" {<br>
>   *   2. Extension definition and helper structs<br>
>   *   3. IOCTL's Query structs in the order of the Query's entries.<br>
>   *   4. The rest of IOCTL structs in the order of IOCTL declaration.<br>
> - *   5. uEvents<br>
>   */<br>
>  <br>
>  /**<br>
> @@ -1341,16 +1340,6 @@ struct drm_xe_wait_user_fence {<br>
>       __u64 reserved[2];<br>
>  };<br>
>  <br>
> -/**<br>
> - * DOC: uevent generated by xe on it's pci node.<br>
> - *<br>
> - * DRM_XE_RESET_FAILED_UEVENT - Event is generated when attempt to reset gt<br>
> - * fails. The value supplied with the event is always "NEEDS_RESET".<br>
> - * Additional information supplied is tile id and gt id of the gt unit for<br>
> - * which reset has failed.<br>
> - */<br>
> -#define DRM_XE_RESET_FAILED_UEVENT "DEVICE_STATUS"<br>
> -<br>
>  #if defined(__cplusplus)<br>
>  }<br>
>  #endif<br>
</blockquote></div></div>