diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2022-04-07 15:36:56 +0200 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2022-05-04 15:55:23 +0200 |
| commit | 5ebfdeb21223e0406a711decbb2baceb5ec8c416 (patch) | |
| tree | 3aa3603a0e9fe710245aa8ba53be30c2f144be4d /hw/virtio/vhost-user.c | |
| parent | 31009d13cc505b9569971fbbe50266c1727ba5db (diff) | |
| download | focaccia-qemu-5ebfdeb21223e0406a711decbb2baceb5ec8c416.tar.gz focaccia-qemu-5ebfdeb21223e0406a711decbb2baceb5ec8c416.zip | |
libvhost-user: Fix extra vu_add/rem_mem_reg reply
Outside of postcopy mode, neither VHOST_USER_ADD_MEM_REG nor VHOST_USER_REM_MEM_REG are supposed to send a reply unless explicitly requested with the need_reply flag. Their current implementation always sends a reply, even if it isn't requested. This confuses the master because it will interpret the reply as a reply for the next message for which it actually expects a reply. need_reply is already handled correctly by vu_dispatch(), so just don't send a reply in the non-postcopy part of the message handler for these two commands. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Message-Id: <20220407133657.155281-3-kwolf@redhat.com> Reviewed-by: Raphael Norwitz <raphael.norwitz@nutanix.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'hw/virtio/vhost-user.c')
0 files changed, 0 insertions, 0 deletions