| Message ID | 1438797568-9994-1-git-send-email-paulo.r.zanoni@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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
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) { /*
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(-)