drm/i915/gem: Remove duplicated call to ops->pread
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 16 Jun 2021 09:03:50 +0000 (11:03 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 17 Jun 2021 08:39:14 +0000 (10:39 +0200)
Between

commit ae30af84edb5b7cc95485922e43afd909a892e1b
Author: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Date:   Tue Mar 23 16:50:00 2021 +0100

    drm/i915: Disable userptr pread/pwrite support.

and

commit 0049b688459b846f819b6e51c24cd0781fcfde41
Author: Matthew Auld <matthew.auld@intel.com>
Date:   Thu Nov 5 15:49:33 2020 +0000

    drm/i915/gem: Allow backends to override pread implementation

this accidentally landed twice.

Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Cc: Matthew Auld <matthew.auld@intel.com
Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210616090350.828696-1-daniel.vetter@ffwll.ch
drivers/gpu/drm/i915/i915_gem.c

index 6a0a3f0..07aa807 100644 (file)
@@ -469,12 +469,6 @@ i915_gem_pread_ioctl(struct drm_device *dev, void *data,
        if (ret != -ENODEV)
                goto out;
 
-       ret = -ENODEV;
-       if (obj->ops->pread)
-               ret = obj->ops->pread(obj, args);
-       if (ret != -ENODEV)
-               goto out;
-
        ret = i915_gem_object_wait(obj,
                                   I915_WAIT_INTERRUPTIBLE,
                                   MAX_SCHEDULE_TIMEOUT);