Bug 68630

Summary: igt/gem_mmap_gtt/access fails
Product: DRI Reporter: lu hua <huax.lu>
Component: DRM/IntelAssignee: Intel GFX Bugs mailing list <intel-gfx-bugs>
Status: CLOSED FIXED QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: All   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
dmesg none

Description lu hua 2013-08-28 02:25:09 UTC

    
Comment 1 lu hua 2013-08-28 02:28:41 UTC
System Environment:
--------------------------
Platform:    Pinveview/Ironlake/Sandybridge/Ivybridge/Haswell
Kernel:      (drm-intel-nightly) aaf0996cd776c5131d108f08a2e587ab0baaa6d4

Bug detailed description:
-----------------------------
It fails on Pinveview/Ironlake/Sandybridge/Ivybridge/Haswell with -queued,-fixes,-nightly kernel. It's a new case.

output:
Test assertion failure function test_access, file gem_mmap_gtt.c:100:
Failed assertion: !mmap64(0, OBJECT_SIZE, PROT_READ | PROT_WRITE, MAP_SHARED, fd2, mmap_arg.offset) && errno == EACCES
Subtest access: FAIL

Reproduce steps:
----------------------------
1. xinit
2. ./gem_mmap_gtt --run-subtest access
Comment 2 lu hua 2013-08-28 02:29:39 UTC
Created attachment 84757 [details]
dmesg
Comment 3 Daniel Vetter 2013-08-28 09:10:58 UTC
Should be fixed now with

commit ca481c9b2a3ae3598453535b8f0369f1f875d52f
Author: David Herrmann <dh.herrmann@gmail.com>
Date:   Sun Aug 25 18:28:58 2013 +0200

    drm/gem: implement vma access management

Note that this patch is only in drm-next, so you can only verify on -nightly. -fixes and dinq will still be broken.
Comment 4 lu hua 2013-08-30 01:51:53 UTC
It still fails on -nightly commit d0af9e5ee868b1e948a1c2488b261762763eabff.
Comment 5 lu hua 2013-08-30 07:31:01 UTC
It still fails on -nightly commit aa7d3c6086b3b9eaab93e69882018be73d80ddb5.
Comment 6 Daniel Vetter 2013-08-30 07:33:11 UTC
Oops, small bug in the testcase, should be fixed with:

commit f73724c3628b61c41642837a709ac8427d699b99
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Fri Aug 30 08:15:32 2013 +0200

    tests/gem_mmap_gtt: fix access checks
Comment 7 lu hua 2013-09-02 06:32:08 UTC
Fixed on -nightly branch.
It still fails on -fixes branch.
ouput:
Test assertion failure function test_access, file gem_mmap_gtt.c:99:
Failed assertion: mmap64(0, OBJECT_SIZE, PROT_READ | PROT_WRITE, MAP_SHARED, fd2, mmap_arg.offset) == MAP_FAILED
Subtest access: FAIL
Comment 8 Daniel Vetter 2013-09-02 07:27:01 UTC
 Subtest access: FAIL(In reply to comment #7)
> Fixed on -nightly branch.
> It still fails on -fixes branch.
> ouput:
> Test assertion failure function test_access, file gem_mmap_gtt.c:99:
> Failed assertion: mmap64(0, OBJECT_SIZE, PROT_READ | PROT_WRITE, MAP_SHARED,
> fd2, mmap_arg.offset) == MAP_FAILED
> Subtest access: FAIL

See my note in comment #3

"Note that this patch is only in drm-next, so you can only verify on -nightly. -fixes and dinq will still be broken."
Comment 9 lu hua 2013-09-02 07:34:58 UTC
Verified.Fixed.
Comment 10 Elizabeth 2017-10-06 14:43:43 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.