[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 15/17] blockdev: Remove 'media' parameter from block
From: |
Kevin Wolf |
Subject: |
[Qemu-devel] [PATCH 15/17] blockdev: Remove 'media' parameter from blockdev_init() |
Date: |
Fri, 20 Sep 2013 13:54:28 +0200 |
The remaining users shouldn't be there with blockdev-add and are easy to
move to drive_init().
Bonus bug fix: As a side effect, CD-ROM drives can now use block drivers
on the read-only whitelist without explicitly specifying read-only=on,
even if a format is explicitly specified.
Signed-off-by: Kevin Wolf <address@hidden>
---
blockdev.c | 40 +++++++++++++++-------------------------
1 file changed, 15 insertions(+), 25 deletions(-)
diff --git a/blockdev.c b/blockdev.c
index ed4ba65..afdeb34 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -306,8 +306,7 @@ typedef enum { MEDIA_DISK, MEDIA_CDROM } DriveMediaType;
/* Takes the ownership of bs_opts */
static DriveInfo *blockdev_init(QDict *bs_opts,
- BlockInterfaceType type,
- DriveMediaType media)
+ BlockInterfaceType type)
{
const char *buf;
const char *file = NULL;
@@ -489,22 +488,6 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
bdrv_set_io_limits(dinfo->bdrv, &cfg);
}
- switch(type) {
- case IF_IDE:
- case IF_SCSI:
- case IF_XEN:
- case IF_NONE:
- dinfo->media_cd = media == MEDIA_CDROM;
- break;
- case IF_SD:
- case IF_FLOPPY:
- case IF_PFLASH:
- case IF_MTD:
- case IF_VIRTIO:
- break;
- default:
- abort();
- }
if (!file || !*file) {
if (has_driver_specific_opts) {
file = NULL;
@@ -526,11 +509,6 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
bdrv_flags |= BDRV_O_INCOMING;
}
- if (media == MEDIA_CDROM) {
- /* CDROM is fine for any interface, don't check. */
- ro = 1;
- }
-
bdrv_flags |= ro ? 0 : BDRV_O_RDWR;
if (ro && copy_on_read) {
@@ -714,6 +692,7 @@ DriveInfo *drive_init(QemuOpts *all_opts,
BlockInterfaceType block_default_type)
media = MEDIA_DISK;
} else if (!strcmp(value, "cdrom")) {
media = MEDIA_CDROM;
+ qdict_put(bs_opts, "read-only", qstring_from_str("on"));
} else {
error_report("'%s' invalid media", value);
goto fail;
@@ -861,7 +840,7 @@ DriveInfo *drive_init(QemuOpts *all_opts,
BlockInterfaceType block_default_type)
}
/* Actual block device init: Functionality shared with blockdev-add */
- dinfo = blockdev_init(bs_opts, type, media);
+ dinfo = blockdev_init(bs_opts, type);
if (dinfo == NULL) {
goto fail;
}
@@ -879,6 +858,17 @@ DriveInfo *drive_init(QemuOpts *all_opts,
BlockInterfaceType block_default_type)
dinfo->unit = unit_id;
dinfo->devaddr = devaddr;
+ switch(type) {
+ case IF_IDE:
+ case IF_SCSI:
+ case IF_XEN:
+ case IF_NONE:
+ dinfo->media_cd = media == MEDIA_CDROM;
+ break;
+ default:
+ break;
+ }
+
fail:
qemu_opts_del(legacy_opts);
return dinfo;
@@ -2173,7 +2163,7 @@ void qmp_blockdev_add(BlockdevOptions *options, Error
**errp)
qdict_flatten(qdict);
- dinfo = blockdev_init(qdict, IF_NONE, MEDIA_DISK);
+ dinfo = blockdev_init(qdict, IF_NONE);
if (!dinfo) {
error_setg(errp, "Could not open image");
goto fail;
--
1.8.1.4
- Re: [Qemu-devel] [PATCH 10/17] blockdev: Move parsing of 'boot' option to drive_init, (continued)
- [Qemu-devel] [PATCH 11/17] blockdev: Move bus/unit/index processing to drive_init, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 12/17] blockdev: Move virtio-blk device creation to drive_init, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 13/17] blockdev: Remove IF_* check for read-only blockdev_init, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 14/17] qemu-iotests: Check autodel behaviour for device_del, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 15/17] blockdev: Remove 'media' parameter from blockdev_init(),
Kevin Wolf <=
- [Qemu-devel] [PATCH 16/17] blockdev: Don't disable COR automatically with blockdev-add, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 17/17] blockdev: blockdev_init() error conversion, Kevin Wolf, 2013/09/20