On 31/8/23 09:20, Laszlo Ersek wrote:
On 8/30/23 15:40, Laszlo Ersek wrote:
In order to avoid a forward-declaration for "vhost_user_write_sync" in a
subsequent patch, hoist "vhost_user_write_sync" ->
"vhost_user_get_features" -> "vhost_user_get_u64" just above
"vhost_set_vring".
This is purely code movement -- no observable change.
Cc: "Michael S. Tsirkin" <m...@redhat.com> (supporter:vhost)
Cc: Eugenio Perez Martin <epere...@redhat.com>
Cc: German Maglione <gmagli...@redhat.com>
Cc: Liu Jiang <ge...@linux.alibaba.com>
Cc: Sergio Lopez Pascual <s...@redhat.com>
Cc: Stefano Garzarella <sgarz...@redhat.com>
Signed-off-by: Laszlo Ersek <ler...@redhat.com>
Reviewed-by: Stefano Garzarella <sgarz...@redhat.com>
---
Notes:
v2:
- pick up R-b from Stefano
- rename "vhost_user_write_msg" to "vhost_user_write_sync" (in code and
commit message) [Stefano]
hw/virtio/vhost-user.c | 170 ++++++++++----------
1 file changed, 85 insertions(+), 85 deletions(-)
Phil reviewed v1:
98150923-39ef-7581-6144-8d0ad8d4dd52@linaro.org">http://mid.mail-archive.com/98150923-39ef-7581-6144-8d0ad8d4dd52@linaro.org
and I would've kept his R-b (similar to Stefano's) across the
vhost_user_write_msg->vhost_user_write_sync rename in v2; so I'm copying
it here:
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
Hope that's OK.
Sure! (same for patch 2/7)
Thanks :)
Phil.