Bug 74953 - igt/gem_madvise/dontneed-before-pwrite fails on -fixes kernel
Summary: igt/gem_madvise/dontneed-before-pwrite fails on -fixes kernel
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: medium normal
Assignee: Daniel Vetter
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-14 01:40 UTC by lu hua
Modified: 2017-10-06 14:40 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:

Attachments
dmesg (99.18 KB, text/plain)
2014-02-14 01:40 UTC, lu hua 		no flags Details
View All

Description lu hua 2014-02-14 01:40:21 UTC 
Created attachment 94035 [details]
dmesg

System Environment:
--------------------------
Platform: Pineview/Ironlake/Sandybridge/Ivybridge/Haswell/
Kernel: (drm-intel-fixes) 2f589112609b0a964b3d78c99c0f3a83ac16add6

Bug detailed description:
-----------------------------
New case igt/gem_madvise/dontneed-before-pwrite fails on all platforms with -fixes kernel. It works well on -nightly kernel.

output:
IGT-Version: 1.5-g9597836 (x86_64) (Linux: 3.13.0-rc8_drm-intel-fixes_2f5891_20140214+ x86_64)
Test assertion failure function dontneed_before_pwrite, file gem_madvise.c:111:
Last errno: 22, Invalid argument
Failed assertion: errno == EFAULT
Subtest dontneed-before-pwrite: FAIL

Reproduce steps:
----------------------------
1. ./gem_madvise --run-subtest dontneed-before-pwrite
Comment 1 Daniel Vetter 2014-02-14 08:05:27 UTC 
This test just checks for a bit of api clarification we've done in dinq with

commit 8c99e57d3926959dd940e834da6fa707601ba7e5
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Fri Jan 31 11:34:58 2014 +0000

    drm/i915: Treat using a purged buffer as a source of EFAULT

-fixes obviously still has the old behaviour. Since this is just userspace/kernel interface cleanup no need to backport (as long as -nightly works well ofc).
Comment 2 lu hua 2014-03-12 03:28:36 UTC 
It still fails on latest -fixes kernel.
Comment 3 Daniel Vetter 2014-03-27 20:58:57 UTC 
(In reply to comment #2)
> It still fails on latest -fixes kernel.

And I explained in comment #1 that this is fully expected and totally ok. You only need to verify on dinq.
Comment 4 lu hua 2014-05-06 07:38:24 UTC 
Verified.Fixed.
Comment 5 Elizabeth 2017-10-06 14:40:06 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.