[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 09/12] block: update return value from bdrv_s
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 09/12] block: update return value from bdrv_snapshot_create |
Date: |
Tue, 26 Mar 2013 15:54:56 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130311 Thunderbird/17.0.4 |
On 03/22/2013 07:16 AM, Pavel Hrdina wrote:
> If we provide error message, we should also provide a return code.
> In some cases we could not care about any error message and the return
> code is enough for as.
>
> Signed-off-by: Pavel Hrdina <address@hidden>
> ---
> +++ b/qemu-img.c
> @@ -1943,6 +1943,7 @@ static int img_snapshot(int argc, char **argv)
> int action = 0;
> qemu_timeval tv;
> bool quiet = false;
> + Error *local_err;
>
> bdrv_oflags = BDRV_O_FLAGS | BDRV_O_RDWR;
> /* Parse commandline parameters */
> @@ -2019,10 +2020,10 @@ static int img_snapshot(int argc, char **argv)
> sn.date_sec = tv.tv_sec;
> sn.date_nsec = tv.tv_usec * 1000;
>
> - ret = bdrv_snapshot_create(bs, &sn, NULL);
> + local_err = NULL;
> + ret = bdrv_snapshot_create(bs, &sn, &local_err);
> if (ret) {
Isn't it true that 'ret' is non-zero iff 'local_err' is no longer NULL?
In which case, why not have the functions return 'void' instead of an
integer, and use the null-ness of local_err be your test of whether
bdrv_snapshot_create failed.
> - error_report("Could not create snapshot '%s': %d (%s)",
> - snapshot_name, ret, strerror(-ret));
> + qemu_img_handle_error(local_err);
Same question as in 8/12 - you have already guaranteed that
qemu_img_handle_error will only be called if local_err is nonnull, so
does that helper function have to do a sanity check?
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH v2 04/12] savevm: add error parameter to qemu_savevm_state_iterate(), (continued)
- [Qemu-devel] [PATCH v2 07/12] qapi: Convert savevm, Pavel Hrdina, 2013/03/22
- [Qemu-devel] [PATCH v2 06/12] savevm: add error parameter to qemu_savevm_state(), Pavel Hrdina, 2013/03/22
- [Qemu-devel] [PATCH v2 08/12] qemu-img: introduce qemu_img_handle_error, Pavel Hrdina, 2013/03/22
- [Qemu-devel] [PATCH v2 10/12] savevm: update return value from qemu_savevm_state, Pavel Hrdina, 2013/03/22
- [Qemu-devel] [PATCH v2 09/12] block: update return value from bdrv_snapshot_create, Pavel Hrdina, 2013/03/22
- Re: [Qemu-devel] [PATCH v2 09/12] block: update return value from bdrv_snapshot_create,
Eric Blake <=
- [Qemu-devel] [PATCH v2 12/12] savevm: icrease the IO_BUF_SIZE to improve the speed of savevm, Pavel Hrdina, 2013/03/22
- [Qemu-devel] [PATCH v2 11/12] vm-snapshot-save: add force parameter, Pavel Hrdina, 2013/03/22