Re: [igt-dev] ✓ Fi.CI.BAT: success for runner: Make sure output is still collected when killing test due to taint

Petri Latvala
2020-01-29T13:24:59Z

On Tue, Jan 28, 2020 at 03:56:11PM +0000, Chris Wilson wrote:
> Quoting Patchwork (2020-01-28 15:30:42)
> > == Series Details ==
> >
> > Series: runner: Make sure output is still collected when killing test due to taint
> > URL : https://patchwork.freedesktop.org/series/72657/
> > State : success
> >
> > == Summary ==
> >
> > CI Bug Log - changes from IGT_5394 -> IGTPW_4016
> > ====================================================
> >
> > Summary
> > -------
> >
> > **WARNING**
> >
> > Minor unknown changes coming with IGTPW_4016 need to be verified
> > manually.
> >
> > If you think the reported changes have nothing to do with the changes
> > introduced in IGTPW_4016, please notify your bug team to allow them
> > to document this new failure mode, which will reduce false positives in CI.
> >
> > External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4016/index.html
> >
> > Possible new issues
> > -------------------
> >
> > Here are the unknown changes that may have been introduced in IGTPW_4016:
> >
> > ### IGT changes ###
> >
> > #### Warnings ####
> >
> > * igt@gem_exec_suspend@basic-s3:
> > - fi-cml-s: [FAIL][1] ([fdo#103375]) -> [TIMEOUT][2]
> > [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5394/fi-cml-s/igt@gem_exec_suspend@basic-s3.html
> > [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4016/fi-cml-s/igt@gem_exec_suspend@basic-s3.html
>
> Incomplete or Timeout?
>
> Starting subtest: basic-S3
> [160.045053] Killing the test because the kernel is tainted.
> [160.119964] Aborting: Kernel badly tainted (0x240) (check dmesg for details):
> (0x200) TAINT_WARN: WARN_ON has happened.
>
> I'd rather have stern purple than whimsical blue.

I can probably make that happen and it's almost going to be easy.

Do you mind if I do it in a followup patch though? I'd like to get
this in, the questions about incomplete output are increasing every
day...

--
Petri Latvala