[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] vhost-user: Skip unnecessary duplicated VHOST_USER_SET_LOG_B
From: |
Alex Bennée |
Subject: |
Re: [PATCH] vhost-user: Skip unnecessary duplicated VHOST_USER_SET_LOG_BASE requests |
Date: |
Mon, 01 Jul 2024 09:49:09 +0100 |
项文成 <xiangwencheng@dayudpu.com> writes:
> From: BillXiang <xiangwencheng@dayudpu.com>
>
> The VHOST_USER_SET_LOG_BASE requests should be categorized into
> non-vring specific messages, and should be sent only once.
> If send more than once, dpdk will munmap old log_addr which may has
> been used and cause segmentation fault.
This looks fine to me but looking at the vhost-user.rst we don't seem to
make any explicit statements about how many times given messages should
be sent.
>
> Signed-off-by: BillXiang <xiangwencheng@dayudpu.com>
> ---
> hw/virtio/vhost-user.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
> index cdf9af4a4b..41e34edd49 100644
> --- a/hw/virtio/vhost-user.c
> +++ b/hw/virtio/vhost-user.c
> @@ -371,6 +371,7 @@ static bool
> vhost_user_per_device_request(VhostUserRequest request)
> case VHOST_USER_RESET_DEVICE:
> case VHOST_USER_ADD_MEM_REG:
> case VHOST_USER_REM_MEM_REG:
> + case VHOST_USER_SET_LOG_BASE:
> return true;
> default:
> return false;
--
Alex Bennée
Virtualisation Tech Lead @ Linaro