Bug 84268 - [HSW/BDW]igt/kms_cursor_crc some subcases fail
Summary: [HSW/BDW]igt/kms_cursor_crc some subcases fail
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Paulo Zanoni
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-09-24 08:06 UTC by Guo Jinxian
Modified: 2017-10-06 14:35 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
dmesg (93.25 KB, text/plain)
2014-09-24 08:06 UTC, Guo Jinxian
no flags Details

Description Guo Jinxian 2014-09-24 08:06:04 UTC
Created attachment 106766 [details]
dmesg

==System Environment==
--------------------------
Regression: No.
They are new cases.

Non-working platforms: HSW BDW

==kernel==
--------------------------
origin/drm-intel-nightly: 1c38dbc94bc61e685c72994cc9e634d4eb8c4da1(fails)
    drm-intel-nightly: 2014y-09m-23d-14h-26m-24s UTC integration manifest
origin/drm-intel-next-queued: 9adccc6063d1cf6ba38a5a26b87001554105be18(fails)
    drm/i915: add SW tracking to FBC enabling
origin/drm-intel-fixes: c84db77010877da6c5da119868ed54c43d59e726(fails)
    drm/i915/hdmi: fix hdmi audio state readout

==Bug detailed description==
-----------------------------
igt/kms_cursor_crc some subcases fail

Case lise:
igt/kms_cursor_crc/cursor-128x42-offscreen
igt/kms_cursor_crc/cursor-128x42-onscreen
igt/kms_cursor_crc/cursor-128x42-random
igt/kms_cursor_crc/cursor-128x42-sliding
igt/kms_cursor_crc/cursor-256x85-offscreen
igt/kms_cursor_crc/cursor-256x85-onscreen
igt/kms_cursor_crc/cursor-256x85-random
igt/kms_cursor_crc/cursor-256x85-sliding
igt/kms_cursor_crc/cursor-64x21-offscreen
igt/kms_cursor_crc/cursor-64x21-onscreen
igt/kms_cursor_crc/cursor-64x21-random
igt/kms_cursor_crc/cursor-64x21-sliding
igt/kms_cursor_crc/cursor-128x128-offscreen
igt/kms_cursor_crc/cursor-128x128-onscreen
igt/kms_cursor_crc/cursor-128x128-random
igt/kms_cursor_crc/cursor-128x128-sliding
igt/kms_cursor_crc/cursor-256x256-offscreen
igt/kms_cursor_crc/cursor-256x256-onscreen
igt/kms_cursor_crc/cursor-256x256-random
igt/kms_cursor_crc/cursor-256x256-sliding
igt/kms_cursor_crc/cursor-64x64-offscreen
igt/kms_cursor_crc/cursor-64x64-onscreen
igt/kms_cursor_crc/cursor-64x64-random
igt/kms_cursor_crc/cursor-64x64-sliding

Output:
root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests# ./kms_cursor_crc --run-subtest cursor-128x42-offscreen
IGT-Version: 1.8-g4b81e9c (x86_64) (Linux: 3.17.0-rc6_drm-intel-nightly_1c38db_20140924+ x86_64)
Beginning cursor-128x42-offscreen on pipe A, connector eDP-1
.Test assertion failure function igt_cursor_commit_legacy, file igt_kms.c:1333:
Failed assertion: ret == 0
Last errno: 22, Invalid argument
Subtest cursor-128x42-offscreen: FAIL (0.122s)

==Reproduce steps==
---------------------------- 
1. ./kms_cursor_crc --run-subtest cursor-128x42-offscreen
Comment 1 Guo Jinxian 2014-09-24 08:35:45 UTC
Update case list:
igt/kms_cursor_crc/cursor-128x42-offscreen
igt/kms_cursor_crc/cursor-128x42-onscreen
igt/kms_cursor_crc/cursor-128x42-random
igt/kms_cursor_crc/cursor-128x42-sliding
igt/kms_cursor_crc/cursor-256x85-offscreen
igt/kms_cursor_crc/cursor-256x85-onscreen
igt/kms_cursor_crc/cursor-256x85-random
igt/kms_cursor_crc/cursor-256x85-sliding
igt/kms_cursor_crc/cursor-64x21-offscreen
igt/kms_cursor_crc/cursor-64x21-onscreen
igt/kms_cursor_crc/cursor-64x21-random
igt/kms_cursor_crc/cursor-64x21-sliding

