[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/4] drm/i915/guc: rename __create/destroy_doorbell (rev2)

[Daniele Ceraolo Spurio]

On 17/10/18 12:10, Patchwork wrote:
> == Series Details ==
> 
> Series: series starting with [1/4] drm/i915/guc: rename __create/destroy_doorbell (rev2)
> URL   : https://patchwork.freedesktop.org/series/51090/
> State : failure
> 
> == Summary ==
> 
> = CI Bug Log - changes from CI_DRM_5000 -> Patchwork_10492 =
> 
> == Summary - FAILURE ==
> 
>    Serious unknown changes coming with Patchwork_10492 absolutely need to be
>    verified manually.
>    
>    If you think the reported changes have nothing to do with the changes
>    introduced in Patchwork_10492, please notify your bug team to allow them
>    to document this new failure mode, which will reduce false positives in CI.
> 
>    External URL: https://patchwork.freedesktop.org/api/1.0/series/51090/revisions/2/mbox/
> 
> == Possible new issues ==
> 
>    Here are the unknown changes that may have been introduced in Patchwork_10492:
> 
>    === IGT changes ===
> 
>      ==== Possible regressions ====
> 
>      igt at drv_selftest@live_execlists:
>        fi-skl-iommu:       PASS -> INCOMPLETE
> 
>      igt at drv_selftest@live_guc:
>        fi-kbl-7567u:       PASS -> INCOMPLETE
>        fi-skl-6600u:       PASS -> INCOMPLETE
>        fi-skl-6260u:       PASS -> INCOMPLETE
>        fi-skl-6700k2:      PASS -> INCOMPLETE
>        fi-whl-u:           PASS -> INCOMPLETE
>        fi-skl-6770hq:      PASS -> INCOMPLETE
>        fi-kbl-7560u:       PASS -> INCOMPLETE
>        fi-kbl-8809g:       PASS -> INCOMPLETE
>        fi-kbl-r:           PASS -> INCOMPLETE
>        fi-kbl-x1275:       PASS -> INCOMPLETE
>        fi-skl-6700hq:      PASS -> INCOMPLETE
>        fi-cfl-s3:          PASS -> INCOMPLETE
>        fi-cfl-8109u:       PASS -> INCOMPLETE
>        fi-kbl-7500u:       PASS -> INCOMPLETE
>        fi-cfl-8700k:       PASS -> INCOMPLETE
> 

This is the invalid doorbell selfest trying to poll a doorbell register 
that doesn't exist, now caught because of the new BUG_ON. Given that it 
is a pure GuC FW interface test I don't think it is worth updating the 
code to detect the invalid case and act on it, so I'm just going to 
remove it as Michal suggested. We do have coverage for the i915 code 
running out of doorbells in the guc_doorbells selftest.

Daniele

>      igt at drv_selftest@live_hangcheck:
>        fi-skl-6600u:       PASS -> DMESG-WARN
> 
>      
> == Known issues ==
> 
>    Here are the changes found in Patchwork_10492 that come from known issues:
> 
>    === IGT changes ===
> 
>      ==== Issues hit ====
> 
>      igt at drv_selftest@live_guc:
>        fi-skl-gvtdvm:      PASS -> INCOMPLETE (fdo#105600)
>        fi-bxt-dsi:         PASS -> INCOMPLETE (fdo#103927)
>        fi-bxt-j4205:       PASS -> INCOMPLETE (fdo#103927)
> 
>      igt at kms_pipe_crc_basic@nonblocking-crc-pipe-a:
>        fi-byt-clapper:     PASS -> FAIL (fdo#107362)
> 
>      
>      ==== Possible fixes ====
> 
>      igt at kms_pipe_crc_basic@nonblocking-crc-pipe-b:
>        fi-byt-clapper:     FAIL (fdo#107362) -> PASS
> 
>      
>      ==== Warnings ====
> 
>      igt at drv_selftest@live_guc:
>        fi-apl-guc:         DMESG-WARN (fdo#107258) -> INCOMPLETE (fdo#106693)
> 
>      
>    fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
>    fdo#105600 https://bugs.freedesktop.org/show_bug.cgi?id=105600
>    fdo#106693 https://bugs.freedesktop.org/show_bug.cgi?id=106693
>    fdo#107258 https://bugs.freedesktop.org/show_bug.cgi?id=107258
>    fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
> 
> 
> == Participating hosts (47 -> 41) ==
> 
>    Missing    (6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-u
> 
> 
> == Build changes ==
> 
>      * Linux: CI_DRM_5000 -> Patchwork_10492
> 
>    CI_DRM_5000: b9543c130d4f6edd76ec98090c46044ba6d9493e @ git://anongit.freedesktop.org/gfx-ci/linux
>    IGT_4683: 7766b1e2348b32cc8ed58a972c6fd53b20279549 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
>    Patchwork_10492: 880bdc1de4f866988489380c18c9a051f3ab01d7 @ git://anongit.freedesktop.org/gfx-ci/linux
> 
> 
> == Linux commits ==
> 
> 880bdc1de4f8 HAX enable GuC for CI
> 61e396e88d17 drm/i915/guc: doorbell checking cleanup
> 641964fa483b drm/i915/guc: reserve the doorbell before selecting the cacheline
> f79da532bd81 drm/i915/guc: rename __create/destroy_doorbell
> 
> == Logs ==
> 
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10492/issues.html
>