Re: [PATCH] drm/xe/guc: Destroy LR exec queue directly if GuC is not running

[Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>]

On 10/13/2025 8:36 PM, Shuicheng Lin wrote:
> During LR exec queue cleanup, if the GuC firmware is not running,
> the driver cannot communicate with the GuC to properly deregister
> the exec queue. In this case, directly destroy the exec queue
> instead of attempting deregistration.
>
> This prevents schedule disable failure and GuC ID resource leaks as
> below dmesg log:
> "
> [   50.242564] pci 0000:03:00.0: [drm] GT0: Schedule disable failed to respond, guc_id=2
> [   50.242568] ------------[ cut here ]------------
> [   50.242584] pci 0000:03:00.0: [drm] Assertion `ret` failed!
> ...
> [   50.244942] pci 0000:03:00.0: [drm] *ERROR* GT0: GUC ID manager unclean (1/65535)
> [   50.244970] pci 0000:03:00.0: [drm] GT0:     total 65535
> [   50.245002] pci 0000:03:00.0: [drm] GT0:     used 1
> [   50.245032] pci 0000:03:00.0: [drm] GT0:     range 2..2 (1)
> "
>
> Fixes: 8ae8a2e8dd21 ("drm/xe: Long running job update")
> Cc: Matthew Brost <matthew.brost@intel.com>
> Signed-off-by: Shuicheng Lin <shuicheng.lin@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_guc_submit.c | 10 +++++++++-
>   1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
> index 0ef67d3523a7..d2dfbdc82920 100644
> --- a/drivers/gpu/drm/xe/xe_guc_submit.c
> +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
> @@ -47,6 +47,8 @@
>   #include "xe_uc_fw.h"
>   #include "xe_vm.h"
>   
> +static void __guc_exec_queue_destroy(struct xe_guc *guc, struct xe_exec_queue *q);
> +
>   static struct xe_guc *
>   exec_queue_to_guc(struct xe_exec_queue *q)
>   {
> @@ -1060,10 +1062,15 @@ static void xe_guc_exec_queue_lr_cleanup(struct work_struct *w)
>   	 * state.
>   	 */
>   	if (!wedged && exec_queue_registered(q) && !exec_queue_destroyed(q)) {
> -		struct xe_guc *guc = exec_queue_to_guc(q);
>   		int ret;
>   
>   		set_exec_queue_banned(q);
> +		/* If GuC is not running, just destroy the exec queue as we can't communicate with it */
> +		if (!xe_uc_fw_is_running(&guc->fw)) {

This doesn't cover the case where the GuC is reset after we send the 
message but before we get a reply. There is also already a check below 
and inside disable_scheduling_deregister to cover the case where the GuC 
is not responding (guc_read_stopped()), which should also cover the case 
where the GuC has already been reset. That flow is probably not working 
as expected, so it might be worth having a look at fixing that instead.

Daniele

> +			__guc_exec_queue_destroy(guc, q);
> +			goto skip_deregister;
> +		}
> +
>   		disable_scheduling_deregister(guc, q);
>   
>   		/*
> @@ -1088,6 +1095,7 @@ static void xe_guc_exec_queue_lr_cleanup(struct work_struct *w)
>   		}
>   	}
>   
> +skip_deregister:
>   	if (!exec_queue_killed(q) && !xe_lrc_ring_is_idle(q->lrc[0]))
>   		xe_devcoredump(q, NULL, "LR job cleanup, guc_id=%d", q->guc->id);
>