Re: [PATCH] vhost-user: add NEED_REPLY flag

2024-09-10 Thread Michael S. Tsirkin
On Sun, Aug 04, 2024 at 11:48:59PM +0800, luzhixing12345 wrote: > Front-end message requests which need reply should set NEED_REPLY_MASK > in flag, and response from slave need clear NEED_REPLY_MASK flag. neither this. > --- > hw/virtio/vhost-user.c| 2 +- > subprojects/libv

Re: [PATCH] vhost-user: add NEED_REPLY flag

2024-09-05 Thread luzhixing12345
Signed-off-by: luzhixing12345 >On Mon, Aug 12, 2024 at 12:53:19PM GMT, 陆知行 wrote: >>Hi, can someone review this patch? >>I find requests which call vhost_user_get_u64 does not set NEED_REPLY flag > >Can you provide an example to trigger this issue? > >Also, with this change all calls to vhost_us

Re: [PATCH] vhost-user: add NEED_REPLY flag

2024-08-27 Thread Stefano Garzarella
On Mon, Aug 12, 2024 at 12:53:19PM GMT, 陆知行 wrote: Hi, can someone review this patch? I find requests which call vhost_user_get_u64 does not set NEED_REPLY flag Can you provide an example to trigger this issue? Also, with this change all calls to vhost_user_get_u64() will set that flag, is t

Re: [PATCH] vhost-user: add NEED_REPLY flag

2024-08-11 Thread 陆知行
Hi, can someone review this patch? I find requests which call vhost_user_get_u64 does not set NEED_REPLY flag luzhixing12345 于2024年8月4日周日 23:50写道: > Front-end message requests which need reply should set NEED_REPLY_MASK > in flag, and response from slave need clear NEED_REPLY_MASK flag. > > ---

[PATCH] vhost-user: add NEED_REPLY flag

2024-08-04 Thread luzhixing12345
Front-end message requests which need reply should set NEED_REPLY_MASK in flag, and response from slave need clear NEED_REPLY_MASK flag. --- hw/virtio/vhost-user.c| 2 +- subprojects/libvhost-user/libvhost-user.c | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --g