[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