About another cases, here have another bug 84269 to track.
Comment 2 Rodrigo Vivi 2014-09-24 21:23:01 UTC
Arent these 2 duplicate of: https://bugs.freedesktop.org/show_bug.cgi?id=83640 ?

I mean, did you tried that bisected point?
Comment 3 Guo Jinxian 2014-09-28 06:22:09 UTC
(In reply to comment #2)
> Arent these 2 duplicate of:
> https://bugs.freedesktop.org/show_bug.cgi?id=83640 ?
> 
> I mean, did you tried that bisected point?

The bisect result about bug 83640 is invalid. The bug 83640 always exist. which duplicate with bug 81377.

About cases list in this bug we didn't test them before. They should be new cases.
Comment 4 Paulo Zanoni 2014-10-10 20:39:15 UTC
I can't really reproduce this "Invalid argument" error, only the igt_crc_equal() assertion (which is bug 72864). Does this bug still happen for you on drm-intel-nightly?
Comment 5 Guo Jinxian 2014-10-13 02:59:06 UTC
(In reply to Paulo Zanoni from comment #4)
> I can't really reproduce this "Invalid argument" error, only the
> igt_crc_equal() assertion (which is bug 72864). Does this bug still happen
> for you on drm-intel-nightly?

The failure still able to reproduce on latest -nightly(ead7d71357628dc1231535886ec09853e9164d6d)

root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests# ./kms_cursor_crc --run-subtest cursor-128x42-offscreen
IGT-Version: 1.8-gb7d80d1 (x86_64) (Linux: 3.17.0_drm-intel-nightly_ead7d7_20141013+ x86_64)
Beginning cursor-128x42-offscreen on pipe A, connector eDP-1
.Test assertion failure function igt_cursor_commit_legacy, file igt_kms.c:1333:
Failed assertion: ret == 0
Last errno: 22, Invalid argument
Subtest cursor-128x42-offscreen: FAIL (0.136s)
Comment 6 Paulo Zanoni 2014-10-14 17:06:58 UTC
Ok, I rechecked and indeed I can reproduce the bug.

This patch fixes the problem for me: http://patchwork.freedesktop.org/patch/35067/
Comment 7 Paulo Zanoni 2014-10-17 22:03:21 UTC
(In reply to Paulo Zanoni from comment #6)
> Ok, I rechecked and indeed I can reproduce the bug.
> 
> This patch fixes the problem for me:
> http://patchwork.freedesktop.org/patch/35067/

Patch committed to IGT. Closing bug.

When verifying, please make sure your IGT contains the patch.
Comment 8 Guo Jinxian 2014-10-21 07:07:00 UTC
(In reply to Paulo Zanoni from comment #7)
> (In reply to Paulo Zanoni from comment #6)
> > Ok, I rechecked and indeed I can reproduce the bug.
> > 
> > This patch fixes the problem for me:
> > http://patchwork.freedesktop.org/patch/35067/
> 
> Patch committed to IGT. Closing bug.
> 
> When verifying, please make sure your IGT contains the patch.

Test skips on latest igt

root@x-bdw05:/GFX/Test/Intel_gpu_tools/intel-gpu-tools/tests# ./kms_cursor_crc --run-subtest cursor-128x42-offscreen
IGT-Version: 1.8-gbba1cd0 (x86_64) (Linux: 3.18.0-rc1_drm-intel-nightly_f2bcbc_20141021+ x86_64)
Test requirement not met in function run_test_generic, file kms_cursor_crc.c:466:
Test requirement: has_nonsquare_cursors(data->devid)
Subtest cursor-128x42-offscreen: SKIP (0.000s)


commit bba1cd0091b3e8e987f15dac1f38d233b5b0ee78
Author:     Paulo Zanoni <paulo.r.zanoni@intel.com>
AuthorDate: Tue Oct 14 13:52:58 2014 -0300
Commit:     Paulo Zanoni <paulo.r.zanoni@intel.com>
CommitDate: Fri Oct 17 19:00:50 2014 -0300

    tests/kms_cursor_crc: HSW/BDW only have square cursors

    When I look at cursor_size_ok() (from the Kernel's intel_display.c), I
    see that only 845g and i865g support non-square cursors, so SKIP the
    tests on HSW/BDW instead of failing them.

    This problem happened because support for non-square cursors was being
    developed for HSW/BDW, but it ended up not being merged due to a small
    problem and priority changes. Let's make those tests SKIP until we
    have proper support on the Kernel: after that, we can revert this
    patch.

    v2: Improve commit message.

    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=84268
    Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Comment 9 Elizabeth 2017-10-06 14:35:22 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.