Bug 85310 - [IVB Bisected]igt/gem_exec_parse some subcases fail
Summary: [IVB Bisected]igt/gem_exec_parse some subcases fail
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: Other All
: high normal
Assignee: Intel GFX Bugs mailing list
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-22 05:31 UTC by Guo Jinxian
Modified: 2017-10-06 14:34 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
dmesg (86.02 KB, text/plain)
2014-10-22 05:31 UTC, Guo Jinxian
no flags Details

Description Guo Jinxian 2014-10-22 05:31:28 UTC
Created attachment 108218 [details]
dmesg

==System Environment==
--------------------------
Regression: Yes.
Good commit on -fixes: bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9(2014_10_17)

Non-working platforms: IVB HSW

==kernel==
--------------------------
origin/drm-intel-nightly: 1e61c930ec1745ec3cfc77be1cdd023c409de26d(works)
    drm-intel-nightly: 2014y-10m-21d-17h-46m-36s UTC integration manifest
origin/drm-intel-next-queued: f3b190654cbceb0788bfc3b67e4f38ad870dfb84(works)
    drm/i915: add runtime PM get/put call in i915_execlists
origin/drm-intel-fixes: f114040e3ea6e07372334ade75d1ee0775c355e1(fails)
    Linux 3.18-rc1

==Bug detailed description==
igt/gem_exec_parse some subcases fail

Case list:
igt/gem_exec_parse/basic-rejected
igt/gem_exec_parse/batch-without-end
igt/gem_exec_parse/bitmasks
igt/gem_exec_parse/oacontrol-tracking
igt/gem_exec_parse/registers


Output:
[root@x-ivb6 tests]# ./gem_exec_parse --run-subtest basic-rejected
IGT-Version: 1.8-gbba1cd0 (x86_64) (Linux: 3.18.0-rc1_drm-intel-fixes_f11404_20141022+ x86_64)
Test assertion failure function exec_batch, file gem_exec_parse.c:135:
Failed assertion: __gem_execbuf(fd, &execbuf) == expected_ret
error: 0 != -22
Subtest basic-rejected: FAIL (0.000s)

==Reproduce steps==
---------------------------- 
1. ./gem_exec_parse --run-subtest basic-rejected
Comment 1 Paulo Zanoni 2014-10-22 13:15:42 UTC
Can you please bisect this for us?
Comment 2 Guo Jinxian 2014-10-24 07:31:16 UTC
Test passes on latest -fixes(a0d036b074b4a5a933e37fcb9bdd6b3cc80a0387)

[root@x-hsw27 tests]# ./gem_exec_parse --run-subtest basic-rejected
IGT-Version: 1.8-gac20795 (x86_64) (Linux: 3.16.0-rc6_drm-intel-fixes_a0d036_20140723+ x86_64)
Subtest basic-rejected: SUCCESS (0.000s)
[root@x-hsw27 tests]# ./gem_exec_parse --run-subtest basic-rejected
IGT-Version: 1.8-gac20795 (x86_64) (Linux: 3.16.0-rc6_drm-intel-fixes_a0d036_20140723+ x86_64)
Subtest basic-rejected: SUCCESS (0.000s)
[root@x-hsw27 tests]# ./gem_exec_parse --run-subtest batch-without-end
IGT-Version: 1.8-gac20795 (x86_64) (Linux: 3.16.0-rc6_drm-intel-fixes_a0d036_20140723+ x86_64)
Subtest batch-without-end: SUCCESS (0.000s)
[root@x-hsw27 tests]# ./gem_exec_parse --run-subtest bitmasks
IGT-Version: 1.8-gac20795 (x86_64) (Linux: 3.16.0-rc6_drm-intel-fixes_a0d036_20140723+ x86_64)
Subtest bitmasks: SUCCESS (0.000s)
[root@x-hsw27 tests]# ./gem_exec_parse --run-subtest oacontrol-tracking
IGT-Version: 1.8-gac20795 (x86_64) (Linux: 3.16.0-rc6_drm-intel-fixes_a0d036_20140723+ x86_64)
Subtest oacontrol-tracking: SUCCESS (0.000s)
[root@x-hsw27 tests]# ./gem_exec_parse --run-subtest registers
IGT-Version: 1.8-gac20795 (x86_64) (Linux: 3.16.0-rc6_drm-intel-fixes_a0d036_20140723+ x86_64)
Subtest registers: SUCCESS (0.000s)
Comment 3 Guo Jinxian 2014-10-24 08:49:35 UTC
Test passes on HSW, but still fails on IVB. 
Here is the bisect result below:

896ab1a5d54269b463a24194c2e4a369103b46d8 is the first bad commit
commit 6c5566a82c6fb1da9e13a294f23d4cd85a08cb30
Author:     Daniel Vetter <daniel.vetter@ffwll.ch>
AuthorDate: Wed Aug 6 15:04:50 2014 +0200
Commit:     Daniel Vetter <daniel.vetter@ffwll.ch>
CommitDate: Wed Aug 13 14:23:30 2014 +0200

    drm/i915: Allow i915_gem_setup_global_gtt to fail

    We already needs this just as a safety check in case the preallocation
    reservation dance fails. But we definitely need this to be able to
    move tha aliasing ppgtt setup back out of the context code to this
    place, where it belongs.

    Reviewed-by: Michel Thierry <michel.thierry@intel.com>
    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Comment 4 Daniel Vetter 2014-11-14 08:37:44 UTC
(In reply to Guo Jinxian from comment #3)
> Test passes on HSW, but still fails on IVB. 
> Here is the bisect result below:
> 
> 896ab1a5d54269b463a24194c2e4a369103b46d8 is the first bad commit
> commit 6c5566a82c6fb1da9e13a294f23d4cd85a08cb30

So which on is it? 89.. or 6c.. Please double-check.

In any case this is fixed in

commit 9beb0ccb66ec5d011ac13a02d3b1098035195251
Author: Brad Volkin <bradley.d.volkin@intel.com>
Date:   Thu Sep 18 16:26:26 2014 -0700

    drm/i915: Re-enable the command parser when using PPGTT
Comment 5 Guo Jinxian 2014-11-17 03:41:15 UTC
Verified on latest -nightly(3b6da75500fb10e59dfb153eba9845c05f8d82d6)


[root@x-hsw24 tests]# ./gem_exec_parse --run-subtest basic-rejected
IGT-Version: 1.8-gaa63fc7 (x86_64) (Linux: 3.18.0-rc4_drm-intel-nightly_3b6da7_20141116+ x86_64)
Subtest basic-rejected: SUCCESS (0.000s)
Comment 6 Elizabeth 2017-10-06 14:34:32 UTC
Closing old verified.


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.