|
 |
9ae3a8 |
From 50ff46f7848a999267c7f15dac727fe690dabfd7 Mon Sep 17 00:00:00 2001
|
|
|
9ae3a8 |
From: Max Reitz <mreitz@redhat.com>
|
|
|
9ae3a8 |
Date: Mon, 4 Nov 2013 22:32:17 +0100
|
|
|
9ae3a8 |
Subject: [PATCH 24/87] block/raw-win32: Employ error parameter
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
RH-Author: Max Reitz <mreitz@redhat.com>
|
|
|
9ae3a8 |
Message-id: <1383604354-12743-27-git-send-email-mreitz@redhat.com>
|
|
|
9ae3a8 |
Patchwork-id: 55326
|
|
|
9ae3a8 |
O-Subject: [RHEL-7.0 qemu-kvm PATCH 26/43] block/raw-win32: Employ error parameter
|
|
|
9ae3a8 |
Bugzilla: 1026524
|
|
|
9ae3a8 |
RH-Acked-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Fam Zheng <famz@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
BZ: 1026524
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Make use of the error parameter in the opening and creating functions in
|
|
|
9ae3a8 |
block/raw-win32.c.
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Max Reitz <mreitz@redhat.com>
|
|
|
9ae3a8 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
9ae3a8 |
(cherry picked from commit c6252b7cea0dfa893cf1f49de3a58f222e910783)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Max Reitz <mreitz@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
block/raw-win32.c | 16 +++++++++++-----
|
|
|
9ae3a8 |
1 file changed, 11 insertions(+), 5 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
block/raw-win32.c | 16 +++++++++++-----
|
|
|
9ae3a8 |
1 files changed, 11 insertions(+), 5 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/block/raw-win32.c b/block/raw-win32.c
|
|
|
9ae3a8 |
index 9ebb083..5a60ca5 100644
|
|
|
9ae3a8 |
--- a/block/raw-win32.c
|
|
|
9ae3a8 |
+++ b/block/raw-win32.c
|
|
|
9ae3a8 |
@@ -250,8 +250,7 @@ static int raw_open(BlockDriverState *bs, QDict *options, int flags,
|
|
|
9ae3a8 |
opts = qemu_opts_create_nofail(&raw_runtime_opts);
|
|
|
9ae3a8 |
qemu_opts_absorb_qdict(opts, options, &local_err);
|
|
|
9ae3a8 |
if (error_is_set(&local_err)) {
|
|
|
9ae3a8 |
- qerror_report_err(local_err);
|
|
|
9ae3a8 |
- error_free(local_err);
|
|
|
9ae3a8 |
+ error_propagate(errp, local_err);
|
|
|
9ae3a8 |
ret = -EINVAL;
|
|
|
9ae3a8 |
goto fail;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
@@ -263,6 +262,7 @@ static int raw_open(BlockDriverState *bs, QDict *options, int flags,
|
|
|
9ae3a8 |
if ((flags & BDRV_O_NATIVE_AIO) && aio == NULL) {
|
|
|
9ae3a8 |
aio = win32_aio_init();
|
|
|
9ae3a8 |
if (aio == NULL) {
|
|
|
9ae3a8 |
+ error_setg(errp, "Could not initialize AIO");
|
|
|
9ae3a8 |
ret = -EINVAL;
|
|
|
9ae3a8 |
goto fail;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
@@ -279,6 +279,7 @@ static int raw_open(BlockDriverState *bs, QDict *options, int flags,
|
|
|
9ae3a8 |
} else {
|
|
|
9ae3a8 |
ret = -EINVAL;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
+ error_setg_errno(errp, -ret, "Could not open file");
|
|
|
9ae3a8 |
goto fail;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
@@ -286,6 +287,7 @@ static int raw_open(BlockDriverState *bs, QDict *options, int flags,
|
|
|
9ae3a8 |
ret = win32_aio_attach(aio, s->hfile);
|
|
|
9ae3a8 |
if (ret < 0) {
|
|
|
9ae3a8 |
CloseHandle(s->hfile);
|
|
|
9ae3a8 |
+ error_setg_errno(errp, -ret, "Could not enable AIO");
|
|
|
9ae3a8 |
goto fail;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
s->aio = aio;
|
|
|
9ae3a8 |
@@ -437,8 +439,10 @@ static int raw_create(const char *filename, QEMUOptionParameter *options,
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
fd = qemu_open(filename, O_WRONLY | O_CREAT | O_TRUNC | O_BINARY,
|
|
|
9ae3a8 |
0644);
|
|
|
9ae3a8 |
- if (fd < 0)
|
|
|
9ae3a8 |
+ if (fd < 0) {
|
|
|
9ae3a8 |
+ error_setg_errno(errp, errno, "Could not create file");
|
|
|
9ae3a8 |
return -EIO;
|
|
|
9ae3a8 |
+ }
|
|
|
9ae3a8 |
set_sparse(fd);
|
|
|
9ae3a8 |
ftruncate(fd, total_size * 512);
|
|
|
9ae3a8 |
qemu_close(fd);
|
|
|
9ae3a8 |
@@ -547,8 +551,7 @@ static int hdev_open(BlockDriverState *bs, QDict *options, int flags,
|
|
|
9ae3a8 |
QemuOpts *opts = qemu_opts_create_nofail(&raw_runtime_opts);
|
|
|
9ae3a8 |
qemu_opts_absorb_qdict(opts, options, &local_err);
|
|
|
9ae3a8 |
if (error_is_set(&local_err)) {
|
|
|
9ae3a8 |
- qerror_report_err(local_err);
|
|
|
9ae3a8 |
- error_free(local_err);
|
|
|
9ae3a8 |
+ error_propagate(errp, local_err);
|
|
|
9ae3a8 |
ret = -EINVAL;
|
|
|
9ae3a8 |
goto done;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
@@ -557,6 +560,7 @@ static int hdev_open(BlockDriverState *bs, QDict *options, int flags,
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
if (strstart(filename, "/dev/cdrom", NULL)) {
|
|
|
9ae3a8 |
if (find_cdrom(device_name, sizeof(device_name)) < 0) {
|
|
|
9ae3a8 |
+ error_setg(errp, "Could not open CD-ROM drive");
|
|
|
9ae3a8 |
ret = -ENOENT;
|
|
|
9ae3a8 |
goto done;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
@@ -583,8 +587,10 @@ static int hdev_open(BlockDriverState *bs, QDict *options, int flags,
|
|
|
9ae3a8 |
int err = GetLastError();
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
if (err == ERROR_ACCESS_DENIED) {
|
|
|
9ae3a8 |
+ error_setg_errno(errp, EACCES, "Could not open device");
|
|
|
9ae3a8 |
ret = -EACCES;
|
|
|
9ae3a8 |
} else {
|
|
|
9ae3a8 |
+ error_setg(errp, "Could not open device");
|
|
|
9ae3a8 |
ret = -1;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
goto done;
|
|
|
9ae3a8 |
--
|
|
|
9ae3a8 |
1.7.1
|
|
|
9ae3a8 |
|