diff mbox series

vhsot-user: Remove redundant judgment

Message ID 20240710074041.85984-1-xiangwencheng@dayudpu.com
State New
Headers show
Series vhsot-user: Remove redundant judgment | expand

Commit Message

BillXiang July 10, 2024, 7:40 a.m. UTC
From: BillXiang <xiangwencheng@dayudpu.com>

Send only once has been supported in vhost_user_write by vhost_user_per_device_request

Signed-off-by: BillXiang <xiangwencheng@dayudpu.com>
---
 hw/virtio/vhost-user.c | 5 -----
 1 file changed, 5 deletions(-)

Comments

Michael S. Tsirkin July 20, 2024, 7:13 p.m. UTC | #1
On Wed, Jul 10, 2024 at 03:40:41PM +0800, BillXiang wrote:
> From: BillXiang <xiangwencheng@dayudpu.com>
> 
> Send only once has been supported in vhost_user_write by vhost_user_per_device_request
> 
> Signed-off-by: BillXiang <xiangwencheng@dayudpu.com>

I am confused by all the patches. Is it on top of your
other patch or instead of? Because that one added
the vq index check.
If we are going to check vq index it's likely cleaner to
always check it.

And again, typos do not inspire confidence. How was this tested?

> ---
>  hw/virtio/vhost-user.c | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
> index 00561daa06..04e3568b7e 100644
> --- a/hw/virtio/vhost-user.c
> +++ b/hw/virtio/vhost-user.c
> @@ -446,11 +446,6 @@ static int vhost_user_set_log_base(struct vhost_dev *dev, uint64_t base,
>          .hdr.size = sizeof(msg.payload.log),
>      };
>  
> -    /* Send only once with first queue pair */
> -    if (dev->vq_index != 0) {
> -        return 0;
> -    }
> -
>      if (shmfd && log->fd != -1) {
>          fds[fd_num++] = log->fd;
>      }
> -- 
> 2.30.0
BillXiang July 21, 2024, 3:33 a.m. UTC | #2
> From: "Michael S. Tsirkin"<mst@redhat.com>
> Date:  Sun, Jul 21, 2024, 03:14
> Subject:  Re: [PATCH] vhsot-user: Remove redundant judgment
> To: "BillXiang"<xiangwencheng@dayudpu.com>
> Cc: <qemu-devel@nongnu.org>, <alex.bennee@linaro.org>
> On Wed, Jul 10, 2024 at 03:40:41PM +0800, BillXiang wrote:
> > From: BillXiang <xiangwencheng@dayudpu.com>
> > 
> > Send only once has been supported in vhost_user_write by vhost_user_per_device_request
> > 
> > Signed-off-by: BillXiang <xiangwencheng@dayudpu.com>

> I am confused by all the patches. Is it on top of your
> other patch or instead of? Because that one added
> the vq index check.
> If we are going to check vq index it's likely cleaner to
> always check it.


Yes, it's on top of commit 7c211eb078c4 
("vhost-user: Skip unnecessary duplicated VHOST_USER_SET_LOG_BASE requests") ,
and the check has been done in vhost_user_write 
after VHOST_USER_SET_LOG_BASE be added 
to vhost_user_per_device_request.

> And again, typos do not inspire confidence. How was this tested?

> > ---
> >  hw/virtio/vhost-user.c | 5 -----
> >  1 file changed, 5 deletions(-)
> > 
> > diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
> > index 00561daa06..04e3568b7e 100644
> > --- a/hw/virtio/vhost-user.c
> > +++ b/hw/virtio/vhost-user.c
> > @@ -446,11 +446,6 @@ static int vhost_user_set_log_base(struct vhost_dev *dev, uint64_t base,
> >          .hdr.size = sizeof(msg.payload.log),
> >      };
> >  
> > -    /* Send only once with first queue pair */
> > -    if (dev->vq_index != 0) {
> > -        return 0;
> > -    }
> > -
> >      if (shmfd && log->fd != -1) {
> >          fds[fd_num++] = log->fd;
> >      }
> > -- 
> > 2.30.0
diff mbox series

Patch

diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
index 00561daa06..04e3568b7e 100644
--- a/hw/virtio/vhost-user.c
+++ b/hw/virtio/vhost-user.c
@@ -446,11 +446,6 @@  static int vhost_user_set_log_base(struct vhost_dev *dev, uint64_t base,
         .hdr.size = sizeof(msg.payload.log),
     };
 
-    /* Send only once with first queue pair */
-    if (dev->vq_index != 0) {
-        return 0;
-    }
-
     if (shmfd && log->fd != -1) {
         fds[fd_num++] = log->fd;
     }