diff options
author | Francois Dugast <francois.dugast@intel.com> | 2023-07-17 11:20:18 +0300 |
---|---|---|
committer | Rodrigo Vivi <rodrigo.vivi@intel.com> | 2023-12-21 19:37:30 +0300 |
commit | b8c1ba831e675005ff871cd4a4e04ff90326b4ae (patch) | |
tree | de3632bd0b6e46956f00b8c302385e2b3fd5adae /drivers/gpu/drm/xe/xe_bo.c | |
parent | 5ce58303440b7efb21c554cb0b6614482aab8fe9 (diff) | |
download | linux-b8c1ba831e675005ff871cd4a4e04ff90326b4ae.tar.xz |
drm/xe: Prevent flooding the kernel log with XE_IOCTL_ERR
Lower log level of XE_IOCTL_ERR macro to debug in order to prevent flooding
kernel log.
v2: Rename XE_IOCTL_ERR to XE_IOCTL_DBG (Rodrigo Vivi)
v3: Rebase
v4: Fix style, remove unrelated change about __FILE__ and __LINE__
Link: https://lists.freedesktop.org/archives/intel-xe/2023-May/004704.html
Signed-off-by: Francois Dugast <francois.dugast@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Diffstat (limited to 'drivers/gpu/drm/xe/xe_bo.c')
-rw-r--r-- | drivers/gpu/drm/xe/xe_bo.c | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/drivers/gpu/drm/xe/xe_bo.c b/drivers/gpu/drm/xe/xe_bo.c index 00b8b5e7f197..1031cb69219d 100644 --- a/drivers/gpu/drm/xe/xe_bo.c +++ b/drivers/gpu/drm/xe/xe_bo.c @@ -1724,35 +1724,35 @@ int xe_gem_create_ioctl(struct drm_device *dev, void *data, u32 handle; int err; - if (XE_IOCTL_ERR(xe, args->extensions) || XE_IOCTL_ERR(xe, args->pad) || - XE_IOCTL_ERR(xe, args->reserved[0] || args->reserved[1])) + if (XE_IOCTL_DBG(xe, args->extensions) || XE_IOCTL_DBG(xe, args->pad) || + XE_IOCTL_DBG(xe, args->reserved[0] || args->reserved[1])) return -EINVAL; - if (XE_IOCTL_ERR(xe, args->flags & + if (XE_IOCTL_DBG(xe, args->flags & ~(XE_GEM_CREATE_FLAG_DEFER_BACKING | XE_GEM_CREATE_FLAG_SCANOUT | xe->info.mem_region_mask))) return -EINVAL; /* at least one memory type must be specified */ - if (XE_IOCTL_ERR(xe, !(args->flags & xe->info.mem_region_mask))) + if (XE_IOCTL_DBG(xe, !(args->flags & xe->info.mem_region_mask))) return -EINVAL; - if (XE_IOCTL_ERR(xe, args->handle)) + if (XE_IOCTL_DBG(xe, args->handle)) return -EINVAL; - if (XE_IOCTL_ERR(xe, !args->size)) + if (XE_IOCTL_DBG(xe, !args->size)) return -EINVAL; - if (XE_IOCTL_ERR(xe, args->size > SIZE_MAX)) + if (XE_IOCTL_DBG(xe, args->size > SIZE_MAX)) return -EINVAL; - if (XE_IOCTL_ERR(xe, args->size & ~PAGE_MASK)) + if (XE_IOCTL_DBG(xe, args->size & ~PAGE_MASK)) return -EINVAL; if (args->vm_id) { vm = xe_vm_lookup(xef, args->vm_id); - if (XE_IOCTL_ERR(xe, !vm)) + if (XE_IOCTL_DBG(xe, !vm)) return -ENOENT; err = xe_vm_lock(vm, &ww, 0, true); if (err) { @@ -1795,15 +1795,15 @@ int xe_gem_mmap_offset_ioctl(struct drm_device *dev, void *data, struct drm_xe_gem_mmap_offset *args = data; struct drm_gem_object *gem_obj; - if (XE_IOCTL_ERR(xe, args->extensions) || - XE_IOCTL_ERR(xe, args->reserved[0] || args->reserved[1])) + if (XE_IOCTL_DBG(xe, args->extensions) || + XE_IOCTL_DBG(xe, args->reserved[0] || args->reserved[1])) return -EINVAL; - if (XE_IOCTL_ERR(xe, args->flags)) + if (XE_IOCTL_DBG(xe, args->flags)) return -EINVAL; gem_obj = drm_gem_object_lookup(file, args->handle); - if (XE_IOCTL_ERR(xe, !gem_obj)) + if (XE_IOCTL_DBG(xe, !gem_obj)) return -ENOENT; /* The mmap offset was set up at BO allocation time. */ |