Bug 78936 - [BDW Regression]igt/gem_render_copy_redux fails
Summary: [BDW Regression]igt/gem_render_copy_redux fails
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: Other All
: high normal
Assignee: Jesse Barnes
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on: 78935
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-20 05:04 UTC by Guo Jinxian
Modified: 2017-09-04 10:24 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
dmesg (86.19 KB, text/plain)
2014-05-20 05:04 UTC, Guo Jinxian
no flags Details

Description Guo Jinxian 2014-05-20 05:04:27 UTC
Created attachment 99377 [details]
dmesg

==System Environment==
--------------------------
Regression: Yes. 
Good commit on -next-queued(b7c0d9df97c10ec5693a838df2fd53058f8e9e96)

Non-working platforms: BDW

==kernel==
--------------------------
-nightly: f79ba79cf037eea9ee757ad37730b00f43d5ef80 (fails)
-queued: d3b448d9917a3d6531e499d88bfb13ea5e31e4ad (fails)
    Author: Chris Wilson <chris@chris-wilson.co.uk>
    Date:   Fri May 16 18:59:00 2014 +0100

    drm/i915: Only unpin the default ctx object if it exists

    Since commit 691e6415c891b8b2b082a120b896b443531c4d45
    Author: Chris Wilson <chris@chris-wilson.co.uk>
    Date:   Wed Apr 9 09:07:36 2014 +0100

        drm/i915: Always use kref tracking for all contexts.

    we have contexts everywhere, and so we must be careful to distinguish
    fake contexts, which do not have an associated bo, and real ones, which
    do. In particular, we now need to be careful not to dereference NULL
    pointers.

    This is one such example, as the commit highlighted above failed to move
    the unpinning of the default ctx object into the real-context-only
    branch.

    Reported-by: Daniel Vetter <daniel.vetter@ffwll.ch>
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78792
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
    Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
    Cc: Ben Widawsky <benjamin.widawsky@intel.com>
    Cc: Mika Kuoppala <mika.kuoppala@intel.com>
    Cc: Jani Nikula <jani.nikula@intel.com>
    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

-fixes: e95a2f7509f5219177d6821a0a8754f93892ca56 (works)
    Author: Ville Syrjälä <ville.syrjala@linux.intel.com>
    Date:   Thu May 8 15:09:19 2014 +0300

    drm/i915: Increase WM memory latency values on SNB

    On SNB the BIOS provided WM memory latency values seem insufficient to
    handle high resolution displays.

    In this particular case the display mode was a 2560x1440@60Hz, which
    makes the pixel clock 241.5 MHz. It was empirically found that a memory
    latency value if 1.2 usec is enough to avoid underruns, whereas the BIOS
    provided value of 0.7 usec was clearly too low. Incidentally 1.2 usec
    is what the typical BIOS provided values are on IVB systems.

    Increase the WM memory latency values to at least 1.2 usec on SNB.
    Hopefully this won't have a significant effect on power consumption.

    v2: Increase the latency values regardless of the pixel clock

    Cc: Robert N <crshman@gmail.com>
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70254
    Tested-by: Robert Navarro <crshman@gmail.com>
    Tested-by: Vitaly Minko <vitaly.minko@gmail.com>
    Cc: stable@vger.kernel.org
    Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
    Signed-off-by: Jani Nikula <jani.nikula@intel.com>

==Bug detailed description==
-----------------------------
igt/gem_render_copy_redux fails

Output:
./gem_render_copy_redux
IGT-Version: 1.6-gd71add5 (x86_64) (Linux: 3.15.0-rc3_drm-intel-nightly_f79ba7_20140519+ x86_64)
Test assertion failure function scratch_buf_check, file gem_render_copy_redux.c:133:
Last errno: 0, Success
Failed assertion: val == color
Expected 0xffff00ff, found 0xfff0ff00 at (502,502)
Subtest normal: FAIL
Test assertion failure function scratch_buf_check, file gem_render_copy_redux.c:133:
Last errno: 0, Success
Failed assertion: val == color
Expected 0xffff00ff, found 0xfff0ff00 at (502,502)
Subtest interruptible: FAIL
Test assertion failure function scratch_buf_check, file gem_render_copy_redux.c:133:
Last errno: 0, Success
Failed assertion: val == color
Expected 0xffff00ff, found 0xfff0ff00 at (502,502)
Subtest flink: FAIL
gem_render_copy_redux: igt_core.c:769: __igt_fork_helper: Assertion `!proc->running' failed.
Aborted

==Reproduce steps==
---------------------------- 
1. ./gem_render_copy_redux
Comment 1 Chris Wilson 2014-05-20 06:25:20 UTC
This depends upon gem_render_copy working first.
Comment 2 Guo Jinxian 2014-05-20 08:43:00 UTC
Bisected was blocked by Bug 78274.
Comment 3 Ben Widawsky 2014-05-28 23:09:32 UTC
Please test:
http://patchwork.freedesktop.org/patch/26784/
Comment 4 Ben Widawsky 2014-05-29 06:34:23 UTC
I changed machines and am no longer seeing this test passing. Please ignore the request to test for now.
Comment 5 Ben Widawsky 2014-06-03 21:56:39 UTC
Please test: IGT patch http://patchwork.freedesktop.org/patch/27088/
Comment 6 Guo Jinxian 2014-06-04 01:37:24 UTC
Test on latest -nightly(455a8fc4304af51a913e33763b72dd2849c11d0c) use igt with this patch. the result was pass.

Output:
./gem_render_copy_redux
IGT-Version: 1.6-g3c70e6a (x86_64) (Linux: 3.15.0-rc7_drm-intel-nightly_455a8f_20140603+ x86_64)
Subtest normal: SUCCESS
Subtest interruptible: SUCCESS
Subtest flink: SUCCESS
Subtest flink-interruptible: SUCCESS
Comment 7 Guo Jinxian 2014-06-12 01:29:02 UTC
Verified on latest -nightly(8148006fc6cc97fea2b792c91c0968a038612355)
Comment 8 Jari Tahvanainen 2017-09-04 10:24:16 UTC
Closing old verified+fixed.


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.