* [RESEND] drm/edid/firmware: stop using a throwaway platform device
@ 2022-11-14 11:17 ` Jani Nikula
0 siblings, 0 replies; 15+ messages in thread
From: Jani Nikula @ 2022-11-14 11:17 UTC (permalink / raw)
To: dri-devel, intel-gfx; +Cc: jani.nikula, Matthieu CHARETTE
We've used a temporary platform device for firmware EDID loading since
it was introduced in commit da0df92b5731 ("drm: allow loading an EDID as
firmware to override broken monitor"), but there's no explanation why.
Using a temporary device does not play well with CONFIG_FW_CACHE=y,
which caches firmware images (e.g. on suspend) so that drivers can
request firmware when the system is not ready for it, and return the
images from the cache (e.g. during resume). This works automatically for
regular devices, but obviously not for a temporarily created device.
Stop using the throwaway platform device, and use the drm device
instead.
Note that this may still be problematic for cases where the display was
plugged in during suspend, and the firmware wasn't loaded and therefore
not cached before suspend.
References: https://lore.kernel.org/r/20220727074152.43059-1-matthieu.charette@gmail.com
Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/2061
Reported-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
Tested-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
Resend with a proper commit message; patch itself is unchanged.
---
drivers/gpu/drm/drm_edid_load.c | 13 +------------
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c
index ef4ab59d6935..5d9ef267ebb3 100644
--- a/drivers/gpu/drm/drm_edid_load.c
+++ b/drivers/gpu/drm/drm_edid_load.c
@@ -172,20 +172,9 @@ static const struct drm_edid *edid_load(struct drm_connector *connector, const c
fwdata = generic_edid[builtin];
fwsize = sizeof(generic_edid[builtin]);
} else {
- struct platform_device *pdev;
int err;
- pdev = platform_device_register_simple(connector->name, -1, NULL, 0);
- if (IS_ERR(pdev)) {
- drm_err(connector->dev,
- "[CONNECTOR:%d:%s] Failed to register EDID firmware platform device for connector \"%s\"\n",
- connector->base.id, connector->name,
- connector->name);
- return ERR_CAST(pdev);
- }
-
- err = request_firmware(&fw, name, &pdev->dev);
- platform_device_unregister(pdev);
+ err = request_firmware(&fw, name, connector->dev->dev);
if (err) {
drm_err(connector->dev,
"[CONNECTOR:%d:%s] Requesting EDID firmware \"%s\" failed (err=%d)\n",
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [Intel-gfx] [RESEND] drm/edid/firmware: stop using a throwaway platform device
@ 2022-11-14 11:17 ` Jani Nikula
0 siblings, 0 replies; 15+ messages in thread
From: Jani Nikula @ 2022-11-14 11:17 UTC (permalink / raw)
To: dri-devel, intel-gfx; +Cc: jani.nikula, Matthieu CHARETTE
We've used a temporary platform device for firmware EDID loading since
it was introduced in commit da0df92b5731 ("drm: allow loading an EDID as
firmware to override broken monitor"), but there's no explanation why.
Using a temporary device does not play well with CONFIG_FW_CACHE=y,
which caches firmware images (e.g. on suspend) so that drivers can
request firmware when the system is not ready for it, and return the
images from the cache (e.g. during resume). This works automatically for
regular devices, but obviously not for a temporarily created device.
Stop using the throwaway platform device, and use the drm device
instead.
Note that this may still be problematic for cases where the display was
plugged in during suspend, and the firmware wasn't loaded and therefore
not cached before suspend.
References: https://lore.kernel.org/r/20220727074152.43059-1-matthieu.charette@gmail.com
Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/2061
Reported-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
Tested-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
Resend with a proper commit message; patch itself is unchanged.
---
drivers/gpu/drm/drm_edid_load.c | 13 +------------
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c
index ef4ab59d6935..5d9ef267ebb3 100644
--- a/drivers/gpu/drm/drm_edid_load.c
+++ b/drivers/gpu/drm/drm_edid_load.c
@@ -172,20 +172,9 @@ static const struct drm_edid *edid_load(struct drm_connector *connector, const c
fwdata = generic_edid[builtin];
fwsize = sizeof(generic_edid[builtin]);
} else {
- struct platform_device *pdev;
int err;
- pdev = platform_device_register_simple(connector->name, -1, NULL, 0);
- if (IS_ERR(pdev)) {
- drm_err(connector->dev,
- "[CONNECTOR:%d:%s] Failed to register EDID firmware platform device for connector \"%s\"\n",
- connector->base.id, connector->name,
- connector->name);
- return ERR_CAST(pdev);
- }
-
- err = request_firmware(&fw, name, &pdev->dev);
- platform_device_unregister(pdev);
+ err = request_firmware(&fw, name, connector->dev->dev);
if (err) {
drm_err(connector->dev,
"[CONNECTOR:%d:%s] Requesting EDID firmware \"%s\" failed (err=%d)\n",
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/edid/firmware: stop using a throwaway platform device
2022-11-14 11:17 ` [Intel-gfx] " Jani Nikula
(?)
@ 2022-11-14 14:22 ` Patchwork
-1 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2022-11-14 14:22 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx
== Series Details ==
Series: drm/edid/firmware: stop using a throwaway platform device
URL : https://patchwork.freedesktop.org/series/110859/
State : warning
== Summary ==
Error: dim checkpatch failed
185b6fb3e97a drm/edid/firmware: stop using a throwaway platform device
-:26: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#26:
References: https://lore.kernel.org/r/20220727074152.43059-1-matthieu.charette@gmail.com
total: 0 errors, 1 warnings, 0 checks, 21 lines checked
^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/edid/firmware: stop using a throwaway platform device
2022-11-14 11:17 ` [Intel-gfx] " Jani Nikula
(?)
(?)
@ 2022-11-14 14:44 ` Patchwork
2022-11-14 15:33 ` Jani Nikula
-1 siblings, 1 reply; 15+ messages in thread
From: Patchwork @ 2022-11-14 14:44 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 6361 bytes --]
== Series Details ==
Series: drm/edid/firmware: stop using a throwaway platform device
URL : https://patchwork.freedesktop.org/series/110859/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_12376 -> Patchwork_110859v1
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_110859v1 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_110859v1, 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_110859v1/index.html
Participating hosts (37 -> 37)
------------------------------
Additional (2): bat-kbl-2 fi-bsw-nick
Missing (2): fi-bdw-samus bat-dg1-5
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_110859v1:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live@mman:
- fi-rkl-guc: [PASS][1] -> [TIMEOUT][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-guc/igt@i915_selftest@live@mman.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-guc/igt@i915_selftest@live@mman.html
Known issues
------------
Here are the changes found in Patchwork_110859v1 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_lmem_swapping@basic:
- fi-apl-guc: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#4613]) +3 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@gem_lmem_swapping@basic.html
* igt@gem_lmem_swapping@parallel-random-engines:
- fi-bsw-nick: NOTRUN -> [SKIP][4] ([fdo#109271]) +38 similar issues
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nick/igt@gem_lmem_swapping@parallel-random-engines.html
* igt@i915_pm_rpm@basic-pci-d3-state:
- bat-adlp-4: [PASS][5] -> [DMESG-WARN][6] ([i915#7077])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-4/igt@i915_pm_rpm@basic-pci-d3-state.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-4/igt@i915_pm_rpm@basic-pci-d3-state.html
* igt@kms_chamelium@common-hpd-after-suspend:
- fi-rkl-11600: NOTRUN -> [SKIP][7] ([fdo#111827])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-11600/igt@kms_chamelium@common-hpd-after-suspend.html
* igt@kms_chamelium@hdmi-crc-fast:
- fi-apl-guc: NOTRUN -> [SKIP][8] ([fdo#109271] / [fdo#111827]) +8 similar issues
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@kms_chamelium@hdmi-crc-fast.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-bsw-nick: NOTRUN -> [SKIP][9] ([fdo#109271] / [fdo#111827]) +8 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nick/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_force_connector_basic@force-connector-state:
- fi-apl-guc: NOTRUN -> [SKIP][10] ([fdo#109271]) +11 similar issues
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@kms_force_connector_basic@force-connector-state.html
#### Possible fixes ####
* igt@gem_render_tiled_blits@basic:
- fi-apl-guc: [INCOMPLETE][11] ([i915#7056]) -> [PASS][12]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-apl-guc/igt@gem_render_tiled_blits@basic.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@gem_render_tiled_blits@basic.html
* igt@i915_selftest@live@migrate:
- {bat-adlp-6}: [INCOMPLETE][13] ([i915#7348]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-6/igt@i915_selftest@live@migrate.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-6/igt@i915_selftest@live@migrate.html
* igt@i915_selftest@live@slpc:
- {bat-adln-1}: [DMESG-FAIL][15] ([i915#6997]) -> [PASS][16]
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adln-1/igt@i915_selftest@live@slpc.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adln-1/igt@i915_selftest@live@slpc.html
* igt@i915_suspend@basic-s3-without-i915:
- fi-rkl-11600: [INCOMPLETE][17] ([i915#4817]) -> [PASS][18]
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.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#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4817]: https://gitlab.freedesktop.org/drm/intel/issues/4817
[i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
[i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
[i915#6997]: https://gitlab.freedesktop.org/drm/intel/issues/6997
[i915#7056]: https://gitlab.freedesktop.org/drm/intel/issues/7056
[i915#7077]: https://gitlab.freedesktop.org/drm/intel/issues/7077
[i915#7348]: https://gitlab.freedesktop.org/drm/intel/issues/7348
Build changes
-------------
* Linux: CI_DRM_12376 -> Patchwork_110859v1
CI-20190529: 20190529
CI_DRM_12376: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7054: 5a0fd5b1011658f7ba117a45537e770a80d257fd @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_110859v1: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
### Linux commits
90c2b72dc380 drm/edid/firmware: stop using a throwaway platform device
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/index.html
[-- Attachment #2: Type: text/html, Size: 7413 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/edid/firmware: stop using a throwaway platform device
2022-11-14 14:44 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
@ 2022-11-14 15:33 ` Jani Nikula
2022-11-14 18:49 ` Vudum, Lakshminarayana
0 siblings, 1 reply; 15+ messages in thread
From: Jani Nikula @ 2022-11-14 15:33 UTC (permalink / raw)
To: Patchwork, Vudum, Lakshminarayana; +Cc: intel-gfx
On Mon, 14 Nov 2022, Patchwork <patchwork@emeril.freedesktop.org> wrote:
> == Series Details ==
>
> Series: drm/edid/firmware: stop using a throwaway platform device
> URL : https://patchwork.freedesktop.org/series/110859/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_12376 -> Patchwork_110859v1
> ====================================================
>
> Summary
> -------
>
> **FAILURE**
>
> Serious unknown changes coming with Patchwork_110859v1 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_110859v1, 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_110859v1/index.html
>
> Participating hosts (37 -> 37)
> ------------------------------
>
> Additional (2): bat-kbl-2 fi-bsw-nick
> Missing (2): fi-bdw-samus bat-dg1-5
>
> Possible new issues
> -------------------
>
> Here are the unknown changes that may have been introduced in Patchwork_110859v1:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
> * igt@i915_selftest@live@mman:
> - fi-rkl-guc: [PASS][1] -> [TIMEOUT][2]
> [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-guc/igt@i915_selftest@live@mman.html
> [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-guc/igt@i915_selftest@live@mman.html
Completely unrelated.
BR,
Jani.
>
>
> Known issues
> ------------
>
> Here are the changes found in Patchwork_110859v1 that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
> * igt@gem_lmem_swapping@basic:
> - fi-apl-guc: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#4613]) +3 similar issues
> [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@gem_lmem_swapping@basic.html
>
> * igt@gem_lmem_swapping@parallel-random-engines:
> - fi-bsw-nick: NOTRUN -> [SKIP][4] ([fdo#109271]) +38 similar issues
> [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nick/igt@gem_lmem_swapping@parallel-random-engines.html
>
> * igt@i915_pm_rpm@basic-pci-d3-state:
> - bat-adlp-4: [PASS][5] -> [DMESG-WARN][6] ([i915#7077])
> [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-4/igt@i915_pm_rpm@basic-pci-d3-state.html
> [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-4/igt@i915_pm_rpm@basic-pci-d3-state.html
>
> * igt@kms_chamelium@common-hpd-after-suspend:
> - fi-rkl-11600: NOTRUN -> [SKIP][7] ([fdo#111827])
> [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-11600/igt@kms_chamelium@common-hpd-after-suspend.html
>
> * igt@kms_chamelium@hdmi-crc-fast:
> - fi-apl-guc: NOTRUN -> [SKIP][8] ([fdo#109271] / [fdo#111827]) +8 similar issues
> [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@kms_chamelium@hdmi-crc-fast.html
>
> * igt@kms_chamelium@hdmi-hpd-fast:
> - fi-bsw-nick: NOTRUN -> [SKIP][9] ([fdo#109271] / [fdo#111827]) +8 similar issues
> [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nick/igt@kms_chamelium@hdmi-hpd-fast.html
>
> * igt@kms_force_connector_basic@force-connector-state:
> - fi-apl-guc: NOTRUN -> [SKIP][10] ([fdo#109271]) +11 similar issues
> [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@kms_force_connector_basic@force-connector-state.html
>
>
> #### Possible fixes ####
>
> * igt@gem_render_tiled_blits@basic:
> - fi-apl-guc: [INCOMPLETE][11] ([i915#7056]) -> [PASS][12]
> [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-apl-guc/igt@gem_render_tiled_blits@basic.html
> [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@gem_render_tiled_blits@basic.html
>
> * igt@i915_selftest@live@migrate:
> - {bat-adlp-6}: [INCOMPLETE][13] ([i915#7348]) -> [PASS][14]
> [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-6/igt@i915_selftest@live@migrate.html
> [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-6/igt@i915_selftest@live@migrate.html
>
> * igt@i915_selftest@live@slpc:
> - {bat-adln-1}: [DMESG-FAIL][15] ([i915#6997]) -> [PASS][16]
> [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adln-1/igt@i915_selftest@live@slpc.html
> [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adln-1/igt@i915_selftest@live@slpc.html
>
> * igt@i915_suspend@basic-s3-without-i915:
> - fi-rkl-11600: [INCOMPLETE][17] ([i915#4817]) -> [PASS][18]
> [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html
> [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.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#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
> [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
> [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
> [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
> [i915#4817]: https://gitlab.freedesktop.org/drm/intel/issues/4817
> [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
> [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
> [i915#6997]: https://gitlab.freedesktop.org/drm/intel/issues/6997
> [i915#7056]: https://gitlab.freedesktop.org/drm/intel/issues/7056
> [i915#7077]: https://gitlab.freedesktop.org/drm/intel/issues/7077
> [i915#7348]: https://gitlab.freedesktop.org/drm/intel/issues/7348
>
>
> Build changes
> -------------
>
> * Linux: CI_DRM_12376 -> Patchwork_110859v1
>
> CI-20190529: 20190529
> CI_DRM_12376: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_7054: 5a0fd5b1011658f7ba117a45537e770a80d257fd @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> Patchwork_110859v1: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
>
>
> ### Linux commits
>
> 90c2b72dc380 drm/edid/firmware: stop using a throwaway platform device
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/index.html
--
Jani Nikula, Intel Open Source Graphics Center
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/edid/firmware: stop using a throwaway platform device
2022-11-14 15:33 ` Jani Nikula
@ 2022-11-14 18:49 ` Vudum, Lakshminarayana
0 siblings, 0 replies; 15+ messages in thread
From: Vudum, Lakshminarayana @ 2022-11-14 18:49 UTC (permalink / raw)
To: Nikula, Jani, Patchwork; +Cc: intel-gfx
Issue si related to https://gitlab.freedesktop.org/drm/intel/-/issues/6794.
Re-reported.
Lakshmi.
-----Original Message-----
From: Nikula, Jani <jani.nikula@intel.com>
Sent: Monday, November 14, 2022 5:33 PM
To: Patchwork <patchwork@emeril.freedesktop.org>; Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: ✗ Fi.CI.BAT: failure for drm/edid/firmware: stop using a throwaway platform device
On Mon, 14 Nov 2022, Patchwork <patchwork@emeril.freedesktop.org> wrote:
> == Series Details ==
>
> Series: drm/edid/firmware: stop using a throwaway platform device
> URL : https://patchwork.freedesktop.org/series/110859/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_12376 -> Patchwork_110859v1
> ====================================================
>
> Summary
> -------
>
> **FAILURE**
>
> Serious unknown changes coming with Patchwork_110859v1 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_110859v1, 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_110859v1/index.html
>
> Participating hosts (37 -> 37)
> ------------------------------
>
> Additional (2): bat-kbl-2 fi-bsw-nick
> Missing (2): fi-bdw-samus bat-dg1-5
>
> Possible new issues
> -------------------
>
> Here are the unknown changes that may have been introduced in Patchwork_110859v1:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
> * igt@i915_selftest@live@mman:
> - fi-rkl-guc: [PASS][1] -> [TIMEOUT][2]
> [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-guc/igt@i915_selftest@live@mman.html
> [2]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-guc
> /igt@i915_selftest@live@mman.html
Completely unrelated.
BR,
Jani.
>
>
> Known issues
> ------------
>
> Here are the changes found in Patchwork_110859v1 that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
> * igt@gem_lmem_swapping@basic:
> - fi-apl-guc: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#4613]) +3 similar issues
> [3]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc
> /igt@gem_lmem_swapping@basic.html
>
> * igt@gem_lmem_swapping@parallel-random-engines:
> - fi-bsw-nick: NOTRUN -> [SKIP][4] ([fdo#109271]) +38 similar issues
> [4]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nic
> k/igt@gem_lmem_swapping@parallel-random-engines.html
>
> * igt@i915_pm_rpm@basic-pci-d3-state:
> - bat-adlp-4: [PASS][5] -> [DMESG-WARN][6] ([i915#7077])
> [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-4/igt@i915_pm_rpm@basic-pci-d3-state.html
> [6]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-4
> /igt@i915_pm_rpm@basic-pci-d3-state.html
>
> * igt@kms_chamelium@common-hpd-after-suspend:
> - fi-rkl-11600: NOTRUN -> [SKIP][7] ([fdo#111827])
> [7]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-116
> 00/igt@kms_chamelium@common-hpd-after-suspend.html
>
> * igt@kms_chamelium@hdmi-crc-fast:
> - fi-apl-guc: NOTRUN -> [SKIP][8] ([fdo#109271] / [fdo#111827]) +8 similar issues
> [8]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc
> /igt@kms_chamelium@hdmi-crc-fast.html
>
> * igt@kms_chamelium@hdmi-hpd-fast:
> - fi-bsw-nick: NOTRUN -> [SKIP][9] ([fdo#109271] / [fdo#111827]) +8 similar issues
> [9]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nic
> k/igt@kms_chamelium@hdmi-hpd-fast.html
>
> * igt@kms_force_connector_basic@force-connector-state:
> - fi-apl-guc: NOTRUN -> [SKIP][10] ([fdo#109271]) +11 similar issues
> [10]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc
> /igt@kms_force_connector_basic@force-connector-state.html
>
>
> #### Possible fixes ####
>
> * igt@gem_render_tiled_blits@basic:
> - fi-apl-guc: [INCOMPLETE][11] ([i915#7056]) -> [PASS][12]
> [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-apl-guc/igt@gem_render_tiled_blits@basic.html
> [12]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc
> /igt@gem_render_tiled_blits@basic.html
>
> * igt@i915_selftest@live@migrate:
> - {bat-adlp-6}: [INCOMPLETE][13] ([i915#7348]) -> [PASS][14]
> [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-6/igt@i915_selftest@live@migrate.html
> [14]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-6
> /igt@i915_selftest@live@migrate.html
>
> * igt@i915_selftest@live@slpc:
> - {bat-adln-1}: [DMESG-FAIL][15] ([i915#6997]) -> [PASS][16]
> [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adln-1/igt@i915_selftest@live@slpc.html
> [16]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adln-1
> /igt@i915_selftest@live@slpc.html
>
> * igt@i915_suspend@basic-s3-without-i915:
> - fi-rkl-11600: [INCOMPLETE][17] ([i915#4817]) -> [PASS][18]
> [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html
> [18]:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-116
> 00/igt@i915_suspend@basic-s3-without-i915.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#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
> [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
> [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
> [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
> [i915#4817]: https://gitlab.freedesktop.org/drm/intel/issues/4817
> [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
> [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
> [i915#6997]: https://gitlab.freedesktop.org/drm/intel/issues/6997
> [i915#7056]: https://gitlab.freedesktop.org/drm/intel/issues/7056
> [i915#7077]: https://gitlab.freedesktop.org/drm/intel/issues/7077
> [i915#7348]: https://gitlab.freedesktop.org/drm/intel/issues/7348
>
>
> Build changes
> -------------
>
> * Linux: CI_DRM_12376 -> Patchwork_110859v1
>
> CI-20190529: 20190529
> CI_DRM_12376: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_7054: 5a0fd5b1011658f7ba117a45537e770a80d257fd @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> Patchwork_110859v1: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @
> git://anongit.freedesktop.org/gfx-ci/linux
>
>
> ### Linux commits
>
> 90c2b72dc380 drm/edid/firmware: stop using a throwaway platform device
>
> == Logs ==
>
> For more details see:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/index.html
--
Jani Nikula, Intel Open Source Graphics Center
^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/edid/firmware: stop using a throwaway platform device
2022-11-14 11:17 ` [Intel-gfx] " Jani Nikula
` (2 preceding siblings ...)
(?)
@ 2022-11-14 18:56 ` Patchwork
-1 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2022-11-14 18:56 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 7890 bytes --]
== Series Details ==
Series: drm/edid/firmware: stop using a throwaway platform device
URL : https://patchwork.freedesktop.org/series/110859/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_12376 -> Patchwork_110859v1
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_110859v1 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_110859v1, 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_110859v1/index.html
Participating hosts (37 -> 38)
------------------------------
Additional (3): fi-kbl-soraka bat-kbl-2 fi-bsw-nick
Missing (2): fi-bdw-samus bat-dg1-5
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_110859v1:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live@mman:
- fi-kbl-soraka: NOTRUN -> [INCOMPLETE][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@i915_selftest@live@mman.html
Known issues
------------
Here are the changes found in Patchwork_110859v1 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_gttfill@basic:
- fi-kbl-soraka: NOTRUN -> [SKIP][2] ([fdo#109271]) +8 similar issues
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@gem_exec_gttfill@basic.html
* igt@gem_huc_copy@huc-copy:
- fi-kbl-soraka: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#2190])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html
* igt@gem_lmem_swapping@basic:
- fi-apl-guc: NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#4613]) +3 similar issues
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@gem_lmem_swapping@basic.html
- fi-kbl-soraka: NOTRUN -> [SKIP][5] ([fdo#109271] / [i915#4613]) +3 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html
* igt@gem_lmem_swapping@parallel-random-engines:
- fi-bsw-nick: NOTRUN -> [SKIP][6] ([fdo#109271]) +38 similar issues
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nick/igt@gem_lmem_swapping@parallel-random-engines.html
* igt@i915_pm_rpm@basic-pci-d3-state:
- bat-adlp-4: [PASS][7] -> [DMESG-WARN][8] ([i915#7077])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-4/igt@i915_pm_rpm@basic-pci-d3-state.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-4/igt@i915_pm_rpm@basic-pci-d3-state.html
* igt@i915_selftest@live@gt_pm:
- fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][9] ([i915#1886])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html
* igt@i915_selftest@live@mman:
- fi-rkl-guc: [PASS][10] -> [TIMEOUT][11] ([i915#6794])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-guc/igt@i915_selftest@live@mman.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-guc/igt@i915_selftest@live@mman.html
* igt@kms_chamelium@common-hpd-after-suspend:
- fi-rkl-11600: NOTRUN -> [SKIP][12] ([fdo#111827])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-11600/igt@kms_chamelium@common-hpd-after-suspend.html
* igt@kms_chamelium@hdmi-crc-fast:
- fi-apl-guc: NOTRUN -> [SKIP][13] ([fdo#109271] / [fdo#111827]) +8 similar issues
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@kms_chamelium@hdmi-crc-fast.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-bsw-nick: NOTRUN -> [SKIP][14] ([fdo#109271] / [fdo#111827]) +8 similar issues
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nick/igt@kms_chamelium@hdmi-hpd-fast.html
- fi-kbl-soraka: NOTRUN -> [SKIP][15] ([fdo#109271] / [fdo#111827]) +7 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_force_connector_basic@force-connector-state:
- fi-apl-guc: NOTRUN -> [SKIP][16] ([fdo#109271]) +11 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@kms_force_connector_basic@force-connector-state.html
#### Possible fixes ####
* igt@gem_render_tiled_blits@basic:
- fi-apl-guc: [INCOMPLETE][17] ([i915#7056]) -> [PASS][18]
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-apl-guc/igt@gem_render_tiled_blits@basic.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@gem_render_tiled_blits@basic.html
* igt@i915_selftest@live@migrate:
- {bat-adlp-6}: [INCOMPLETE][19] ([i915#7348]) -> [PASS][20]
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-6/igt@i915_selftest@live@migrate.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-6/igt@i915_selftest@live@migrate.html
* igt@i915_selftest@live@slpc:
- {bat-adln-1}: [DMESG-FAIL][21] ([i915#6997]) -> [PASS][22]
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adln-1/igt@i915_selftest@live@slpc.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adln-1/igt@i915_selftest@live@slpc.html
* igt@i915_suspend@basic-s3-without-i915:
- fi-rkl-11600: [INCOMPLETE][23] ([i915#4817]) -> [PASS][24]
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.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#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886
[i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
[i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4817]: https://gitlab.freedesktop.org/drm/intel/issues/4817
[i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
[i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
[i915#6794]: https://gitlab.freedesktop.org/drm/intel/issues/6794
[i915#6997]: https://gitlab.freedesktop.org/drm/intel/issues/6997
[i915#7056]: https://gitlab.freedesktop.org/drm/intel/issues/7056
[i915#7077]: https://gitlab.freedesktop.org/drm/intel/issues/7077
[i915#7348]: https://gitlab.freedesktop.org/drm/intel/issues/7348
Build changes
-------------
* Linux: CI_DRM_12376 -> Patchwork_110859v1
CI-20190529: 20190529
CI_DRM_12376: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7054: 5a0fd5b1011658f7ba117a45537e770a80d257fd @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_110859v1: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
### Linux commits
90c2b72dc380 drm/edid/firmware: stop using a throwaway platform device
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/index.html
[-- Attachment #2: Type: text/html, Size: 9480 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/edid/firmware: stop using a throwaway platform device
2022-11-14 11:17 ` [Intel-gfx] " Jani Nikula
` (3 preceding siblings ...)
(?)
@ 2022-11-14 19:10 ` Patchwork
-1 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2022-11-14 19:10 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 7449 bytes --]
== Series Details ==
Series: drm/edid/firmware: stop using a throwaway platform device
URL : https://patchwork.freedesktop.org/series/110859/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12376 -> Patchwork_110859v1
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/index.html
Participating hosts (37 -> 38)
------------------------------
Additional (3): fi-kbl-soraka bat-kbl-2 fi-bsw-nick
Missing (2): fi-bdw-samus bat-dg1-5
Known issues
------------
Here are the changes found in Patchwork_110859v1 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_gttfill@basic:
- fi-kbl-soraka: NOTRUN -> [SKIP][1] ([fdo#109271]) +8 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@gem_exec_gttfill@basic.html
* igt@gem_huc_copy@huc-copy:
- fi-kbl-soraka: NOTRUN -> [SKIP][2] ([fdo#109271] / [i915#2190])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html
* igt@gem_lmem_swapping@basic:
- fi-apl-guc: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#4613]) +3 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@gem_lmem_swapping@basic.html
- fi-kbl-soraka: NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#4613]) +3 similar issues
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html
* igt@gem_lmem_swapping@parallel-random-engines:
- fi-bsw-nick: NOTRUN -> [SKIP][5] ([fdo#109271]) +38 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nick/igt@gem_lmem_swapping@parallel-random-engines.html
* igt@i915_pm_rpm@basic-pci-d3-state:
- bat-adlp-4: [PASS][6] -> [DMESG-WARN][7] ([i915#7077])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-4/igt@i915_pm_rpm@basic-pci-d3-state.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-4/igt@i915_pm_rpm@basic-pci-d3-state.html
* igt@i915_selftest@live@gt_pm:
- fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][8] ([i915#1886])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html
* igt@i915_selftest@live@mman:
- fi-rkl-guc: [PASS][9] -> [TIMEOUT][10] ([i915#6794])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-guc/igt@i915_selftest@live@mman.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-guc/igt@i915_selftest@live@mman.html
- fi-kbl-soraka: NOTRUN -> [INCOMPLETE][11] ([i915#7320])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@i915_selftest@live@mman.html
* igt@kms_chamelium@common-hpd-after-suspend:
- fi-rkl-11600: NOTRUN -> [SKIP][12] ([fdo#111827])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-11600/igt@kms_chamelium@common-hpd-after-suspend.html
* igt@kms_chamelium@hdmi-crc-fast:
- fi-apl-guc: NOTRUN -> [SKIP][13] ([fdo#109271] / [fdo#111827]) +8 similar issues
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@kms_chamelium@hdmi-crc-fast.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-bsw-nick: NOTRUN -> [SKIP][14] ([fdo#109271] / [fdo#111827]) +8 similar issues
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-bsw-nick/igt@kms_chamelium@hdmi-hpd-fast.html
- fi-kbl-soraka: NOTRUN -> [SKIP][15] ([fdo#109271] / [fdo#111827]) +7 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-kbl-soraka/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_force_connector_basic@force-connector-state:
- fi-apl-guc: NOTRUN -> [SKIP][16] ([fdo#109271]) +11 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@kms_force_connector_basic@force-connector-state.html
#### Possible fixes ####
* igt@gem_render_tiled_blits@basic:
- fi-apl-guc: [INCOMPLETE][17] ([i915#7056]) -> [PASS][18]
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-apl-guc/igt@gem_render_tiled_blits@basic.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-apl-guc/igt@gem_render_tiled_blits@basic.html
* igt@i915_selftest@live@migrate:
- {bat-adlp-6}: [INCOMPLETE][19] ([i915#7348]) -> [PASS][20]
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adlp-6/igt@i915_selftest@live@migrate.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adlp-6/igt@i915_selftest@live@migrate.html
* igt@i915_selftest@live@slpc:
- {bat-adln-1}: [DMESG-FAIL][21] ([i915#6997]) -> [PASS][22]
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/bat-adln-1/igt@i915_selftest@live@slpc.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/bat-adln-1/igt@i915_selftest@live@slpc.html
* igt@i915_suspend@basic-s3-without-i915:
- fi-rkl-11600: [INCOMPLETE][23] ([i915#4817]) -> [PASS][24]
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.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#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886
[i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
[i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4817]: https://gitlab.freedesktop.org/drm/intel/issues/4817
[i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
[i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
[i915#6794]: https://gitlab.freedesktop.org/drm/intel/issues/6794
[i915#6997]: https://gitlab.freedesktop.org/drm/intel/issues/6997
[i915#7056]: https://gitlab.freedesktop.org/drm/intel/issues/7056
[i915#7077]: https://gitlab.freedesktop.org/drm/intel/issues/7077
[i915#7320]: https://gitlab.freedesktop.org/drm/intel/issues/7320
[i915#7348]: https://gitlab.freedesktop.org/drm/intel/issues/7348
Build changes
-------------
* Linux: CI_DRM_12376 -> Patchwork_110859v1
CI-20190529: 20190529
CI_DRM_12376: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7054: 5a0fd5b1011658f7ba117a45537e770a80d257fd @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_110859v1: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
### Linux commits
90c2b72dc380 drm/edid/firmware: stop using a throwaway platform device
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/index.html
[-- Attachment #2: Type: text/html, Size: 9021 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/edid/firmware: stop using a throwaway platform device
2022-11-14 11:17 ` [Intel-gfx] " Jani Nikula
` (4 preceding siblings ...)
(?)
@ 2022-11-14 22:03 ` Patchwork
-1 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2022-11-14 22:03 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 44328 bytes --]
== Series Details ==
Series: drm/edid/firmware: stop using a throwaway platform device
URL : https://patchwork.freedesktop.org/series/110859/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_12376_full -> Patchwork_110859v1_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_110859v1_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_110859v1_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
Participating hosts (11 -> 11)
------------------------------
No changes in participating hosts
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_110859v1_full:
### IGT changes ###
#### Possible regressions ####
* igt@kms_cursor_crc@cursor-sliding-64x64@pipe-b-edp-1:
- shard-tglb: [PASS][1] -> [INCOMPLETE][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-tglb1/igt@kms_cursor_crc@cursor-sliding-64x64@pipe-b-edp-1.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-tglb8/igt@kms_cursor_crc@cursor-sliding-64x64@pipe-b-edp-1.html
* igt@kms_flip@busy-flip@a-edp1:
- shard-skl: NOTRUN -> [FAIL][3]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl2/igt@kms_flip@busy-flip@a-edp1.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@gem_ctx_isolation@preservation-s3@vcs1:
- {shard-dg1}: NOTRUN -> [INCOMPLETE][4] +1 similar issue
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-dg1-13/igt@gem_ctx_isolation@preservation-s3@vcs1.html
Known issues
------------
Here are the changes found in Patchwork_110859v1_full that come from known issues:
### CI changes ###
#### Issues hit ####
* boot:
- shard-glk: ([PASS][5], [PASS][6], [PASS][7], [PASS][8], [PASS][9], [PASS][10], [PASS][11], [PASS][12], [PASS][13], [PASS][14], [PASS][15], [PASS][16], [PASS][17], [PASS][18], [PASS][19], [PASS][20], [PASS][21], [PASS][22], [PASS][23], [PASS][24], [PASS][25], [PASS][26], [PASS][27], [PASS][28], [PASS][29]) -> ([PASS][30], [PASS][31], [PASS][32], [FAIL][33], [PASS][34], [PASS][35], [PASS][36], [PASS][37], [PASS][38], [PASS][39], [PASS][40], [PASS][41], [PASS][42], [PASS][43], [PASS][44], [PASS][45], [PASS][46], [PASS][47], [PASS][48], [PASS][49], [PASS][50], [PASS][51], [PASS][52], [PASS][53], [PASS][54]) ([i915#4392])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk1/boot.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk9/boot.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk9/boot.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk9/boot.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk8/boot.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk8/boot.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk8/boot.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk7/boot.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk7/boot.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk7/boot.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk6/boot.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk6/boot.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk6/boot.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk5/boot.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk5/boot.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk5/boot.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk1/boot.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk3/boot.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk3/boot.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk1/boot.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk3/boot.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk2/boot.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk2/boot.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk2/boot.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk2/boot.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk5/boot.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk9/boot.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk9/boot.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk9/boot.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk9/boot.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk8/boot.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk8/boot.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk8/boot.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk7/boot.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk7/boot.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk7/boot.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk6/boot.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk6/boot.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk6/boot.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk1/boot.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk1/boot.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk1/boot.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk2/boot.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk2/boot.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk2/boot.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk5/boot.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk3/boot.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk3/boot.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk3/boot.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk5/boot.html
### IGT changes ###
#### Issues hit ####
* igt@dmabuf@all@dma_fence_chain:
- shard-skl: NOTRUN -> [INCOMPLETE][55] ([i915#6949])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@dmabuf@all@dma_fence_chain.html
* igt@gem_create@create-massive:
- shard-skl: NOTRUN -> [DMESG-WARN][56] ([i915#4991])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@gem_create@create-massive.html
* igt@gem_ctx_persistence@hang:
- shard-skl: NOTRUN -> [SKIP][57] ([fdo#109271]) +136 similar issues
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@gem_ctx_persistence@hang.html
* igt@gem_exec_balancer@parallel-bb-first:
- shard-iclb: [PASS][58] -> [SKIP][59] ([i915#4525])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb1/igt@gem_exec_balancer@parallel-bb-first.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb3/igt@gem_exec_balancer@parallel-bb-first.html
* igt@gem_exec_balancer@parallel-ordering:
- shard-iclb: NOTRUN -> [SKIP][60] ([i915#4525])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@gem_exec_balancer@parallel-ordering.html
* igt@gem_exec_fair@basic-deadline:
- shard-skl: NOTRUN -> [FAIL][61] ([i915#2846])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@gem_exec_fair@basic-deadline.html
* igt@gem_exec_fair@basic-none@vcs0:
- shard-glk: [PASS][62] -> [FAIL][63] ([i915#2842]) +1 similar issue
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk7/igt@gem_exec_fair@basic-none@vcs0.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk2/igt@gem_exec_fair@basic-none@vcs0.html
* igt@gem_huc_copy@huc-copy:
- shard-skl: NOTRUN -> [SKIP][64] ([fdo#109271] / [i915#2190])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl4/igt@gem_huc_copy@huc-copy.html
* igt@gem_lmem_swapping@heavy-verify-multi:
- shard-skl: NOTRUN -> [SKIP][65] ([fdo#109271] / [i915#4613]) +1 similar issue
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@gem_lmem_swapping@heavy-verify-multi.html
* igt@gem_lmem_swapping@verify-random:
- shard-iclb: NOTRUN -> [SKIP][66] ([i915#4613])
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@gem_lmem_swapping@verify-random.html
* igt@gem_pxp@verify-pxp-stale-buf-optout-execution:
- shard-iclb: NOTRUN -> [SKIP][67] ([i915#4270]) +1 similar issue
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@gem_pxp@verify-pxp-stale-buf-optout-execution.html
* igt@gen9_exec_parse@allowed-all:
- shard-iclb: NOTRUN -> [SKIP][68] ([i915#2856]) +1 similar issue
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@gen9_exec_parse@allowed-all.html
* igt@i915_pm_rc6_residency@rc6-idle@vcs0:
- shard-skl: [PASS][69] -> [WARN][70] ([i915#1804])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl9/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl2/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
* igt@i915_selftest@live@gt_pm:
- shard-skl: NOTRUN -> [DMESG-FAIL][71] ([i915#1886])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl6/igt@i915_selftest@live@gt_pm.html
* igt@i915_suspend@basic-s2idle-without-i915:
- shard-skl: [PASS][72] -> [DMESG-WARN][73] ([i915#1982])
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl9/igt@i915_suspend@basic-s2idle-without-i915.html
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl4/igt@i915_suspend@basic-s2idle-without-i915.html
* igt@kms_atomic_transition@plane-all-modeset-transition@pipe-b-hdmi-a-2:
- shard-glk: NOTRUN -> [INCOMPLETE][74] ([i915#5584])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk8/igt@kms_atomic_transition@plane-all-modeset-transition@pipe-b-hdmi-a-2.html
* igt@kms_big_fb@4-tiled-addfb-size-overflow:
- shard-iclb: NOTRUN -> [SKIP][75] ([i915#5286])
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_big_fb@4-tiled-addfb-size-overflow.html
* igt@kms_ccs@pipe-b-bad-rotation-90-y_tiled_gen12_mc_ccs:
- shard-iclb: NOTRUN -> [SKIP][76] ([fdo#109278] / [i915#3886]) +1 similar issue
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_ccs@pipe-b-bad-rotation-90-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-c-random-ccs-data-y_tiled_gen12_mc_ccs:
- shard-skl: NOTRUN -> [SKIP][77] ([fdo#109271] / [i915#3886]) +5 similar issues
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl2/igt@kms_ccs@pipe-c-random-ccs-data-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-d-crc-primary-rotation-180-4_tiled_dg2_rc_ccs:
- shard-iclb: NOTRUN -> [SKIP][78] ([fdo#109278]) +8 similar issues
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_ccs@pipe-d-crc-primary-rotation-180-4_tiled_dg2_rc_ccs.html
* igt@kms_chamelium@hdmi-hpd:
- shard-skl: NOTRUN -> [SKIP][79] ([fdo#109271] / [fdo#111827]) +7 similar issues
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl2/igt@kms_chamelium@hdmi-hpd.html
* igt@kms_chamelium@hdmi-hpd-with-enabled-mode:
- shard-iclb: NOTRUN -> [SKIP][80] ([fdo#109284] / [fdo#111827])
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_chamelium@hdmi-hpd-with-enabled-mode.html
* igt@kms_cursor_crc@cursor-sliding-512x512:
- shard-iclb: NOTRUN -> [SKIP][81] ([i915#3359])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_cursor_crc@cursor-sliding-512x512.html
* igt@kms_cursor_crc@cursor-suspend@pipe-c-edp-1:
- shard-skl: [PASS][82] -> [INCOMPLETE][83] ([i915#6951])
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl9/igt@kms_cursor_crc@cursor-suspend@pipe-c-edp-1.html
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl4/igt@kms_cursor_crc@cursor-suspend@pipe-c-edp-1.html
* igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size:
- shard-iclb: [PASS][84] -> [FAIL][85] ([i915#2346]) +1 similar issue
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb6/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size.html
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb7/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size.html
* igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-1:
- shard-glk: NOTRUN -> [SKIP][86] ([fdo#109271])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk3/igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-1.html
* igt@kms_flip@2x-flip-vs-wf_vblank:
- shard-iclb: NOTRUN -> [SKIP][87] ([fdo#109274]) +3 similar issues
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_flip@2x-flip-vs-wf_vblank.html
* igt@kms_flip@flip-vs-suspend@b-dp1:
- shard-apl: [PASS][88] -> [DMESG-WARN][89] ([i915#180]) +3 similar issues
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-apl8/igt@kms_flip@flip-vs-suspend@b-dp1.html
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-apl3/igt@kms_flip@flip-vs-suspend@b-dp1.html
* igt@kms_flip@plain-flip-fb-recreate@a-edp1:
- shard-skl: [PASS][90] -> [FAIL][91] ([i915#2122]) +1 similar issue
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl4/igt@kms_flip@plain-flip-fb-recreate@a-edp1.html
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl6/igt@kms_flip@plain-flip-fb-recreate@a-edp1.html
* igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-upscaling@pipe-a-valid-mode:
- shard-iclb: NOTRUN -> [SKIP][92] ([i915#2587] / [i915#2672]) +3 similar issues
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb5/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-upscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-32bpp-yftile-upscaling@pipe-a-default-mode:
- shard-iclb: NOTRUN -> [SKIP][93] ([i915#2672]) +5 similar issues
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-32bpp-yftile-upscaling@pipe-a-default-mode.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-blt:
- shard-iclb: NOTRUN -> [SKIP][94] ([fdo#109280]) +9 similar issues
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-render:
- shard-iclb: NOTRUN -> [FAIL][95] ([i915#2546])
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-render.html
* igt@kms_frontbuffer_tracking@psr-suspend:
- shard-skl: [PASS][96] -> [INCOMPLETE][97] ([i915#7255])
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl6/igt@kms_frontbuffer_tracking@psr-suspend.html
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl9/igt@kms_frontbuffer_tracking@psr-suspend.html
* igt@kms_hdr@static-toggle-dpms:
- shard-iclb: NOTRUN -> [SKIP][98] ([i915#3555]) +1 similar issue
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_hdr@static-toggle-dpms.html
* igt@kms_plane_alpha_blend@alpha-transparent-fb@pipe-c-edp-1:
- shard-skl: NOTRUN -> [FAIL][99] ([i915#4573]) +2 similar issues
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@kms_plane_alpha_blend@alpha-transparent-fb@pipe-c-edp-1.html
* igt@kms_plane_scaling@plane-downscale-with-pixel-format-factor-0-5@pipe-b-edp-1:
- shard-iclb: [PASS][100] -> [SKIP][101] ([i915#5176]) +2 similar issues
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb3/igt@kms_plane_scaling@plane-downscale-with-pixel-format-factor-0-5@pipe-b-edp-1.html
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb2/igt@kms_plane_scaling@plane-downscale-with-pixel-format-factor-0-5@pipe-b-edp-1.html
* igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf:
- shard-skl: NOTRUN -> [SKIP][102] ([fdo#109271] / [i915#658])
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl2/igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf.html
* igt@kms_psr2_su@page_flip-nv12@pipe-b-edp-1:
- shard-iclb: NOTRUN -> [FAIL][103] ([i915#5939]) +2 similar issues
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb2/igt@kms_psr2_su@page_flip-nv12@pipe-b-edp-1.html
* igt@kms_psr@psr2_cursor_plane_move:
- shard-iclb: NOTRUN -> [SKIP][104] ([fdo#109441])
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_psr@psr2_cursor_plane_move.html
* igt@kms_psr@psr2_primary_blt:
- shard-iclb: [PASS][105] -> [SKIP][106] ([fdo#109441]) +2 similar issues
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb2/igt@kms_psr@psr2_primary_blt.html
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb3/igt@kms_psr@psr2_primary_blt.html
* igt@kms_psr_stress_test@invalidate-primary-flip-overlay:
- shard-tglb: [PASS][107] -> [SKIP][108] ([i915#5519])
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-tglb8/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-tglb1/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html
- shard-iclb: NOTRUN -> [SKIP][109] ([i915#5519])
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html
* igt@kms_writeback@writeback-check-output:
- shard-skl: NOTRUN -> [SKIP][110] ([fdo#109271] / [i915#2437])
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl2/igt@kms_writeback@writeback-check-output.html
* igt@perf_pmu@interrupts:
- shard-skl: [PASS][111] -> [FAIL][112] ([i915#7318])
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl3/igt@perf_pmu@interrupts.html
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl10/igt@perf_pmu@interrupts.html
* igt@prime_vgem@coherency-gtt:
- shard-iclb: NOTRUN -> [SKIP][113] ([fdo#109292] / [fdo#109295])
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb6/igt@prime_vgem@coherency-gtt.html
* igt@sysfs_clients@fair-3:
- shard-skl: NOTRUN -> [SKIP][114] ([fdo#109271] / [i915#2994])
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@sysfs_clients@fair-3.html
#### Possible fixes ####
* igt@fbdev@nullptr:
- {shard-rkl}: [SKIP][115] ([i915#2582]) -> [PASS][116] +1 similar issue
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@fbdev@nullptr.html
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-6/igt@fbdev@nullptr.html
* igt@gem_ctx_exec@basic-nohangcheck:
- {shard-rkl}: [FAIL][117] ([i915#6268]) -> [PASS][118]
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@gem_ctx_exec@basic-nohangcheck.html
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-5/igt@gem_ctx_exec@basic-nohangcheck.html
- shard-tglb: [FAIL][119] ([i915#6268]) -> [PASS][120]
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-tglb6/igt@gem_ctx_exec@basic-nohangcheck.html
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-tglb6/igt@gem_ctx_exec@basic-nohangcheck.html
* igt@gem_ctx_isolation@preservation-s3@rcs0:
- {shard-dg1}: [INCOMPLETE][121] ([i915#4793]) -> [PASS][122]
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-dg1-19/igt@gem_ctx_isolation@preservation-s3@rcs0.html
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-dg1-13/igt@gem_ctx_isolation@preservation-s3@rcs0.html
* igt@gem_eio@in-flight-suspend:
- {shard-rkl}: [FAIL][123] ([fdo#103375]) -> [PASS][124] +2 similar issues
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-3/igt@gem_eio@in-flight-suspend.html
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-1/igt@gem_eio@in-flight-suspend.html
* igt@gem_eio@suspend:
- {shard-rkl}: [FAIL][125] ([i915#7052]) -> [PASS][126]
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@gem_eio@suspend.html
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-5/igt@gem_eio@suspend.html
* igt@gem_exec_balancer@parallel-balancer:
- shard-iclb: [SKIP][127] ([i915#4525]) -> [PASS][128]
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb5/igt@gem_exec_balancer@parallel-balancer.html
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb1/igt@gem_exec_balancer@parallel-balancer.html
* igt@gem_exec_fair@basic-flow@rcs0:
- shard-tglb: [FAIL][129] ([i915#2842]) -> [PASS][130] +1 similar issue
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-tglb3/igt@gem_exec_fair@basic-flow@rcs0.html
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-tglb2/igt@gem_exec_fair@basic-flow@rcs0.html
* igt@gem_exec_fair@basic-pace-share@rcs0:
- shard-glk: [FAIL][131] ([i915#2842]) -> [PASS][132]
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk8/igt@gem_exec_fair@basic-pace-share@rcs0.html
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk6/igt@gem_exec_fair@basic-pace-share@rcs0.html
- {shard-rkl}: [FAIL][133] ([i915#2842]) -> [PASS][134] +1 similar issue
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-5/igt@gem_exec_fair@basic-pace-share@rcs0.html
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-5/igt@gem_exec_fair@basic-pace-share@rcs0.html
* igt@gem_exec_reloc@basic-write-read-noreloc:
- {shard-rkl}: [SKIP][135] ([i915#3281]) -> [PASS][136] +5 similar issues
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@gem_exec_reloc@basic-write-read-noreloc.html
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-5/igt@gem_exec_reloc@basic-write-read-noreloc.html
* igt@gen9_exec_parse@bb-chained:
- {shard-rkl}: [SKIP][137] ([i915#2527]) -> [PASS][138] +1 similar issue
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@gen9_exec_parse@bb-chained.html
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-5/igt@gen9_exec_parse@bb-chained.html
* igt@kms_async_flips@alternate-sync-async-flip@pipe-b-edp-1:
- shard-skl: [FAIL][139] ([i915#2521]) -> [PASS][140] +2 similar issues
[139]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl6/igt@kms_async_flips@alternate-sync-async-flip@pipe-b-edp-1.html
[140]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@kms_async_flips@alternate-sync-async-flip@pipe-b-edp-1.html
* igt@kms_atomic_transition@plane-all-modeset-transition@pipe-b-hdmi-a-1:
- shard-glk: [INCOMPLETE][141] ([i915#5584]) -> [PASS][142]
[141]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk5/igt@kms_atomic_transition@plane-all-modeset-transition@pipe-b-hdmi-a-1.html
[142]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk8/igt@kms_atomic_transition@plane-all-modeset-transition@pipe-b-hdmi-a-1.html
* igt@kms_cursor_legacy@cursor-vs-flip@atomic-transitions:
- shard-skl: [INCOMPLETE][143] ([i915#1982] / [i915#7096]) -> [PASS][144]
[143]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl3/igt@kms_cursor_legacy@cursor-vs-flip@atomic-transitions.html
[144]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl10/igt@kms_cursor_legacy@cursor-vs-flip@atomic-transitions.html
* igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size:
- shard-glk: [FAIL][145] ([i915#2346]) -> [PASS][146] +1 similar issue
[145]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk5/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size.html
[146]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk8/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size.html
* igt@kms_cursor_legacy@flip-vs-cursor@legacy:
- shard-skl: [FAIL][147] ([i915#2346]) -> [PASS][148]
[147]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl3/igt@kms_cursor_legacy@flip-vs-cursor@legacy.html
[148]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl4/igt@kms_cursor_legacy@flip-vs-cursor@legacy.html
* igt@kms_fbcon_fbt@psr-suspend:
- {shard-rkl}: [SKIP][149] ([i915#3955]) -> [PASS][150]
[149]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@kms_fbcon_fbt@psr-suspend.html
[150]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-6/igt@kms_fbcon_fbt@psr-suspend.html
- shard-skl: [INCOMPLETE][151] -> [PASS][152]
[151]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl7/igt@kms_fbcon_fbt@psr-suspend.html
[152]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@kms_fbcon_fbt@psr-suspend.html
* igt@kms_flip@2x-flip-vs-expired-vblank@bc-hdmi-a1-hdmi-a2:
- shard-glk: [FAIL][153] ([i915#79]) -> [PASS][154]
[153]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk9/igt@kms_flip@2x-flip-vs-expired-vblank@bc-hdmi-a1-hdmi-a2.html
[154]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk6/igt@kms_flip@2x-flip-vs-expired-vblank@bc-hdmi-a1-hdmi-a2.html
* igt@kms_flip@2x-plain-flip-fb-recreate@ab-hdmi-a1-hdmi-a2:
- shard-glk: [FAIL][155] ([i915#2122]) -> [PASS][156]
[155]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk5/igt@kms_flip@2x-plain-flip-fb-recreate@ab-hdmi-a1-hdmi-a2.html
[156]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk7/igt@kms_flip@2x-plain-flip-fb-recreate@ab-hdmi-a1-hdmi-a2.html
* igt@kms_flip@flip-vs-blocking-wf-vblank@b-edp1:
- shard-skl: [FAIL][157] ([i915#2122]) -> [PASS][158] +2 similar issues
[157]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl2/igt@kms_flip@flip-vs-blocking-wf-vblank@b-edp1.html
[158]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl6/igt@kms_flip@flip-vs-blocking-wf-vblank@b-edp1.html
* igt@kms_flip@flip-vs-expired-vblank@a-edp1:
- shard-skl: [FAIL][159] ([i915#79]) -> [PASS][160] +1 similar issue
[159]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl10/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
[160]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl10/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
* igt@kms_flip@flip-vs-suspend@c-dp1:
- shard-apl: [DMESG-WARN][161] ([i915#180]) -> [PASS][162]
[161]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-apl8/igt@kms_flip@flip-vs-suspend@c-dp1.html
[162]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-apl3/igt@kms_flip@flip-vs-suspend@c-dp1.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-indfb-draw-render:
- {shard-rkl}: [SKIP][163] ([i915#1849] / [i915#4098]) -> [PASS][164] +10 similar issues
[163]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-indfb-draw-render.html
[164]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-6/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-indfb-draw-render.html
* igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-5@pipe-a-edp-1:
- shard-iclb: [SKIP][165] ([i915#5235]) -> [PASS][166] +2 similar issues
[165]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb2/igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-5@pipe-a-edp-1.html
[166]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb3/igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-5@pipe-a-edp-1.html
* igt@kms_psr@primary_render:
- {shard-rkl}: [SKIP][167] ([i915#1072]) -> [PASS][168]
[167]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@kms_psr@primary_render.html
[168]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-6/igt@kms_psr@primary_render.html
* igt@kms_psr@psr2_sprite_plane_onoff:
- shard-iclb: [SKIP][169] ([fdo#109441]) -> [PASS][170] +2 similar issues
[169]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb1/igt@kms_psr@psr2_sprite_plane_onoff.html
[170]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb2/igt@kms_psr@psr2_sprite_plane_onoff.html
* igt@kms_vblank@pipe-b-ts-continuation-idle:
- {shard-rkl}: [SKIP][171] ([i915#1845] / [i915#4098]) -> [PASS][172] +11 similar issues
[171]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@kms_vblank@pipe-b-ts-continuation-idle.html
[172]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-6/igt@kms_vblank@pipe-b-ts-continuation-idle.html
* igt@perf@polling:
- shard-skl: [FAIL][173] ([i915#1542]) -> [PASS][174]
[173]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl3/igt@perf@polling.html
[174]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl4/igt@perf@polling.html
* igt@prime_vgem@basic-fence-flip:
- {shard-rkl}: [SKIP][175] ([fdo#109295] / [i915#3708] / [i915#4098]) -> [PASS][176]
[175]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@prime_vgem@basic-fence-flip.html
[176]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-6/igt@prime_vgem@basic-fence-flip.html
* igt@testdisplay:
- {shard-rkl}: [SKIP][177] ([i915#4098]) -> [PASS][178]
[177]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-rkl-4/igt@testdisplay.html
[178]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-rkl-6/igt@testdisplay.html
#### Warnings ####
* igt@gem_pwrite@basic-exhaustion:
- shard-glk: [INCOMPLETE][179] ([i915#7248]) -> [WARN][180] ([i915#2658])
[179]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-glk9/igt@gem_pwrite@basic-exhaustion.html
[180]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-glk8/igt@gem_pwrite@basic-exhaustion.html
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-move:
- shard-skl: [INCOMPLETE][181] -> [SKIP][182] ([fdo#109271])
[181]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl3/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-move.html
[182]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl4/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-move.html
* igt@kms_plane_alpha_blend@alpha-basic@pipe-c-dp-1:
- shard-apl: [FAIL][183] ([i915#4573]) -> [DMESG-FAIL][184] ([IGT#6]) +1 similar issue
[183]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-apl7/igt@kms_plane_alpha_blend@alpha-basic@pipe-c-dp-1.html
[184]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-apl2/igt@kms_plane_alpha_blend@alpha-basic@pipe-c-dp-1.html
* igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-fully-sf:
- shard-iclb: [SKIP][185] ([i915#2920]) -> [SKIP][186] ([i915#658])
[185]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb2/igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-fully-sf.html
[186]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb3/igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-fully-sf.html
* igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf:
- shard-iclb: [SKIP][187] ([i915#658]) -> [SKIP][188] ([i915#2920]) +1 similar issue
[187]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb1/igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf.html
[188]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb2/igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf.html
* igt@kms_psr2_sf@overlay-plane-update-continuous-sf:
- shard-iclb: [SKIP][189] ([fdo#111068] / [i915#658]) -> [SKIP][190] ([i915#2920])
[189]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-iclb3/igt@kms_psr2_sf@overlay-plane-update-continuous-sf.html
[190]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-iclb2/igt@kms_psr2_sf@overlay-plane-update-continuous-sf.html
* igt@runner@aborted:
- shard-skl: [FAIL][191] ([i915#4312]) -> ([FAIL][192], [FAIL][193], [FAIL][194]) ([i915#3002] / [i915#4312] / [i915#6949])
[191]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12376/shard-skl2/igt@runner@aborted.html
[192]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@runner@aborted.html
[193]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl7/igt@runner@aborted.html
[194]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/shard-skl2/igt@runner@aborted.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[IGT#6]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/6
[fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109279]: https://bugs.freedesktop.org/show_bug.cgi?id=109279
[fdo#109280]: https://bugs.freedesktop.org/show_bug.cgi?id=109280
[fdo#109283]: https://bugs.freedesktop.org/show_bug.cgi?id=109283
[fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284
[fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
[fdo#109292]: https://bugs.freedesktop.org/show_bug.cgi?id=109292
[fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#110723]: https://bugs.freedesktop.org/show_bug.cgi?id=110723
[fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
[fdo#111614]: https://bugs.freedesktop.org/show_bug.cgi?id=111614
[fdo#111615]: https://bugs.freedesktop.org/show_bug.cgi?id=111615
[fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[fdo#112054]: https://bugs.freedesktop.org/show_bug.cgi?id=112054
[i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
[i915#132]: https://gitlab.freedesktop.org/drm/intel/issues/132
[i915#1397]: https://gitlab.freedesktop.org/drm/intel/issues/1397
[i915#1542]: https://gitlab.freedesktop.org/drm/intel/issues/1542
[i915#1722]: https://gitlab.freedesktop.org/drm/intel/issues/1722
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#1804]: https://gitlab.freedesktop.org/drm/intel/issues/1804
[i915#1825]: https://gitlab.freedesktop.org/drm/intel/issues/1825
[i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
[i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849
[i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
[i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
[i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
[i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437
[i915#2521]: https://gitlab.freedesktop.org/drm/intel/issues/2521
[i915#2527]: https://gitlab.freedesktop.org/drm/intel/issues/2527
[i915#2546]: https://gitlab.freedesktop.org/drm/intel/issues/2546
[i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
[i915#2587]: https://gitlab.freedesktop.org/drm/intel/issues/2587
[i915#2658]: https://gitlab.freedesktop.org/drm/intel/issues/2658
[i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
[i915#280]: https://gitlab.freedesktop.org/drm/intel/issues/280
[i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
[i915#2846]: https://gitlab.freedesktop.org/drm/intel/issues/2846
[i915#2856]: https://gitlab.freedesktop.org/drm/intel/issues/2856
[i915#2920]: https://gitlab.freedesktop.org/drm/intel/issues/2920
[i915#2994]: https://gitlab.freedesktop.org/drm/intel/issues/2994
[i915#3002]: https://gitlab.freedesktop.org/drm/intel/issues/3002
[i915#3116]: https://gitlab.freedesktop.org/drm/intel/issues/3116
[i915#3281]: https://gitlab.freedesktop.org/drm/intel/issues/3281
[i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
[i915#3297]: https://gitlab.freedesktop.org/drm/intel/issues/3297
[i915#3299]: https://gitlab.freedesktop.org/drm/intel/issues/3299
[i915#3359]: https://gitlab.freedesktop.org/drm/intel/issues/3359
[i915#3458]: https://gitlab.freedesktop.org/drm/intel/issues/3458
[i915#3536]: https://gitlab.freedesktop.org/drm/intel/issues/3536
[i915#3539]: https://gitlab.freedesktop.org/drm/intel/issues/3539
[i915#3546]: https://gitlab.freedesktop.org/drm/intel/issues/3546
[i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
[i915#3558]: https://gitlab.freedesktop.org/drm/intel/issues/3558
[i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
[i915#3638]: https://gitlab.freedesktop.org/drm/intel/issues/3638
[i915#3689]: https://gitlab.freedesktop.org/drm/intel/issues/3689
[i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
[i915#3734]: https://gitlab.freedesktop.org/drm/intel/issues/3734
[i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886
[i915#3952]: https://gitlab.freedesktop.org/drm/intel/issues/3952
[i915#3955]: https://gitlab.freedesktop.org/drm/intel/issues/3955
[i915#4070]: https://gitlab.freedesktop.org/drm/intel/issues/4070
[i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
[i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
[i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098
[i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4392]: https://gitlab.freedesktop.org/drm/intel/issues/4392
[i915#4525]: https://gitlab.freedesktop.org/drm/intel/issues/4525
[i915#4538]: https://gitlab.freedesktop.org/drm/intel/issues/4538
[i915#4573]: https://gitlab.freedesktop.org/drm/intel/issues/4573
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4767]: https://gitlab.freedesktop.org/drm/intel/issues/4767
[i915#4793]: https://gitlab.freedesktop.org/drm/intel/issues/4793
[i915#4833]: https://gitlab.freedesktop.org/drm/intel/issues/4833
[i915#4852]: https://gitlab.freedesktop.org/drm/intel/issues/4852
[i915#4880]: https://gitlab.freedesktop.org/drm/intel/issues/4880
[i915#4991]: https://gitlab.freedesktop.org/drm/intel/issues/4991
[i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176
[i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235
[i915#5286]: https://gitlab.freedesktop.org/drm/intel/issues/5286
[i915#5288]: https://gitlab.freedesktop.org/drm/intel/issues/5288
[i915#5289]: https://gitlab.freedesktop.org/drm/intel/issues/5289
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
[i915#5439]: https://gitlab.freedesktop.org/drm/intel/issues/5439
[i915#5519]: https://gitlab.freedesktop.org/drm/intel/issues/5519
[i915#5563]: https://gitlab.freedesktop.org/drm/intel/issues/5563
[i915#5584]: https://gitlab.freedesktop.org/drm/intel/issues/5584
[i915#5939]: https://gitlab.freedesktop.org/drm/intel/issues/5939
[i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095
[i915#6248]: https://gitlab.freedesktop.org/drm/intel/issues/6248
[i915#6268]: https://gitlab.freedesktop.org/drm/intel/issues/6268
[i915#6497]: https://gitlab.freedesktop.org/drm/intel/issues/6497
[i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
[i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621
[i915#6768]: https://gitlab.freedesktop.org/drm/intel/issues/6768
[i915#6949]: https://gitlab.freedesktop.org/drm/intel/issues/6949
[i915#6951]: https://gitlab.freedesktop.org/drm/intel/issues/6951
[i915#7052]: https://gitlab.freedesktop.org/drm/intel/issues/7052
[i915#7096]: https://gitlab.freedesktop.org/drm/intel/issues/7096
[i915#7118]: https://gitlab.freedesktop.org/drm/intel/issues/7118
[i915#7248]: https://gitlab.freedesktop.org/drm/intel/issues/7248
[i915#7255]: https://gitlab.freedesktop.org/drm/intel/issues/7255
[i915#7318]: https://gitlab.freedesktop.org/drm/intel/issues/7318
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
Build changes
-------------
* Linux: CI_DRM_12376 -> Patchwork_110859v1
CI-20190529: 20190529
CI_DRM_12376: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7054: 5a0fd5b1011658f7ba117a45537e770a80d257fd @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_110859v1: ceb77a7b0d107112b1b4197917d8fdec9a3d0ca8 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110859v1/index.html
[-- Attachment #2: Type: text/html, Size: 47298 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RESEND] drm/edid/firmware: stop using a throwaway platform device
2022-11-14 11:17 ` [Intel-gfx] " Jani Nikula
@ 2022-11-16 9:33 ` Thomas Zimmermann
-1 siblings, 0 replies; 15+ messages in thread
From: Thomas Zimmermann @ 2022-11-16 9:33 UTC (permalink / raw)
To: Jani Nikula, dri-devel, intel-gfx; +Cc: Matthieu CHARETTE
[-- Attachment #1.1: Type: text/plain, Size: 3065 bytes --]
Hi
Am 14.11.22 um 12:17 schrieb Jani Nikula:
> We've used a temporary platform device for firmware EDID loading since
> it was introduced in commit da0df92b5731 ("drm: allow loading an EDID as
> firmware to override broken monitor"), but there's no explanation why.
>
> Using a temporary device does not play well with CONFIG_FW_CACHE=y,
> which caches firmware images (e.g. on suspend) so that drivers can
> request firmware when the system is not ready for it, and return the
> images from the cache (e.g. during resume). This works automatically for
> regular devices, but obviously not for a temporarily created device.
>
> Stop using the throwaway platform device, and use the drm device
> instead.
>
> Note that this may still be problematic for cases where the display was
> plugged in during suspend, and the firmware wasn't loaded and therefore
> not cached before suspend.
>
> References: https://lore.kernel.org/r/20220727074152.43059-1-matthieu.charette@gmail.com
> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/2061
> Reported-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
> Tested-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
I looked through request_firmware() but did not see any signs that it
somehow depends on a platform device. I assume that this might only
affect the device name in the error message.
Best regards
Thomas
>
> ---
>
> Resend with a proper commit message; patch itself is unchanged.
> ---
> drivers/gpu/drm/drm_edid_load.c | 13 +------------
> 1 file changed, 1 insertion(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c
> index ef4ab59d6935..5d9ef267ebb3 100644
> --- a/drivers/gpu/drm/drm_edid_load.c
> +++ b/drivers/gpu/drm/drm_edid_load.c
> @@ -172,20 +172,9 @@ static const struct drm_edid *edid_load(struct drm_connector *connector, const c
> fwdata = generic_edid[builtin];
> fwsize = sizeof(generic_edid[builtin]);
> } else {
> - struct platform_device *pdev;
> int err;
>
> - pdev = platform_device_register_simple(connector->name, -1, NULL, 0);
> - if (IS_ERR(pdev)) {
> - drm_err(connector->dev,
> - "[CONNECTOR:%d:%s] Failed to register EDID firmware platform device for connector \"%s\"\n",
> - connector->base.id, connector->name,
> - connector->name);
> - return ERR_CAST(pdev);
> - }
> -
> - err = request_firmware(&fw, name, &pdev->dev);
> - platform_device_unregister(pdev);
> + err = request_firmware(&fw, name, connector->dev->dev);
> if (err) {
> drm_err(connector->dev,
> "[CONNECTOR:%d:%s] Requesting EDID firmware \"%s\" failed (err=%d)\n",
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Ivo Totev
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [RESEND] drm/edid/firmware: stop using a throwaway platform device
@ 2022-11-16 9:33 ` Thomas Zimmermann
0 siblings, 0 replies; 15+ messages in thread
From: Thomas Zimmermann @ 2022-11-16 9:33 UTC (permalink / raw)
To: Jani Nikula, dri-devel, intel-gfx; +Cc: Matthieu CHARETTE
[-- Attachment #1.1: Type: text/plain, Size: 3065 bytes --]
Hi
Am 14.11.22 um 12:17 schrieb Jani Nikula:
> We've used a temporary platform device for firmware EDID loading since
> it was introduced in commit da0df92b5731 ("drm: allow loading an EDID as
> firmware to override broken monitor"), but there's no explanation why.
>
> Using a temporary device does not play well with CONFIG_FW_CACHE=y,
> which caches firmware images (e.g. on suspend) so that drivers can
> request firmware when the system is not ready for it, and return the
> images from the cache (e.g. during resume). This works automatically for
> regular devices, but obviously not for a temporarily created device.
>
> Stop using the throwaway platform device, and use the drm device
> instead.
>
> Note that this may still be problematic for cases where the display was
> plugged in during suspend, and the firmware wasn't loaded and therefore
> not cached before suspend.
>
> References: https://lore.kernel.org/r/20220727074152.43059-1-matthieu.charette@gmail.com
> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/2061
> Reported-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
> Tested-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
I looked through request_firmware() but did not see any signs that it
somehow depends on a platform device. I assume that this might only
affect the device name in the error message.
Best regards
Thomas
>
> ---
>
> Resend with a proper commit message; patch itself is unchanged.
> ---
> drivers/gpu/drm/drm_edid_load.c | 13 +------------
> 1 file changed, 1 insertion(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c
> index ef4ab59d6935..5d9ef267ebb3 100644
> --- a/drivers/gpu/drm/drm_edid_load.c
> +++ b/drivers/gpu/drm/drm_edid_load.c
> @@ -172,20 +172,9 @@ static const struct drm_edid *edid_load(struct drm_connector *connector, const c
> fwdata = generic_edid[builtin];
> fwsize = sizeof(generic_edid[builtin]);
> } else {
> - struct platform_device *pdev;
> int err;
>
> - pdev = platform_device_register_simple(connector->name, -1, NULL, 0);
> - if (IS_ERR(pdev)) {
> - drm_err(connector->dev,
> - "[CONNECTOR:%d:%s] Failed to register EDID firmware platform device for connector \"%s\"\n",
> - connector->base.id, connector->name,
> - connector->name);
> - return ERR_CAST(pdev);
> - }
> -
> - err = request_firmware(&fw, name, &pdev->dev);
> - platform_device_unregister(pdev);
> + err = request_firmware(&fw, name, connector->dev->dev);
> if (err) {
> drm_err(connector->dev,
> "[CONNECTOR:%d:%s] Requesting EDID firmware \"%s\" failed (err=%d)\n",
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Ivo Totev
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RESEND] drm/edid/firmware: stop using a throwaway platform device
2022-11-16 9:33 ` [Intel-gfx] " Thomas Zimmermann
@ 2022-11-16 13:32 ` Jani Nikula
-1 siblings, 0 replies; 15+ messages in thread
From: Jani Nikula @ 2022-11-16 13:32 UTC (permalink / raw)
To: Thomas Zimmermann, dri-devel, intel-gfx; +Cc: Matthieu CHARETTE
On Wed, 16 Nov 2022, Thomas Zimmermann <tzimmermann@suse.de> wrote:
> Hi
>
> Am 14.11.22 um 12:17 schrieb Jani Nikula:
>> We've used a temporary platform device for firmware EDID loading since
>> it was introduced in commit da0df92b5731 ("drm: allow loading an EDID as
>> firmware to override broken monitor"), but there's no explanation why.
>>
>> Using a temporary device does not play well with CONFIG_FW_CACHE=y,
>> which caches firmware images (e.g. on suspend) so that drivers can
>> request firmware when the system is not ready for it, and return the
>> images from the cache (e.g. during resume). This works automatically for
>> regular devices, but obviously not for a temporarily created device.
>>
>> Stop using the throwaway platform device, and use the drm device
>> instead.
>>
>> Note that this may still be problematic for cases where the display was
>> plugged in during suspend, and the firmware wasn't loaded and therefore
>> not cached before suspend.
>>
>> References: https://lore.kernel.org/r/20220727074152.43059-1-matthieu.charette@gmail.com
>> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/2061
>> Reported-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
>> Tested-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
>> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>
> Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
>
> I looked through request_firmware() but did not see any signs that it
> somehow depends on a platform device. I assume that this might only
> affect the device name in the error message.
Thanks, pushed to drm-misc-next.
Matthieu, thanks for you patience and the report as well!
BR,
Jani.
>
> Best regards
> Thomas
>
>>
>> ---
>>
>> Resend with a proper commit message; patch itself is unchanged.
>> ---
>> drivers/gpu/drm/drm_edid_load.c | 13 +------------
>> 1 file changed, 1 insertion(+), 12 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c
>> index ef4ab59d6935..5d9ef267ebb3 100644
>> --- a/drivers/gpu/drm/drm_edid_load.c
>> +++ b/drivers/gpu/drm/drm_edid_load.c
>> @@ -172,20 +172,9 @@ static const struct drm_edid *edid_load(struct drm_connector *connector, const c
>> fwdata = generic_edid[builtin];
>> fwsize = sizeof(generic_edid[builtin]);
>> } else {
>> - struct platform_device *pdev;
>> int err;
>>
>> - pdev = platform_device_register_simple(connector->name, -1, NULL, 0);
>> - if (IS_ERR(pdev)) {
>> - drm_err(connector->dev,
>> - "[CONNECTOR:%d:%s] Failed to register EDID firmware platform device for connector \"%s\"\n",
>> - connector->base.id, connector->name,
>> - connector->name);
>> - return ERR_CAST(pdev);
>> - }
>> -
>> - err = request_firmware(&fw, name, &pdev->dev);
>> - platform_device_unregister(pdev);
>> + err = request_firmware(&fw, name, connector->dev->dev);
>> if (err) {
>> drm_err(connector->dev,
>> "[CONNECTOR:%d:%s] Requesting EDID firmware \"%s\" failed (err=%d)\n",
--
Jani Nikula, Intel Open Source Graphics Center
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [RESEND] drm/edid/firmware: stop using a throwaway platform device
@ 2022-11-16 13:32 ` Jani Nikula
0 siblings, 0 replies; 15+ messages in thread
From: Jani Nikula @ 2022-11-16 13:32 UTC (permalink / raw)
To: Thomas Zimmermann, dri-devel, intel-gfx; +Cc: Matthieu CHARETTE
On Wed, 16 Nov 2022, Thomas Zimmermann <tzimmermann@suse.de> wrote:
> Hi
>
> Am 14.11.22 um 12:17 schrieb Jani Nikula:
>> We've used a temporary platform device for firmware EDID loading since
>> it was introduced in commit da0df92b5731 ("drm: allow loading an EDID as
>> firmware to override broken monitor"), but there's no explanation why.
>>
>> Using a temporary device does not play well with CONFIG_FW_CACHE=y,
>> which caches firmware images (e.g. on suspend) so that drivers can
>> request firmware when the system is not ready for it, and return the
>> images from the cache (e.g. during resume). This works automatically for
>> regular devices, but obviously not for a temporarily created device.
>>
>> Stop using the throwaway platform device, and use the drm device
>> instead.
>>
>> Note that this may still be problematic for cases where the display was
>> plugged in during suspend, and the firmware wasn't loaded and therefore
>> not cached before suspend.
>>
>> References: https://lore.kernel.org/r/20220727074152.43059-1-matthieu.charette@gmail.com
>> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/2061
>> Reported-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
>> Tested-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
>> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>
> Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
>
> I looked through request_firmware() but did not see any signs that it
> somehow depends on a platform device. I assume that this might only
> affect the device name in the error message.
Thanks, pushed to drm-misc-next.
Matthieu, thanks for you patience and the report as well!
BR,
Jani.
>
> Best regards
> Thomas
>
>>
>> ---
>>
>> Resend with a proper commit message; patch itself is unchanged.
>> ---
>> drivers/gpu/drm/drm_edid_load.c | 13 +------------
>> 1 file changed, 1 insertion(+), 12 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c
>> index ef4ab59d6935..5d9ef267ebb3 100644
>> --- a/drivers/gpu/drm/drm_edid_load.c
>> +++ b/drivers/gpu/drm/drm_edid_load.c
>> @@ -172,20 +172,9 @@ static const struct drm_edid *edid_load(struct drm_connector *connector, const c
>> fwdata = generic_edid[builtin];
>> fwsize = sizeof(generic_edid[builtin]);
>> } else {
>> - struct platform_device *pdev;
>> int err;
>>
>> - pdev = platform_device_register_simple(connector->name, -1, NULL, 0);
>> - if (IS_ERR(pdev)) {
>> - drm_err(connector->dev,
>> - "[CONNECTOR:%d:%s] Failed to register EDID firmware platform device for connector \"%s\"\n",
>> - connector->base.id, connector->name,
>> - connector->name);
>> - return ERR_CAST(pdev);
>> - }
>> -
>> - err = request_firmware(&fw, name, &pdev->dev);
>> - platform_device_unregister(pdev);
>> + err = request_firmware(&fw, name, connector->dev->dev);
>> if (err) {
>> drm_err(connector->dev,
>> "[CONNECTOR:%d:%s] Requesting EDID firmware \"%s\" failed (err=%d)\n",
--
Jani Nikula, Intel Open Source Graphics Center
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RESEND] drm/edid/firmware: stop using a throwaway platform device
2022-11-16 13:32 ` [Intel-gfx] " Jani Nikula
@ 2022-11-16 18:17 ` Matthieu CHARETTE
-1 siblings, 0 replies; 15+ messages in thread
From: Matthieu CHARETTE @ 2022-11-16 18:17 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx, dri-devel, Thomas Zimmermann
Thank you everyone for your work!
Matthieu.
On Wed, Nov 16 2022 at 03:32:01 PM +0200, Jani Nikula
<jani.nikula@intel.com> wrote:
> On Wed, 16 Nov 2022, Thomas Zimmermann <tzimmermann@suse.de> wrote:
>> Hi
>>
>> Am 14.11.22 um 12:17 schrieb Jani Nikula:
>>> We've used a temporary platform device for firmware EDID loading
>>> since
>>> it was introduced in commit da0df92b5731 ("drm: allow loading an
>>> EDID as
>>> firmware to override broken monitor"), but there's no explanation
>>> why.
>>>
>>> Using a temporary device does not play well with CONFIG_FW_CACHE=y,
>>> which caches firmware images (e.g. on suspend) so that drivers can
>>> request firmware when the system is not ready for it, and return
>>> the
>>> images from the cache (e.g. during resume). This works
>>> automatically for
>>> regular devices, but obviously not for a temporarily created
>>> device.
>>>
>>> Stop using the throwaway platform device, and use the drm device
>>> instead.
>>>
>>> Note that this may still be problematic for cases where the
>>> display was
>>> plugged in during suspend, and the firmware wasn't loaded and
>>> therefore
>>> not cached before suspend.
>>>
>>> References:
>>> https://lore.kernel.org/r/20220727074152.43059-1-matthieu.charette@gmail.com
>>> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/2061
>>> Reported-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
>>> Tested-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
>>> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
>>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>>
>> Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
>>
>> I looked through request_firmware() but did not see any signs that
>> it
>> somehow depends on a platform device. I assume that this might only
>> affect the device name in the error message.
>
> Thanks, pushed to drm-misc-next.
>
> Matthieu, thanks for you patience and the report as well!
>
> BR,
> Jani.
>
>
>>
>> Best regards
>> Thomas
>>
>>>
>>> ---
>>>
>>> Resend with a proper commit message; patch itself is unchanged.
>>> ---
>>> drivers/gpu/drm/drm_edid_load.c | 13 +------------
>>> 1 file changed, 1 insertion(+), 12 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/drm_edid_load.c
>>> b/drivers/gpu/drm/drm_edid_load.c
>>> index ef4ab59d6935..5d9ef267ebb3 100644
>>> --- a/drivers/gpu/drm/drm_edid_load.c
>>> +++ b/drivers/gpu/drm/drm_edid_load.c
>>> @@ -172,20 +172,9 @@ static const struct drm_edid
>>> *edid_load(struct drm_connector *connector, const c
>>> fwdata = generic_edid[builtin];
>>> fwsize = sizeof(generic_edid[builtin]);
>>> } else {
>>> - struct platform_device *pdev;
>>> int err;
>>>
>>> - pdev = platform_device_register_simple(connector->name, -1,
>>> NULL, 0);
>>> - if (IS_ERR(pdev)) {
>>> - drm_err(connector->dev,
>>> - "[CONNECTOR:%d:%s] Failed to register EDID firmware platform
>>> device for connector \"%s\"\n",
>>> - connector->base.id, connector->name,
>>> - connector->name);
>>> - return ERR_CAST(pdev);
>>> - }
>>> -
>>> - err = request_firmware(&fw, name, &pdev->dev);
>>> - platform_device_unregister(pdev);
>>> + err = request_firmware(&fw, name, connector->dev->dev);
>>> if (err) {
>>> drm_err(connector->dev,
>>> "[CONNECTOR:%d:%s] Requesting EDID firmware \"%s\" failed
>>> (err=%d)\n",
>
> --
> Jani Nikula, Intel Open Source Graphics Center
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [RESEND] drm/edid/firmware: stop using a throwaway platform device
@ 2022-11-16 18:17 ` Matthieu CHARETTE
0 siblings, 0 replies; 15+ messages in thread
From: Matthieu CHARETTE @ 2022-11-16 18:17 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx, dri-devel, Thomas Zimmermann
Thank you everyone for your work!
Matthieu.
On Wed, Nov 16 2022 at 03:32:01 PM +0200, Jani Nikula
<jani.nikula@intel.com> wrote:
> On Wed, 16 Nov 2022, Thomas Zimmermann <tzimmermann@suse.de> wrote:
>> Hi
>>
>> Am 14.11.22 um 12:17 schrieb Jani Nikula:
>>> We've used a temporary platform device for firmware EDID loading
>>> since
>>> it was introduced in commit da0df92b5731 ("drm: allow loading an
>>> EDID as
>>> firmware to override broken monitor"), but there's no explanation
>>> why.
>>>
>>> Using a temporary device does not play well with CONFIG_FW_CACHE=y,
>>> which caches firmware images (e.g. on suspend) so that drivers can
>>> request firmware when the system is not ready for it, and return
>>> the
>>> images from the cache (e.g. during resume). This works
>>> automatically for
>>> regular devices, but obviously not for a temporarily created
>>> device.
>>>
>>> Stop using the throwaway platform device, and use the drm device
>>> instead.
>>>
>>> Note that this may still be problematic for cases where the
>>> display was
>>> plugged in during suspend, and the firmware wasn't loaded and
>>> therefore
>>> not cached before suspend.
>>>
>>> References:
>>> https://lore.kernel.org/r/20220727074152.43059-1-matthieu.charette@gmail.com
>>> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/2061
>>> Reported-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
>>> Tested-by: Matthieu CHARETTE <matthieu.charette@gmail.com>
>>> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
>>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>>
>> Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
>>
>> I looked through request_firmware() but did not see any signs that
>> it
>> somehow depends on a platform device. I assume that this might only
>> affect the device name in the error message.
>
> Thanks, pushed to drm-misc-next.
>
> Matthieu, thanks for you patience and the report as well!
>
> BR,
> Jani.
>
>
>>
>> Best regards
>> Thomas
>>
>>>
>>> ---
>>>
>>> Resend with a proper commit message; patch itself is unchanged.
>>> ---
>>> drivers/gpu/drm/drm_edid_load.c | 13 +------------
>>> 1 file changed, 1 insertion(+), 12 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/drm_edid_load.c
>>> b/drivers/gpu/drm/drm_edid_load.c
>>> index ef4ab59d6935..5d9ef267ebb3 100644
>>> --- a/drivers/gpu/drm/drm_edid_load.c
>>> +++ b/drivers/gpu/drm/drm_edid_load.c
>>> @@ -172,20 +172,9 @@ static const struct drm_edid
>>> *edid_load(struct drm_connector *connector, const c
>>> fwdata = generic_edid[builtin];
>>> fwsize = sizeof(generic_edid[builtin]);
>>> } else {
>>> - struct platform_device *pdev;
>>> int err;
>>>
>>> - pdev = platform_device_register_simple(connector->name, -1,
>>> NULL, 0);
>>> - if (IS_ERR(pdev)) {
>>> - drm_err(connector->dev,
>>> - "[CONNECTOR:%d:%s] Failed to register EDID firmware platform
>>> device for connector \"%s\"\n",
>>> - connector->base.id, connector->name,
>>> - connector->name);
>>> - return ERR_CAST(pdev);
>>> - }
>>> -
>>> - err = request_firmware(&fw, name, &pdev->dev);
>>> - platform_device_unregister(pdev);
>>> + err = request_firmware(&fw, name, connector->dev->dev);
>>> if (err) {
>>> drm_err(connector->dev,
>>> "[CONNECTOR:%d:%s] Requesting EDID firmware \"%s\" failed
>>> (err=%d)\n",
>
> --
> Jani Nikula, Intel Open Source Graphics Center
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2022-11-16 18:17 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-14 11:17 [RESEND] drm/edid/firmware: stop using a throwaway platform device Jani Nikula
2022-11-14 11:17 ` [Intel-gfx] " Jani Nikula
2022-11-14 14:22 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2022-11-14 14:44 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
2022-11-14 15:33 ` Jani Nikula
2022-11-14 18:49 ` Vudum, Lakshminarayana
2022-11-14 18:56 ` Patchwork
2022-11-14 19:10 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2022-11-14 22:03 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2022-11-16 9:33 ` [RESEND] " Thomas Zimmermann
2022-11-16 9:33 ` [Intel-gfx] " Thomas Zimmermann
2022-11-16 13:32 ` Jani Nikula
2022-11-16 13:32 ` [Intel-gfx] " Jani Nikula
2022-11-16 18:17 ` Matthieu CHARETTE
2022-11-16 18:17 ` [Intel-gfx] " Matthieu CHARETTE
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.