[CI][DRMTIP] igt@kms_* - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | (((0xB2)) << 0) | ((((sizeof(struct drm_mode_create_dumb)))) << ((0+8)+8)))), (&create)) == 0
Submitted by Martin Peres @mupuf
Assigned to Stanislav Lisovskiy
Link to original bug (#109495)
Description
https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_197/fi-cfl-8109u/igt@kms_cursor_crc@cursor-64x21-random.html
Starting subtest: cursor-64x21-random
(kms_cursor_crc:1169) igt_kms-CRITICAL: Test assertion failure function kmstest_dumb_create, file ../lib/igt_kms.c:608:
(kms_cursor_crc:1169) igt_kms-CRITICAL: Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | (((0xB2)) << 0) | ((((sizeof(struct drm_mode_create_dumb)))) << ((0+8)+8)))), (&create)) == 0
(kms_cursor_crc:1169) igt_kms-CRITICAL: Last errno: 22, Invalid argument
(kms_cursor_crc:1169) igt_kms-CRITICAL: error: -1 != 0
Activity
added CI feature: display/Other platform: CFL platform: ICL platform: KBL platform: TGL priority::medium severity::normal + 1 deleted label
The CI Bug Log issue associated to this bug has been updated.
New filters associated
* KBL CFL: all tests - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ..., Last errno: 22, Invalid argument
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_190/fi-cfl-8109u/igt@kms_vblank@pipe-c-query-busy.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_190/fi-cfl-8109u/igt@kms_universal_plane@cursor-fb-leak-pipe-a.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_190/fi-cfl-8109u/igt@kms_plane@plane-panning-bottom-right-pipe-a-planes.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_196/fi-cfl-8109u/igt@kms_plane@plane-position-hole-dpms-pipe-c-planes.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_196/fi-cfl-8109u/igt@kms_color@pipe-b-gamma.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_193/fi-cfl-8109u/igt@kms_cursor_crc@cursor-64x21-sliding.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_197/fi-cfl-8109u/igt@kms_vblank@pipe-a-query-forked-busy-hang.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_197/fi-cfl-8109u/igt@kms_cursor_crc@cursor-64x21-offscreen.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_197/fi-cfl-8109u/igt@kms_cursor_crc@cursor-64x21-random.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_197/fi-cfl-8109u/igt@kms_pipe_crc_basic@nonblocking-crc-pipe-c.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_197/fi-cfl-8109u/igt@kms_vblank@pipe-b-ts-continuation-modeset-hang.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_197/fi-kbl-7500u/igt@kms_lease@simple_lease.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_198/fi-cfl-8109u/igt@kms_color@pipe-a-ctm-max.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_198/fi-cfl-8109u/igt@kms_vblank@pipe-b-accuracy-idle.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_198/fi-kbl-7500u/igt@kms_lease@simple_lease.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_201/fi-cfl-8109u/igt@kms_plane_multiple@atomic-pipe-b-tiling-yf.html
- https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_201/fi-cfl-8109u/igt@kms_universal_plane@universal-plane-pipe-a-functional.htmlA CI Bug Log filter associated to this bug has been updated:
KBL CFL: all tests - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ..., Last errno: 22, Invalid argument
SKL KBL CFL: all tests - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ..., Last errno: 22, Invalid argument
New failures caught by the filter:
A CI Bug Log filter associated to this bug has been updated:
SKL KBL CFL: all tests - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ..., Last errno: 22, Invalid argument
SKL KBL CFL: all tests - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ...
No new failures caught with the new filterA CI Bug Log filter associated to this bug has been updated:
SKL KBL CFL: all tests - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ...
SKL KBL CFL ICL: all tests - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ...
New failures caught by the filter:
@l4kshmisaid:(In reply to CI Bug Log from comment 5)
A CI Bug Log filter associated to this bug has been updated:
{- SKL KBL CFL: all tests - fail - Failed assertion: igt_ioctl((fd),
((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ...
-}
{+ SKL KBL CFL ICL: all tests - fail - Failed assertion: igt_ioctl((fd),
((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ...
+}
New failures caught by the filter:
*
https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_352/fi-icl-u4/
igt@kms_vblank@pipe-b-wait-busy.html
*
https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_352/fi-icl-u4/
igt@kms_cursor_crc@pipe-c-cursor-256x256-random.html
Also on ICL.Sure it fails - it is called with size=0, width=0 and height = 0 as args :)
igt_fb-DEBUG: igt_create_fb_with_bo_size(width=0, height=0, format=XR24(0x34325258), modifier=0x0, size=0)
(kms_cursor_crc:1512) igt_kms-CRITICAL: Test assertion failure function kmstest_dumb_create, file ../lib/igt_kms.c:820:
...
(kms_cursor_crc:1512) igt_kms-CRITICAL: Last errno: 22, Invalid argument
(kms_cursor_crc:1512) igt_kms-CRITICAL: error: -1 != 0
And first check in drm_mode_create_dumb immediately fails:
if (!args->width || !args->height || !args->bpp)
<------>``<------>return -EINVAL;(In reply to Stanislav Lisovskiy from comment 8)
So the issue 99% is in test, I will check it
That was wrong. The issue seems might be severe and in both cases happens because
set_fb_on_crtc function in test doesn't get a correct mode, then tries to create fb with all params set to 0, causing above mentioned assertion:
igt_fb-DEBUG: igt_create_fb_with_bo_size(width=0, height=0, format=XR24(0x34325258), modifier=0x0, size=0)
(kms_cursor_crc:1512) igt_kms-CRITICAL: Test assertion failure function kmstest_dumb_create, file ../lib/igt_kms.c:820:
In both cases this happens after probe fails for that connector:
`<7>`[ 30.889091] [drm:intel_dp_detect [i915]] [CONNECTOR:95:DP-2]
`<7>`[ 30.921069] [drm:drm_dp_dpcd_access] Too many retries, giving up. First error: -5
`<7>`[ 30.921079] [drm:drm_helper_probe_single_connector_modes] [CONNECTOR:95:DP-2] status updated from connected to disconnected
`<7>`[ 30.921169] [drm:drm_helper_probe_single_connector_modes] [CONNECTOR:95:DP-2] disconnected
`<7>`[ 30.921598] [drm:drm_helper_probe_single_connector_modes] [CONNECTOR:100:HDMI-A-1]
Also this could be related to this bug I got today:
https://bugs.freedesktop.org/show_bug.cgi?id=111250Update:
With the help of Chamelium, I can now easily reproduce this anytime and currently the issue consists of two separate issues:
- There seems to be some hardware bouncing involved for hpd pins, like very display goind to unplug and plug in a very short time. The kernel seems to not able filter those as those events are not exceeding the required threshold for irq storm detection in a fixed period of time, i.e those anyway get processed like connector going to disconnected and/or connected state quickly.
2) Sometimes connector goes to disconnected state in "unlucky" time, when igt does refresh the output, because currently it refreshes the outputs periodically in igt_output_pipe_select, during that time kernel manages to return some kind of garbage mode, containing all 0, which is then saved by igt and used during framebuffer creation => causing the asserting as drm_mode_create_dumb rejects creation of 0 size, 0 height and width buffer.
IMHO this should have fixes both in kernel(against small very short hpd pin bounces, not detectable by storm detection) and also in igt, as it should not refresh the outputs on the fly during test execution, but use cached version for
outputs, if output happens to be disconnected in the middle of a test, this will at least fail on commit, i.e in a controlled way.- There seems to be some hardware bouncing involved for hpd pins, like very display goind to unplug and plug in a very short time. The kernel seems to not able filter those as those events are not exceeding the required threshold for irq storm detection in a fixed period of time, i.e those anyway get processed like connector going to disconnected and/or connected state quickly.
A CI Bug Log filter associated to this bug has been updated:
SKL KBL CFL ICL: all tests - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ...
SKL KBL CFL ICL TGL: all tests - fail - Failed assertion: igt_ioctl((fd), ((((2U|1U) << (((0+8)+8)+14)) | ((('d')) << (0+8)) | ...
New failures caught by the filter:
The issue seems to be severe as it may cause userspace to crash in a similar way as IGT does. There are seems to be some issue either with the hardware or the way how we handle hpd irqs - as the display seem to sporadically dissapear causing issues if it happens in the middle of a test case. So we definitely need to investigate this further.
