[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 6/7] vhost-user-blk: Factor out vhost_user_blk_realize_connec
From: |
Raphael Norwitz |
Subject: |
Re: [PATCH 6/7] vhost-user-blk: Factor out vhost_user_blk_realize_connect() |
Date: |
Fri, 11 Jun 2021 20:11:17 +0000 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Wed, Jun 09, 2021 at 05:46:57PM +0200, Kevin Wolf wrote:
> This function is the part that we will want to retry if the connection
> is lost during initialisation, so factor it out to keep the following
> patch simpler.
>
> The error path for vhost_dev_get_config() forgot disconnecting the
> chardev, add this while touching the code.
>
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Raphael Norwitz <raphael.norwitz@nutanix.com>
> ---
> hw/block/vhost-user-blk.c | 48 ++++++++++++++++++++++++++-------------
> 1 file changed, 32 insertions(+), 16 deletions(-)
>
> diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c
> index 3770f715da..e49d2e4c83 100644
> --- a/hw/block/vhost-user-blk.c
> +++ b/hw/block/vhost-user-blk.c
> @@ -423,6 +423,36 @@ static void vhost_user_blk_event(void *opaque,
> QEMUChrEvent event)
> }
> }
>
> +static int vhost_user_blk_realize_connect(VHostUserBlk *s, Error **errp)
> +{
> + DeviceState *dev = &s->parent_obj.parent_obj;
> + int ret;
> +
> + s->connected = false;
> +
> + ret = qemu_chr_fe_wait_connected(&s->chardev, errp);
> + if (ret < 0) {
> + return ret;
> + }
> +
> + ret = vhost_user_blk_connect(dev, errp);
> + if (ret < 0) {
> + qemu_chr_fe_disconnect(&s->chardev);
> + return ret;
> + }
> + assert(s->connected);
> +
> + ret = vhost_dev_get_config(&s->dev, (uint8_t *)&s->blkcfg,
> + sizeof(struct virtio_blk_config), errp);
> + if (ret < 0) {
> + qemu_chr_fe_disconnect(&s->chardev);
> + vhost_dev_cleanup(&s->dev);
> + return ret;
> + }
> +
> + return 0;
> +}
> +
> static void vhost_user_blk_device_realize(DeviceState *dev, Error **errp)
> {
> VirtIODevice *vdev = VIRTIO_DEVICE(dev);
> @@ -467,22 +497,10 @@ static void vhost_user_blk_device_realize(DeviceState
> *dev, Error **errp)
>
> s->inflight = g_new0(struct vhost_inflight, 1);
> s->vhost_vqs = g_new0(struct vhost_virtqueue, s->num_queues);
> - s->connected = false;
> -
> - if (qemu_chr_fe_wait_connected(&s->chardev, errp) < 0) {
> - goto virtio_err;
> - }
>
> - if (vhost_user_blk_connect(dev, errp) < 0) {
> - qemu_chr_fe_disconnect(&s->chardev);
> - goto virtio_err;
> - }
> - assert(s->connected);
> -
> - ret = vhost_dev_get_config(&s->dev, (uint8_t *)&s->blkcfg,
> - sizeof(struct virtio_blk_config), errp);
> + ret = vhost_user_blk_realize_connect(s, errp);
> if (ret < 0) {
> - goto vhost_err;
> + goto virtio_err;
> }
>
> /* we're fully initialized, now we can operate, so add the handler */
> @@ -491,8 +509,6 @@ static void vhost_user_blk_device_realize(DeviceState
> *dev, Error **errp)
> NULL, true);
> return;
>
> -vhost_err:
> - vhost_dev_cleanup(&s->dev);
> virtio_err:
> g_free(s->vhost_vqs);
> s->vhost_vqs = NULL;
> --
> 2.30.2
>
- Re: [PATCH 3/7] vhost: Return 0/-errno in vhost_dev_init(), (continued)
- [PATCH 5/7] vhost: Distinguish errors in vhost_dev_get_config(), Kevin Wolf, 2021/06/09
- [PATCH 4/7] vhost-user-blk: Add Error parameter to vhost_user_blk_start(), Kevin Wolf, 2021/06/09
- [PATCH 6/7] vhost-user-blk: Factor out vhost_user_blk_realize_connect(), Kevin Wolf, 2021/06/09
- [PATCH 7/7] vhost-user-blk: Implement reconnection during realize, Kevin Wolf, 2021/06/09
- Re: [PATCH 0/7] vhost-user-blk: Implement reconnection during realize, Kevin Wolf, 2021/06/30