[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 0/7] block: bdrv_img_create(): propagate errors
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [RFC 0/7] block: bdrv_img_create(): propagate errors |
Date: |
Fri, 12 Oct 2012 10:31:45 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121009 Thunderbird/16.0 |
Il 11/10/2012 23:26, Luiz Capitulino ha scritto:
> I'm calling this an RFC because I did it on hurry and it's almost untested,
> but I wanted to drop it for early review while I'm out for a public holiday :)
>
> This should improve qmp_transaction() error messages on bdrv_img_create()
> failure quite a bit. Also, the "formatting" message is not printed to stdout
> anymore when in QMP.
>
> Luiz Capitulino (6):
> block: bdrv_img_create(): add param_ret argument
> block: bdrv_img_create(): move param printing to qemu-img
> block: bdrv_img_create(): add Error ** argument
> qemu-img: img_create(): use Error object
> qmp: qmp_transaction(): pass Error object to bdrv_img_create()
> block: bdrv_img_create(): drop unused code
>
> Paolo Bonzini (1):
> error: add error_set_errno and error_setg_errno
>
> block.c | 69
> +++++++++++++++++++++++++++-----------------------------------
> block.h | 7 ++++---
> blockdev.c | 13 ++++++------
> error.c | 28 +++++++++++++++++++++++++
> error.h | 9 ++++++++
> qemu-img.c | 18 +++++++++++++---
> 6 files changed, 93 insertions(+), 51 deletions(-)
>
Looks good. We could debate endlessly how to order the patches, but the
idea is fine.
Paolo
- [Qemu-devel] [RFC 0/7] block: bdrv_img_create(): propagate errors, Luiz Capitulino, 2012/10/11
- [Qemu-devel] [RFC 1/7] error: add error_set_errno and error_setg_errno, Luiz Capitulino, 2012/10/11
- [Qemu-devel] [RFC 2/7] block: bdrv_img_create(): add param_ret argument, Luiz Capitulino, 2012/10/11
- [Qemu-devel] [RFC 3/7] block: bdrv_img_create(): move param printing to qemu-img, Luiz Capitulino, 2012/10/11
- [Qemu-devel] [RFC 5/7] qemu-img: img_create(): use Error object, Luiz Capitulino, 2012/10/11
- [Qemu-devel] [RFC 4/7] block: bdrv_img_create(): add Error ** argument, Luiz Capitulino, 2012/10/11
- [Qemu-devel] [RFC 6/7] qmp: qmp_transaction(): pass Error object to bdrv_img_create(), Luiz Capitulino, 2012/10/11
- [Qemu-devel] [RFC 7/7] block: bdrv_img_create(): drop unused code, Luiz Capitulino, 2012/10/11
- Re: [Qemu-devel] [RFC 0/7] block: bdrv_img_create(): propagate errors,
Paolo Bonzini <=