accel/qaic: Fix GEM import path code
authorPranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
Fri, 8 Dec 2023 16:31:00 +0000 (09:31 -0700)
committerJeffrey Hugo <quic_jhugo@quicinc.com>
Fri, 15 Dec 2023 17:38:56 +0000 (10:38 -0700)
Do not modify the size of dmabuf as it is immutable.

Fixes: ff13be830333 ("accel/qaic: Add datapath")
Signed-off-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
Reviewed-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
Signed-off-by: Jeffrey Hugo <quic_jhugo@quicinc.com>
Reviewed-by: Jacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20231208163101.1295769-2-quic_jhugo@quicinc.com
drivers/accel/qaic/qaic_data.c

index 4a8e43a7a6a408b89d1c5bf67cfde9ad458a06f2..d42f002bc0cffaa69cc6203f9fc2a0e1b73e26a3 100644 (file)
@@ -777,7 +777,6 @@ struct drm_gem_object *qaic_gem_prime_import(struct drm_device *dev, struct dma_
        struct dma_buf_attachment *attach;
        struct drm_gem_object *obj;
        struct qaic_bo *bo;
-       size_t size;
        int ret;
 
        bo = qaic_alloc_init_bo();
@@ -795,13 +794,12 @@ struct drm_gem_object *qaic_gem_prime_import(struct drm_device *dev, struct dma_
                goto attach_fail;
        }
 
-       size = PAGE_ALIGN(attach->dmabuf->size);
-       if (size == 0) {
+       if (!attach->dmabuf->size) {
                ret = -EINVAL;
                goto size_align_fail;
        }
 
-       drm_gem_private_object_init(dev, obj, size);
+       drm_gem_private_object_init(dev, obj, attach->dmabuf->size);
        /*
         * skipping dma_buf_map_attachment() as we do not know the direction
         * just yet. Once the direction is known in the subsequent IOCTL to