On 5/5/25 11:59, Manos Pitsidianakis wrote: > diff --git a/drivers/gpu/drm/virtio/virtgpu_kms.c > b/drivers/gpu/drm/virtio/virtgpu_kms.c > index > 7b3c4d314f8eee692e2842a7056d6dc64936fc2f..a8b751179332b9ec2fbba1392a6ee0e638a5192e > 100644 > --- a/drivers/gpu/drm/virtio/virtgpu_kms.c > +++ b/drivers/gpu/drm/virtio/virtgpu_kms.c > @@ -286,6 +286,10 @@ void virtio_gpu_deinit(struct drm_device *dev) > flush_work(&vgdev->cursorq.dequeue_work); > flush_work(&vgdev->config_changed_work); > virtio_reset_device(vgdev->vdev); > + spin_lock(&vgdev->display_info_lock); > + drmm_kfree(dev, vgdev->capsets); > + vgdev->capsets = NULL; > + spin_unlock(&vgdev->display_info_lock);
Isn't this lock superfluous? -- Best regards, Dmitry
