[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 5/5] blockdev: use error_set_file_open_failed
From: |
Alon Levy |
Subject: |
Re: [Qemu-devel] [PATCH v2 5/5] blockdev: use error_set_file_open_failed |
Date: |
Thu, 15 Mar 2012 17:23:42 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Mar 15, 2012 at 11:56:29AM -0300, Luiz Capitulino wrote:
> On Thu, 15 Mar 2012 00:55:14 +0200
> Alon Levy <address@hidden> wrote:
>
> > This is a little trickier, since those calls chain in many fun ways and
> > produce sometimes their own return values reusing existing errno values
> > for similar meanings. In that respect error_set_file_open_failed
> > specifically ignores EINVAL, ENOTSUP and ENOENT. The first two simply
> > are not returned by open (2), but the last is but I chose to ignore it
> > to allow easy reuse in blockdev to avoid confusion when it is used
> > internally by the create functions.
>
> Please, just drop this from this series as it's completely unrelated to
> the screendump command.
After I went to all the trouble jumping through blockdev.c ? OK :)
>
> >
> > Signed-off-by: Alon Levy <address@hidden>
> > ---
> > blockdev.c | 11 +++++++----
> > 1 file changed, 7 insertions(+), 4 deletions(-)
> >
> > diff --git a/blockdev.c b/blockdev.c
> > index 1a500b8..544d067 100644
> > --- a/blockdev.c
> > +++ b/blockdev.c
> > @@ -777,7 +777,7 @@ void qmp_transaction(BlockdevActionList *dev_list,
> > Error **errp)
> > states->old_bs->drv->format_name,
> > NULL, -1, flags);
> > if (ret) {
> > - error_set(errp, QERR_OPEN_FILE_FAILED, new_image_file);
> > + error_set_file_open_failed(errp, new_image_file, -ret);
> > goto delete_and_fail;
> > }
> > }
> > @@ -787,7 +787,7 @@ void qmp_transaction(BlockdevActionList *dev_list,
> > Error **errp)
> > ret = bdrv_open(states->new_bs, new_image_file,
> > flags | BDRV_O_NO_BACKING, drv);
> > if (ret != 0) {
> > - error_set(errp, QERR_OPEN_FILE_FAILED, new_image_file);
> > + error_set_file_open_failed(errp, new_image_file, -ret);
> > goto delete_and_fail;
> > }
> > }
> > @@ -881,8 +881,11 @@ static void qmp_bdrv_open_encrypted(BlockDriverState
> > *bs, const char *filename,
> > int bdrv_flags, BlockDriver *drv,
> > const char *password, Error **errp)
> > {
> > - if (bdrv_open(bs, filename, bdrv_flags, drv) < 0) {
> > - error_set(errp, QERR_OPEN_FILE_FAILED, filename);
> > + int ret;
> > +
> > + ret = bdrv_open(bs, filename, bdrv_flags, drv);
> > + if (ret < 0) {
> > + error_set_file_open_failed(errp, filename, ret);
> > return;
> > }
> >
>
- [Qemu-devel] [PATCH v2 0/5] screendump qapi convertion, Alon Levy, 2012/03/14
- [Qemu-devel] [PATCH v2 2/5] vga_hw_screen_dump: add Error** param, Alon Levy, 2012/03/14
- [Qemu-devel] [PATCH v2 3/5] qapi: convert screendump, Alon Levy, 2012/03/14
- [Qemu-devel] [PATCH v2 4/5] vga: ppm_save(): Return error on failure, Alon Levy, 2012/03/14
- [Qemu-devel] [PATCH v2 5/5] blockdev: use error_set_file_open_failed, Alon Levy, 2012/03/14
- [Qemu-devel] [PATCH v2 1/5] error: add error_set_file_open_failed, Alon Levy, 2012/03/14
- [Qemu-devel] [PATCH v3 0/5] screendump qapi convertion, Alon Levy, 2012/03/18
- [Qemu-devel] [PATCH v3 5/5] vga: ppm_save(): Return error on failure, Alon Levy, 2012/03/18