Commit 2f4494b3 authored by Pranjal Ramajor Asha Kanojiya's avatar Pranjal Ramajor Asha Kanojiya Committed by Jeff Hugo
Browse files

accel/qaic: Call drm_gem_create_mmap_offset() once for each BO 



Every time QAIC_MMAP_BO ioctl is called for a BO,
drm_gem_create_mmap_offset() is called. Calling
drm_gem_create_mmap_offset() more then once for a BO seems redundant.

Signed-off-by: default avatarPranjal Ramajor Asha Kanojiya <quic_pkanojiy@quicinc.com>
Reviewed-by: default avatarJeffrey Hugo <quic_jhugo@quicinc.com>
Signed-off-by: default avatarJeffrey Hugo <quic_jhugo@quicinc.com>
Reviewed-by: default avatarJacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20231208163457.1295993-6-quic_jhugo@quicinc.com
parent ce5fed9a

drivers/accel/qaic/qaic_data.c

+6 −4
Original line number Diff line number Diff line
@@ -718,6 +718,10 @@ int qaic_create_bo_ioctl(struct drm_device *dev, void *data, struct drm_file *fi 
	if (ret)

		goto free_bo;



	ret = drm_gem_create_mmap_offset(obj);

	if (ret)

		goto free_bo;



	ret = drm_gem_handle_create(file_priv, obj, &args->handle);

	if (ret)

		goto free_bo;
@@ -745,7 +749,7 @@ int qaic_mmap_bo_ioctl(struct drm_device *dev, void *data, struct drm_file *file 
	struct drm_gem_object *obj;

	struct qaic_device *qdev;

	struct qaic_user *usr;

	int ret;

	int ret = 0;



	usr = file_priv->driver_priv;

	usr_rcu_id = srcu_read_lock(&usr->qddev_lock);
@@ -767,8 +771,6 @@ int qaic_mmap_bo_ioctl(struct drm_device *dev, void *data, struct drm_file *file 
		goto unlock_dev_srcu;

	}



	ret = drm_gem_create_mmap_offset(obj);

	if (ret == 0)

	args->offset = drm_vma_node_offset_addr(&obj->vma_node);



	drm_gem_object_put(obj);