mbox series

[v2,0/4] drm/edid & drm/i915: vendor and product id logging improvements

Message ID cover.1712655867.git.jani.nikula@intel.com (mailing list archive)
Headers show
Series drm/edid & drm/i915: vendor and product id logging improvements | expand

Message

Jani Nikula April 9, 2024, 9:46 a.m. UTC 
v2 with __be16 manufacturer_name in patch 1 and revamped date printing
in patch 2.

Jani Nikula (4):
  drm/edid: add drm_edid_get_product_id()
  drm/edid: add drm_edid_print_product_id()
  drm/i915/bios: switch to struct drm_edid and struct
    drm_edid_product_id
  drm/i915/bios: return drm_edid_product_id from get_lvds_pnp_id()

 drivers/gpu/drm/drm_edid.c                | 59 +++++++++++++++++++++++
 drivers/gpu/drm/i915/display/intel_bios.c | 49 ++++++++-----------
 include/drm/drm_edid.h                    | 28 +++++++++--
 3 files changed, 103 insertions(+), 33 deletions(-)

Comments

Jani Nikula April 10, 2024, 10:08 a.m. UTC | #1 
On Wed, 10 Apr 2024, Patchwork <patchwork@emeril.freedesktop.org> wrote:
> == Series Details ==
>
> Series: drm/edid & drm/i915: vendor and product id logging improvements (rev3)
> URL   : https://patchwork.freedesktop.org/series/131414/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_14552 -> Patchwork_131414v3
> ====================================================
>
> Summary
> -------
>
>   **FAILURE**
>
>   Serious unknown changes coming with Patchwork_131414v3 absolutely need to be
>   verified manually.
>   
>   If you think the reported changes have nothing to do with the changes
>   introduced in Patchwork_131414v3, please notify your bug team (I915-ci-infra@lists.freedesktop.org) 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_131414v3/index.html
>
> Participating hosts (37 -> 34)
> ------------------------------
>
>   Additional (2): fi-rkl-11600 fi-cfl-8109u 
>   Missing    (5): bat-dg1-7 fi-elk-e7500 fi-blb-e6850 bat-jsl-1 bat-arls-3 
>
> Possible new issues
> -------------------
>
>   Here are the unknown changes that may have been introduced in Patchwork_131414v3:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
>   * igt@i915_selftest@live@gt_timelines:
>     - bat-arls-2:         [PASS][1] -> [ABORT][2]
>    [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-arls-2/igt@i915_selftest@live@gt_timelines.html
>    [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-arls-2/igt@i915_selftest@live@gt_timelines.html

I don't see how this could result from the changes. Please re-report.

Thanks,
Jani.

>
>   
> Known issues
> ------------
>
>   Here are the changes found in Patchwork_131414v3 that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
>   * igt@debugfs_test@basic-hwmon:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][3] ([i915#9318])
>    [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@debugfs_test@basic-hwmon.html
>
>   * igt@gem_huc_copy@huc-copy:
>     - fi-cfl-8109u:       NOTRUN -> [SKIP][4] ([i915#2190])
>    [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-cfl-8109u/igt@gem_huc_copy@huc-copy.html
>     - fi-rkl-11600:       NOTRUN -> [SKIP][5] ([i915#2190])
>    [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@gem_huc_copy@huc-copy.html
>
>   * igt@gem_lmem_swapping@basic@lmem0:
>     - bat-dg2-8:          [PASS][6] -> [FAIL][7] ([i915#10378])
>    [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-dg2-8/igt@gem_lmem_swapping@basic@lmem0.html
>    [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-dg2-8/igt@gem_lmem_swapping@basic@lmem0.html
>
>   * igt@gem_lmem_swapping@parallel-random-engines:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][8] ([i915#4613]) +3 other tests skip
>    [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@gem_lmem_swapping@parallel-random-engines.html
>
>   * igt@gem_lmem_swapping@verify-random:
>     - fi-cfl-8109u:       NOTRUN -> [SKIP][9] ([i915#4613]) +3 other tests skip
>    [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-cfl-8109u/igt@gem_lmem_swapping@verify-random.html
>
>   * igt@gem_tiled_pread_basic:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][10] ([i915#3282])
>    [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@gem_tiled_pread_basic.html
>
>   * igt@i915_selftest@live@gt_engines:
>     - bat-adls-6:         [PASS][11] -> [TIMEOUT][12] ([i915#10026])
>    [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-adls-6/igt@i915_selftest@live@gt_engines.html
>    [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-adls-6/igt@i915_selftest@live@gt_engines.html
>
>   * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][13] ([i915#4103]) +1 other test skip
>    [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
>
>   * igt@kms_dsc@dsc-basic:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][14] ([i915#3555] / [i915#3840])
>    [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@kms_dsc@dsc-basic.html
>
>   * igt@kms_force_connector_basic@force-load-detect:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][15]
>    [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@kms_force_connector_basic@force-load-detect.html
>
>   * igt@kms_pm_backlight@basic-brightness:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][16] ([i915#5354])
>    [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@kms_pm_backlight@basic-brightness.html
>     - fi-cfl-8109u:       NOTRUN -> [SKIP][17] +11 other tests skip
>    [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-cfl-8109u/igt@kms_pm_backlight@basic-brightness.html
>
>   * igt@kms_psr@psr-sprite-plane-onoff:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][18] ([i915#1072] / [i915#9732]) +3 other tests skip
>    [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@kms_psr@psr-sprite-plane-onoff.html
>
>   * igt@kms_setmode@basic-clone-single-crtc:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][19] ([i915#3555])
>    [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@kms_setmode@basic-clone-single-crtc.html
>
>   * igt@prime_vgem@basic-read:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][20] ([i915#3291] / [i915#3708]) +2 other tests skip
>    [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@prime_vgem@basic-read.html
>
>   
> #### Possible fixes ####
>
>   * igt@gem_lmem_swapping@basic@lmem0:
>     - bat-dg2-11:         [FAIL][21] ([i915#10378]) -> [PASS][22]
>    [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-dg2-11/igt@gem_lmem_swapping@basic@lmem0.html
>    [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-dg2-11/igt@gem_lmem_swapping@basic@lmem0.html
>     - bat-dg2-9:          [FAIL][23] ([i915#10378]) -> [PASS][24]
>    [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-dg2-9/igt@gem_lmem_swapping@basic@lmem0.html
>    [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-dg2-9/igt@gem_lmem_swapping@basic@lmem0.html
>
>   * igt@i915_selftest@live@gt_contexts:
>     - bat-dg2-8:          [ABORT][25] ([i915#10366]) -> [PASS][26]
>    [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-dg2-8/igt@i915_selftest@live@gt_contexts.html
>    [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-dg2-8/igt@i915_selftest@live@gt_contexts.html
>
>   
>   [i915#10026]: https://gitlab.freedesktop.org/drm/intel/issues/10026
>   [i915#10366]: https://gitlab.freedesktop.org/drm/intel/issues/10366
>   [i915#10378]: https://gitlab.freedesktop.org/drm/intel/issues/10378
>   [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
>   [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
>   [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
>   [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
>   [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
>   [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
>   [i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840
>   [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
>   [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
>   [i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
>   [i915#9318]: https://gitlab.freedesktop.org/drm/intel/issues/9318
>   [i915#9732]: https://gitlab.freedesktop.org/drm/intel/issues/9732
>
>
> Build changes
> -------------
>
>   * Linux: CI_DRM_14552 -> Patchwork_131414v3
>
>   CI-20190529: 20190529
>   CI_DRM_14552: 057ec21a54cddd595a7725fa8731eb4c5bd5abff @ git://anongit.freedesktop.org/gfx-ci/linux
>   IGT_7803: 9669a17ae56f1dcd22ba4c5cb39b3cd334a46862 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
>   Patchwork_131414v3: 057ec21a54cddd595a7725fa8731eb4c5bd5abff @ git://anongit.freedesktop.org/gfx-ci/linux
>
>
> ### Linux commits
>
> 423abf498a49 drm/i915/bios: return drm_edid_product_id from get_lvds_pnp_id()
> bec9e40d51c6 drm/i915/bios: switch to struct drm_edid and struct drm_edid_product_id
> 9198292942f9 drm/edid: add drm_edid_print_product_id()
> fe86e819efa5 drm/edid: add drm_edid_get_product_id()
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/index.html
Illipilli, TejasreeX April 11, 2024, 7:59 a.m. UTC | #2 
Hi,

https://patchwork.freedesktop.org/series/131414/ - Re-reported.

Thanks,
Tejasree

-----Original Message-----
From: Nikula, Jani <jani.nikula@intel.com> 
Sent: Wednesday, April 10, 2024 3:38 PM
To: Patchwork <patchwork@emeril.freedesktop.org>; LGCI Bug Filing <lgci.bug.filing@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: ✗ Fi.CI.BAT: failure for drm/edid & drm/i915: vendor and product id logging improvements (rev3)

On Wed, 10 Apr 2024, Patchwork <patchwork@emeril.freedesktop.org> wrote:
> == Series Details ==
>
> Series: drm/edid & drm/i915: vendor and product id logging improvements (rev3)
> URL   : https://patchwork.freedesktop.org/series/131414/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_14552 -> Patchwork_131414v3 
> ====================================================
>
> Summary
> -------
>
>   **FAILURE**
>
>   Serious unknown changes coming with Patchwork_131414v3 absolutely need to be
>   verified manually.
>   
>   If you think the reported changes have nothing to do with the changes
>   introduced in Patchwork_131414v3, please notify your bug team (I915-ci-infra@lists.freedesktop.org) 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_131414v3/index.html
>
> Participating hosts (37 -> 34)
> ------------------------------
>
>   Additional (2): fi-rkl-11600 fi-cfl-8109u 
>   Missing    (5): bat-dg1-7 fi-elk-e7500 fi-blb-e6850 bat-jsl-1 bat-arls-3 
>
> Possible new issues
> -------------------
>
>   Here are the unknown changes that may have been introduced in Patchwork_131414v3:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
>   * igt@i915_selftest@live@gt_timelines:
>     - bat-arls-2:         [PASS][1] -> [ABORT][2]
>    [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-arls-2/igt@i915_selftest@live@gt_timelines.html
>    [2]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-arls-2
> /igt@i915_selftest@live@gt_timelines.html

I don't see how this could result from the changes. Please re-report.

Thanks,
Jani.

>
>   
> Known issues
> ------------
>
>   Here are the changes found in Patchwork_131414v3 that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
>   * igt@debugfs_test@basic-hwmon:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][3] ([i915#9318])
>    [3]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@debugfs_test@basic-hwmon.html
>
>   * igt@gem_huc_copy@huc-copy:
>     - fi-cfl-8109u:       NOTRUN -> [SKIP][4] ([i915#2190])
>    [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-cfl-8109u/igt@gem_huc_copy@huc-copy.html
>     - fi-rkl-11600:       NOTRUN -> [SKIP][5] ([i915#2190])
>    [5]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@gem_huc_copy@huc-copy.html
>
>   * igt@gem_lmem_swapping@basic@lmem0:
>     - bat-dg2-8:          [PASS][6] -> [FAIL][7] ([i915#10378])
>    [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-dg2-8/igt@gem_lmem_swapping@basic@lmem0.html
>    [7]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-dg2-8/
> igt@gem_lmem_swapping@basic@lmem0.html
>
>   * igt@gem_lmem_swapping@parallel-random-engines:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][8] ([i915#4613]) +3 other tests skip
>    [8]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@gem_lmem_swapping@parallel-random-engines.html
>
>   * igt@gem_lmem_swapping@verify-random:
>     - fi-cfl-8109u:       NOTRUN -> [SKIP][9] ([i915#4613]) +3 other tests skip
>    [9]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-cfl-810
> 9u/igt@gem_lmem_swapping@verify-random.html
>
>   * igt@gem_tiled_pread_basic:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][10] ([i915#3282])
>    [10]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@gem_tiled_pread_basic.html
>
>   * igt@i915_selftest@live@gt_engines:
>     - bat-adls-6:         [PASS][11] -> [TIMEOUT][12] ([i915#10026])
>    [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-adls-6/igt@i915_selftest@live@gt_engines.html
>    [12]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-adls-6
> /igt@i915_selftest@live@gt_engines.html
>
>   * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][13] ([i915#4103]) +1 other test skip
>    [13]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
>
>   * igt@kms_dsc@dsc-basic:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][14] ([i915#3555] / [i915#3840])
>    [14]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@kms_dsc@dsc-basic.html
>
>   * igt@kms_force_connector_basic@force-load-detect:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][15]
>    [15]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@kms_force_connector_basic@force-load-detect.html
>
>   * igt@kms_pm_backlight@basic-brightness:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][16] ([i915#5354])
>    [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-11600/igt@kms_pm_backlight@basic-brightness.html
>     - fi-cfl-8109u:       NOTRUN -> [SKIP][17] +11 other tests skip
>    [17]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-cfl-810
> 9u/igt@kms_pm_backlight@basic-brightness.html
>
>   * igt@kms_psr@psr-sprite-plane-onoff:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][18] ([i915#1072] / [i915#9732]) +3 other tests skip
>    [18]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@kms_psr@psr-sprite-plane-onoff.html
>
>   * igt@kms_setmode@basic-clone-single-crtc:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][19] ([i915#3555])
>    [19]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@kms_setmode@basic-clone-single-crtc.html
>
>   * igt@prime_vgem@basic-read:
>     - fi-rkl-11600:       NOTRUN -> [SKIP][20] ([i915#3291] / [i915#3708]) +2 other tests skip
>    [20]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/fi-rkl-116
> 00/igt@prime_vgem@basic-read.html
>
>   
> #### Possible fixes ####
>
>   * igt@gem_lmem_swapping@basic@lmem0:
>     - bat-dg2-11:         [FAIL][21] ([i915#10378]) -> [PASS][22]
>    [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-dg2-11/igt@gem_lmem_swapping@basic@lmem0.html
>    [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-dg2-11/igt@gem_lmem_swapping@basic@lmem0.html
>     - bat-dg2-9:          [FAIL][23] ([i915#10378]) -> [PASS][24]
>    [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-dg2-9/igt@gem_lmem_swapping@basic@lmem0.html
>    [24]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-dg2-9/
> igt@gem_lmem_swapping@basic@lmem0.html
>
>   * igt@i915_selftest@live@gt_contexts:
>     - bat-dg2-8:          [ABORT][25] ([i915#10366]) -> [PASS][26]
>    [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/bat-dg2-8/igt@i915_selftest@live@gt_contexts.html
>    [26]: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/bat-dg2-8/
> igt@i915_selftest@live@gt_contexts.html
>
>   
>   [i915#10026]: https://gitlab.freedesktop.org/drm/intel/issues/10026
>   [i915#10366]: https://gitlab.freedesktop.org/drm/intel/issues/10366
>   [i915#10378]: https://gitlab.freedesktop.org/drm/intel/issues/10378
>   [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
>   [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
>   [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
>   [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
>   [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
>   [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
>   [i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840
>   [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
>   [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
>   [i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
>   [i915#9318]: https://gitlab.freedesktop.org/drm/intel/issues/9318
>   [i915#9732]: https://gitlab.freedesktop.org/drm/intel/issues/9732
>
>
> Build changes
> -------------
>
>   * Linux: CI_DRM_14552 -> Patchwork_131414v3
>
>   CI-20190529: 20190529
>   CI_DRM_14552: 057ec21a54cddd595a7725fa8731eb4c5bd5abff @ git://anongit.freedesktop.org/gfx-ci/linux
>   IGT_7803: 9669a17ae56f1dcd22ba4c5cb39b3cd334a46862 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
>   Patchwork_131414v3: 057ec21a54cddd595a7725fa8731eb4c5bd5abff @ 
> git://anongit.freedesktop.org/gfx-ci/linux
>
>
> ### Linux commits
>
> 423abf498a49 drm/i915/bios: return drm_edid_product_id from 
> get_lvds_pnp_id()
> bec9e40d51c6 drm/i915/bios: switch to struct drm_edid and struct 
> drm_edid_product_id
> 9198292942f9 drm/edid: add drm_edid_print_product_id()
> fe86e819efa5 drm/edid: add drm_edid_get_product_id()
>
> == Logs ==
>
> For more details see: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/index.html

--
Jani Nikula, Intel
Jani Nikula April 12, 2024, 8:06 a.m. UTC | #3 
On Thu, 11 Apr 2024, Patchwork <patchwork@emeril.freedesktop.org> wrote:
> == Series Details ==
>
> Series: drm/edid & drm/i915: vendor and product id logging improvements (rev3)
> URL   : https://patchwork.freedesktop.org/series/131414/
> State : failure

[snip]

> #### Possible regressions ####
>
>   * igt@kms_cursor_legacy@cursor-vs-flip-atomic-transitions:
>     - shard-dg1:          NOTRUN -> [INCOMPLETE][1]
>    [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/shard-dg1-17/igt@kms_cursor_legacy@cursor-vs-flip-atomic-transitions.html
>
>   * igt@kms_plane@pixel-format@pipe-a:
>     - shard-tglu:         [PASS][2] -> [INCOMPLETE][3]
>    [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/shard-tglu-8/igt@kms_plane@pixel-format@pipe-a.html
>    [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/shard-tglu-3/igt@kms_plane@pixel-format@pipe-a.html
>     - shard-mtlp:         [PASS][4] -> [ABORT][5]
>    [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/shard-mtlp-1/igt@kms_plane@pixel-format@pipe-a.html
>    [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/shard-mtlp-3/igt@kms_plane@pixel-format@pipe-a.html

I don't see how this could be an issue with this patch, but it looks
serious:

<7> [451.620816] [drm:eb_lookup_vmas [i915]] EINVAL at eb_validate_vma:510
<4> [451.624224] ------------[ cut here ]------------
<4> [451.624309] list_del corruption. prev->next should be ffff88810afbf3d0, but was ffff88810afbb410. (prev=ffff88810afbb410)
<4> [451.624332] WARNING: CPU: 1 PID: 1374 at lib/list_debug.c:62 __list_del_entry_valid_or_report+0xb7/0xe0

etc.

BR,
Jani.

>
>   * igt@kms_sequence@queue-busy@pipe-c-hdmi-a-2:
>     - shard-glk:          [PASS][6] -> [INCOMPLETE][7]
>    [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/shard-glk5/igt@kms_sequence@queue-busy@pipe-c-hdmi-a-2.html
>    [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/shard-glk4/igt@kms_sequence@queue-busy@pipe-c-hdmi-a-2.html
Rodrigo Vivi April 16, 2024, 1:13 a.m. UTC | #4 
On Fri, Apr 12, 2024 at 11:06:09AM +0300, Jani Nikula wrote:
> On Thu, 11 Apr 2024, Patchwork <patchwork@emeril.freedesktop.org> wrote:
> > == Series Details ==
> >
> > Series: drm/edid & drm/i915: vendor and product id logging improvements (rev3)
> > URL   : https://patchwork.freedesktop.org/series/131414/
> > State : failure
> 
> [snip]
> 
> > #### Possible regressions ####
> >
> >   * igt@kms_cursor_legacy@cursor-vs-flip-atomic-transitions:
> >     - shard-dg1:          NOTRUN -> [INCOMPLETE][1]
> >    [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/shard-dg1-17/igt@kms_cursor_legacy@cursor-vs-flip-atomic-transitions.html
> >
> >   * igt@kms_plane@pixel-format@pipe-a:
> >     - shard-tglu:         [PASS][2] -> [INCOMPLETE][3]
> >    [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/shard-tglu-8/igt@kms_plane@pixel-format@pipe-a.html
> >    [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/shard-tglu-3/igt@kms_plane@pixel-format@pipe-a.html
> >     - shard-mtlp:         [PASS][4] -> [ABORT][5]
> >    [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/shard-mtlp-1/igt@kms_plane@pixel-format@pipe-a.html
> >    [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/shard-mtlp-3/igt@kms_plane@pixel-format@pipe-a.html
> 
> I don't see how this could be an issue with this patch, but it looks
> serious:

yeap!

 likely this one:

https://gitlab.freedesktop.org/drm/intel/-/issues/10608

with a possible fix candidate from Janusz:

https://lore.kernel.org/all/20240415195310.165934-2-janusz.krzysztofik@linux.intel.com/

> 
> <7> [451.620816] [drm:eb_lookup_vmas [i915]] EINVAL at eb_validate_vma:510
> <4> [451.624224] ------------[ cut here ]------------
> <4> [451.624309] list_del corruption. prev->next should be ffff88810afbf3d0, but was ffff88810afbb410. (prev=ffff88810afbb410)
> <4> [451.624332] WARNING: CPU: 1 PID: 1374 at lib/list_debug.c:62 __list_del_entry_valid_or_report+0xb7/0xe0
> 
> etc.
> 
> BR,
> Jani.
> 
> >
> >   * igt@kms_sequence@queue-busy@pipe-c-hdmi-a-2:
> >     - shard-glk:          [PASS][6] -> [INCOMPLETE][7]
> >    [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14552/shard-glk5/igt@kms_sequence@queue-busy@pipe-c-hdmi-a-2.html
> >    [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_131414v3/shard-glk4/igt@kms_sequence@queue-busy@pipe-c-hdmi-a-2.html
> 
> -- 
> Jani Nikula, Intel