Re: ✗ Fi.CI.BAT: failure for drm/i915: Handle fb->offsets[] and rewrite fb rotation handling to be more generic (v4)

Ville Syrjälä
2016-05-27T11:43:35Z

On Tue, May 03, 2016 at 04:25:20PM -0000, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915: Handle fb->offsets[] and rewrite fb rotation handling to be more generic (v4)
> URL : https://patchwork.freedesktop.org/series/6688/
> State : failure
>
> == Summary ==
>
> Series 6688v1 drm/i915: Handle fb->offsets[] and rewrite fb rotation handling to be more generic (v4)
> http://patchwork.freedesktop.org/api/1.0/series/6688/revisions/1/mbox/
>
> Test gem_exec_flush:
> Subgroup basic-uc-pro-default-interruptible:
> pass -> FAIL (bsw-nuc-2)

These things fail on gmch mahcines all the time.

This one more or less:
https://bugs.freedesktop.org/show_bug.cgi?id=95382

> Subgroup basic-uc-prw-default:
> fail -> PASS (bsw-nuc-2)
> Test kms_flip:
> Subgroup basic-plain-flip:
> incomplete -> PASS (hsw-brixbox)
> Test kms_pipe_crc_basic:
> Subgroup read-crc-pipe-a-frame-sequence:
> pass -> FAIL (ilk-hp8440p)

(kms_pipe_crc_basic:7867) INFO: read-crc-pipe-A-frame-sequence: Testing connector VGA-1 using pipe A
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: VGA-1: set_pipe(A)
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: commit {
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: VGA-1: Selecting pipe A
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: eDP-1: Selecting pipe B
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: VGA-1: SetCrtc pipe A, disabling
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: }
(kms_pipe_crc_basic:7867) DEBUG: Clearing the fb with color (0.00,1.00,0.00)
(kms_pipe_crc_basic:7867) igt-fb-DEBUG: igt_create_fb_with_bo_size(width=1024, height=768, format=0x34325258, tiling=0x0, size=0)
(kms_pipe_crc_basic:7867) igt-fb-DEBUG: igt_create_fb_with_bo_size(handle=1, pitch=4096)
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: A.0: plane_set_fb(56)
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: commit {
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: eDP-1: Selecting pipe A
(kms_pipe_crc_basic:7867) igt-kms-DEBUG: display: eDP-1: SetCrtc pipe A, fb 56, panning (0, 0), mode 1600x900
(kms_pipe_crc_basic:7867) igt-kms-CRITICAL: Test assertion failure function igt_primary_plane_commit_legacy, file igt_kms.c:1919:
(kms_pipe_crc_basic:7867) igt-kms-CRITICAL: Failed assertion: ret == 0
(kms_pipe_crc_basic:7867) igt-kms-CRITICAL: Last errno: 28, No space left on device

After staring at the dmesg for a while I decided that it's a
problem with leaking the forced connector state between tests
and random hotplugs. Hopefully Chris's idea [1] of using sysfs
instead of debugfs for the connector forcing will fix most of these.

[1] https://lists.freedesktop.org/archives/intel-gfx/2016-May/096712.html
>
> bdw-nuci7-2 total:221 pass:209 dwarn:0 dfail:0 fail:0 skip:12
> bdw-ultra total:221 pass:196 dwarn:0 dfail:0 fail:0 skip:25
> bsw-nuc-2 total:220 pass:176 dwarn:0 dfail:0 fail:3 skip:41
> byt-nuc total:220 pass:175 dwarn:0 dfail:0 fail:4 skip:41
> hsw-brixbox total:221 pass:195 dwarn:0 dfail:0 fail:0 skip:26
> hsw-gt2 total:221 pass:199 dwarn:0 dfail:0 fail:1 skip:21
> ilk-hp8440p total:221 pass:157 dwarn:0 dfail:0 fail:3 skip:61
> ivb-t430s total:221 pass:190 dwarn:0 dfail:0 fail:0 skip:31
> skl-i7k-2 total:221 pass:194 dwarn:0 dfail:0 fail:0 skip:27
> skl-nuci5 total:221 pass:210 dwarn:0 dfail:0 fail:0 skip:11
> snb-dellxps total:221 pass:179 dwarn:0 dfail:0 fail:0 skip:42
> snb-x220t total:221 pass:179 dwarn:0 dfail:0 fail:1 skip:41
>
> Results at /archive/results/CI_IGT_test/Patchwork_2132/
>
> e10fc3de61f519461c4a05c9ef28c67b14f297b0 drm-intel-nightly: 2016y-05m-03d-13h-50m-23s UTC integration manifest
> b7a0248 drm/i915: Make sure fb offset is (macro)pixel aligned
> eb345c5 drm/i915: Deal with NV12 CbCr plane AUX surface on SKL+
> eb20e0d drm/i915: Compute display surface offset in the plane check hook for SKL+
> 14e5ba0 drm/i915: Make intel_adjust_tile_offset() work for linear buffers
> df8e4dd drm/i915: Allow calling intel_adjust_tile_offset() multiple times
> c886764 drm/i915: Limit fb x offset due to fences
> 2ab89d9 drm/i915: Adjust obj tiling vs. fb modifier rules
> c52138e drm/i915: Use fb modifiers for display tiling decisions
> e197f8e drm/i915: Pass around plane_state instead of fb+rotation
> b781379 drm/i915: Move SKL hw stride calculation into a helper
> c473f16 drm/i915: Don't pass pitch to intel_compute_page_offset()
> b4e52ec drm/i915: Rewrite fb rotation GTT handling

--
Ville Syrjälä
Intel OTC