Bug 68630

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

Created attachment 84757 [details]
dmesg

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.

It still fails on -nightly commit d0af9e5ee868b1e948a1c2488b261762763eabff.

It still fails on -nightly commit aa7d3c6086b3b9eaab93e69882018be73d80ddb5.

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

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

 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."

Verified.Fixed.

Closing old verified.