Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for i915: Simplify mmio handling & add new DG2 shadow table (rev2)

[Matt Roper <matthew.d.roper@intel.com>]

On Fri, Sep 10, 2021 at 08:56:20PM +0000, Patchwork wrote:
> == Series Details ==
> 
> Series: i915: Simplify mmio handling & add new DG2 shadow table (rev2)
> URL   : https://patchwork.freedesktop.org/series/94534/
> State : failure
> 
> == Summary ==
> 
> CI Bug Log - changes from CI_DRM_10570 -> Patchwork_21017
> ====================================================
> 
> Summary
> -------
> 
>   **FAILURE**
> 
>   Serious unknown changes coming with Patchwork_21017 absolutely need to be
>   verified manually.
>   
>   If you think the reported changes have nothing to do with the changes
>   introduced in Patchwork_21017, 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/Patchwork_21017/index.html
> 
> Possible new issues
> -------------------
> 
>   Here are the unknown changes that may have been introduced in Patchwork_21017:
> 
> ### IGT changes ###
> 
> #### Possible regressions ####
> 
>   * igt@kms_busy@basic:
>     - fi-rkl-guc:         NOTRUN -> [SKIP][1]
>    [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21017/fi-rkl-guc/igt@kms_busy@basic.html

KMS test skip due to no suitable display; not related to the uncore mmio
changes in this series.


Matt

> 
>   
> Known issues
> ------------
> 
>   Here are the changes found in Patchwork_21017 that come from known issues:
> 
> ### IGT changes ###
> 
> #### Issues hit ####
> 
>   * igt@gem_exec_suspend@basic-s3:
>     - fi-tgl-1115g4:      [PASS][2] -> [FAIL][3] ([i915#1888])
>    [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10570/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s3.html
>    [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21017/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s3.html
> 
>   * igt@runner@aborted:
>     - fi-kbl-7500u:       NOTRUN -> [FAIL][4] ([i915#1814] / [i915#3363])
>    [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21017/fi-kbl-7500u/igt@runner@aborted.html
> 
>   
> #### Possible fixes ####
> 
>   * igt@i915_pm_rpm@module-reload:
>     - {fi-hsw-gt1}:       [INCOMPLETE][5] ([i915#151]) -> [PASS][6]
>    [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10570/fi-hsw-gt1/igt@i915_pm_rpm@module-reload.html
>    [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21017/fi-hsw-gt1/igt@i915_pm_rpm@module-reload.html
> 
>   * igt@kms_flip@basic-flip-vs-modeset@c-dp1:
>     - fi-cfl-8109u:       [FAIL][7] -> [PASS][8] +1 similar issue
>    [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10570/fi-cfl-8109u/igt@kms_flip@basic-flip-vs-modeset@c-dp1.html
>    [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21017/fi-cfl-8109u/igt@kms_flip@basic-flip-vs-modeset@c-dp1.html
> 
>   * igt@kms_flip@basic-flip-vs-wf_vblank@a-vga1:
>     - fi-bwr-2160:        [FAIL][9] ([i915#2122]) -> [PASS][10]
>    [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10570/fi-bwr-2160/igt@kms_flip@basic-flip-vs-wf_vblank@a-vga1.html
>    [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21017/fi-bwr-2160/igt@kms_flip@basic-flip-vs-wf_vblank@a-vga1.html
> 
>   * igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b:
>     - fi-cfl-8109u:       [DMESG-WARN][11] ([i915#295]) -> [PASS][12] +18 similar issues
>    [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10570/fi-cfl-8109u/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b.html
>    [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21017/fi-cfl-8109u/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b.html
> 
>   
>   {name}: This element is suppressed. This means it is ignored when computing
>           the status of the difference (SUCCESS, WARNING, or FAILURE).
> 
>   [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
>   [fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315
>   [i915#151]: https://gitlab.freedesktop.org/drm/intel/issues/151
>   [i915#1814]: https://gitlab.freedesktop.org/drm/intel/issues/1814
>   [i915#1888]: https://gitlab.freedesktop.org/drm/intel/issues/1888
>   [i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
>   [i915#295]: https://gitlab.freedesktop.org/drm/intel/issues/295
>   [i915#3363]: https://gitlab.freedesktop.org/drm/intel/issues/3363
> 
> 
> Participating hosts (44 -> 39)
> ------------------------------
> 
>   Additional (1): fi-kbl-7500u 
>   Missing    (6): bat-dg1-6 bat-dg1-5 fi-bsw-cyan bat-adlp-4 bat-jsl-2 fi-bdw-samus 
> 
> 
> Build changes
> -------------
> 
>   * Linux: CI_DRM_10570 -> Patchwork_21017
> 
>   CI-20190529: 20190529
>   CI_DRM_10570: 8ba36ce2437426b91de6f03d30e75629108ea22b @ git://anongit.freedesktop.org/gfx-ci/linux
>   IGT_6203: 64452a46b57ca4ef35eb65a547df8ed1b131e8f0 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
>   Patchwork_21017: f4b1bdfbfb22fecdc3bfb40d2fb944a26edd8bf3 @ git://anongit.freedesktop.org/gfx-ci/linux
> 
> 
> == Linux commits ==
> 
> f4b1bdfbfb22 drm/i915/dg2: Add DG2-specific shadow register table
> 9eb63bad3b52 drm/i915/uncore: Drop gen11 mmio read handlers
> 63aa9ddb2b4f drm/i915/uncore: Drop gen11/gen12 mmio write handlers
> ff723b7086b1 drm/i915/uncore: Replace gen8 write functions with general fwtable
> b8d30ae4eb04 drm/i915/uncore: Associate shadow table with uncore
> 728596c7a8e0 drm/i915/uncore: Convert gen6/gen7 read operations to fwtable
> 
> == Logs ==
> 
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21017/index.html

-- 
Matt Roper
Graphics Software Engineer
VTT-OSGC Platform Enablement
Intel Corporation
(916) 356-2795