* [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1
@ 2023-01-04 12:44 Juha-Pekka Heikkila
2023-01-04 13:23 ` Jani Nikula
` (8 more replies)
0 siblings, 9 replies; 16+ messages in thread
From: Juha-Pekka Heikkila @ 2023-01-04 12:44 UTC (permalink / raw)
To: intel-gfx; +Cc: Drew Davenport
intel_adjusted_rate() didn't take into account src rectangle
can be less than 1 in width or height.
Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
---
drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
index 10e1fc9d0698..a9948e8d3543 100644
--- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
@@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src,
const struct drm_rect *dst,
unsigned int rate)
{
- unsigned int src_w, src_h, dst_w, dst_h;
+ unsigned int src_w, src_h, dst_w, dst_h, dst_wh;
src_w = drm_rect_width(src) >> 16;
src_h = drm_rect_height(src) >> 16;
@@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src,
dst_w = min(src_w, dst_w);
dst_h = min(src_h, dst_h);
- return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h),
- dst_w * dst_h);
+ /* in case src contained only fractional part */
+ dst_wh = max(dst_w * dst_h, (unsigned) 1);
+
+ return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh);
}
unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state,
--
2.37.3
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila @ 2023-01-04 13:23 ` Jani Nikula 2023-01-04 19:12 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork ` (7 subsequent siblings) 8 siblings, 0 replies; 16+ messages in thread From: Jani Nikula @ 2023-01-04 13:23 UTC (permalink / raw) To: Juha-Pekka Heikkila, intel-gfx; +Cc: Drew Davenport On Wed, 04 Jan 2023, Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> wrote: > intel_adjusted_rate() didn't take into account src rectangle > can be less than 1 in width or height. > > Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> > --- > drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > index 10e1fc9d0698..a9948e8d3543 100644 > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > @@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > const struct drm_rect *dst, > unsigned int rate) > { > - unsigned int src_w, src_h, dst_w, dst_h; > + unsigned int src_w, src_h, dst_w, dst_h, dst_wh; > > src_w = drm_rect_width(src) >> 16; > src_h = drm_rect_height(src) >> 16; > @@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > dst_w = min(src_w, dst_w); > dst_h = min(src_h, dst_h); > > - return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), > - dst_w * dst_h); > + /* in case src contained only fractional part */ > + dst_wh = max(dst_w * dst_h, (unsigned) 1); The options are to use 1U or max_t(), but please don't cast the parameters of max(). Side note, the explicit "unsigned int" is always preferred over the implicit "unsigned". BR, Jani. > + > + return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh); > } > > unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state, -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila 2023-01-04 13:23 ` Jani Nikula @ 2023-01-04 19:12 ` Patchwork 2023-01-04 19:39 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork ` (6 subsequent siblings) 8 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2023-01-04 19:12 UTC (permalink / raw) To: Juha-Pekka Heikkila; +Cc: intel-gfx == Series Details == Series: drm/i915/display: assume some pixelrate for src smaller than 1 URL : https://patchwork.freedesktop.org/series/112396/ State : warning == Summary == Error: dim checkpatch failed 2c7cc185eea1 drm/i915/display: assume some pixelrate for src smaller than 1 -:32: CHECK:SPACING: No space is necessary after a cast #32: FILE: drivers/gpu/drm/i915/display/intel_atomic_plane.c:159: + dst_wh = max(dst_w * dst_h, (unsigned) 1); -:32: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned' #32: FILE: drivers/gpu/drm/i915/display/intel_atomic_plane.c:159: + dst_wh = max(dst_w * dst_h, (unsigned) 1); total: 0 errors, 1 warnings, 1 checks, 20 lines checked ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila 2023-01-04 13:23 ` Jani Nikula 2023-01-04 19:12 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork @ 2023-01-04 19:39 ` Patchwork 2023-01-05 1:54 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork ` (5 subsequent siblings) 8 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2023-01-04 19:39 UTC (permalink / raw) To: Juha-Pekka Heikkila; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 4890 bytes --] == Series Details == Series: drm/i915/display: assume some pixelrate for src smaller than 1 URL : https://patchwork.freedesktop.org/series/112396/ State : success == Summary == CI Bug Log - changes from CI_DRM_12546 -> Patchwork_112396v1 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/index.html Participating hosts (43 -> 42) ------------------------------ Additional (1): fi-kbl-soraka Missing (2): fi-bsw-kefka fi-snb-2520m Known issues ------------ Here are the changes found in Patchwork_112396v1 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_gttfill@basic: - fi-kbl-soraka: NOTRUN -> [SKIP][1] ([fdo#109271]) +7 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/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_112396v1/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html * igt@gem_lmem_swapping@basic: - fi-kbl-soraka: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#4613]) +3 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html * igt@i915_selftest@live@gt_pm: - fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][4] ([i915#1886]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html * igt@i915_selftest@live@guc_hang: - fi-kbl-soraka: NOTRUN -> [INCOMPLETE][5] ([i915#7640]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/fi-kbl-soraka/igt@i915_selftest@live@guc_hang.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-kbl-soraka: NOTRUN -> [SKIP][6] ([fdo#109271] / [fdo#111827]) +7 similar issues [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/fi-kbl-soraka/igt@kms_chamelium@hdmi-hpd-fast.html * igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions: - fi-bsw-n3050: [PASS][7] -> [FAIL][8] ([i915#6298]) +1 similar issue [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/fi-bsw-n3050/igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/fi-bsw-n3050/igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions.html #### Possible fixes #### * igt@gem_exec_suspend@basic-s0@smem: - {bat-adlp-6}: [DMESG-WARN][9] ([i915#2867]) -> [PASS][10] +2 similar issues [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/bat-adlp-6/igt@gem_exec_suspend@basic-s0@smem.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/bat-adlp-6/igt@gem_exec_suspend@basic-s0@smem.html * igt@i915_selftest@live@reset: - {bat-rpls-1}: [DMESG-FAIL][11] ([i915#4983]) -> [PASS][12] [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/bat-rpls-1/igt@i915_selftest@live@reset.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/bat-rpls-1/igt@i915_selftest@live@reset.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#2867]: https://gitlab.freedesktop.org/drm/intel/issues/2867 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983 [i915#6298]: https://gitlab.freedesktop.org/drm/intel/issues/6298 [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367 [i915#7336]: https://gitlab.freedesktop.org/drm/intel/issues/7336 [i915#7359]: https://gitlab.freedesktop.org/drm/intel/issues/7359 [i915#7640]: https://gitlab.freedesktop.org/drm/intel/issues/7640 Build changes ------------- * Linux: CI_DRM_12546 -> Patchwork_112396v1 CI-20190529: 20190529 CI_DRM_12546: 07a684fbd4d0f5e284e8a782e0298f772fc4164e @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7107: 4f22b49ee353406c14ce8bb3151ebe3ce4e6e9be @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_112396v1: 07a684fbd4d0f5e284e8a782e0298f772fc4164e @ git://anongit.freedesktop.org/gfx-ci/linux ### Linux commits afe40f0c6249 drm/i915/display: assume some pixelrate for src smaller than 1 == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/index.html [-- Attachment #2: Type: text/html, Size: 5722 bytes --] ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila ` (2 preceding siblings ...) 2023-01-04 19:39 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork @ 2023-01-05 1:54 ` Patchwork 2023-01-08 11:30 ` [Intel-gfx] [PATCH v2] " Juha-Pekka Heikkila ` (4 subsequent siblings) 8 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2023-01-05 1:54 UTC (permalink / raw) To: Juha-Pekka Heikkila; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 20801 bytes --] == Series Details == Series: drm/i915/display: assume some pixelrate for src smaller than 1 URL : https://patchwork.freedesktop.org/series/112396/ State : success == Summary == CI Bug Log - changes from CI_DRM_12546_full -> Patchwork_112396v1_full ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/index.html Participating hosts (13 -> 9) ------------------------------ Missing (4): pig-skl-6260u pig-kbl-iris shard-tglu-9 pig-glk-j5005 Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_112396v1_full: ### IGT changes ### #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@kms_atomic_transition@plane-all-modeset-transition: - {shard-rkl}: [SKIP][1] ([i915#1845] / [i915#4098]) -> [SKIP][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-3/igt@kms_atomic_transition@plane-all-modeset-transition.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-6/igt@kms_atomic_transition@plane-all-modeset-transition.html Known issues ------------ Here are the changes found in Patchwork_112396v1_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-glk: [PASS][3] -> [FAIL][4] ([i915#2842]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-glk7/igt@gem_exec_fair@basic-pace-solo@rcs0.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-glk8/igt@gem_exec_fair@basic-pace-solo@rcs0.html * igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions: - shard-glk: [PASS][5] -> [FAIL][6] ([i915#2346]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-glk6/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-glk7/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions.html * igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-1: - shard-glk: NOTRUN -> [SKIP][7] ([fdo#109271]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-glk6/igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-1.html #### Possible fixes #### * igt@drm_fdinfo@virtual-idle: - {shard-rkl}: [FAIL][8] ([i915#7742]) -> [PASS][9] [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-1/igt@drm_fdinfo@virtual-idle.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-5/igt@drm_fdinfo@virtual-idle.html * igt@gem_eio@suspend: - {shard-rkl}: [FAIL][10] ([i915#7052]) -> [PASS][11] [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-4/igt@gem_eio@suspend.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-2/igt@gem_eio@suspend.html * igt@gem_exec_fair@basic-none-rrul@rcs0: - {shard-rkl}: [FAIL][12] ([i915#2842]) -> [PASS][13] +1 similar issue [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-1/igt@gem_exec_fair@basic-none-rrul@rcs0.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-5/igt@gem_exec_fair@basic-none-rrul@rcs0.html * igt@gem_exec_fair@basic-none@rcs0: - shard-glk: [FAIL][14] ([i915#2842]) -> [PASS][15] +1 similar issue [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-glk2/igt@gem_exec_fair@basic-none@rcs0.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-glk5/igt@gem_exec_fair@basic-none@rcs0.html * igt@gem_exec_reloc@basic-gtt-wc-noreloc: - {shard-rkl}: [SKIP][16] ([i915#3281]) -> [PASS][17] +8 similar issues [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-1/igt@gem_exec_reloc@basic-gtt-wc-noreloc.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-5/igt@gem_exec_reloc@basic-gtt-wc-noreloc.html * igt@gem_mmap_gtt@coherency: - {shard-rkl}: [SKIP][18] ([fdo#111656]) -> [PASS][19] [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-1/igt@gem_mmap_gtt@coherency.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-5/igt@gem_mmap_gtt@coherency.html * igt@gem_partial_pwrite_pread@reads-display: - {shard-rkl}: [SKIP][20] ([i915#3282]) -> [PASS][21] +2 similar issues [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-1/igt@gem_partial_pwrite_pread@reads-display.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-5/igt@gem_partial_pwrite_pread@reads-display.html * igt@gen9_exec_parse@unaligned-access: - {shard-rkl}: [SKIP][22] ([i915#2527]) -> [PASS][23] +2 similar issues [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-1/igt@gen9_exec_parse@unaligned-access.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-5/igt@gen9_exec_parse@unaligned-access.html * igt@i915_pm_rc6_residency@rc6-idle@rcs0: - {shard-dg1}: [FAIL][24] ([i915#3591]) -> [PASS][25] [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-dg1-16/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-dg1-19/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html * igt@i915_pm_rpm@drm-resources-equal: - {shard-rkl}: [SKIP][26] ([fdo#109308]) -> [PASS][27] [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-2/igt@i915_pm_rpm@drm-resources-equal.html [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-6/igt@i915_pm_rpm@drm-resources-equal.html * igt@i915_pm_rpm@fences: - {shard-rkl}: [SKIP][28] ([i915#1849]) -> [PASS][29] [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-3/igt@i915_pm_rpm@fences.html [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-6/igt@i915_pm_rpm@fences.html * igt@i915_selftest@live@gt_heartbeat: - shard-glk: [DMESG-FAIL][30] ([i915#5334]) -> [PASS][31] [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-glk5/igt@i915_selftest@live@gt_heartbeat.html [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-glk1/igt@i915_selftest@live@gt_heartbeat.html * igt@i915_selftest@live@gt_pm: - {shard-rkl}: [DMESG-FAIL][32] ([i915#4258]) -> [PASS][33] [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-6/igt@i915_selftest@live@gt_pm.html [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-3/igt@i915_selftest@live@gt_pm.html * igt@kms_atomic@atomic_plane_damage: - {shard-rkl}: [SKIP][34] ([i915#4098]) -> [PASS][35] +1 similar issue [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-2/igt@kms_atomic@atomic_plane_damage.html [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-6/igt@kms_atomic@atomic_plane_damage.html * igt@kms_big_fb@x-tiled-32bpp-rotate-0: - {shard-rkl}: [SKIP][36] ([i915#1845] / [i915#4098]) -> [PASS][37] +22 similar issues [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-5/igt@kms_big_fb@x-tiled-32bpp-rotate-0.html [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-6/igt@kms_big_fb@x-tiled-32bpp-rotate-0.html * igt@kms_big_fb@y-tiled-64bpp-rotate-180: - {shard-tglu}: [SKIP][38] ([i915#1845] / [i915#7651]) -> [PASS][39] +2 similar issues [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-tglu-6/igt@kms_big_fb@y-tiled-64bpp-rotate-180.html [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-tglu-8/igt@kms_big_fb@y-tiled-64bpp-rotate-180.html * igt@kms_ccs@pipe-d-crc-primary-basic-y_tiled_gen12_rc_ccs_cc: - {shard-tglu}: [SKIP][40] ([i915#7651]) -> [PASS][41] +10 similar issues [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-tglu-6/igt@kms_ccs@pipe-d-crc-primary-basic-y_tiled_gen12_rc_ccs_cc.html [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-tglu-8/igt@kms_ccs@pipe-d-crc-primary-basic-y_tiled_gen12_rc_ccs_cc.html * igt@kms_frontbuffer_tracking@fbc-tiling-linear: - {shard-rkl}: [SKIP][42] ([i915#1849] / [i915#4098]) -> [PASS][43] +17 similar issues [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-5/igt@kms_frontbuffer_tracking@fbc-tiling-linear.html [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-6/igt@kms_frontbuffer_tracking@fbc-tiling-linear.html * igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a-planes: - {shard-tglu}: [SKIP][44] ([i915#1849]) -> [PASS][45] +3 similar issues [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-tglu-6/igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a-planes.html [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-tglu-8/igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a-planes.html * igt@kms_properties@plane-properties-atomic: - {shard-tglu}: [SKIP][46] -> [PASS][47] [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-tglu-6/igt@kms_properties@plane-properties-atomic.html [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-tglu-8/igt@kms_properties@plane-properties-atomic.html * igt@kms_psr@primary_mmap_gtt: - {shard-rkl}: [SKIP][48] ([i915#1072]) -> [PASS][49] +3 similar issues [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-5/igt@kms_psr@primary_mmap_gtt.html [49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-6/igt@kms_psr@primary_mmap_gtt.html * igt@kms_psr_stress_test@invalidate-primary-flip-overlay: - {shard-rkl}: [SKIP][50] ([i915#5461]) -> [PASS][51] [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-2/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html [51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-6/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html * igt@kms_universal_plane@cursor-fb-leak-pipe-b: - {shard-tglu}: [SKIP][52] ([fdo#109274]) -> [PASS][53] [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-tglu-6/igt@kms_universal_plane@cursor-fb-leak-pipe-b.html [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-tglu-8/igt@kms_universal_plane@cursor-fb-leak-pipe-b.html * igt@kms_universal_plane@universal-plane-pipe-a-functional: - {shard-rkl}: [SKIP][54] ([i915#1845] / [i915#4070] / [i915#4098]) -> [PASS][55] [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12546/shard-rkl-2/igt@kms_universal_plane@universal-plane-pipe-a-functional.html [55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v1/shard-rkl-6/igt@kms_universal_plane@universal-plane-pipe-a-functional.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [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#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#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289 [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295 [fdo#109303]: https://bugs.freedesktop.org/show_bug.cgi?id=109303 [fdo#109308]: https://bugs.freedesktop.org/show_bug.cgi?id=109308 [fdo#109312]: https://bugs.freedesktop.org/show_bug.cgi?id=109312 [fdo#109313]: https://bugs.freedesktop.org/show_bug.cgi?id=109313 [fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315 [fdo#109506]: https://bugs.freedesktop.org/show_bug.cgi?id=109506 [fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642 [fdo#110189]: https://bugs.freedesktop.org/show_bug.cgi?id=110189 [fdo#110542]: https://bugs.freedesktop.org/show_bug.cgi?id=110542 [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#111644]: https://bugs.freedesktop.org/show_bug.cgi?id=111644 [fdo#111656]: https://bugs.freedesktop.org/show_bug.cgi?id=111656 [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 [fdo#112283]: https://bugs.freedesktop.org/show_bug.cgi?id=112283 [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#1722]: https://gitlab.freedesktop.org/drm/intel/issues/1722 [i915#1825]: https://gitlab.freedesktop.org/drm/intel/issues/1825 [i915#1839]: https://gitlab.freedesktop.org/drm/intel/issues/1839 [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845 [i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849 [i915#1902]: https://gitlab.freedesktop.org/drm/intel/issues/1902 [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346 [i915#2527]: https://gitlab.freedesktop.org/drm/intel/issues/2527 [i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575 [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#2681]: https://gitlab.freedesktop.org/drm/intel/issues/2681 [i915#2705]: https://gitlab.freedesktop.org/drm/intel/issues/2705 [i915#280]: https://gitlab.freedesktop.org/drm/intel/issues/280 [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842 [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#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#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291 [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#3528]: https://gitlab.freedesktop.org/drm/intel/issues/3528 [i915#3546]: https://gitlab.freedesktop.org/drm/intel/issues/3546 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3591]: https://gitlab.freedesktop.org/drm/intel/issues/3591 [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#3742]: https://gitlab.freedesktop.org/drm/intel/issues/3742 [i915#3804]: https://gitlab.freedesktop.org/drm/intel/issues/3804 [i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840 [i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886 [i915#3955]: https://gitlab.freedesktop.org/drm/intel/issues/3955 [i915#3989]: https://gitlab.freedesktop.org/drm/intel/issues/3989 [i915#404]: https://gitlab.freedesktop.org/drm/intel/issues/404 [i915#4070]: https://gitlab.freedesktop.org/drm/intel/issues/4070 [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077 [i915#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078 [i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098 [i915#4258]: https://gitlab.freedesktop.org/drm/intel/issues/4258 [i915#426]: https://gitlab.freedesktop.org/drm/intel/issues/426 [i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270 [i915#4538]: https://gitlab.freedesktop.org/drm/intel/issues/4538 [i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#4767]: https://gitlab.freedesktop.org/drm/intel/issues/4767 [i915#4833]: https://gitlab.freedesktop.org/drm/intel/issues/4833 [i915#4877]: https://gitlab.freedesktop.org/drm/intel/issues/4877 [i915#4958]: https://gitlab.freedesktop.org/drm/intel/issues/4958 [i915#5030]: https://gitlab.freedesktop.org/drm/intel/issues/5030 [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#5325]: https://gitlab.freedesktop.org/drm/intel/issues/5325 [i915#5327]: https://gitlab.freedesktop.org/drm/intel/issues/5327 [i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533 [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334 [i915#5439]: https://gitlab.freedesktop.org/drm/intel/issues/5439 [i915#5461]: https://gitlab.freedesktop.org/drm/intel/issues/5461 [i915#5563]: https://gitlab.freedesktop.org/drm/intel/issues/5563 [i915#5723]: https://gitlab.freedesktop.org/drm/intel/issues/5723 [i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095 [i915#6227]: https://gitlab.freedesktop.org/drm/intel/issues/6227 [i915#6230]: https://gitlab.freedesktop.org/drm/intel/issues/6230 [i915#6245]: https://gitlab.freedesktop.org/drm/intel/issues/6245 [i915#6248]: https://gitlab.freedesktop.org/drm/intel/issues/6248 [i915#6301]: https://gitlab.freedesktop.org/drm/intel/issues/6301 [i915#6334]: https://gitlab.freedesktop.org/drm/intel/issues/6334 [i915#6344]: https://gitlab.freedesktop.org/drm/intel/issues/6344 [i915#6433]: https://gitlab.freedesktop.org/drm/intel/issues/6433 [i915#6497]: https://gitlab.freedesktop.org/drm/intel/issues/6497 [i915#6524]: https://gitlab.freedesktop.org/drm/intel/issues/6524 [i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658 [i915#6768]: https://gitlab.freedesktop.org/drm/intel/issues/6768 [i915#6944]: https://gitlab.freedesktop.org/drm/intel/issues/6944 [i915#7052]: https://gitlab.freedesktop.org/drm/intel/issues/7052 [i915#7116]: https://gitlab.freedesktop.org/drm/intel/issues/7116 [i915#7118]: https://gitlab.freedesktop.org/drm/intel/issues/7118 [i915#7456]: https://gitlab.freedesktop.org/drm/intel/issues/7456 [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561 [i915#7651]: https://gitlab.freedesktop.org/drm/intel/issues/7651 [i915#7681]: https://gitlab.freedesktop.org/drm/intel/issues/7681 [i915#7697]: https://gitlab.freedesktop.org/drm/intel/issues/7697 [i915#7707]: https://gitlab.freedesktop.org/drm/intel/issues/7707 [i915#7742]: https://gitlab.freedesktop.org/drm/intel/issues/7742 Build changes ------------- * Linux: CI_DRM_12546 -> Patchwork_112396v1 * Piglit: piglit_4509 -> None CI-20190529: 20190529 CI_DRM_12546: 07a684fbd4d0f5e284e8a782e0298f772fc4164e @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7107: 4f22b49ee353406c14ce8bb3151ebe3ce4e6e9be @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_112396v1: 07a684fbd4d0f5e284e8a782e0298f772fc4164e @ 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_112396v1/index.html [-- Attachment #2: Type: text/html, Size: 15650 bytes --] ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] [PATCH v2] drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila ` (3 preceding siblings ...) 2023-01-05 1:54 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork @ 2023-01-08 11:30 ` Juha-Pekka Heikkila 2023-01-10 21:27 ` Imre Deak 2023-01-08 13:13 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/display: assume some pixelrate for src smaller than 1 (rev2) Patchwork ` (3 subsequent siblings) 8 siblings, 1 reply; 16+ messages in thread From: Juha-Pekka Heikkila @ 2023-01-08 11:30 UTC (permalink / raw) To: intel-gfx intel_adjusted_rate() didn't take into account src rectangle can be less than 1 in with or height. Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> --- drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c index 10e1fc9d0698..cd24d069b6eb 100644 --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c @@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, const struct drm_rect *dst, unsigned int rate) { - unsigned int src_w, src_h, dst_w, dst_h; + unsigned int src_w, src_h, dst_w, dst_h, dst_wh; src_w = drm_rect_width(src) >> 16; src_h = drm_rect_height(src) >> 16; @@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, dst_w = min(src_w, dst_w); dst_h = min(src_h, dst_h); - return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), - dst_w * dst_h); + /* in case src contained only fractional part */ + dst_wh = max(dst_w * dst_h, 1U); + + return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh); } unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state, -- 2.39.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH v2] drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-08 11:30 ` [Intel-gfx] [PATCH v2] " Juha-Pekka Heikkila @ 2023-01-10 21:27 ` Imre Deak 2023-01-11 0:31 ` Imre Deak 0 siblings, 1 reply; 16+ messages in thread From: Imre Deak @ 2023-01-10 21:27 UTC (permalink / raw) To: Juha-Pekka Heikkila; +Cc: intel-gfx On Sun, Jan 08, 2023 at 01:30:44PM +0200, Juha-Pekka Heikkila wrote: > intel_adjusted_rate() didn't take into account src rectangle > can be less than 1 in with or height. Thanks for catching this. > Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> > --- > drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > index 10e1fc9d0698..cd24d069b6eb 100644 > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > @@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > const struct drm_rect *dst, > unsigned int rate) > { > - unsigned int src_w, src_h, dst_w, dst_h; > + unsigned int src_w, src_h, dst_w, dst_h, dst_wh; > > src_w = drm_rect_width(src) >> 16; > src_h = drm_rect_height(src) >> 16; > @@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > dst_w = min(src_w, dst_w); > dst_h = min(src_h, dst_h); > > - return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), > - dst_w * dst_h); > + /* in case src contained only fractional part */ > + dst_wh = max(dst_w * dst_h, 1U); > + > + return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh); The div-by-zero is avoided, but we'd return a 0 rate which doesn't look ok to me. I'd round up instead of down when converting src_w/h from fixed point to int above. > } > > unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state, > -- > 2.39.0 > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH v2] drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-10 21:27 ` Imre Deak @ 2023-01-11 0:31 ` Imre Deak 0 siblings, 0 replies; 16+ messages in thread From: Imre Deak @ 2023-01-11 0:31 UTC (permalink / raw) To: Juha-Pekka Heikkila, intel-gfx On Tue, Jan 10, 2023 at 11:27:33PM +0200, Imre Deak wrote: > On Sun, Jan 08, 2023 at 01:30:44PM +0200, Juha-Pekka Heikkila wrote: > > intel_adjusted_rate() didn't take into account src rectangle > > can be less than 1 in with or height. > > Thanks for catching this. > > > Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> > > --- > > drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++--- > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > index 10e1fc9d0698..cd24d069b6eb 100644 > > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > @@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > > const struct drm_rect *dst, > > unsigned int rate) > > { > > - unsigned int src_w, src_h, dst_w, dst_h; > > + unsigned int src_w, src_h, dst_w, dst_h, dst_wh; > > > > src_w = drm_rect_width(src) >> 16; > > src_h = drm_rect_height(src) >> 16; > > @@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > > dst_w = min(src_w, dst_w); > > dst_h = min(src_h, dst_h); > > > > - return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), > > - dst_w * dst_h); > > + /* in case src contained only fractional part */ > > + dst_wh = max(dst_w * dst_h, 1U); > > + > > + return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh); > > The div-by-zero is avoided, but we'd return a 0 rate which doesn't look > ok to me. I'd round up instead of down when converting src_w/h from > fixed point to int above. Looking at this more, src height < 1 isn't supported by the HW. I think this config should be rejected already by skl_check_main_surface(), as it's done on all other platforms. > > } > > > > unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state, > > -- > > 2.39.0 > > ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/display: assume some pixelrate for src smaller than 1 (rev2) 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila ` (4 preceding siblings ...) 2023-01-08 11:30 ` [Intel-gfx] [PATCH v2] " Juha-Pekka Heikkila @ 2023-01-08 13:13 ` Patchwork 2023-01-08 15:57 ` [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: assume some pixelrate for src smaller than 1 (rev3) Patchwork ` (2 subsequent siblings) 8 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2023-01-08 13:13 UTC (permalink / raw) To: Juha-Pekka Heikkila; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 6557 bytes --] == Series Details == Series: drm/i915/display: assume some pixelrate for src smaller than 1 (rev2) URL : https://patchwork.freedesktop.org/series/112396/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12556 -> Patchwork_112396v2 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_112396v2 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_112396v2, 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_112396v2/index.html Participating hosts (39 -> 39) ------------------------------ Additional (2): fi-kbl-soraka bat-rpls-2 Missing (2): fi-bsw-kefka fi-snb-2520m Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_112396v2: ### IGT changes ### #### Possible regressions #### * igt@i915_selftest@live@evict: - fi-kbl-soraka: NOTRUN -> [INCOMPLETE][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/fi-kbl-soraka/igt@i915_selftest@live@evict.html Known issues ------------ Here are the changes found in Patchwork_112396v2 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_gttfill@basic: - fi-kbl-soraka: NOTRUN -> [SKIP][2] ([fdo#109271]) +7 similar issues [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/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_112396v2/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html * igt@gem_lmem_swapping@basic: - fi-kbl-soraka: NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#4613]) +3 similar issues [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html * igt@i915_selftest@live@gt_heartbeat: - fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][5] ([i915#5334]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/fi-kbl-soraka/igt@i915_selftest@live@gt_heartbeat.html * igt@i915_selftest@live@gt_pm: - fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][6] ([i915#1886]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html * igt@i915_selftest@live@migrate: - bat-adlp-4: [PASS][7] -> [DMESG-FAIL][8] ([i915#7699]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/bat-adlp-4/igt@i915_selftest@live@migrate.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/bat-adlp-4/igt@i915_selftest@live@migrate.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-kbl-soraka: NOTRUN -> [SKIP][9] ([fdo#109271] / [fdo#111827]) +7 similar issues [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/fi-kbl-soraka/igt@kms_chamelium@hdmi-hpd-fast.html * igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions-varying-size: - fi-bsw-n3050: [PASS][10] -> [FAIL][11] ([i915#6298]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/fi-bsw-n3050/igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions-varying-size.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/fi-bsw-n3050/igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions-varying-size.html #### Possible fixes #### * igt@i915_selftest@live@gt_lrc: - {bat-rpls-1}: [INCOMPLETE][12] ([i915#4983]) -> [PASS][13] [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/bat-rpls-1/igt@i915_selftest@live@gt_lrc.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/bat-rpls-1/igt@i915_selftest@live@gt_lrc.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#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [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#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582 [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637 [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708 [i915#4258]: https://gitlab.freedesktop.org/drm/intel/issues/4258 [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983 [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334 [i915#6298]: https://gitlab.freedesktop.org/drm/intel/issues/6298 [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367 [i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621 [i915#7456]: https://gitlab.freedesktop.org/drm/intel/issues/7456 [i915#7467]: https://gitlab.freedesktop.org/drm/intel/issues/7467 [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561 [i915#7699]: https://gitlab.freedesktop.org/drm/intel/issues/7699 Build changes ------------- * Linux: CI_DRM_12556 -> Patchwork_112396v2 CI-20190529: 20190529 CI_DRM_12556: ac04152253dccfb02dcedfa0c57443122cf79314 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7110: db10a19b94d1d7ae5ba62eb48d52c47ccb27766f @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_112396v2: ac04152253dccfb02dcedfa0c57443122cf79314 @ git://anongit.freedesktop.org/gfx-ci/linux ### Linux commits 4c6f0f1de7c1 drm/i915/display: assume some pixelrate for src smaller than 1 == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v2/index.html [-- Attachment #2: Type: text/html, Size: 6466 bytes --] ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: assume some pixelrate for src smaller than 1 (rev3) 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila ` (5 preceding siblings ...) 2023-01-08 13:13 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/display: assume some pixelrate for src smaller than 1 (rev2) Patchwork @ 2023-01-08 15:57 ` Patchwork 2023-01-08 17:18 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 2023-01-11 14:19 ` [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Ville Syrjälä 8 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2023-01-08 15:57 UTC (permalink / raw) To: Juha-Pekka Heikkila; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 6144 bytes --] == Series Details == Series: drm/i915/display: assume some pixelrate for src smaller than 1 (rev3) URL : https://patchwork.freedesktop.org/series/112396/ State : success == Summary == CI Bug Log - changes from CI_DRM_12556 -> Patchwork_112396v3 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/index.html Participating hosts (39 -> 38) ------------------------------ Additional (2): fi-kbl-soraka bat-rpls-2 Missing (3): fi-bsw-kefka fi-apl-guc fi-snb-2520m Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_112396v3: ### IGT changes ### #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@i915_selftest@live@gt_heartbeat: - {bat-kbl-2}: [PASS][1] -> [DMESG-FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/bat-kbl-2/igt@i915_selftest@live@gt_heartbeat.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/bat-kbl-2/igt@i915_selftest@live@gt_heartbeat.html * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-3: - {bat-dg2-9}: [PASS][3] -> [FAIL][4] [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/bat-dg2-9/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-3.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/bat-dg2-9/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-3.html Known issues ------------ Here are the changes found in Patchwork_112396v3 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_gttfill@basic: - fi-kbl-soraka: NOTRUN -> [SKIP][5] ([fdo#109271]) +7 similar issues [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/fi-kbl-soraka/igt@gem_exec_gttfill@basic.html * igt@gem_huc_copy@huc-copy: - fi-kbl-soraka: NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#2190]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html * igt@gem_lmem_swapping@basic: - fi-kbl-soraka: NOTRUN -> [SKIP][7] ([fdo#109271] / [i915#4613]) +3 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html * igt@i915_selftest@live@gt_heartbeat: - fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][8] ([i915#5334]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/fi-kbl-soraka/igt@i915_selftest@live@gt_heartbeat.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_112396v3/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html * igt@i915_selftest@live@guc_multi_lrc: - fi-kbl-soraka: NOTRUN -> [INCOMPLETE][10] ([i915#7640]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/fi-kbl-soraka/igt@i915_selftest@live@guc_multi_lrc.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-kbl-soraka: NOTRUN -> [SKIP][11] ([fdo#109271] / [fdo#111827]) +7 similar issues [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/fi-kbl-soraka/igt@kms_chamelium@hdmi-hpd-fast.html #### Possible fixes #### * igt@i915_selftest@live@gt_lrc: - {bat-rpls-1}: [INCOMPLETE][12] ([i915#4983]) -> [PASS][13] [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/bat-rpls-1/igt@i915_selftest@live@gt_lrc.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/bat-rpls-1/igt@i915_selftest@live@gt_lrc.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#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [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#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582 [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637 [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708 [i915#4258]: https://gitlab.freedesktop.org/drm/intel/issues/4258 [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983 [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334 [i915#6257]: https://gitlab.freedesktop.org/drm/intel/issues/6257 [i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621 [i915#7456]: https://gitlab.freedesktop.org/drm/intel/issues/7456 [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561 [i915#7640]: https://gitlab.freedesktop.org/drm/intel/issues/7640 Build changes ------------- * Linux: CI_DRM_12556 -> Patchwork_112396v3 CI-20190529: 20190529 CI_DRM_12556: ac04152253dccfb02dcedfa0c57443122cf79314 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7110: db10a19b94d1d7ae5ba62eb48d52c47ccb27766f @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_112396v3: ac04152253dccfb02dcedfa0c57443122cf79314 @ git://anongit.freedesktop.org/gfx-ci/linux ### Linux commits 96155caa152a drm/i915/display: assume some pixelrate for src smaller than 1 == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/index.html [-- Attachment #2: Type: text/html, Size: 6123 bytes --] ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/display: assume some pixelrate for src smaller than 1 (rev3) 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila ` (6 preceding siblings ...) 2023-01-08 15:57 ` [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: assume some pixelrate for src smaller than 1 (rev3) Patchwork @ 2023-01-08 17:18 ` Patchwork 2023-01-11 14:19 ` [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Ville Syrjälä 8 siblings, 0 replies; 16+ messages in thread From: Patchwork @ 2023-01-08 17:18 UTC (permalink / raw) To: Juha-Pekka Heikkila; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 18612 bytes --] == Series Details == Series: drm/i915/display: assume some pixelrate for src smaller than 1 (rev3) URL : https://patchwork.freedesktop.org/series/112396/ State : success == Summary == CI Bug Log - changes from CI_DRM_12556_full -> Patchwork_112396v3_full ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/index.html Participating hosts (14 -> 11) ------------------------------ Missing (3): pig-skl-6260u pig-kbl-iris pig-glk-j5005 Known issues ------------ Here are the changes found in Patchwork_112396v3_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_fair@basic-deadline: - shard-glk: [PASS][1] -> [FAIL][2] ([i915#2846]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-glk2/igt@gem_exec_fair@basic-deadline.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-glk6/igt@gem_exec_fair@basic-deadline.html * igt@gem_exec_fair@basic-none@vcs0: - shard-glk: [PASS][3] -> [FAIL][4] ([i915#2842]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-glk6/igt@gem_exec_fair@basic-none@vcs0.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-glk3/igt@gem_exec_fair@basic-none@vcs0.html * igt@perf@stress-open-close: - shard-glk: [PASS][5] -> [INCOMPLETE][6] ([i915#5213]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-glk3/igt@perf@stress-open-close.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-glk4/igt@perf@stress-open-close.html * igt@runner@aborted: - shard-glk: NOTRUN -> [FAIL][7] ([i915#4312]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-glk4/igt@runner@aborted.html #### Possible fixes #### * igt@drm_read@short-buffer-block: - {shard-rkl}: [SKIP][8] ([i915#4098]) -> [PASS][9] [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-5/igt@drm_read@short-buffer-block.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@drm_read@short-buffer-block.html * igt@fbdev@info: - {shard-rkl}: [SKIP][10] ([i915#2582]) -> [PASS][11] [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-4/igt@fbdev@info.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@fbdev@info.html * igt@gem_eio@reset-stress: - {shard-dg1}: [FAIL][12] ([i915#5784]) -> [PASS][13] [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-dg1-17/igt@gem_eio@reset-stress.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-dg1-16/igt@gem_eio@reset-stress.html * igt@gem_eio@suspend: - {shard-rkl}: [FAIL][14] ([i915#7052]) -> [PASS][15] [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-3/igt@gem_eio@suspend.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-1/igt@gem_eio@suspend.html * igt@gem_exec_fair@basic-flow@rcs0: - {shard-rkl}: [FAIL][16] ([i915#2842]) -> [PASS][17] [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-2/igt@gem_exec_fair@basic-flow@rcs0.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@gem_exec_fair@basic-flow@rcs0.html * igt@gem_exec_reloc@basic-gtt-cpu: - {shard-rkl}: [SKIP][18] ([i915#3281]) -> [PASS][19] +1 similar issue [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-4/igt@gem_exec_reloc@basic-gtt-cpu.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-5/igt@gem_exec_reloc@basic-gtt-cpu.html * igt@gem_tiled_pread_basic: - {shard-rkl}: [SKIP][20] ([i915#3282]) -> [PASS][21] +1 similar issue [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-4/igt@gem_tiled_pread_basic.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-5/igt@gem_tiled_pread_basic.html * igt@gen9_exec_parse@bb-start-far: - {shard-rkl}: [SKIP][22] ([i915#2527]) -> [PASS][23] [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-4/igt@gen9_exec_parse@bb-start-far.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-5/igt@gen9_exec_parse@bb-start-far.html * igt@i915_hangman@gt-engine-error@bcs0: - {shard-rkl}: [SKIP][24] ([i915#6258]) -> [PASS][25] [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-5/igt@i915_hangman@gt-engine-error@bcs0.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-4/igt@i915_hangman@gt-engine-error@bcs0.html * igt@i915_pm_rpm@drm-resources-equal: - {shard-rkl}: [SKIP][26] ([fdo#109308]) -> [PASS][27] [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-4/igt@i915_pm_rpm@drm-resources-equal.html [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@i915_pm_rpm@drm-resources-equal.html * igt@i915_pm_rpm@modeset-lpsp-stress: - {shard-rkl}: [SKIP][28] ([i915#1397]) -> [PASS][29] +1 similar issue [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-2/igt@i915_pm_rpm@modeset-lpsp-stress.html [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@i915_pm_rpm@modeset-lpsp-stress.html * igt@kms_atomic@plane-immutable-zpos: - {shard-rkl}: [SKIP][30] ([i915#1845] / [i915#4098]) -> [PASS][31] +26 similar issues [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-2/igt@kms_atomic@plane-immutable-zpos.html [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@kms_atomic@plane-immutable-zpos.html * igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2: - shard-glk: [FAIL][32] ([i915#2122]) -> [PASS][33] [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-glk8/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2.html [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-glk9/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2.html * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-blt: - {shard-rkl}: [SKIP][34] ([i915#1849] / [i915#4098]) -> [PASS][35] +17 similar issues [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-4/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-blt.html [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-blt.html * igt@kms_psr@sprite_plane_onoff: - {shard-rkl}: [SKIP][36] ([i915#1072]) -> [PASS][37] +2 similar issues [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-4/igt@kms_psr@sprite_plane_onoff.html [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@kms_psr@sprite_plane_onoff.html * igt@kms_psr_stress_test@invalidate-primary-flip-overlay: - {shard-rkl}: [SKIP][38] ([i915#5461]) -> [PASS][39] [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-4/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html * igt@kms_universal_plane@disable-primary-vs-flip-pipe-a: - {shard-rkl}: [SKIP][40] ([i915#1845] / [i915#4070] / [i915#4098]) -> [PASS][41] [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-2/igt@kms_universal_plane@disable-primary-vs-flip-pipe-a.html [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-6/igt@kms_universal_plane@disable-primary-vs-flip-pipe-a.html * igt@perf@gen12-unprivileged-single-ctx-counters: - {shard-rkl}: [SKIP][42] ([fdo#109289]) -> [PASS][43] [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-5/igt@perf@gen12-unprivileged-single-ctx-counters.html [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_112396v3/shard-rkl-4/igt@perf@gen12-unprivileged-single-ctx-counters.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274 [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#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289 [fdo#109291]: https://bugs.freedesktop.org/show_bug.cgi?id=109291 [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295 [fdo#109302]: https://bugs.freedesktop.org/show_bug.cgi?id=109302 [fdo#109307]: https://bugs.freedesktop.org/show_bug.cgi?id=109307 [fdo#109308]: https://bugs.freedesktop.org/show_bug.cgi?id=109308 [fdo#109312]: https://bugs.freedesktop.org/show_bug.cgi?id=109312 [fdo#109313]: https://bugs.freedesktop.org/show_bug.cgi?id=109313 [fdo#109314]: https://bugs.freedesktop.org/show_bug.cgi?id=109314 [fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315 [fdo#109506]: https://bugs.freedesktop.org/show_bug.cgi?id=109506 [fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642 [fdo#110189]: https://bugs.freedesktop.org/show_bug.cgi?id=110189 [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#111644]: https://bugs.freedesktop.org/show_bug.cgi?id=111644 [fdo#111656]: https://bugs.freedesktop.org/show_bug.cgi?id=111656 [fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [fdo#112283]: https://bugs.freedesktop.org/show_bug.cgi?id=112283 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [i915#1257]: https://gitlab.freedesktop.org/drm/intel/issues/1257 [i915#132]: https://gitlab.freedesktop.org/drm/intel/issues/132 [i915#1397]: https://gitlab.freedesktop.org/drm/intel/issues/1397 [i915#1755]: https://gitlab.freedesktop.org/drm/intel/issues/1755 [i915#1825]: https://gitlab.freedesktop.org/drm/intel/issues/1825 [i915#1839]: https://gitlab.freedesktop.org/drm/intel/issues/1839 [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845 [i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849 [i915#1850]: https://gitlab.freedesktop.org/drm/intel/issues/1850 [i915#1902]: https://gitlab.freedesktop.org/drm/intel/issues/1902 [i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122 [i915#2434]: https://gitlab.freedesktop.org/drm/intel/issues/2434 [i915#2435]: https://gitlab.freedesktop.org/drm/intel/issues/2435 [i915#2436]: https://gitlab.freedesktop.org/drm/intel/issues/2436 [i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437 [i915#2527]: https://gitlab.freedesktop.org/drm/intel/issues/2527 [i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575 [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#2681]: https://gitlab.freedesktop.org/drm/intel/issues/2681 [i915#2705]: https://gitlab.freedesktop.org/drm/intel/issues/2705 [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#3116]: https://gitlab.freedesktop.org/drm/intel/issues/3116 [i915#315]: https://gitlab.freedesktop.org/drm/intel/issues/315 [i915#3281]: https://gitlab.freedesktop.org/drm/intel/issues/3281 [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282 [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291 [i915#3297]: https://gitlab.freedesktop.org/drm/intel/issues/3297 [i915#3299]: https://gitlab.freedesktop.org/drm/intel/issues/3299 [i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301 [i915#3318]: https://gitlab.freedesktop.org/drm/intel/issues/3318 [i915#3323]: https://gitlab.freedesktop.org/drm/intel/issues/3323 [i915#3359]: https://gitlab.freedesktop.org/drm/intel/issues/3359 [i915#3458]: https://gitlab.freedesktop.org/drm/intel/issues/3458 [i915#3469]: https://gitlab.freedesktop.org/drm/intel/issues/3469 [i915#3528]: https://gitlab.freedesktop.org/drm/intel/issues/3528 [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#3742]: https://gitlab.freedesktop.org/drm/intel/issues/3742 [i915#3804]: https://gitlab.freedesktop.org/drm/intel/issues/3804 [i915#3825]: https://gitlab.freedesktop.org/drm/intel/issues/3825 [i915#3826]: https://gitlab.freedesktop.org/drm/intel/issues/3826 [i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840 [i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886 [i915#4070]: https://gitlab.freedesktop.org/drm/intel/issues/4070 [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077 [i915#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078 [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079 [i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098 [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103 [i915#426]: https://gitlab.freedesktop.org/drm/intel/issues/426 [i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270 [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312 [i915#4349]: https://gitlab.freedesktop.org/drm/intel/issues/4349 [i915#4525]: https://gitlab.freedesktop.org/drm/intel/issues/4525 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#4767]: https://gitlab.freedesktop.org/drm/intel/issues/4767 [i915#4833]: https://gitlab.freedesktop.org/drm/intel/issues/4833 [i915#4852]: https://gitlab.freedesktop.org/drm/intel/issues/4852 [i915#4877]: https://gitlab.freedesktop.org/drm/intel/issues/4877 [i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176 [i915#5213]: https://gitlab.freedesktop.org/drm/intel/issues/5213 [i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235 [i915#5286]: https://gitlab.freedesktop.org/drm/intel/issues/5286 [i915#5289]: https://gitlab.freedesktop.org/drm/intel/issues/5289 [i915#5325]: https://gitlab.freedesktop.org/drm/intel/issues/5325 [i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533 [i915#5439]: https://gitlab.freedesktop.org/drm/intel/issues/5439 [i915#5461]: https://gitlab.freedesktop.org/drm/intel/issues/5461 [i915#5784]: https://gitlab.freedesktop.org/drm/intel/issues/5784 [i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095 [i915#6227]: https://gitlab.freedesktop.org/drm/intel/issues/6227 [i915#6230]: https://gitlab.freedesktop.org/drm/intel/issues/6230 [i915#6245]: https://gitlab.freedesktop.org/drm/intel/issues/6245 [i915#6248]: https://gitlab.freedesktop.org/drm/intel/issues/6248 [i915#6258]: https://gitlab.freedesktop.org/drm/intel/issues/6258 [i915#6268]: https://gitlab.freedesktop.org/drm/intel/issues/6268 [i915#6301]: https://gitlab.freedesktop.org/drm/intel/issues/6301 [i915#6433]: https://gitlab.freedesktop.org/drm/intel/issues/6433 [i915#6497]: https://gitlab.freedesktop.org/drm/intel/issues/6497 [i915#6524]: https://gitlab.freedesktop.org/drm/intel/issues/6524 [i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658 [i915#6768]: https://gitlab.freedesktop.org/drm/intel/issues/6768 [i915#6944]: https://gitlab.freedesktop.org/drm/intel/issues/6944 [i915#6946]: https://gitlab.freedesktop.org/drm/intel/issues/6946 [i915#6953]: https://gitlab.freedesktop.org/drm/intel/issues/6953 [i915#7037]: https://gitlab.freedesktop.org/drm/intel/issues/7037 [i915#7052]: https://gitlab.freedesktop.org/drm/intel/issues/7052 [i915#7116]: https://gitlab.freedesktop.org/drm/intel/issues/7116 [i915#7118]: https://gitlab.freedesktop.org/drm/intel/issues/7118 [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561 [i915#7651]: https://gitlab.freedesktop.org/drm/intel/issues/7651 [i915#7679]: https://gitlab.freedesktop.org/drm/intel/issues/7679 [i915#7697]: https://gitlab.freedesktop.org/drm/intel/issues/7697 [i915#7701]: https://gitlab.freedesktop.org/drm/intel/issues/7701 [i915#7711]: https://gitlab.freedesktop.org/drm/intel/issues/7711 [i915#7742]: https://gitlab.freedesktop.org/drm/intel/issues/7742 Build changes ------------- * Linux: CI_DRM_12556 -> Patchwork_112396v3 * Piglit: piglit_4509 -> None CI-20190529: 20190529 CI_DRM_12556: ac04152253dccfb02dcedfa0c57443122cf79314 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7110: db10a19b94d1d7ae5ba62eb48d52c47ccb27766f @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_112396v3: ac04152253dccfb02dcedfa0c57443122cf79314 @ 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_112396v3/index.html [-- Attachment #2: Type: text/html, Size: 12159 bytes --] ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila ` (7 preceding siblings ...) 2023-01-08 17:18 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork @ 2023-01-11 14:19 ` Ville Syrjälä 2023-01-11 18:28 ` Drew Davenport 8 siblings, 1 reply; 16+ messages in thread From: Ville Syrjälä @ 2023-01-11 14:19 UTC (permalink / raw) To: Juha-Pekka Heikkila; +Cc: Drew Davenport, intel-gfx On Wed, Jan 04, 2023 at 02:44:48PM +0200, Juha-Pekka Heikkila wrote: > intel_adjusted_rate() didn't take into account src rectangle > can be less than 1 in width or height. This should not get called in those cases. What does the backtrace look like? > > Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> > --- > drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > index 10e1fc9d0698..a9948e8d3543 100644 > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > @@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > const struct drm_rect *dst, > unsigned int rate) > { > - unsigned int src_w, src_h, dst_w, dst_h; > + unsigned int src_w, src_h, dst_w, dst_h, dst_wh; > > src_w = drm_rect_width(src) >> 16; > src_h = drm_rect_height(src) >> 16; > @@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > dst_w = min(src_w, dst_w); > dst_h = min(src_h, dst_h); > > - return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), > - dst_w * dst_h); > + /* in case src contained only fractional part */ > + dst_wh = max(dst_w * dst_h, (unsigned) 1); > + > + return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh); > } > > unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state, > -- > 2.37.3 -- Ville Syrjälä Intel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-11 14:19 ` [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Ville Syrjälä @ 2023-01-11 18:28 ` Drew Davenport 2023-01-11 19:39 ` Ville Syrjälä 0 siblings, 1 reply; 16+ messages in thread From: Drew Davenport @ 2023-01-11 18:28 UTC (permalink / raw) To: Ville Syrjälä; +Cc: intel-gfx On Wed, Jan 11, 2023 at 04:19:00PM +0200, Ville Syrjälä wrote: > On Wed, Jan 04, 2023 at 02:44:48PM +0200, Juha-Pekka Heikkila wrote: > > intel_adjusted_rate() didn't take into account src rectangle > > can be less than 1 in width or height. > > This should not get called in those cases. What does the > backtrace look like? In my repro of this issue, the backtrace looks as follows: [ 180.798331] RIP: 0010:intel_plane_pixel_rate+0x4a/0x53 [ 180.798336] Code: <snip long line> [ 180.798338] RSP: 0018:ffffb080ce4179b8 EFLAGS: 00010246 [ 180.798341] RAX: ffffffffffffffff RBX: ffff98cd22a24000 RCX: 0000000000000a00 [ 180.798343] RDX: 0000000000000000 RSI: ffff98cccbae7000 RDI: 0000000000000000 [ 180.798346] RBP: ffffb080ce4179b8 R08: 0000000000087780 R09: 0000000000000002 [ 180.798348] R10: 0000000000000a00 R11: 0000000000000000 R12: 0000000000000000 [ 180.798350] R13: ffff98cd0e495400 R14: ffff98ccc34e0000 R15: ffff98cccbae7000 [ 180.798352] FS: 00007b84119b5000(0000) GS:ffff98d02f900000(0000) knlGS:0000000000000000 [ 180.798354] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 180.798357] CR2: 00007ffc2d5e4080 CR3: 00000001042ee006 CR4: 0000000000770ee0 [ 180.798359] PKRU: 55555554 [ 180.798361] Call Trace: [ 180.798364] <TASK> [ 180.798366] intel_plane_atomic_check_with_state+0x1fd/0x6ea [ 180.798370] ? intel_plane_atomic_check+0x11b/0x145 [ 180.798373] intel_atomic_check_planes+0x263/0x7ce [ 180.798376] ? drm_atomic_helper_check_modeset+0x189/0x923 [ 180.798380] intel_atomic_check+0x14e4/0x184d [ 180.798382] ? intel_rps_mark_interactive+0x23/0x6a [ 180.798386] drm_atomic_check_only+0x3ec/0x98f [ 180.798391] drm_atomic_commit+0xa2/0x105 [ 180.798394] ? drm_atomic_set_fb_for_plane+0x96/0xa5 [ 180.798397] drm_atomic_helper_update_plane+0xdc/0x11f [ 180.798400] drm_mode_setplane+0x236/0x30c [ 180.798404] ? drm_any_plane_has_format+0x51/0x51 [ 180.798407] drm_ioctl_kernel+0xda/0x14d [ 180.798411] drm_ioctl+0x27e/0x3b4 [ 180.798414] ? drm_any_plane_has_format+0x51/0x51 [ 180.798418] __se_sys_ioctl+0x7a/0xbc [ 180.798421] do_syscall_64+0x55/0x9d [ 180.798424] ? exit_to_user_mode_prepare+0x3c/0x8b [ 180.798427] entry_SYSCALL_64_after_hwframe+0x61/0xcb If this function shouldn't be called in such a case, then perhaps I should revist my original attempt at fixing this in https://patchwork.freedesktop.org/patch/516060 by rejecting such a configuration? I'll respond to Alan on that thread. > > > > > Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> > > --- > > drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++--- > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > index 10e1fc9d0698..a9948e8d3543 100644 > > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > @@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > > const struct drm_rect *dst, > > unsigned int rate) > > { > > - unsigned int src_w, src_h, dst_w, dst_h; > > + unsigned int src_w, src_h, dst_w, dst_h, dst_wh; > > > > src_w = drm_rect_width(src) >> 16; > > src_h = drm_rect_height(src) >> 16; > > @@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > > dst_w = min(src_w, dst_w); > > dst_h = min(src_h, dst_h); > > > > - return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), > > - dst_w * dst_h); > > + /* in case src contained only fractional part */ > > + dst_wh = max(dst_w * dst_h, (unsigned) 1); > > + > > + return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh); > > } > > > > unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state, > > -- > > 2.37.3 > > -- > Ville Syrjälä > Intel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-11 18:28 ` Drew Davenport @ 2023-01-11 19:39 ` Ville Syrjälä 2023-01-11 20:09 ` Drew Davenport 0 siblings, 1 reply; 16+ messages in thread From: Ville Syrjälä @ 2023-01-11 19:39 UTC (permalink / raw) To: Drew Davenport; +Cc: intel-gfx On Wed, Jan 11, 2023 at 11:28:51AM -0700, Drew Davenport wrote: > On Wed, Jan 11, 2023 at 04:19:00PM +0200, Ville Syrjälä wrote: > > On Wed, Jan 04, 2023 at 02:44:48PM +0200, Juha-Pekka Heikkila wrote: > > > intel_adjusted_rate() didn't take into account src rectangle > > > can be less than 1 in width or height. > > > > This should not get called in those cases. What does the > > backtrace look like? > > In my repro of this issue, the backtrace looks as follows: > > [ 180.798331] RIP: 0010:intel_plane_pixel_rate+0x4a/0x53 > [ 180.798336] Code: <snip long line> > [ 180.798338] RSP: 0018:ffffb080ce4179b8 EFLAGS: 00010246 > [ 180.798341] RAX: ffffffffffffffff RBX: ffff98cd22a24000 RCX: 0000000000000a00 > [ 180.798343] RDX: 0000000000000000 RSI: ffff98cccbae7000 RDI: 0000000000000000 > [ 180.798346] RBP: ffffb080ce4179b8 R08: 0000000000087780 R09: 0000000000000002 > [ 180.798348] R10: 0000000000000a00 R11: 0000000000000000 R12: 0000000000000000 > [ 180.798350] R13: ffff98cd0e495400 R14: ffff98ccc34e0000 R15: ffff98cccbae7000 > [ 180.798352] FS: 00007b84119b5000(0000) GS:ffff98d02f900000(0000) knlGS:0000000000000000 > [ 180.798354] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 > [ 180.798357] CR2: 00007ffc2d5e4080 CR3: 00000001042ee006 CR4: 0000000000770ee0 > [ 180.798359] PKRU: 55555554 > [ 180.798361] Call Trace: > [ 180.798364] <TASK> > [ 180.798366] intel_plane_atomic_check_with_state+0x1fd/0x6ea > [ 180.798370] ? intel_plane_atomic_check+0x11b/0x145 > [ 180.798373] intel_atomic_check_planes+0x263/0x7ce > [ 180.798376] ? drm_atomic_helper_check_modeset+0x189/0x923 > [ 180.798380] intel_atomic_check+0x14e4/0x184d > [ 180.798382] ? intel_rps_mark_interactive+0x23/0x6a > [ 180.798386] drm_atomic_check_only+0x3ec/0x98f > [ 180.798391] drm_atomic_commit+0xa2/0x105 > [ 180.798394] ? drm_atomic_set_fb_for_plane+0x96/0xa5 > [ 180.798397] drm_atomic_helper_update_plane+0xdc/0x11f > [ 180.798400] drm_mode_setplane+0x236/0x30c > [ 180.798404] ? drm_any_plane_has_format+0x51/0x51 > [ 180.798407] drm_ioctl_kernel+0xda/0x14d > [ 180.798411] drm_ioctl+0x27e/0x3b4 > [ 180.798414] ? drm_any_plane_has_format+0x51/0x51 > [ 180.798418] __se_sys_ioctl+0x7a/0xbc > [ 180.798421] do_syscall_64+0x55/0x9d > [ 180.798424] ? exit_to_user_mode_prepare+0x3c/0x8b > [ 180.798427] entry_SYSCALL_64_after_hwframe+0x61/0xcb > > If this function shouldn't be called in such a case, then perhaps > I should revist my original attempt at fixing this in > https://patchwork.freedesktop.org/patch/516060 by rejecting such a > configuration? I'm saying that this should be impossible already. At least I can't immediately see anything that could call this with an invisible plane. > > I'll respond to Alan on that thread. > > > > > > > > > Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> > > > --- > > > drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++--- > > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > > index 10e1fc9d0698..a9948e8d3543 100644 > > > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > > @@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > > > const struct drm_rect *dst, > > > unsigned int rate) > > > { > > > - unsigned int src_w, src_h, dst_w, dst_h; > > > + unsigned int src_w, src_h, dst_w, dst_h, dst_wh; > > > > > > src_w = drm_rect_width(src) >> 16; > > > src_h = drm_rect_height(src) >> 16; > > > @@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > > > dst_w = min(src_w, dst_w); > > > dst_h = min(src_h, dst_h); > > > > > > - return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), > > > - dst_w * dst_h); > > > + /* in case src contained only fractional part */ > > > + dst_wh = max(dst_w * dst_h, (unsigned) 1); > > > + > > > + return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh); > > > } > > > > > > unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state, > > > -- > > > 2.37.3 > > > > -- > > Ville Syrjälä > > Intel -- Ville Syrjälä Intel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-11 19:39 ` Ville Syrjälä @ 2023-01-11 20:09 ` Drew Davenport 2023-01-11 21:25 ` Juha-Pekka Heikkila 0 siblings, 1 reply; 16+ messages in thread From: Drew Davenport @ 2023-01-11 20:09 UTC (permalink / raw) To: Ville Syrjälä; +Cc: intel-gfx On Wed, Jan 11, 2023 at 09:39:26PM +0200, Ville Syrjälä wrote: > On Wed, Jan 11, 2023 at 11:28:51AM -0700, Drew Davenport wrote: > > On Wed, Jan 11, 2023 at 04:19:00PM +0200, Ville Syrjälä wrote: > > > On Wed, Jan 04, 2023 at 02:44:48PM +0200, Juha-Pekka Heikkila wrote: > > > > intel_adjusted_rate() didn't take into account src rectangle > > > > can be less than 1 in width or height. > > > > > > This should not get called in those cases. What does the > > > backtrace look like? > > > > In my repro of this issue, the backtrace looks as follows: > > > > [ 180.798331] RIP: 0010:intel_plane_pixel_rate+0x4a/0x53 > > [ 180.798336] Code: <snip long line> > > [ 180.798338] RSP: 0018:ffffb080ce4179b8 EFLAGS: 00010246 > > [ 180.798341] RAX: ffffffffffffffff RBX: ffff98cd22a24000 RCX: 0000000000000a00 > > [ 180.798343] RDX: 0000000000000000 RSI: ffff98cccbae7000 RDI: 0000000000000000 > > [ 180.798346] RBP: ffffb080ce4179b8 R08: 0000000000087780 R09: 0000000000000002 > > [ 180.798348] R10: 0000000000000a00 R11: 0000000000000000 R12: 0000000000000000 > > [ 180.798350] R13: ffff98cd0e495400 R14: ffff98ccc34e0000 R15: ffff98cccbae7000 > > [ 180.798352] FS: 00007b84119b5000(0000) GS:ffff98d02f900000(0000) knlGS:0000000000000000 > > [ 180.798354] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 > > [ 180.798357] CR2: 00007ffc2d5e4080 CR3: 00000001042ee006 CR4: 0000000000770ee0 > > [ 180.798359] PKRU: 55555554 > > [ 180.798361] Call Trace: > > [ 180.798364] <TASK> > > [ 180.798366] intel_plane_atomic_check_with_state+0x1fd/0x6ea > > [ 180.798370] ? intel_plane_atomic_check+0x11b/0x145 > > [ 180.798373] intel_atomic_check_planes+0x263/0x7ce > > [ 180.798376] ? drm_atomic_helper_check_modeset+0x189/0x923 > > [ 180.798380] intel_atomic_check+0x14e4/0x184d > > [ 180.798382] ? intel_rps_mark_interactive+0x23/0x6a > > [ 180.798386] drm_atomic_check_only+0x3ec/0x98f > > [ 180.798391] drm_atomic_commit+0xa2/0x105 > > [ 180.798394] ? drm_atomic_set_fb_for_plane+0x96/0xa5 > > [ 180.798397] drm_atomic_helper_update_plane+0xdc/0x11f > > [ 180.798400] drm_mode_setplane+0x236/0x30c > > [ 180.798404] ? drm_any_plane_has_format+0x51/0x51 > > [ 180.798407] drm_ioctl_kernel+0xda/0x14d > > [ 180.798411] drm_ioctl+0x27e/0x3b4 > > [ 180.798414] ? drm_any_plane_has_format+0x51/0x51 > > [ 180.798418] __se_sys_ioctl+0x7a/0xbc > > [ 180.798421] do_syscall_64+0x55/0x9d > > [ 180.798424] ? exit_to_user_mode_prepare+0x3c/0x8b > > [ 180.798427] entry_SYSCALL_64_after_hwframe+0x61/0xcb > > > > If this function shouldn't be called in such a case, then perhaps > > I should revist my original attempt at fixing this in > > https://patchwork.freedesktop.org/patch/516060 by rejecting such a > > configuration? > > I'm saying that this should be impossible already. At least > I can't immediately see anything that could call this with > an invisible plane. In my repro case, I called drmModeSetPlane with the src_h parameter set to 65535 (so the largest 16.16 number that's less than one). This got through any existing checks on the height of the src rect, resulting in the divide-by-zero error in intel_plane_pixel_rate. While investigating this, I tried setting src_h to 0, but this configuration got rejected somewhere along the line before it got through the intel_plane_pixel_rate. > > > > > I'll respond to Alan on that thread. > > > > > > > > > > > > > Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> > > > > --- > > > > drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++--- > > > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > > > > > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > > > index 10e1fc9d0698..a9948e8d3543 100644 > > > > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > > > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c > > > > @@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > > > > const struct drm_rect *dst, > > > > unsigned int rate) > > > > { > > > > - unsigned int src_w, src_h, dst_w, dst_h; > > > > + unsigned int src_w, src_h, dst_w, dst_h, dst_wh; > > > > > > > > src_w = drm_rect_width(src) >> 16; > > > > src_h = drm_rect_height(src) >> 16; > > > > @@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, > > > > dst_w = min(src_w, dst_w); > > > > dst_h = min(src_h, dst_h); > > > > > > > > - return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), > > > > - dst_w * dst_h); > > > > + /* in case src contained only fractional part */ > > > > + dst_wh = max(dst_w * dst_h, (unsigned) 1); > > > > + > > > > + return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh); > > > > } > > > > > > > > unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state, > > > > -- > > > > 2.37.3 > > > > > > -- > > > Ville Syrjälä > > > Intel > > -- > Ville Syrjälä > Intel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 2023-01-11 20:09 ` Drew Davenport @ 2023-01-11 21:25 ` Juha-Pekka Heikkila 0 siblings, 0 replies; 16+ messages in thread From: Juha-Pekka Heikkila @ 2023-01-11 21:25 UTC (permalink / raw) To: Drew Davenport, Ville Syrjälä; +Cc: intel-gfx On 11.1.2023 22.09, Drew Davenport wrote: > On Wed, Jan 11, 2023 at 09:39:26PM +0200, Ville Syrjälä wrote: >> On Wed, Jan 11, 2023 at 11:28:51AM -0700, Drew Davenport wrote: >>> On Wed, Jan 11, 2023 at 04:19:00PM +0200, Ville Syrjälä wrote: >>>> On Wed, Jan 04, 2023 at 02:44:48PM +0200, Juha-Pekka Heikkila wrote: >>>>> intel_adjusted_rate() didn't take into account src rectangle >>>>> can be less than 1 in width or height. >>>> >>>> This should not get called in those cases. What does the >>>> backtrace look like? >>> >>> In my repro of this issue, the backtrace looks as follows: >>> >>> [ 180.798331] RIP: 0010:intel_plane_pixel_rate+0x4a/0x53 >>> [ 180.798336] Code: <snip long line> >>> [ 180.798338] RSP: 0018:ffffb080ce4179b8 EFLAGS: 00010246 >>> [ 180.798341] RAX: ffffffffffffffff RBX: ffff98cd22a24000 RCX: 0000000000000a00 >>> [ 180.798343] RDX: 0000000000000000 RSI: ffff98cccbae7000 RDI: 0000000000000000 >>> [ 180.798346] RBP: ffffb080ce4179b8 R08: 0000000000087780 R09: 0000000000000002 >>> [ 180.798348] R10: 0000000000000a00 R11: 0000000000000000 R12: 0000000000000000 >>> [ 180.798350] R13: ffff98cd0e495400 R14: ffff98ccc34e0000 R15: ffff98cccbae7000 >>> [ 180.798352] FS: 00007b84119b5000(0000) GS:ffff98d02f900000(0000) knlGS:0000000000000000 >>> [ 180.798354] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 >>> [ 180.798357] CR2: 00007ffc2d5e4080 CR3: 00000001042ee006 CR4: 0000000000770ee0 >>> [ 180.798359] PKRU: 55555554 >>> [ 180.798361] Call Trace: >>> [ 180.798364] <TASK> >>> [ 180.798366] intel_plane_atomic_check_with_state+0x1fd/0x6ea >>> [ 180.798370] ? intel_plane_atomic_check+0x11b/0x145 >>> [ 180.798373] intel_atomic_check_planes+0x263/0x7ce >>> [ 180.798376] ? drm_atomic_helper_check_modeset+0x189/0x923 >>> [ 180.798380] intel_atomic_check+0x14e4/0x184d >>> [ 180.798382] ? intel_rps_mark_interactive+0x23/0x6a >>> [ 180.798386] drm_atomic_check_only+0x3ec/0x98f >>> [ 180.798391] drm_atomic_commit+0xa2/0x105 >>> [ 180.798394] ? drm_atomic_set_fb_for_plane+0x96/0xa5 >>> [ 180.798397] drm_atomic_helper_update_plane+0xdc/0x11f >>> [ 180.798400] drm_mode_setplane+0x236/0x30c >>> [ 180.798404] ? drm_any_plane_has_format+0x51/0x51 >>> [ 180.798407] drm_ioctl_kernel+0xda/0x14d >>> [ 180.798411] drm_ioctl+0x27e/0x3b4 >>> [ 180.798414] ? drm_any_plane_has_format+0x51/0x51 >>> [ 180.798418] __se_sys_ioctl+0x7a/0xbc >>> [ 180.798421] do_syscall_64+0x55/0x9d >>> [ 180.798424] ? exit_to_user_mode_prepare+0x3c/0x8b >>> [ 180.798427] entry_SYSCALL_64_after_hwframe+0x61/0xcb >>> >>> If this function shouldn't be called in such a case, then perhaps >>> I should revist my original attempt at fixing this in >>> https://patchwork.freedesktop.org/patch/516060 by rejecting such a >>> configuration? >> >> I'm saying that this should be impossible already. At least >> I can't immediately see anything that could call this with >> an invisible plane. > > In my repro case, I called drmModeSetPlane with the src_h parameter set > to 65535 (so the largest 16.16 number that's less than one). This got > through any existing checks on the height of the src rect, resulting in > the divide-by-zero error in intel_plane_pixel_rate. > > While investigating this, I tried setting src_h to 0, but this > configuration got rejected somewhere along the line before it got > through the intel_plane_pixel_rate. > Here's one of the igt tests I had come up when debugging this: --- #include "igt.h" igt_main { igt_subtest_f("testi") { int drm_fd = drm_open_driver_master(DRIVER_INTEL); igt_display_t display; igt_plane_t* plane; igt_output_t *output; igt_fb_t fb; kmstest_set_vt_graphics_mode(); igt_display_require(&display, drm_fd); igt_display_require_output(&display); output = igt_get_single_output_for_pipe(&display, PIPE_A); igt_output_set_pipe(output, PIPE_A); igt_display_commit_atomic(&display, DRM_MODE_ATOMIC_ALLOW_MODESET, NULL); igt_create_pattern_fb(drm_fd, 256, 256, DRM_FORMAT_XRGB8888, DRM_FORMAT_MOD_NONE, &fb); plane = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY); igt_plane_set_position(plane, 0, 0); igt_plane_set_fb(plane, &fb); igt_plane_set_prop_value(plane, IGT_PLANE_SRC_H, IGT_FIXED(0, 30)); igt_plane_set_size(plane, 256, 8); igt_display_commit_atomic(&display, DRM_MODE_ATOMIC_ALLOW_MODESET, NULL); } } --- This should give callstack ending same way as what Drew had seen. With my patch in place this test will die with einval coming from kernel from skl_update_scaler(..) where it doesn't pass range checks (and will note on dmesg about it) /Juha-Pekka >> >>> >>> I'll respond to Alan on that thread. >>> >>>> >>>>> >>>>> Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> >>>>> --- >>>>> drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++++--- >>>>> 1 file changed, 5 insertions(+), 3 deletions(-) >>>>> >>>>> diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c >>>>> index 10e1fc9d0698..a9948e8d3543 100644 >>>>> --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c >>>>> +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c >>>>> @@ -144,7 +144,7 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, >>>>> const struct drm_rect *dst, >>>>> unsigned int rate) >>>>> { >>>>> - unsigned int src_w, src_h, dst_w, dst_h; >>>>> + unsigned int src_w, src_h, dst_w, dst_h, dst_wh; >>>>> >>>>> src_w = drm_rect_width(src) >> 16; >>>>> src_h = drm_rect_height(src) >> 16; >>>>> @@ -155,8 +155,10 @@ unsigned int intel_adjusted_rate(const struct drm_rect *src, >>>>> dst_w = min(src_w, dst_w); >>>>> dst_h = min(src_h, dst_h); >>>>> >>>>> - return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), >>>>> - dst_w * dst_h); >>>>> + /* in case src contained only fractional part */ >>>>> + dst_wh = max(dst_w * dst_h, (unsigned) 1); >>>>> + >>>>> + return DIV_ROUND_UP_ULL(mul_u32_u32(rate, src_w * src_h), dst_wh); >>>>> } >>>>> >>>>> unsigned int intel_plane_pixel_rate(const struct intel_crtc_state *crtc_state, >>>>> -- >>>>> 2.37.3 >>>> >>>> -- >>>> Ville Syrjälä >>>> Intel >> >> -- >> Ville Syrjälä >> Intel ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2023-01-11 21:25 UTC | newest] Thread overview: 16+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-01-04 12:44 [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Juha-Pekka Heikkila 2023-01-04 13:23 ` Jani Nikula 2023-01-04 19:12 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork 2023-01-04 19:39 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2023-01-05 1:54 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 2023-01-08 11:30 ` [Intel-gfx] [PATCH v2] " Juha-Pekka Heikkila 2023-01-10 21:27 ` Imre Deak 2023-01-11 0:31 ` Imre Deak 2023-01-08 13:13 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/display: assume some pixelrate for src smaller than 1 (rev2) Patchwork 2023-01-08 15:57 ` [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: assume some pixelrate for src smaller than 1 (rev3) Patchwork 2023-01-08 17:18 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 2023-01-11 14:19 ` [Intel-gfx] [PATCH] drm/i915/display: assume some pixelrate for src smaller than 1 Ville Syrjälä 2023-01-11 18:28 ` Drew Davenport 2023-01-11 19:39 ` Ville Syrjälä 2023-01-11 20:09 ` Drew Davenport 2023-01-11 21:25 ` Juha-Pekka Heikkila
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox