|
From: | Hanna Czenczek |
Subject: | Re: [PATCH] vhost-user: Fix protocol feature bit conflict |
Date: | Mon, 16 Oct 2023 12:40:48 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.15.1 |
On 16.10.23 10:45, Manos Pitsidianakis wrote:
On Mon, 16 Oct 2023 11:32, Hanna Czenczek <hreitz@redhat.com> wrote:diff --git a/include/hw/virtio/vhost-user.h b/include/hw/virtio/vhost-user.hindex 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.pitsidianakis@linaro.org>
Thanks! Hanna
[Prev in Thread] | Current Thread | [Next in Thread] |