[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] virtio-gpu: call cleanup mapping function in re
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH] virtio-gpu: call cleanup mapping function in resource destroy |
Date: |
Tue, 29 Nov 2016 04:58:58 +0000 |
On Tue, Nov 29, 2016 at 6:29 AM Li Qiang <address@hidden> wrote:
> If the guest destroy the resource before detach banking, the 'iov'
> and 'addrs' field in resource is not freed thus leading memory
> leak issue. This patch avoid this.
>
> Signed-off-by: Li Qiang <address@hidden>
>
Reviewed-by: Marc-André Lureau <address@hidden>
---
> hw/display/virtio-gpu.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/hw/display/virtio-gpu.c b/hw/display/virtio-gpu.c
> index 60bce94..98dadf2 100644
> --- a/hw/display/virtio-gpu.c
> +++ b/hw/display/virtio-gpu.c
> @@ -28,6 +28,8 @@
> static struct virtio_gpu_simple_resource*
> virtio_gpu_find_resource(VirtIOGPU *g, uint32_t resource_id);
>
> +static void virtio_gpu_cleanup_mapping(struct virtio_gpu_simple_resource
> *res);
> +
> #ifdef CONFIG_VIRGL
> #include <virglrenderer.h>
> #define VIRGL(_g, _virgl, _simple, ...) \
> @@ -358,6 +360,7 @@ static void virtio_gpu_resource_destroy(VirtIOGPU *g,
> struct virtio_gpu_simple_resource
> *res)
> {
> pixman_image_unref(res->image);
> + virtio_gpu_cleanup_mapping(res);
> QTAILQ_REMOVE(&g->reslist, res, next);
> g_free(res);
> }
> --
> 1.8.3.1
>
> --
Marc-André Lureau