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

[Maxime Coquelin]

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 <cfont...@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.coque...@redhat.com>

Thanks,
Maxime