diff options
| author | Max Reitz <mreitz@redhat.com> | 2015-12-11 16:23:05 +0100 |
|---|---|---|
| committer | Peter Maydell <peter.maydell@linaro.org> | 2015-12-11 15:39:29 +0000 |
| commit | 6e0abc251dd4f8eba1f53656dfede12e5840e83b (patch) | |
| tree | 807cf36d47b1f459d1a86e895dadb9f55f5973b2 /blockdev.c | |
| parent | 3fd3c4b37c116cce7e9810fcc15a0a2cf15115a5 (diff) | |
blockdev: Mark {insert, remove}-medium experimental
While in the long term we want throttling to be its own block filter
BDS, in the short term we want it to be part of the BB instead of a BDS;
even in the long term we may want legacy throttling to be automatically
tied to the BB.
blockdev-insert-medium and blockdev-remove-medium do not retain
throttling information in the BB (deliberately so). Therefore, using
them means tying this information to a BDS, which would break the model
described above. (The same applies to other flags such as
detect_zeroes.) We probably want to move this information to the BB or
its own filter BDS before blockdev-{insert,remove}-medium can be
considered completely stable.
Therefore, mark these functions experimental for the time being.
Suggested-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Acked-by: Markus Armbruster <armbru@redhat.com>
Acked-by: Kevin Wolf <kwolf@redhat.com>
Message-id: 1449847385-13986-2-git-send-email-mreitz@redhat.com
Reviewed-by: Eric Blake <eblake@redhat.com>
[PMM: fixed format nit (underlining) in qmp-commands.hx]
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'blockdev.c')
| -rw-r--r-- | blockdev.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/blockdev.c b/blockdev.c index 313841b0b4..80932e8d8d 100644 --- a/blockdev.c +++ b/blockdev.c @@ -2257,7 +2257,7 @@ void qmp_eject(const char *device, bool has_force, bool force, Error **errp) return; } - qmp_blockdev_remove_medium(device, errp); + qmp_x_blockdev_remove_medium(device, errp); } void qmp_block_passwd(bool has_device, const char *device, @@ -2343,7 +2343,7 @@ void qmp_blockdev_close_tray(const char *device, Error **errp) blk_dev_change_media_cb(blk, true); } -void qmp_blockdev_remove_medium(const char *device, Error **errp) +void qmp_x_blockdev_remove_medium(const char *device, Error **errp) { BlockBackend *blk; BlockDriverState *bs; @@ -2430,8 +2430,8 @@ static void qmp_blockdev_insert_anon_medium(const char *device, QTAILQ_INSERT_TAIL(&bdrv_states, bs, device_list); } -void qmp_blockdev_insert_medium(const char *device, const char *node_name, - Error **errp) +void qmp_x_blockdev_insert_medium(const char *device, const char *node_name, + Error **errp) { BlockDriverState *bs; @@ -2520,7 +2520,7 @@ void qmp_blockdev_change_medium(const char *device, const char *filename, goto fail; } - qmp_blockdev_remove_medium(device, &err); + qmp_x_blockdev_remove_medium(device, &err); if (err) { error_propagate(errp, err); goto fail; |