Commit 90caf425 authored by Danilo Krummrich's avatar Danilo Krummrich Committed by Gerd Hoffmann

drm/virtio: kms: use drm managed resources

Allocate driver structures with drm managed resource allocators in order
to cleanup/simplify the drm driver .release callback.
Signed-off-by: default avatarDanilo Krummrich <dakr@redhat.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20220714130028.2127858-3-dakr@redhat.comSigned-off-by: default avatarGerd Hoffmann <kraxel@redhat.com>
parent 78476288
...@@ -28,6 +28,7 @@ ...@@ -28,6 +28,7 @@
#include <linux/virtio_ring.h> #include <linux/virtio_ring.h>
#include <drm/drm_file.h> #include <drm/drm_file.h>
#include <drm/drm_managed.h>
#include "virtgpu_drv.h" #include "virtgpu_drv.h"
...@@ -66,8 +67,9 @@ static void virtio_gpu_get_capsets(struct virtio_gpu_device *vgdev, ...@@ -66,8 +67,9 @@ static void virtio_gpu_get_capsets(struct virtio_gpu_device *vgdev,
{ {
int i, ret; int i, ret;
bool invalid_capset_id = false; bool invalid_capset_id = false;
struct drm_device *drm = vgdev->ddev;
vgdev->capsets = kcalloc(num_capsets, vgdev->capsets = drmm_kcalloc(drm, num_capsets,
sizeof(struct virtio_gpu_drv_capset), sizeof(struct virtio_gpu_drv_capset),
GFP_KERNEL); GFP_KERNEL);
if (!vgdev->capsets) { if (!vgdev->capsets) {
...@@ -94,7 +96,7 @@ static void virtio_gpu_get_capsets(struct virtio_gpu_device *vgdev, ...@@ -94,7 +96,7 @@ static void virtio_gpu_get_capsets(struct virtio_gpu_device *vgdev,
if (ret == 0 || invalid_capset_id) { if (ret == 0 || invalid_capset_id) {
spin_lock(&vgdev->display_info_lock); spin_lock(&vgdev->display_info_lock);
kfree(vgdev->capsets); drmm_kfree(drm, vgdev->capsets);
vgdev->capsets = NULL; vgdev->capsets = NULL;
spin_unlock(&vgdev->display_info_lock); spin_unlock(&vgdev->display_info_lock);
return; return;
...@@ -126,7 +128,7 @@ int virtio_gpu_init(struct virtio_device *vdev, struct drm_device *dev) ...@@ -126,7 +128,7 @@ int virtio_gpu_init(struct virtio_device *vdev, struct drm_device *dev)
if (!virtio_has_feature(vdev, VIRTIO_F_VERSION_1)) if (!virtio_has_feature(vdev, VIRTIO_F_VERSION_1))
return -ENODEV; return -ENODEV;
vgdev = kzalloc(sizeof(struct virtio_gpu_device), GFP_KERNEL); vgdev = drmm_kzalloc(dev, sizeof(struct virtio_gpu_device), GFP_KERNEL);
if (!vgdev) if (!vgdev)
return -ENOMEM; return -ENOMEM;
...@@ -256,7 +258,6 @@ int virtio_gpu_init(struct virtio_device *vdev, struct drm_device *dev) ...@@ -256,7 +258,6 @@ int virtio_gpu_init(struct virtio_device *vdev, struct drm_device *dev)
vgdev->vdev->config->del_vqs(vgdev->vdev); vgdev->vdev->config->del_vqs(vgdev->vdev);
err_vqs: err_vqs:
dev->dev_private = NULL; dev->dev_private = NULL;
kfree(vgdev);
return ret; return ret;
} }
...@@ -295,9 +296,6 @@ void virtio_gpu_release(struct drm_device *dev) ...@@ -295,9 +296,6 @@ void virtio_gpu_release(struct drm_device *dev)
if (vgdev->has_host_visible) if (vgdev->has_host_visible)
drm_mm_takedown(&vgdev->host_visible_mm); drm_mm_takedown(&vgdev->host_visible_mm);
kfree(vgdev->capsets);
kfree(vgdev);
} }
int virtio_gpu_driver_open(struct drm_device *dev, struct drm_file *file) int virtio_gpu_driver_open(struct drm_device *dev, struct drm_file *file)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment