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

[Ville Syrjälä]

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