lkml.org 
[lkml]   [2017]   [Mar]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] drm: virtio: use kmem_cache
From
Date
On 03/01/2017, 03:09 PM, Gerd Hoffmann wrote:
> @@ -128,12 +96,9 @@ void virtio_gpu_free_vbufs(struct virtio_gpu_device *vgdev)
> {
> struct virtio_gpu_vbuffer *vbuf;
>
> - spin_lock(&vgdev->free_vbufs_lock);
> - BUG_ON(list_empty(&vgdev->free_vbufs));
> - vbuf = list_first_entry(&vgdev->free_vbufs,
> - struct virtio_gpu_vbuffer, list);
> - list_del(&vbuf->list);
> - spin_unlock(&vgdev->free_vbufs_lock);
> + vbuf = kmem_cache_alloc(vgdev->vbufs, GFP_KERNEL);
> + if (IS_ERR(vbuf))
> + return ERR_CAST(vbuf);

AFAICS kmem_cache_alloc returns NULL on error.

> memset(vbuf, 0, VBUFFER_SIZE);
>
> BUG_ON(size > MAX_INLINE_CMD_SIZE);
> @@ -208,9 +173,7 @@ static void free_vbuf(struct virtio_gpu_device *vgdev,
> if (vbuf->resp_size > MAX_INLINE_RESP_SIZE)
> kfree(vbuf->resp_buf);
> kfree(vbuf->data_buf);
> - spin_lock(&vgdev->free_vbufs_lock);
> - list_add(&vbuf->list, &vgdev->free_vbufs);
> - spin_unlock(&vgdev->free_vbufs_lock);
> + kmem_cache_free(vgdev->vbufs, vbuf);
> }
>
> static void reclaim_vbufs(struct virtqueue *vq, struct list_head *reclaim_list)
>

regards,
--
js
suse labs

\
 
 \ /
  Last update: 2017-03-10 19:13    [W:0.032 / U:0.408 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site