Bug 111379 - [CI][DRMTIP] igt@gem_eio@unwedge-stress - fail - Failed assertion: __gem_execbuf(fd, execbuf) == 0
Summary: [CI][DRMTIP] igt@gem_eio@unwedge-stress - fail - Failed assertion: __gem_exec...
Status: RESOLVED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: DRI git
Hardware: Other All
: medium normal
Assignee: Intel GFX Bugs mailing list
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard: ReadyForDev
Keywords:
Depends on:
Blocks:
 
Reported: 2019-08-12 11:48 UTC by Lakshmi
Modified: 2019-08-13 15:14 UTC (History)
1 user (show)

See Also:
i915 platform: BYT
i915 features: GEM/Other


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lakshmi 2019-08-12 11:48:51 UTC
https://intel-gfx-ci.01.org/tree/drm-tip/drmtip_340/fi-byt-j1900/igt@gem_eio@unwedge-stress.html

Starting subtest: unwedge-stress
(gem_eio:1149) ioctl_wrappers-CRITICAL: Test assertion failure function gem_execbuf, file ../lib/ioctl_wrappers.c:609:
(gem_eio:1149) ioctl_wrappers-CRITICAL: Failed assertion: __gem_execbuf(fd, execbuf) == 0
(gem_eio:1149) ioctl_wrappers-CRITICAL: error: -5 != 0
Subtest unwedge-stress failed.
Comment 2 Chris Wilson 2019-08-12 15:09:04 UTC
Hmm,

<7> [297.642048] hangcheck bcs0
<7> [297.642063] hangcheck 	Awake? 4
<7> [297.642074] hangcheck 	Hangcheck: 8000 ms ago
<7> [297.642084] hangcheck 	Reset count: 0 (global 294)
<7> [297.642094] hangcheck 	Requests:
<7> [297.642119] hangcheck 		active  6:12ce2*  @ 8512ms: gem_eio[1149]
<7> [297.642130] hangcheck 		ring->start:  0x007f1000
<7> [297.642140] hangcheck 		ring->head:   0x00003890
<7> [297.642149] hangcheck 		ring->tail:   0x00002a50
<7> [297.642158] hangcheck 		ring->emit:   0x00002a50
<7> [297.642168] hangcheck 		ring->space:  0x00000e00
<7> [297.642177] hangcheck 		ring->hwsp:   0x007eb100

that ring->space suggests we blocked waiting for space (and so hung before we ourselves performed the reset).

reset_stress() expects to submit 21 batches without a problem, that might have been optimistic.
Comment 3 Chris Wilson 2019-08-13 15:14:37 UTC
Should be

commit d62008b45c735c32c5e52dd14c5ad2a4ba57feb5 (upstream/master, origin/master, 
origin/HEAD)
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Mon Aug 12 16:10:49 2019 +0100

    i915/gem_eio: Restrict number of batches of submitted
    
    Make sure we don't block while setting up the stress case before the
    reset by only submitting less batches than would fill the ring.
    
    References: https://bugs.freedesktop.org/show_bug.cgi?id=111379
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Reviewed-by: Andi Shyti <andi.shyti@intel.com>


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.