[igt-dev] [PATCH i-g-t] runner: Make the result an incomplete if a test is killed due to taint
Chris Wilson
chris at chris-wilson.co.uk
Wed Jan 29 14:27:54 UTC 2020
Quoting Mika Kuoppala (2020-01-29 14:13:36)
> Chris Wilson <chris at chris-wilson.co.uk> writes:
>
> > Quoting Petri Latvala (2020-01-29 13:56:19)
> >> If we're checking for taints, we kill the test as soon as we notice a
> >> taint. Out of the box, such killing will get marked as such and yields
> >> a 'timeout' result, which is misleading. The test didn't spend too
> >> much time, it just did nasties.
> >>
> >> Make sure taint-killing results in an 'incomplete' result
> >> instead. It's still not completely truthful for the state of the
> >> testing but closer than a 'timeout'. And stands out more in CI result
> >> analysis.
> >>
> >> Suggested-by: Chris Wilson <chris at chris-wilson.co.uk>
> >> Signed-off-by: Petri Latvala <petri.latvala at intel.com>
> >> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> >
> > Incomplete has been synonymous in my mind with the "kernel oopsed" so it
> > worksforme. Even if the HW dies, everyone blames the kernel for letting
> > it die!
> >
>
> So with this patch, we can now assume 'incomplete' is always
> atleast oops/bug_on or a more catastrophics system hang?
Somewhere between oops and Jenkins!
-Chris
More information about the igt-dev
mailing list