Bug 68133

Summary: [Bisected]igt/gem_pread_after_blit/interruptible-display skipped
Product: DRI Reporter: lu hua <huax.lu>
Component: DRM/IntelAssignee: Daniel Vetter <daniel>
Status: CLOSED NOTABUG QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: major    
Priority: high    
Version: unspecified   
Hardware: All   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
dmesg none

Description lu hua 2013-08-15 06:14:11 UTC
Created attachment 84085 [details]
dmesg

System Environment:
--------------------------
Platform:    Pineview/Ironlake/Sandybridge/Ivybridge/Haswell
Kernel:     (drm-intel-fixes)3d51af04567800a87a04785d375f1258977ef6d7

Bug detailed description:
-----------------------------
It skip on Pineview/Ironlake/Sandybridge/Ivybridge/Haswell with -fixes kernel.
It works well on -queued and -nightly kernel.
igt/gem_pread_after_blit/normal-display also skipped.

Bisect shows:7553ad6e10f76aa703359a10e08138e14501d54d is the first bad commit
commit 7553ad6e10f76aa703359a10e08138e14501d54d
Author:     Daniel Vetter <daniel.vetter@ffwll.ch>
AuthorDate: Mon Aug 12 10:43:59 2013 +0200
Commit:     Daniel Vetter <daniel.vetter@ffwll.ch>
CommitDate: Mon Aug 12 11:17:58 2013 +0200

    tests: use drmtest_skip() in caching ioctl helpers

    This way we can rip out all the skip handling from the test control flow,
    and additionally (by using drmtest_retval()) even get correct exit codes.

    The only tricky part is that when we only want ot skip parts of a test
    (like for gem_pread and gem_pwrite) we need to split out those parts as
    subtests. But no addition of control-flow is required, the set/longjmp
    magic in the helpers all makes it happen.

    Also we make extensive use of the behaviour of drmtest_skip to skip
    all subsequent subtests if it is called outside of a subtest. This allows
    us to re-flatten the control flow a lot.

    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>


Test requirement not met in function gem_set_caching, file drmtest.c:421:
Test requirement: (ret == 0)
Subtest interruptible-display: SKIP


Reproduce steps:
----------------------------
1. ./gem_pread_after_blit --run-subtest interruptible-display
Comment 1 Daniel Vetter 2013-08-15 11:08:55 UTC
This is the intended behaviour - display caching support is new in dinq. So this test should skip on -fixes.

Before the igt patch you've bisected to the test simply skipped silently and returned success. So this is a feature, not a bug ;-)

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.