[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC v5 031/126] xen: introduce ERRP_AUTO_PROPAGATE
From: |
Anthony PERARD |
Subject: |
Re: [RFC v5 031/126] xen: introduce ERRP_AUTO_PROPAGATE |
Date: |
Wed, 20 Nov 2019 15:38:31 +0000 |
User-agent: |
Mutt/1.12.2 (2019-09-21) |
On Fri, Oct 11, 2019 at 07:04:17PM +0300, Vladimir Sementsov-Ogievskiy wrote:
> diff --git a/hw/block/xen-block.c b/hw/block/xen-block.c
> --- a/hw/block/xen-block.c
> +++ b/hw/block/xen-block.c
> @@ -915,15 +903,15 @@ static void xen_block_device_create(XenBackendInstance
> *backend,
> goto fail;
> }
>
> - drive = xen_block_drive_create(vdev, device_type, opts, &local_err);
> + drive = xen_block_drive_create(vdev, device_type, opts, errp);
> if (!drive) {
> - error_propagate_prepend(errp, local_err, "failed to create drive: ");
> + error_prepend(errp, "failed to create drive: ");
> goto fail;
> }
>
> - iothread = xen_block_iothread_create(vdev, &local_err);
> - if (local_err) {
> - error_propagate_prepend(errp, local_err,
> + iothread = xen_block_iothread_create(vdev, errp);
> + if (*errp) {
> + error_prepend(errp,
> "failed to create iothread: ");
These two line could be joined now.
> goto fail;
> }
And there are more indentation issues like that in the patch. It would be
nice to fix, but otherwise the patch looks fine:
Acked-by: Anthony PERARD <address@hidden>
Thanks,
--
Anthony PERARD
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [RFC v5 031/126] xen: introduce ERRP_AUTO_PROPAGATE,
Anthony PERARD <=