drm/Xe: Use EOPNOTSUPP instead of ENOTSUPP
authorJanga Rahul Kumar <janga.rahul.kumar@intel.com>
Tue, 13 Jun 2023 09:37:40 +0000 (15:07 +0530)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Thu, 21 Dec 2023 16:34:42 +0000 (11:34 -0500)
ENOTSUPP is not a standard Unix error should use
EOPNOTSUPP instead.

v2: Update commit description (Aravind)

Reviewed-by: Aravind Iddamsetty <aravind.iddamsetty@intel.com>
Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/xe/xe_engine.c
drivers/gpu/drm/xe/xe_mmio.c
drivers/gpu/drm/xe/xe_sync.c
drivers/gpu/drm/xe/xe_vm.c
drivers/gpu/drm/xe/xe_wait_user_fence.c

index fd39da85944264dc6a17bb4cb74415649a86d160..49d92f089242c35b9179c3098fa17f27a1793d6e 100644 (file)
@@ -612,7 +612,7 @@ int xe_engine_create_ioctl(struct drm_device *dev, void *data,
 
        if (XE_IOCTL_ERR(xe, e->vm && xe_vm_in_compute_mode(e->vm) !=
                         !!(e->flags & ENGINE_FLAG_COMPUTE_MODE))) {
-               err = -ENOTSUPP;
+               err = -EOPNOTSUPP;
                goto put_engine;
        }
 
index f7a7f996b37f0cb6813324eae3a3c6537bc16f55..f1336803b91578c77565e06ccc63c130b1691258 100644 (file)
@@ -495,7 +495,7 @@ int xe_mmio_ioctl(struct drm_device *dev, void *data,
                        fallthrough;
                case DRM_XE_MMIO_8BIT: /* TODO */
                case DRM_XE_MMIO_16BIT: /* TODO */
-                       ret = -ENOTSUPP;
+                       ret = -EOPNOTSUPP;
                        goto exit;
                }
        }
@@ -513,7 +513,7 @@ int xe_mmio_ioctl(struct drm_device *dev, void *data,
                        fallthrough;
                case DRM_XE_MMIO_8BIT: /* TODO */
                case DRM_XE_MMIO_16BIT: /* TODO */
-                       ret = -ENOTSUPP;
+                       ret = -EOPNOTSUPP;
                }
        }
 
index 5acb37a8b2ab1e29e52d4311f6db0f78410aac76..7786b908a3fda0e93160881e59883add2a0c04bd 100644 (file)
@@ -120,7 +120,7 @@ int xe_sync_entry_parse(struct xe_device *xe, struct xe_file *xef,
        switch (sync_in.flags & SYNC_FLAGS_TYPE_MASK) {
        case DRM_XE_SYNC_SYNCOBJ:
                if (XE_IOCTL_ERR(xe, no_dma_fences && signal))
-                       return -ENOTSUPP;
+                       return -EOPNOTSUPP;
 
                if (XE_IOCTL_ERR(xe, upper_32_bits(sync_in.addr)))
                        return -EINVAL;
@@ -138,7 +138,7 @@ int xe_sync_entry_parse(struct xe_device *xe, struct xe_file *xef,
 
        case DRM_XE_SYNC_TIMELINE_SYNCOBJ:
                if (XE_IOCTL_ERR(xe, no_dma_fences && signal))
-                       return -ENOTSUPP;
+                       return -EOPNOTSUPP;
 
                if (XE_IOCTL_ERR(xe, upper_32_bits(sync_in.addr)))
                        return -EINVAL;
@@ -173,7 +173,7 @@ int xe_sync_entry_parse(struct xe_device *xe, struct xe_file *xef,
 
        case DRM_XE_SYNC_USER_FENCE:
                if (XE_IOCTL_ERR(xe, !signal))
-                       return -ENOTSUPP;
+                       return -EOPNOTSUPP;
 
                if (XE_IOCTL_ERR(xe, sync_in.addr & 0x7))
                        return -EINVAL;
index b8a0fe24d1d0c67d83cd33f9a472a5ef3d689000..6edac7d4af870c71932a675e60a6464689f844ca 100644 (file)
@@ -1819,10 +1819,10 @@ static int vm_set_error_capture_address(struct xe_device *xe, struct xe_vm *vm,
                return -EINVAL;
 
        if (XE_IOCTL_ERR(xe, !(vm->flags & XE_VM_FLAG_ASYNC_BIND_OPS)))
-               return -ENOTSUPP;
+               return -EOPNOTSUPP;
 
        if (XE_IOCTL_ERR(xe, vm->async_ops.error_capture.addr))
-               return -ENOTSUPP;
+               return -EOPNOTSUPP;
 
        vm->async_ops.error_capture.mm = current->mm;
        vm->async_ops.error_capture.addr = value;
@@ -3072,7 +3072,7 @@ int xe_vm_bind_ioctl(struct drm_device *dev, void *data, struct drm_file *file)
 
        if (VM_BIND_OP(bind_ops[0].op) == XE_VM_BIND_OP_RESTART) {
                if (XE_IOCTL_ERR(xe, !(vm->flags & XE_VM_FLAG_ASYNC_BIND_OPS)))
-                       err = -ENOTSUPP;
+                       err = -EOPNOTSUPP;
                if (XE_IOCTL_ERR(xe, !err && args->num_syncs))
                        err = EINVAL;
                if (XE_IOCTL_ERR(xe, !err && !vm->async_ops.error))
@@ -3096,7 +3096,7 @@ int xe_vm_bind_ioctl(struct drm_device *dev, void *data, struct drm_file *file)
 
        if (XE_IOCTL_ERR(xe, !vm->async_ops.error &&
                         async != !!(vm->flags & XE_VM_FLAG_ASYNC_BIND_OPS))) {
-               err = -ENOTSUPP;
+               err = -EOPNOTSUPP;
                goto put_engine;
        }
 
index 6c8a60c6008716e519c5fb28f2255042e7b8d8e0..3122374341d6aaa5ba7bb436217715e4b25cbf9e 100644 (file)
@@ -143,7 +143,7 @@ int xe_wait_user_fence_ioctl(struct drm_device *dev, void *data,
 
                if (XE_IOCTL_ERR(xe, !vm->async_ops.error_capture.addr)) {
                        xe_vm_put(vm);
-                       return -ENOTSUPP;
+                       return -EOPNOTSUPP;
                }
 
                addr = vm->async_ops.error_capture.addr;