Bug 104669 - [CI] igt@gem_tiled_fence_blits@normal - CRASH
Summary: [CI] igt@gem_tiled_fence_blits@normal - CRASH
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: DRI git
Hardware: Other All
: medium normal
Assignee: Marta Löfstedt
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard: ReadyForDev
Keywords:
Depends on:
Blocks:
 
Reported: 2018-01-17 07:18 UTC by Marta Löfstedt
Modified: 2018-01-18 07:36 UTC (History)
1 user (show)

See Also:
i915 platform: BXT, KBL
i915 features: GEM/Other


Attachments

Description Marta Löfstedt 2018-01-17 07:18:14 UTC
https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4146/shard-apl6/igt@gem_tiled_fence_blits@normal.html
https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4146/shard-kbl3/igt@gem_tiled_fence_blits@normal.html

Subtest normal: CRASH (2.798s)
	

Received signal SIGSEGV.
Stack trace: 
 #0 [fatal_sig_handler+0xfb]
 #1 [killpg+0x40]
 #2 [run_test+0x157]
 #3 [<unknown>+0x157]
Received signal SIGSEGV.
Stack trace: 
 #0 [fatal_sig_handler+0xfb]
 #1 [killpg+0x40]
 #2 [<unknown>+0x40]
Comment 1 Marta Löfstedt 2018-01-17 08:12:24 UTC
test prints:Using 6143 1MiB buffers, i.e. count = 6143, however:


"static void run_test (int fd, int count)
{
        drm_intel_bo *bo[4096];
        uint32_t bo_start_val[4096];"

bo is pnly 4096.

so first loop goes boom:

"    for (i = 0; i < count; i++) {
                bo[i] = create_bo(fd, start);
"
Comment 2 Marta Löfstedt 2018-01-17 12:36:33 UTC
https://patchwork.freedesktop.org/patch/198256/
Comment 3 Chris Wilson 2018-01-17 12:40:46 UTC
commit 319a1de94c363863d70b1ec8950d55894a903cbb (HEAD, upstream/master)
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Wed Jan 17 11:45:30 2018 +0000

    igt/gem_tiled_fence_blits: Allocate bo array
    
    As we allow more buffers to be allocated to fill larger apertures, we
    may exceed the static allocation of 4096 buffers.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104669
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Comment 4 Marta Löfstedt 2018-01-17 12:46:52 UTC
Thanks Chris, I will close once the results are in.
Comment 5 Marta Löfstedt 2018-01-18 07:36:49 UTC
Fixed used from CI_DRM_3641/, test is green


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.