]> git.karo-electronics.de Git - karo-tx-linux.git/commit
drm/cma-helper: Return ENOENT for "no such gem obj"
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 18 Apr 2017 12:11:20 +0000 (14:11 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 18 Apr 2017 15:57:37 +0000 (17:57 +0200)
commit765393db20e34b8ad9c1c6a71a793d17d23f4aa5
tree1e2ebfff4b5c3b10c8f1ca5f3ed928bd3e76d38a
parentd8f2ebaac650dc35db3bf5cf10e8ee1115b455f8
drm/cma-helper: Return ENOENT for "no such gem obj"

All the error codes we (ab)use are strictly not the right ones (since
they're all for the vfs, and the only thing we're allowed to do from
an ioctl is EINVAL). But ENOENT is the common error code for failed to
look up an object throughout drm, so let's use it in the cma helpers,
too.

Note that this does change uabi for some drivers, but that shouldn't
ba a problem:
- other drivers (e.g. i915) already return ENOENT for invalid gem bo
- userspace tends to only check for errors, not specific ones (except
  when ioctl restarting is needed)
- if the gem bo isn't there a previous step (either creating or
  importing it) failed already, this situation should always be a
  userspace bug.

Changing it does help with debugging userspace bugs though, by making
error reporting more consistent.

Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170418121120.31862-1-daniel.vetter@ffwll.ch
drivers/gpu/drm/drm_fb_cma_helper.c