提交 b681072d 编写于 作者: K Kevin Wolf

blockdev: blockdev_init() error conversion

This gives us meaningful error messages for the blockdev-add QMP
command.
Signed-off-by: NKevin Wolf <kwolf@redhat.com>
Reviewed-by: NMax Reitz <mreitz@redhat.com>
Reviewed-by: NEric Blake <eblake@redhat.com>
上级 0ebd24e0
...@@ -272,7 +272,7 @@ static void bdrv_put_ref_bh_schedule(BlockDriverState *bs) ...@@ -272,7 +272,7 @@ static void bdrv_put_ref_bh_schedule(BlockDriverState *bs)
qemu_bh_schedule(s->bh); qemu_bh_schedule(s->bh);
} }
static int parse_block_error_action(const char *buf, bool is_read) static int parse_block_error_action(const char *buf, bool is_read, Error **errp)
{ {
if (!strcmp(buf, "ignore")) { if (!strcmp(buf, "ignore")) {
return BLOCKDEV_ON_ERROR_IGNORE; return BLOCKDEV_ON_ERROR_IGNORE;
...@@ -283,8 +283,8 @@ static int parse_block_error_action(const char *buf, bool is_read) ...@@ -283,8 +283,8 @@ static int parse_block_error_action(const char *buf, bool is_read)
} else if (!strcmp(buf, "report")) { } else if (!strcmp(buf, "report")) {
return BLOCKDEV_ON_ERROR_REPORT; return BLOCKDEV_ON_ERROR_REPORT;
} else { } else {
error_report("'%s' invalid %s error action", error_setg(errp, "'%s' invalid %s error action",
buf, is_read ? "read" : "write"); buf, is_read ? "read" : "write");
return -1; return -1;
} }
} }
...@@ -309,7 +309,8 @@ typedef enum { MEDIA_DISK, MEDIA_CDROM } DriveMediaType; ...@@ -309,7 +309,8 @@ typedef enum { MEDIA_DISK, MEDIA_CDROM } DriveMediaType;
/* Takes the ownership of bs_opts */ /* Takes the ownership of bs_opts */
static DriveInfo *blockdev_init(QDict *bs_opts, static DriveInfo *blockdev_init(QDict *bs_opts,
BlockInterfaceType type) BlockInterfaceType type,
Error **errp)
{ {
const char *buf; const char *buf;
const char *file = NULL; const char *file = NULL;
...@@ -333,15 +334,13 @@ static DriveInfo *blockdev_init(QDict *bs_opts, ...@@ -333,15 +334,13 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
id = qdict_get_try_str(bs_opts, "id"); id = qdict_get_try_str(bs_opts, "id");
opts = qemu_opts_create(&qemu_common_drive_opts, id, 1, &error); opts = qemu_opts_create(&qemu_common_drive_opts, id, 1, &error);
if (error_is_set(&error)) { if (error_is_set(&error)) {
qerror_report_err(error); error_propagate(errp, error);
error_free(error);
return NULL; return NULL;
} }
qemu_opts_absorb_qdict(opts, bs_opts, &error); qemu_opts_absorb_qdict(opts, bs_opts, &error);
if (error_is_set(&error)) { if (error_is_set(&error)) {
qerror_report_err(error); error_propagate(errp, error);
error_free(error);
return NULL; return NULL;
} }
...@@ -361,7 +360,7 @@ static DriveInfo *blockdev_init(QDict *bs_opts, ...@@ -361,7 +360,7 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
if ((buf = qemu_opt_get(opts, "discard")) != NULL) { if ((buf = qemu_opt_get(opts, "discard")) != NULL) {
if (bdrv_parse_discard_flags(buf, &bdrv_flags) != 0) { if (bdrv_parse_discard_flags(buf, &bdrv_flags) != 0) {
error_report("invalid discard option"); error_setg(errp, "invalid discard option");
return NULL; return NULL;
} }
} }
...@@ -383,7 +382,7 @@ static DriveInfo *blockdev_init(QDict *bs_opts, ...@@ -383,7 +382,7 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
} else if (!strcmp(buf, "threads")) { } else if (!strcmp(buf, "threads")) {
/* this is the default */ /* this is the default */
} else { } else {
error_report("invalid aio option"); error_setg(errp, "invalid aio option");
return NULL; return NULL;
} }
} }
...@@ -399,7 +398,7 @@ static DriveInfo *blockdev_init(QDict *bs_opts, ...@@ -399,7 +398,7 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
drv = bdrv_find_format(buf); drv = bdrv_find_format(buf);
if (!drv) { if (!drv) {
error_report("'%s' invalid format", buf); error_setg(errp, "'%s' invalid format", buf);
return NULL; return NULL;
} }
} }
...@@ -435,20 +434,20 @@ static DriveInfo *blockdev_init(QDict *bs_opts, ...@@ -435,20 +434,20 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
cfg.op_size = qemu_opt_get_number(opts, "throttling.iops-size", 0); cfg.op_size = qemu_opt_get_number(opts, "throttling.iops-size", 0);
if (!check_throttle_config(&cfg, &error)) { if (!check_throttle_config(&cfg, &error)) {
error_report("%s", error_get_pretty(error)); error_propagate(errp, error);
error_free(error);
return NULL; return NULL;
} }
on_write_error = BLOCKDEV_ON_ERROR_ENOSPC; on_write_error = BLOCKDEV_ON_ERROR_ENOSPC;
if ((buf = qemu_opt_get(opts, "werror")) != NULL) { if ((buf = qemu_opt_get(opts, "werror")) != NULL) {
if (type != IF_IDE && type != IF_SCSI && type != IF_VIRTIO && type != IF_NONE) { if (type != IF_IDE && type != IF_SCSI && type != IF_VIRTIO && type != IF_NONE) {
error_report("werror is not supported by this bus type"); error_setg(errp, "werror is not supported by this bus type");
return NULL; return NULL;
} }
on_write_error = parse_block_error_action(buf, 0); on_write_error = parse_block_error_action(buf, 0, &error);
if (on_write_error < 0) { if (error_is_set(&error)) {
error_propagate(errp, error);
return NULL; return NULL;
} }
} }
...@@ -460,8 +459,9 @@ static DriveInfo *blockdev_init(QDict *bs_opts, ...@@ -460,8 +459,9 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
return NULL; return NULL;
} }
on_read_error = parse_block_error_action(buf, 1); on_read_error = parse_block_error_action(buf, 1, &error);
if (on_read_error < 0) { if (error_is_set(&error)) {
error_propagate(errp, error);
return NULL; return NULL;
} }
} }
...@@ -514,8 +514,9 @@ static DriveInfo *blockdev_init(QDict *bs_opts, ...@@ -514,8 +514,9 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
ret = bdrv_open(dinfo->bdrv, file, bs_opts, bdrv_flags, drv, &error); ret = bdrv_open(dinfo->bdrv, file, bs_opts, bdrv_flags, drv, &error);
if (ret < 0) { if (ret < 0) {
error_report("could not open disk image %s: %s", error_setg(errp, "could not open disk image %s: %s",
file ?: dinfo->id, error_get_pretty(error)); file ?: dinfo->id, error_get_pretty(error));
error_free(error);
goto err; goto err;
} }
...@@ -862,9 +863,15 @@ DriveInfo *drive_init(QemuOpts *all_opts, BlockInterfaceType block_default_type) ...@@ -862,9 +863,15 @@ DriveInfo *drive_init(QemuOpts *all_opts, BlockInterfaceType block_default_type)
} }
/* Actual block device init: Functionality shared with blockdev-add */ /* Actual block device init: Functionality shared with blockdev-add */
dinfo = blockdev_init(bs_opts, type); dinfo = blockdev_init(bs_opts, type, &local_err);
if (dinfo == NULL) { if (dinfo == NULL) {
if (error_is_set(&local_err)) {
qerror_report_err(local_err);
error_free(local_err);
}
goto fail; goto fail;
} else {
assert(!error_is_set(&local_err));
} }
/* Set legacy DriveInfo fields */ /* Set legacy DriveInfo fields */
...@@ -2155,7 +2162,6 @@ void qmp_blockdev_add(BlockdevOptions *options, Error **errp) ...@@ -2155,7 +2162,6 @@ void qmp_blockdev_add(BlockdevOptions *options, Error **errp)
QmpOutputVisitor *ov = qmp_output_visitor_new(); QmpOutputVisitor *ov = qmp_output_visitor_new();
QObject *obj; QObject *obj;
QDict *qdict; QDict *qdict;
DriveInfo *dinfo;
Error *local_err = NULL; Error *local_err = NULL;
/* Require an ID in the top level */ /* Require an ID in the top level */
...@@ -2189,9 +2195,9 @@ void qmp_blockdev_add(BlockdevOptions *options, Error **errp) ...@@ -2189,9 +2195,9 @@ void qmp_blockdev_add(BlockdevOptions *options, Error **errp)
qdict_flatten(qdict); qdict_flatten(qdict);
dinfo = blockdev_init(qdict, IF_NONE); blockdev_init(qdict, IF_NONE, &local_err);
if (!dinfo) { if (error_is_set(&local_err)) {
error_setg(errp, "Could not open image"); error_propagate(errp, local_err);
goto fail; goto fail;
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册