Bug 75131 - [PNV Regression]igt/kms_cursor_crc fails
Summary: [PNV Regression]igt/kms_cursor_crc fails
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: high major
Assignee: Damien Lespiau
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-18 06:14 UTC by lu hua
Modified: 2017-10-06 14:39 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
dmesg (78.34 KB, text/plain)
2014-02-18 06:14 UTC, lu hua
no flags Details

Description lu hua 2014-02-18 06:14:26 UTC
Created attachment 94258 [details]
dmesg

System Environment:
--------------------------
Platform: Pieview
kernel:   (drm-intel-nightly)1be8f2b4dd6d3db00af24d4891c82d2650bd282d

Bug detailed description:
---------------------------
It fails on Pineview with -fixes and -nightly kernel.

The latest known good commit: 22533b1eb038dbf4d3dc79945b81cefc4aac64be
The latest known bad commit: 959783684e2010326d91db2aa55f424f7e041c84

run ./kms_cursor_crc --run-subtest cursor-black-invisible-offscreen
output:
IGT-Version: 1.5-g9597836 (i686) (Linux: 3.13.0_drm-intel-nightly_1be8f2_20140218+ i686)
Test assertion failure function igt_wait_for_vblank, file igt_display.c:40:
Last errno: 22, Invalid argument
Failed assertion: drmWaitVBlank(drm_fd, &wait_vbl) == 0
Subtest cursor-black-invisible-offscreen: FAIL
Comment 1 Ville Syrjala 2014-02-18 09:37:21 UTC
Based on a hunch I tested the following commit on my gen2:

commit d435829ae9702c9f777b68227e43eafd702a8031
Author: Damien Lespiau <damien.lespiau@intel.com>
Date:   Wed Feb 5 17:14:12 2014 +0000

    kms_cursor_crc: Port the test to the new modeset API
    
    Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>

and it does look like the culprit. My gen4 still seems to pass, so the problem might be related to the limited pipe->LVDS/VGA connection options on gen2.

Assigning this one to Damien.
Comment 2 Damien Lespiau 2014-02-18 15:24:57 UTC
Oh my, that means I need to setup the PNV I have around.
Comment 3 Daniel Vetter 2014-03-03 14:31:19 UTC
Simply restrict lvds/edp to pipe B for testing ;-)
Comment 4 Damien Lespiau 2014-03-03 14:36:37 UTC
I think I don't handle correctly the error when the encoder doesn't have the pipe in possible_crtcs in that new code.
Comment 5 Ville Syrjala 2014-04-25 13:28:01 UTC
I believe this should be fixed by this:

commit a461515b334ebb7782ee15e31f943e7a6753d845
Author: Ville Syrjälä <ville.syrjala@linux.intel.com>
Date:   Fri Apr 25 15:27:57 2014 +0300

    kms_cursor_crc: Fix the test on platforms where the pipe->port mapping has restrictions

Please re-test.
Comment 6 lu hua 2014-04-28 02:57:32 UTC
It has been fixed.
Comment 7 lu hua 2014-04-28 02:57:50 UTC
Verified.Fixed.
Comment 8 Elizabeth 2017-10-06 14:39:54 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.