Commit 1bc92a51 authored by Jacek Lawrynowicz's avatar Jacek Lawrynowicz
Browse files

accel/ivpu: Remove invalid warnings 



Warn in ivpu_file_priv_put() checks a pointer that is previously
accessed. Warn in ivpu_ipc_fini() can be triggered even in valid cases
where IPC is disabled upon closing the device.

Reviewed-by: default avatarKarol Wachowski <karol.wachowski@intel.com>
Reviewed-by: default avatarJeffrey Hugo <quic_jhugo@quicinc.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240930195322.461209-25-jacek.lawrynowicz@linux.intel.com


Signed-off-by: default avatarJacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>
parent cc3c72c7

drivers/accel/ivpu/ivpu_drv.c

+0 −2
Original line number Diff line number Diff line
@@ -117,8 +117,6 @@ void ivpu_file_priv_put(struct ivpu_file_priv **link) 
	struct ivpu_file_priv *file_priv = *link;

	struct ivpu_device *vdev = file_priv->vdev;



	drm_WARN_ON(&vdev->drm, !file_priv);



	ivpu_dbg(vdev, KREF, "file_priv put: ctx %u refcount %u\n",

		 file_priv->ctx.id, kref_read(&file_priv->ref));

drivers/accel/ivpu/ivpu_ipc.c

+0 −1
Original line number Diff line number Diff line
@@ -533,7 +533,6 @@ void ivpu_ipc_fini(struct ivpu_device *vdev) 
{

	struct ivpu_ipc_info *ipc = vdev->ipc;



	drm_WARN_ON(&vdev->drm, ipc->on);

	drm_WARN_ON(&vdev->drm, !list_empty(&ipc->cons_list));

	drm_WARN_ON(&vdev->drm, !list_empty(&ipc->cb_msg_list));

	drm_WARN_ON(&vdev->drm, atomic_read(&ipc->rx_msg_count) > 0);