Hi Jonathan,
...
> diff --git a/drivers/gpu/drm/i915/gt/selftest_hangcheck.c
> b/drivers/gpu/drm/i915/gt/selftest_hangcheck.c
> index 619c70c54ef9..4f252f704975 100644
> --- a/drivers/gpu/drm/i915/gt/selftest_hangcheck.c
> +++ b/drivers/gpu/drm/i915/gt/selftest_hangcheck.c
> @@ -904,9 +904,7 @@ static void active_engine(struct kthread_work *work)
> arg->result = PTR_ERR(ce[count]);
> pr_err("[%s] Create context #%ld failed: %d!\n",
> engine->name, count, arg->result);
> - if (!count)
> - return;
> - while (--count)
> + while (count--)
This is a good catch, but we still need to decrease count by 1
before entering the loop, right?
Andi
> intel_context_put(ce[count]);
> return;
> }
> --
> 2.43.0
