Re: [PATCH 1/1] vhost: Move fds outside of VhostUserMessage

[Maxime Coquelin <maxime.coquelin@redhat.com>]

Hi Christophe,

On 1/23/22 12:59, Christophe Fontaine wrote:
> FDs at the end of the VhostUserMessage structure limits the size
> of the payload. Move them to an other englobing structure, before
> the header & payload of a VhostUserMessage.
> Also removes a reference to fds in the VHUMsg structure defined in
> drivers/net/virtio/virtio_user/vhost_user.c
> 
> Signed-off-by: Christophe Fontaine <cfontain@redhat.com>
> ---
>   drivers/net/virtio/virtio_user/vhost_user.c |   1 -
>   lib/vhost/vhost_crypto.c                    |  10 +-
>   lib/vhost/vhost_user.c                      | 529 ++++++++++----------
>   lib/vhost/vhost_user.h                      |   7 +-
>   4 files changed, 288 insertions(+), 259 deletions(-)
> 

Thanks for working on this, I agree the FDs should be moved outside the
Vhost-user message structure.

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime