On 8/30/23 15:40, Laszlo Ersek wrote: > In the vhost_user_set_vring_addr() function, we calculate > "reply_supported" unconditionally, even though we'll only need it if > "wait_for_reply" is also true. > > Restrict the scope of "reply_supported" to the minimum. > > This is purely refactoring -- no observable change. > > Cc: "Michael S. Tsirkin" <[email protected]> (supporter:vhost) > Cc: Eugenio Perez Martin <[email protected]> > Cc: German Maglione <[email protected]> > Cc: Liu Jiang <[email protected]> > Cc: Sergio Lopez Pascual <[email protected]> > Cc: Stefano Garzarella <[email protected]> > Signed-off-by: Laszlo Ersek <[email protected]> > Reviewed-by: Stefano Garzarella <[email protected]> > --- > > Notes: > v2: > > - pick up Stefano's R-b > > hw/virtio/vhost-user.c | 11 ++++++----- > 1 file changed, 6 insertions(+), 5 deletions(-)
identical to v1, so: Reviewed-by: Philippe Mathieu-Daudé <[email protected]> from <[email protected]">http://mid.mail-archive.com/[email protected]>. Laszlo > > diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c > index b4b677c1ce66..64eac317bfb2 100644 > --- a/hw/virtio/vhost-user.c > +++ b/hw/virtio/vhost-user.c > @@ -1331,17 +1331,18 @@ static int vhost_user_set_vring_addr(struct vhost_dev > *dev, > .hdr.size = sizeof(msg.payload.addr), > }; > > - bool reply_supported = virtio_has_feature(dev->protocol_features, > - > VHOST_USER_PROTOCOL_F_REPLY_ACK); > - > /* > * wait for a reply if logging is enabled to make sure > * backend is actually logging changes > */ > bool wait_for_reply = addr->flags & (1 << VHOST_VRING_F_LOG); > > - if (reply_supported && wait_for_reply) { > - msg.hdr.flags |= VHOST_USER_NEED_REPLY_MASK; > + if (wait_for_reply) { > + bool reply_supported = virtio_has_feature(dev->protocol_features, > + VHOST_USER_PROTOCOL_F_REPLY_ACK); > + if (reply_supported) { > + msg.hdr.flags |= VHOST_USER_NEED_REPLY_MASK; > + } > } > > ret = vhost_user_write(dev, &msg, NULL, 0); >
