[igt] tests/gem_mmap_wc: don't fail if the Kernel is too old for the getparam flag
diff mbox

Message ID 1438797568-9994-1-git-send-email-paulo.r.zanoni@intel.com
State New
Headers show

Commit Message

Paulo Zanoni Aug. 5, 2015, 5:59 p.m. UTC
Just don't check the drmIoctl() return code: the "if (val)" should be
enough to prevent any problems.

v2: Don't SKIP, just proceed (Chris).

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89739
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
---
 tests/gem_mmap_wc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Chris Wilson Aug. 5, 2015, 6:45 p.m. UTC | #1
On Wed, Aug 05, 2015 at 02:59:28PM -0300, Paulo Zanoni wrote:
> Just don't check the drmIoctl() return code: the "if (val)" should be
> enough to prevent any problems.
> 
> v2: Don't SKIP, just proceed (Chris).
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89739
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris

Patch
diff mbox

diff --git a/tests/gem_mmap_wc.c b/tests/gem_mmap_wc.c
index 6e43465..0a5326e 100644
--- a/tests/gem_mmap_wc.c
+++ b/tests/gem_mmap_wc.c
@@ -104,7 +104,7 @@  test_invalid_flags(int fd)
 	gp.value = &val;
 
 	/* Do we have the new mmap_ioctl? */
-	do_ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
+	drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
 
 	if (val >= 1) {
 		/*