Hi Janusz, On 2026-08-31 at 11:33:08 +0200, Janusz Krzysztofik wrote: > CI extreamly sporadically (five times over last six mohths) reported extreamly -> extremely, mohths -> months
> failures of igt@i915_selftest@live@hangcheck on ARL-S machines. The > selftest was timing out while waiting up to 1 second for a request > supposed to trigger a GPU hang to be started. It occurred possible to > reproduce this issue manually by running the selftest in a loop for a > few hours. > > Among results from 5 occurrences reported by CI so far, and two manual > reproductions, two of them contained messages that suggested a > posssibility of the awaited request to actually start soon after > timeout. With the timeout extended from 1 to 2 seconds, it occurred no > longer possible to trigger the failure, even when running the test in a > loop for 10, then for 12 hours, on the same machine where it was failing > before in 2-3 hours. > > Assume the conditions that prevented the hang requests to start within 1 > second were always transient, and 2 seconds is sufficient for those > requests to start effectively on machines in the CI environment. Fix > the issue by using that value as the timeout. > > Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/work_items/16624 > Signed-off-by: Janusz Krzysztofik <[email protected]> > --- > drivers/gpu/drm/i915/gt/selftest_hangcheck.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/gt/selftest_hangcheck.c > b/drivers/gpu/drm/i915/gt/selftest_hangcheck.c > index 00dfc37221fac..53f3ce5c33a47 100644 > --- a/drivers/gpu/drm/i915/gt/selftest_hangcheck.c > +++ b/drivers/gpu/drm/i915/gt/selftest_hangcheck.c > @@ -278,7 +278,7 @@ static bool wait_until_running(struct hang *h, struct > i915_request *rq) > 10) && > wait_for(i915_seqno_passed(hws_seqno(h, rq), > rq->fence.seqno), > - 1000)); > + 2000)); > } > > static int igt_hang_sanitycheck(void *arg) > -- > 2.54.0 > Patch is trivial, so: Reviewed-by: Krzysztof Karas <[email protected]> though, it would probably need a rebase, since it does not apply cleanly for CI build process. -- Best Regards, Krzysztof
