Bug 68133 - [Bisected]igt/gem_pread_after_blit/interruptible-display skipped
Summary: [Bisected]igt/gem_pread_after_blit/interruptible-display skipped
Status: CLOSED NOTABUG
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: high major
Assignee: Daniel Vetter
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-15 06:14 UTC by lu hua
Modified: 2015-05-13 07:56 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
dmesg (70.70 KB, text/plain)
2013-08-15 06:14 UTC, lu hua
no flags Details

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.