Bug 109493

The CI Bug Log issue associated to this bug has been updated.

### New filters associated

* BWR GDG: igt@gem_cpu_reloc@forked - fail - igt_skip: Assertion `!test_child' failed.
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_197/fi-bwr-2160/igt@gem_cpu_reloc@forked.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_197/fi-gdg-551/igt@gem_cpu_reloc@forked.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_198/fi-bwr-2160/igt@gem_cpu_reloc@forked.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_198/fi-gdg-551/igt@gem_cpu_reloc@forked.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_199/fi-bwr-2160/igt@gem_cpu_reloc@forked.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_199/fi-gdg-551/igt@gem_cpu_reloc@forked.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_200/fi-bwr-2160/igt@gem_cpu_reloc@forked.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_200/fi-gdg-551/igt@gem_cpu_reloc@forked.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_201/fi-bwr-2160/igt@gem_cpu_reloc@forked.html
  - https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_201/fi-gdg-551/igt@gem_cpu_reloc@forked.html

igt mishandling a igt_skip from inside an igt_fork; usually procedure is to move the skip check before the fork.

Hmm, there are no skips here, so the test failed and we got no error message from the child. That's quite a nasty igt bug.

Found the skip, patch sent.

commit 4049adf01014af077df2174def4fadf7cecb066e (HEAD, upstream/master)
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Wed Jan 30 16:18:58 2019 +0000

    i915/gem_cpu_reloc: Do the can-store-dword check at start
    
    igt doesn't handle skipping from inside igt_fork very gracefully and
    crashes instead of reporting the lack of requirements. One solution
    would be to fix igt, but far easier is to just move the requirement
    checking around to do it before we even fork.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109493
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

(In reply to Chris Wilson from comment #5)
> commit 4049adf01014af077df2174def4fadf7cecb066e (HEAD, upstream/master)
> Author: Chris Wilson <chris@chris-wilson.co.uk>
> Date:   Wed Jan 30 16:18:58 2019 +0000
> 
>     i915/gem_cpu_reloc: Do the can-store-dword check at start
>     
>     igt doesn't handle skipping from inside igt_fork very gracefully and
>     crashes instead of reporting the lack of requirements. One solution
>     would be to fix igt, but far easier is to just move the requirement
>     checking around to do it before we even fork.
>     
>     Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109493
>     Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>     Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

Thanks, it definitely did the trick as it was seen twice every single run between drmtip_197 and 203, and nothing for 34 runs :)

The CI Bug Log issue associated to this bug has been archived.

New failures matching the above filters will not be associated to this bug anymore.