Re: [Qemu-devel] [PATCH] vhost-user+postcopy: Use qemu_set_nonblock

[Eric Blake]

On 05/02/2018 05:55 AM, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <address@hidden>
>
> Use qemu_set_nonblock rather than a simple fcntl; cleaner
> and I have no reason to change other flags.
>
> Reported-by: Peter Maydell <address@hidden>
> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
> ---
>   hw/virtio/vhost-user.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Eric Blake <address@hidden>

(wonder if checkpatch.pl can be taught to easily recognize this one)

> diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
> index 38da8692bb..c2bbb622d6 100644
> --- a/hw/virtio/vhost-user.c
> +++ b/hw/virtio/vhost-user.c
> @@ -1076,7 +1076,7 @@ static int vhost_user_postcopy_advise(struct vhost_dev 
> *dev, Error **errp)
>           error_setg(errp, "%s: Failed to get ufd", __func__);
>           return -1;
>       }
> -    fcntl(ufd, F_SETFL, O_NONBLOCK);
> +    qemu_set_nonblock(ufd);
>   
>       /* register ufd with userfault thread */
>       u->postcopy_fd.fd = ufd;

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org