[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 04/12] vhost-user-server: do not set memory fd non-blockin
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v4 04/12] vhost-user-server: do not set memory fd non-blocking |
Date: |
Wed, 8 May 2024 10:00:45 +0100 |
User-agent: |
Mutt/2.2.12 (2023-09-09) |
On Wed, May 08, 2024 at 09:44:48AM +0200, Stefano Garzarella wrote:
> In vhost-user-server we set all fd received from the other peer
> in non-blocking mode. For some of them (e.g. memfd, shm_open, etc.)
> it's not really needed, because we don't use these fd with blocking
> operations, but only to map memory.
>
> In addition, in some systems this operation can fail (e.g. in macOS
> setting an fd returned by shm_open() non-blocking fails with errno
> = ENOTTY).
>
> So, let's avoid setting fd non-blocking for those messages that we
> know carry memory fd (e.g. VHOST_USER_ADD_MEM_REG,
> VHOST_USER_SET_MEM_TABLE).
>
> Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
> ---
> v3:
> - avoiding setting fd non-blocking for messages where we have memory fd
> (Eric)
> ---
> util/vhost-user-server.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PATCH v4 01/12] libvhost-user: set msg.msg_control to NULL when it is empty, (continued)
- [PATCH v4 03/12] libvhost-user: mask F_INFLIGHT_SHMFD if memfd is not supported, Stefano Garzarella, 2024/05/08
- [PATCH v4 02/12] libvhost-user: fail vu_message_write() if sendmsg() is failing, Stefano Garzarella, 2024/05/08
- [PATCH v4 04/12] vhost-user-server: do not set memory fd non-blocking, Stefano Garzarella, 2024/05/08
- Re: [PATCH v4 04/12] vhost-user-server: do not set memory fd non-blocking,
Daniel P . Berrangé <=
- [PATCH v4 05/12] contrib/vhost-user-blk: fix bind() using the right size of the address, Stefano Garzarella, 2024/05/08
- [PATCH v4 07/12] vhost-user: enable frontends on any POSIX system, Stefano Garzarella, 2024/05/08
- [PATCH v4 06/12] contrib/vhost-user-*: use QEMU bswap helper functions, Stefano Garzarella, 2024/05/08
- [PATCH v4 08/12] libvhost-user: enable it on any POSIX system, Stefano Garzarella, 2024/05/08