Bug 72901 - [SNB/IVB/HSW/BYT/BDW Regression]igt/gem_largeobject fails
Summary: [SNB/IVB/HSW/BYT/BDW Regression]igt/gem_largeobject 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: Ben Widawsky
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-20 08:42 UTC by lu hua
Modified: 2017-10-06 14:40 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:

Attachments

Description lu hua 2013-12-20 08:42:50 UTC 
System Environment:
--------------------------
Platform: Sandybridge/Ivybridge/Haswell
Kernel:  (drm-intel-nightly)074fdf374fa296c16a7449b9d2d4ad6ff73c0c8d

Bug detailed description:
-------------------------
It fails on Sandybridge/Ivybridge/Haswell with ppgtt and nightly branch. It works well on -fixes and -queued kernel.

The latest good commit: 02905525c3310ba673676f8041d0ff6be4da0e24 
The latest bad commit: a7c1d426ef335ccfb6bd567a3f616fa232418fa2 

output:
IGT-Version: 1.5-g2a71bff (x86_64) (Linux: 3.13.0-rc4_drm-intel-nightly_074fdf_20131219+ x86_64)
obj size 134217728
Test assertion failure function test_large_object, file gem_largeobject.c:67:
Last errno: 19, No such device
Failed assertion: ioctl(fd, DRM_IOCTL_I915_GEM_PIN, &pin) == 0

Reproduce steps:
----------------------------
1. ./gem_largeobject
Comment 1 lu hua 2013-12-26 06:30:33 UTC 
It also fails on BYT and BDW.
Comment 2 Ben Widawsky 2013-12-26 21:58:00 UTC 
(In reply to comment #1)
> It also fails on BYT and BDW.

This is almost surely due to 02f6bcccf7c324115747aae2f0addd6af5d321cd. Please revert and try again. We can update the tests if you confirm.
Comment 3 lu hua 2013-12-27 03:09:44 UTC 
(In reply to comment #2)
> (In reply to comment #1)
> > It also fails on BYT and BDW.
> 
> This is almost surely due to 02f6bcccf7c324115747aae2f0addd6af5d321cd.
> Please revert and try again. We can update the tests if you confirm.


Revert this commit, It works well.
Comment 4 Daniel Vetter 2014-01-08 20:14:23 UTC 
I think we should just replace the pin in this test with a gtt mmap. Should have the same effect and so not reduce test coverage really.
Comment 5 Chris Wilson 2014-01-08 21:50:40 UTC 
Or just revert the ABI breakage.
Comment 6 Daniel Vetter 2014-01-10 08:23:44 UTC 
commit 2c9e76184518854f21d367296ababef87fb41243
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Fri Jan 10 09:21:54 2014 +0100

    tests/gem_largeobject: Prefault instead of pin
Comment 7 lu hua 2014-01-13 01:57:20 UTC 
Verified.Fixed.
Comment 8 Elizabeth 2017-10-06 14:40:56 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.