* [Intel-gfx] [PATCH] drm/i915/display: check if compressed_llb was allocated
@ 2021-07-01 9:03 Matthew Auld
2021-07-01 10:11 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Matthew Auld @ 2021-07-01 9:03 UTC (permalink / raw)
To: intel-gfx
If we hit the error path here we unconditionally call
i915_gem_stolen_remove_node, even though we only allocate the
compressed_llb on older platforms. Therefore we should first check that
we actually allocated the node before trying to remove it.
References: https://gitlab.freedesktop.org/drm/intel/-/issues/3709
Fixes: 46b2c40e0af3 ("drm/i915/fbc: Allocate llb before cfb")
Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: José Roberto de Souza <jose.souza@intel.com>
---
drivers/gpu/drm/i915/display/intel_fbc.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c b/drivers/gpu/drm/i915/display/intel_fbc.c
index 7dc72e4a4656..82effb64a3b9 100644
--- a/drivers/gpu/drm/i915/display/intel_fbc.c
+++ b/drivers/gpu/drm/i915/display/intel_fbc.c
@@ -516,7 +516,8 @@ static int intel_fbc_alloc_cfb(struct drm_i915_private *dev_priv,
return 0;
err_llb:
- i915_gem_stolen_remove_node(dev_priv, &fbc->compressed_llb);
+ if (drm_mm_node_allocated(&fbc->compressed_llb))
+ i915_gem_stolen_remove_node(dev_priv, &fbc->compressed_llb);
err:
if (drm_mm_initialized(&dev_priv->mm.stolen))
drm_info_once(&dev_priv->drm, "not enough stolen space for compressed buffer (need %d more bytes), disabling. Hint: you may be able to increase stolen memory size in the BIOS to avoid this.\n", size);
--
2.26.3
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: check if compressed_llb was allocated
2021-07-01 9:03 [Intel-gfx] [PATCH] drm/i915/display: check if compressed_llb was allocated Matthew Auld
@ 2021-07-01 10:11 ` Patchwork
2021-07-01 11:19 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2021-07-01 10:11 UTC (permalink / raw)
To: Matthew Auld; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 1980 bytes --]
== Series Details ==
Series: drm/i915/display: check if compressed_llb was allocated
URL : https://patchwork.freedesktop.org/series/92100/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10296 -> Patchwork_20503
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/index.html
Known issues
------------
Here are the changes found in Patchwork_20503 that come from known issues:
### IGT changes ###
#### Possible fixes ####
* igt@kms_flip@basic-flip-vs-wf_vblank@a-dsi1:
- {fi-tgl-dsi}: [DMESG-WARN][1] ([i915#1982]) -> [PASS][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/fi-tgl-dsi/igt@kms_flip@basic-flip-vs-wf_vblank@a-dsi1.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/fi-tgl-dsi/igt@kms_flip@basic-flip-vs-wf_vblank@a-dsi1.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303
Participating hosts (37 -> 33)
------------------------------
Missing (4): fi-kbl-soraka fi-bsw-cyan fi-bdw-samus fi-bsw-n3050
Build changes
-------------
* Linux: CI_DRM_10296 -> Patchwork_20503
CI-20190529: 20190529
CI_DRM_10296: 1ad092a7879a55938159b58a5b662370cab95ec8 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6125: 5bb62cb290fa32777934de4b39e107ccd282f7e8 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_20503: fc420969e0c01726cf0f4bbf2ac12d596f7ceb10 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
fc420969e0c0 drm/i915/display: check if compressed_llb was allocated
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/index.html
[-- Attachment #1.2: Type: text/html, Size: 2514 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/display: check if compressed_llb was allocated
2021-07-01 9:03 [Intel-gfx] [PATCH] drm/i915/display: check if compressed_llb was allocated Matthew Auld
2021-07-01 10:11 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
@ 2021-07-01 11:19 ` Patchwork
2021-07-01 17:14 ` [Intel-gfx] [PATCH] " Souza, Jose
2021-07-01 17:42 ` Ville Syrjälä
3 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2021-07-01 11:19 UTC (permalink / raw)
To: Matthew Auld; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 22147 bytes --]
== Series Details ==
Series: drm/i915/display: check if compressed_llb was allocated
URL : https://patchwork.freedesktop.org/series/92100/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_10296_full -> Patchwork_20503_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_20503_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_20503_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_20503_full:
### IGT changes ###
#### Possible regressions ####
* igt@i915_pm_rps@basic-api:
- shard-tglb: [PASS][1] -> [FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-tglb5/igt@i915_pm_rps@basic-api.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-tglb1/igt@i915_pm_rps@basic-api.html
Known issues
------------
Here are the changes found in Patchwork_20503_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_ctx_persistence@legacy-engines-mixed:
- shard-snb: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#1099]) +1 similar issue
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-snb7/igt@gem_ctx_persistence@legacy-engines-mixed.html
* igt@gem_ctx_persistence@many-contexts:
- shard-tglb: [PASS][4] -> [FAIL][5] ([i915#2410])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-tglb2/igt@gem_ctx_persistence@many-contexts.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-tglb5/igt@gem_ctx_persistence@many-contexts.html
* igt@gem_eio@in-flight-contexts-immediate:
- shard-tglb: [PASS][6] -> [TIMEOUT][7] ([i915#3063])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-tglb6/igt@gem_eio@in-flight-contexts-immediate.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-tglb3/igt@gem_eio@in-flight-contexts-immediate.html
* igt@gem_exec_fair@basic-none-solo@rcs0:
- shard-kbl: [PASS][8] -> [FAIL][9] ([i915#2842]) +1 similar issue
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-kbl2/igt@gem_exec_fair@basic-none-solo@rcs0.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl6/igt@gem_exec_fair@basic-none-solo@rcs0.html
* igt@gem_exec_fair@basic-pace@vcs1:
- shard-tglb: [PASS][10] -> [FAIL][11] ([i915#2842]) +1 similar issue
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-tglb5/igt@gem_exec_fair@basic-pace@vcs1.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-tglb2/igt@gem_exec_fair@basic-pace@vcs1.html
* igt@gem_mmap_gtt@cpuset-medium-copy:
- shard-iclb: [PASS][12] -> [FAIL][13] ([i915#2428])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-iclb7/igt@gem_mmap_gtt@cpuset-medium-copy.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-iclb3/igt@gem_mmap_gtt@cpuset-medium-copy.html
* igt@gem_pread@exhaustion:
- shard-skl: NOTRUN -> [WARN][14] ([i915#2658])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl3/igt@gem_pread@exhaustion.html
* igt@gem_vm_create@destroy-race:
- shard-tglb: [PASS][15] -> [FAIL][16] ([i915#2822])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-tglb7/igt@gem_vm_create@destroy-race.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-tglb6/igt@gem_vm_create@destroy-race.html
* igt@gem_workarounds@suspend-resume-context:
- shard-apl: [PASS][17] -> [DMESG-WARN][18] ([i915#180]) +4 similar issues
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-apl6/igt@gem_workarounds@suspend-resume-context.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl2/igt@gem_workarounds@suspend-resume-context.html
* igt@kms_ccs@pipe-d-crc-primary-rotation-180-yf_tiled_ccs:
- shard-apl: NOTRUN -> [SKIP][19] ([fdo#109271]) +136 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl7/igt@kms_ccs@pipe-d-crc-primary-rotation-180-yf_tiled_ccs.html
* igt@kms_chamelium@hdmi-hpd-with-enabled-mode:
- shard-kbl: NOTRUN -> [SKIP][20] ([fdo#109271] / [fdo#111827])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl3/igt@kms_chamelium@hdmi-hpd-with-enabled-mode.html
* igt@kms_chamelium@vga-edid-read:
- shard-apl: NOTRUN -> [SKIP][21] ([fdo#109271] / [fdo#111827]) +8 similar issues
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl7/igt@kms_chamelium@vga-edid-read.html
* igt@kms_color_chamelium@pipe-a-ctm-blue-to-red:
- shard-snb: NOTRUN -> [SKIP][22] ([fdo#109271] / [fdo#111827]) +6 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-snb7/igt@kms_color_chamelium@pipe-a-ctm-blue-to-red.html
* igt@kms_content_protection@atomic:
- shard-kbl: NOTRUN -> [TIMEOUT][23] ([i915#1319])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl3/igt@kms_content_protection@atomic.html
* igt@kms_content_protection@legacy:
- shard-apl: NOTRUN -> [TIMEOUT][24] ([i915#1319])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl3/igt@kms_content_protection@legacy.html
* igt@kms_content_protection@uevent:
- shard-apl: NOTRUN -> [FAIL][25] ([i915#2105])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl7/igt@kms_content_protection@uevent.html
* igt@kms_cursor_crc@pipe-b-cursor-128x128-random:
- shard-skl: [PASS][26] -> [FAIL][27] ([i915#3444])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl5/igt@kms_cursor_crc@pipe-b-cursor-128x128-random.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl9/igt@kms_cursor_crc@pipe-b-cursor-128x128-random.html
* igt@kms_cursor_crc@pipe-b-cursor-512x512-offscreen:
- shard-skl: NOTRUN -> [SKIP][28] ([fdo#109271]) +27 similar issues
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl3/igt@kms_cursor_crc@pipe-b-cursor-512x512-offscreen.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions:
- shard-skl: [PASS][29] -> [FAIL][30] ([i915#2346])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl6/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl2/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size:
- shard-skl: [PASS][31] -> [FAIL][32] ([i915#2346] / [i915#533])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl10/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl4/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
* igt@kms_flip@plain-flip-fb-recreate@b-edp1:
- shard-skl: [PASS][33] -> [FAIL][34] ([i915#2122])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl10/igt@kms_flip@plain-flip-fb-recreate@b-edp1.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl4/igt@kms_flip@plain-flip-fb-recreate@b-edp1.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-pwrite:
- shard-snb: NOTRUN -> [SKIP][35] ([fdo#109271]) +222 similar issues
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-snb6/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-pwrite.html
* igt@kms_frontbuffer_tracking@psr-rgb101010-draw-mmap-wc:
- shard-kbl: NOTRUN -> [SKIP][36] ([fdo#109271]) +24 similar issues
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl3/igt@kms_frontbuffer_tracking@psr-rgb101010-draw-mmap-wc.html
* igt@kms_pipe_crc_basic@hang-read-crc-pipe-d:
- shard-skl: NOTRUN -> [SKIP][37] ([fdo#109271] / [i915#533])
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl3/igt@kms_pipe_crc_basic@hang-read-crc-pipe-d.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb:
- shard-kbl: NOTRUN -> [FAIL][38] ([fdo#108145] / [i915#265])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl3/igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb.html
* igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
- shard-skl: [PASS][39] -> [FAIL][40] ([fdo#108145] / [i915#265])
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl9/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl1/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
* igt@kms_plane_alpha_blend@pipe-b-alpha-7efc:
- shard-apl: NOTRUN -> [FAIL][41] ([fdo#108145] / [i915#265]) +1 similar issue
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl3/igt@kms_plane_alpha_blend@pipe-b-alpha-7efc.html
* igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-4:
- shard-apl: NOTRUN -> [SKIP][42] ([fdo#109271] / [i915#658]) +3 similar issues
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl7/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-4.html
* igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5:
- shard-skl: NOTRUN -> [SKIP][43] ([fdo#109271] / [i915#658])
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl10/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html
* igt@kms_psr@psr2_dpms:
- shard-iclb: [PASS][44] -> [SKIP][45] ([fdo#109441])
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-iclb2/igt@kms_psr@psr2_dpms.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-iclb8/igt@kms_psr@psr2_dpms.html
* igt@sysfs_clients@create:
- shard-kbl: NOTRUN -> [SKIP][46] ([fdo#109271] / [i915#2994])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl3/igt@sysfs_clients@create.html
* igt@sysfs_clients@sema-50:
- shard-apl: NOTRUN -> [SKIP][47] ([fdo#109271] / [i915#2994]) +1 similar issue
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl7/igt@sysfs_clients@sema-50.html
#### Possible fixes ####
* igt@gem_eio@unwedge-stress:
- shard-tglb: [TIMEOUT][48] ([i915#2369] / [i915#3063] / [i915#3648]) -> [PASS][49]
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-tglb6/igt@gem_eio@unwedge-stress.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-tglb3/igt@gem_eio@unwedge-stress.html
* igt@gem_exec_fair@basic-flow@rcs0:
- shard-tglb: [FAIL][50] ([i915#2842]) -> [PASS][51] +1 similar issue
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-tglb3/igt@gem_exec_fair@basic-flow@rcs0.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-tglb7/igt@gem_exec_fair@basic-flow@rcs0.html
* igt@gem_exec_fair@basic-none-rrul@rcs0:
- shard-glk: [FAIL][52] ([i915#2842]) -> [PASS][53]
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-glk4/igt@gem_exec_fair@basic-none-rrul@rcs0.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-glk5/igt@gem_exec_fair@basic-none-rrul@rcs0.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-iclb: [FAIL][54] ([i915#2842]) -> [PASS][55] +2 similar issues
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-iclb7/igt@gem_exec_fair@basic-none-share@rcs0.html
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-iclb3/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-pace@rcs0:
- shard-kbl: [FAIL][56] ([i915#2842]) -> [PASS][57] +1 similar issue
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-kbl3/igt@gem_exec_fair@basic-pace@rcs0.html
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl1/igt@gem_exec_fair@basic-pace@rcs0.html
* igt@gem_exec_fair@basic-pace@vecs0:
- shard-kbl: [SKIP][58] ([fdo#109271]) -> [PASS][59]
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-kbl3/igt@gem_exec_fair@basic-pace@vecs0.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl1/igt@gem_exec_fair@basic-pace@vecs0.html
* igt@gem_mmap_gtt@cpuset-big-copy-odd:
- shard-iclb: [FAIL][60] ([i915#307]) -> [PASS][61]
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-iclb6/igt@gem_mmap_gtt@cpuset-big-copy-odd.html
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-iclb1/igt@gem_mmap_gtt@cpuset-big-copy-odd.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1:
- shard-skl: [FAIL][62] ([i915#79]) -> [PASS][63]
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl7/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl10/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1.html
* igt@kms_flip@flip-vs-suspend-interruptible@a-dp1:
- shard-kbl: [DMESG-WARN][64] ([i915#180]) -> [PASS][65]
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-kbl7/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl3/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html
* igt@kms_flip@flip-vs-suspend@b-edp1:
- shard-skl: [INCOMPLETE][66] ([i915#146] / [i915#198]) -> [PASS][67]
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl8/igt@kms_flip@flip-vs-suspend@b-edp1.html
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl3/igt@kms_flip@flip-vs-suspend@b-edp1.html
* igt@kms_flip@plain-flip-ts-check-interruptible@a-edp1:
- shard-skl: [FAIL][68] ([i915#2122]) -> [PASS][69]
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl3/igt@kms_flip@plain-flip-ts-check-interruptible@a-edp1.html
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl6/igt@kms_flip@plain-flip-ts-check-interruptible@a-edp1.html
* igt@kms_hdr@bpc-switch:
- shard-skl: [FAIL][70] ([i915#1188]) -> [PASS][71]
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl8/igt@kms_hdr@bpc-switch.html
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl8/igt@kms_hdr@bpc-switch.html
* igt@kms_psr@psr2_sprite_plane_move:
- shard-iclb: [SKIP][72] ([fdo#109441]) -> [PASS][73] +5 similar issues
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-iclb4/igt@kms_psr@psr2_sprite_plane_move.html
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html
* igt@perf@blocking:
- shard-skl: [FAIL][74] ([i915#1542]) -> [PASS][75]
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl3/igt@perf@blocking.html
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl6/igt@perf@blocking.html
#### Warnings ####
* igt@i915_pm_rc6_residency@rc6-fence:
- shard-iclb: [WARN][76] ([i915#2684]) -> [WARN][77] ([i915#1804] / [i915#2684])
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-iclb1/igt@i915_pm_rc6_residency@rc6-fence.html
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-iclb4/igt@i915_pm_rc6_residency@rc6-fence.html
* igt@kms_hdr@bpc-switch-suspend:
- shard-skl: [INCOMPLETE][78] ([i915#198] / [i915#2828]) -> [FAIL][79] ([i915#1188])
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-skl10/igt@kms_hdr@bpc-switch-suspend.html
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-skl4/igt@kms_hdr@bpc-switch-suspend.html
* igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-2:
- shard-iclb: [SKIP][80] ([i915#2920]) -> [SKIP][81] ([i915#658]) +1 similar issue
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-iclb2/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-2.html
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-iclb6/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-2.html
* igt@runner@aborted:
- shard-kbl: ([FAIL][82], [FAIL][83], [FAIL][84]) ([i915#180] / [i915#3002] / [i915#3363]) -> ([FAIL][85], [FAIL][86]) ([i915#3002] / [i915#3363])
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-kbl3/igt@runner@aborted.html
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-kbl7/igt@runner@aborted.html
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-kbl1/igt@runner@aborted.html
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl1/igt@runner@aborted.html
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-kbl4/igt@runner@aborted.html
- shard-apl: ([FAIL][87], [FAIL][88], [FAIL][89]) ([fdo#109271] / [i915#1814] / [i915#3002] / [i915#3363]) -> ([FAIL][90], [FAIL][91], [FAIL][92], [FAIL][93], [FAIL][94], [FAIL][95]) ([fdo#109271] / [i915#1610] / [i915#180] / [i915#2292] / [i915#3002] / [i915#3363])
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-apl8/igt@runner@aborted.html
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-apl7/igt@runner@aborted.html
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10296/shard-apl1/igt@runner@aborted.html
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl1/igt@runner@aborted.html
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl3/igt@runner@aborted.html
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl2/igt@runner@aborted.html
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl8/igt@runner@aborted.html
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl6/igt@runner@aborted.html
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20503/shard-apl8/igt@runner@aborted.html
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#1099]: https://gitlab.freedesktop.org/drm/intel/issues/1099
[i915#1188]: https://gitlab.freedesktop.org/drm/intel/issues/1188
[i915#1319]: https://gitlab.freedesktop.org/drm/intel/issues/1319
[i915#146]: https://gitlab.freedesktop.org/drm/intel/issues/146
[i915#1542]: https://gitlab.freedesktop.org/drm/intel/issues/1542
[i915#1610]: https://gitlab.freedesktop.org/drm/intel/issues/1610
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#1804]: https://gitlab.freedesktop.org/drm/intel/issues/1804
[i915#1814]: https://gitlab.freedesktop.org/drm/intel/issues/1814
[i915#198]: https://gitlab.freedesktop.org/drm/intel/issues/198
[i915#2105]: https://gitlab.freedesktop.org/drm/intel/issues/2105
[i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
[i915#2292]: https://gitlab.freedesktop.org/drm/intel/issues/2292
[i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
[i915#2369]: https://gitlab.freedesktop.org/drm/intel/issues/2369
[i915#2410]: https://gitlab.freedesktop.org/drm/intel/issues/2410
[i915#2428]: https://gitlab.freedesktop.org/drm/intel/issues/2428
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#2658]: https://gitlab.freedesktop.org/drm/intel/issues/2658
[i915#2684]: https://gitlab.freedesktop.org/drm/intel/issues/2684
[i915#2822]: https://gitlab.freedesktop.org/drm/intel/issues/2822
[i915#2828]: https://gitlab.freedesktop.org/drm/intel/issues/2828
[i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
[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#3063]: https://gitlab.freedesktop.org/drm/intel/issues/3063
[i915#307]: https://gitlab.freedesktop.org/drm/intel/issues/307
[i915#3363]: https://gitlab.freedesktop.org/drm/intel/issues/3363
[i915#3444]: https://gitlab.freedesktop.org/drm/intel/issues/3444
[i915#3648]: https://gitlab.freedesktop.org/drm/intel/issues/3648
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
[i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
Participating hosts (10 -> 10)
------------------------------
No changes in participating hosts
Build changes
-------------
* Linux: CI_DRM_10296 -> Patchwork_20503
CI-20190529: 20190529
CI_DRM_10296: 1ad092a7879a55938159b58a5b662370cab95ec8 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6125: 5bb62cb290fa32777934de4b39e107ccd282f7e8 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_20503: fc420969e0c01726cf0f4bbf2ac12d596f7ceb10 @ 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_20503/index.html
[-- Attachment #1.2: Type: text/html, Size: 27494 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: check if compressed_llb was allocated
2021-07-01 9:03 [Intel-gfx] [PATCH] drm/i915/display: check if compressed_llb was allocated Matthew Auld
2021-07-01 10:11 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
2021-07-01 11:19 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
@ 2021-07-01 17:14 ` Souza, Jose
2021-07-01 17:42 ` Ville Syrjälä
3 siblings, 0 replies; 5+ messages in thread
From: Souza, Jose @ 2021-07-01 17:14 UTC (permalink / raw)
To: intel-gfx, Auld, Matthew
On Thu, 2021-07-01 at 10:03 +0100, Matthew Auld wrote:
> If we hit the error path here we unconditionally call
> i915_gem_stolen_remove_node, even though we only allocate the
> compressed_llb on older platforms. Therefore we should first check that
> we actually allocated the node before trying to remove it.
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
> References: https://gitlab.freedesktop.org/drm/intel/-/issues/3709
> Fixes: 46b2c40e0af3 ("drm/i915/fbc: Allocate llb before cfb")
> Signed-off-by: Matthew Auld <matthew.auld@intel.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: José Roberto de Souza <jose.souza@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_fbc.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c b/drivers/gpu/drm/i915/display/intel_fbc.c
> index 7dc72e4a4656..82effb64a3b9 100644
> --- a/drivers/gpu/drm/i915/display/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/display/intel_fbc.c
> @@ -516,7 +516,8 @@ static int intel_fbc_alloc_cfb(struct drm_i915_private *dev_priv,
> return 0;
>
> err_llb:
> - i915_gem_stolen_remove_node(dev_priv, &fbc->compressed_llb);
> + if (drm_mm_node_allocated(&fbc->compressed_llb))
> + i915_gem_stolen_remove_node(dev_priv, &fbc->compressed_llb);
> err:
> if (drm_mm_initialized(&dev_priv->mm.stolen))
> drm_info_once(&dev_priv->drm, "not enough stolen space for compressed buffer (need %d more bytes), disabling. Hint: you may be able to increase stolen memory size in the BIOS to avoid this.\n", size);
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: check if compressed_llb was allocated
2021-07-01 9:03 [Intel-gfx] [PATCH] drm/i915/display: check if compressed_llb was allocated Matthew Auld
` (2 preceding siblings ...)
2021-07-01 17:14 ` [Intel-gfx] [PATCH] " Souza, Jose
@ 2021-07-01 17:42 ` Ville Syrjälä
3 siblings, 0 replies; 5+ messages in thread
From: Ville Syrjälä @ 2021-07-01 17:42 UTC (permalink / raw)
To: Matthew Auld; +Cc: intel-gfx
On Thu, Jul 01, 2021 at 10:03:26AM +0100, Matthew Auld wrote:
> If we hit the error path here we unconditionally call
> i915_gem_stolen_remove_node, even though we only allocate the
> compressed_llb on older platforms. Therefore we should first check that
> we actually allocated the node before trying to remove it.
>
> References: https://gitlab.freedesktop.org/drm/intel/-/issues/3709
Doh! Seems pre-merge testing didn't hit this, or I just suck at reading
the logs :(
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Fixes: 46b2c40e0af3 ("drm/i915/fbc: Allocate llb before cfb")
> Signed-off-by: Matthew Auld <matthew.auld@intel.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: José Roberto de Souza <jose.souza@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_fbc.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c b/drivers/gpu/drm/i915/display/intel_fbc.c
> index 7dc72e4a4656..82effb64a3b9 100644
> --- a/drivers/gpu/drm/i915/display/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/display/intel_fbc.c
> @@ -516,7 +516,8 @@ static int intel_fbc_alloc_cfb(struct drm_i915_private *dev_priv,
> return 0;
>
> err_llb:
> - i915_gem_stolen_remove_node(dev_priv, &fbc->compressed_llb);
> + if (drm_mm_node_allocated(&fbc->compressed_llb))
> + i915_gem_stolen_remove_node(dev_priv, &fbc->compressed_llb);
> err:
> if (drm_mm_initialized(&dev_priv->mm.stolen))
> drm_info_once(&dev_priv->drm, "not enough stolen space for compressed buffer (need %d more bytes), disabling. Hint: you may be able to increase stolen memory size in the BIOS to avoid this.\n", size);
> --
> 2.26.3
--
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-07-01 17:42 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-01 9:03 [Intel-gfx] [PATCH] drm/i915/display: check if compressed_llb was allocated Matthew Auld
2021-07-01 10:11 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
2021-07-01 11:19 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2021-07-01 17:14 ` [Intel-gfx] [PATCH] " Souza, Jose
2021-07-01 17:42 ` Ville Syrjälä
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.