lkml.org 
[lkml]   [2022]   [May]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: [PATCH] virtio_balloon: check virtqueue_add_outbuf() return value
Date
The purpose of this patch is, virtqueue_add_outbuf() adding data to
virtqueue may succeed or fail. If successful, it will call virtqueue_kick()
to notifie the host backend driver, but if the virtqueue_add_outbuf() fails,
the data is not successfully added to the virtqueue, and the host back-end
driver does not need to be notified for processing.

Thanks.

-----邮件原件-----
发件人: Michael S. Tsirkin <mst@redhat.com>
发送时间: 2022年5月27日 18:25
收件人: Bo Liu (刘波)-浪潮信息 <liubo03@inspur.com>
抄送: david@redhat.com; jasowang@redhat.com;
virtualization@lists.linux-foundation.org; linux-kernel@vger.kernel.org
主题: Re: [PATCH] virtio_balloon: check virtqueue_add_outbuf() return value

On Thu, May 26, 2022 at 09:34:01PM -0400, Bo Liu wrote:
> virtqueue_add_outbuf() can fail, when it fails, there is no need to
> call the funciont virtqueue_kick().
> Signed-off-by: Bo Liu <liubo03@inspur.com>

Sorry don't get it.
What's the point of this patch really?

> ---
> drivers/virtio/virtio_balloon.c | 18 +++++++++++-------
> 1 file changed, 11 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/virtio/virtio_balloon.c
> b/drivers/virtio/virtio_balloon.c index b9737da6c4dd..0d7da4d95e1e
> 100644
> --- a/drivers/virtio/virtio_balloon.c
> +++ b/drivers/virtio/virtio_balloon.c
> @@ -153,16 +153,18 @@ static void tell_host(struct virtio_balloon *vb,
> struct virtqueue *vq) {
> struct scatterlist sg;
> unsigned int len;
> + int err;
>
> sg_init_one(&sg, vb->pfns, sizeof(vb->pfns[0]) * vb->num_pfns);
>
> /* We should always be able to add one buffer to an empty queue. */
> - virtqueue_add_outbuf(vq, &sg, 1, vb, GFP_KERNEL);
> - virtqueue_kick(vq);
> -
> - /* When host has read buffer, this completes via balloon_ack */
> - wait_event(vb->acked, virtqueue_get_buf(vq, &len));
> + err = virtqueue_add_outbuf(vq, &sg, 1, vb, GFP_KERNEL);
> + if (!err) {
> + virtqueue_kick(vq);
>
> + /* When host has read buffer, this completes via balloon_ack
*/
> + wait_event(vb->acked, virtqueue_get_buf(vq, &len));
> + }
> }
>
> static int virtballoon_free_page_report(struct
> page_reporting_dev_info *pr_dev_info, @@ -382,6 +384,7 @@ static void
stats_handle_request(struct virtio_balloon *vb)
> struct virtqueue *vq;
> struct scatterlist sg;
> unsigned int len, num_stats;
> + int err;
>
> num_stats = update_balloon_stats(vb);
>
> @@ -389,8 +392,9 @@ static void stats_handle_request(struct virtio_balloon
*vb)
> if (!virtqueue_get_buf(vq, &len))
> return;
> sg_init_one(&sg, vb->stats, sizeof(vb->stats[0]) * num_stats);
> - virtqueue_add_outbuf(vq, &sg, 1, vb, GFP_KERNEL);
> - virtqueue_kick(vq);
> + err = virtqueue_add_outbuf(vq, &sg, 1, vb, GFP_KERNEL);
> + if (!err)
> + virtqueue_kick(vq);
> }
>
> static inline s64 towards_target(struct virtio_balloon *vb)
> --
> 2.27.0

[unhandled content-type:application/pkcs7-signature]
\
 
 \ /
  Last update: 2022-05-27 12:45    [W:0.069 / U:0.332 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site