On 16.10.23 10:45, Manos Pitsidianakis wrote:
On Mon, 16 Oct 2023 11:32, Hanna Czenczek <hre...@redhat.com> wrote:
diff --git a/include/hw/virtio/vhost-user.h
b/include/hw/virtio/vhost-user.h
index 9f9ddf878d..1d4121431b 100644
--- a/include/hw/virtio/vhost-user.h
+++ b/include/hw/virtio/vhost-user.h
@@ -29,7 +29,8 @@ enum VhostUserProtocolFeature {
VHOST_USER_PROTOCOL_F_INBAND_NOTIFICATIONS = 14,
VHOST_USER_PROTOCOL_F_CONFIGURE_MEM_SLOTS = 15,
VHOST_USER_PROTOCOL_F_STATUS = 16,
- VHOST_USER_PROTOCOL_F_SHARED_OBJECT = 17,
+ /* Feature 17 reserved for VHOST_USER_PROTOCOL_F_XEN_MMAP. */
+ VHOST_USER_PROTOCOL_F_SHARED_OBJECT = 18,
VHOST_USER_PROTOCOL_F_MAX
};
May I ask, why not define VHOST_USER_PROTOCOL_F_XEN_MMAP as well
instead of a comment mention?
No particular reason, it’s just what I had planned to do for a while in
other series that would introduce feature bits (e.g.
https://lists.nongnu.org/archive/html/qemu-devel/2023-07/msg02452.html).
I think I took that from libvhost-user, which does this for
VHOST_USER_PROTOCOL_F_STATUS.
Otherwise:
Reviewed-by: Emmanouil Pitsidianakis <manos.pitsidiana...@linaro.org>
Thanks!
Hanna