Re: [PATCH] vhost_user: remove ASSERT() on iovec number

[Stefano Brivio <sbrivio@redhat.com>]

On Mon, 20 Jan 2025 14:15:22 +0100
Laurent Vivier <lvivier@redhat.com> wrote:

> Replace ASSERT() on the number of iovec in the element and on
> the first entry length by a debug() message.
> 
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
>  vu_common.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/vu_common.c b/vu_common.c
> index 431fba6be0c0..d34ae6dc1df3 100644
> --- a/vu_common.c
> +++ b/vu_common.c
> @@ -195,8 +195,12 @@ static void vu_handle_tx(struct vu_dev *vdev, int index,
>  				        hdrlen);
>  		} else {
>  			/* vnet header can be in a separate iovec */
> -			ASSERT(elem[count].out_num == 2);
> -			ASSERT(elem[count].out_sg[0].iov_len == (size_t)hdrlen);
> +			if (elem[count].out_num != 2)
> +				debug("virtio-net tranmit queue contains more than one buffer ([%d]: %u)",

s/tranmit/transmit/

Applied with this change.

-- 
Stefano