Skip to contentSkip to conversation

virtio-blk: miscellaneous changes

PATCH Applied Mark Kanda 2 participants 4 messages Dec 11, 2017 – Dec 12, 2017

lore
Share & tools
lore

Mark Kanda

author
#0 lore raw

v2 cover virtio-blk: miscellaneous changes

v2: add check for maximum queue size [Stefan]

This series is for two minor virtio-blk changes. The first patch makes the virtio-blk queue size user configurable. The second patch rejects logical block size > physical block configurations (similar to a recent change in virtio-scsi).

Mark Kanda (2):
virtio-blk: make queue size configurable
virtio-blk: reject configs with logical block size > physical block
size

 hw/block/virtio-blk.c          | 17 ++++++++++++++++-
 include/hw/virtio/virtio-blk.h |  1 +
 2 files changed, 17 insertions(+), 1 deletion(-)
Signature · 1 line
1.8.3.1
author
#1 lore raw

PATCH v2 1/2 virtio-blk: make queue size configurable

Depending on the configuration, it can be beneficial to adjust the virtio-blk queue size to something other than the current default of 128. Add a new property to make the queue size configurable.

Signed-off-by Mark Kanda <[email protected]>
Reviewed-by Karl Heubaum <[email protected]>
Reviewed-by Martin K. Petersen <[email protected]>
Reviewed-by Ameya More <[email protected]>
---
 hw/block/virtio-blk.c          | 10 +++++++++-
 include/hw/virtio/virtio-blk.h |  1 +
 2 files changed, 10 insertions(+), 1 deletion(-)
hw/block/virtio-blk.c +9 -1
index 05d1440..fb59f57 100644
@@ -928,6 +928,13 @@ static void virtio_blk_device_realize(DeviceState *dev, Error **errp)
         error_setg(errp, "num-queues property must be larger than 0");
         return;
     }
+    if (!is_power_of_2(conf->queue_size) ||
+        conf->queue_size > VIRTQUEUE_MAX_SIZE) {
+        error_setg(errp, "invalid queue-size property (%" PRIu16 "), "
+                   "must be a power of 2 (max %d)",
+                   conf->queue_size, VIRTQUEUE_MAX_SIZE);
+        return;
+    }
 
     blkconf_serial(&conf->conf, &conf->serial);
     blkconf_apply_backend_options(&conf->conf,

@@ -953,7 +960,7 @@ static void virtio_blk_device_realize(DeviceState *dev, Error **errp)
     s->sector_mask = (s->conf.conf.logical_block_size / BDRV_SECTOR_SIZE) - 1;
 
     for (i = 0; i < conf->num_queues; i++) {
-        virtio_add_queue(vdev, 128, virtio_blk_handle_output);
+        virtio_add_queue(vdev, conf->queue_size, virtio_blk_handle_output);
     }
     virtio_blk_data_plane_create(vdev, conf, &s->dataplane, &err);
     if (err != NULL) {

@@ -1012,6 +1019,7 @@ static Property virtio_blk_properties[] = {
     DEFINE_PROP_BIT("request-merging", VirtIOBlock, conf.request_merging, 0,
                     true),
     DEFINE_PROP_UINT16("num-queues", VirtIOBlock, conf.num_queues, 1),
+    DEFINE_PROP_UINT16("queue-size", VirtIOBlock, conf.queue_size, 128),
     DEFINE_PROP_LINK("iothread", VirtIOBlock, conf.iothread, TYPE_IOTHREAD,
                      IOThread *),
     DEFINE_PROP_END_OF_LIST(),
include/hw/virtio/virtio-blk.h +1 -0
index d3c8a6f..5117431 100644
@@ -39,6 +39,7 @@ struct VirtIOBlkConf
     uint32_t config_wce;
     uint32_t request_merging;
     uint16_t num_queues;
+    uint16_t queue_size;
 };
 
 struct VirtIOBlockDataPlane;
Signature · 1 line
1.8.3.1
author
#2 lore raw

PATCH v2 2/2 virtio-blk: reject configs with logical block size > physical block size

In reply to Mark Kanda

virtio-blk logical block size should never be larger than physical block size because it doesn't make sense to have such configurations. QEMU doesn't have a way to effectively express this condition; the best it can do is report the physical block exponent as 0 - indicating the logical block size equals the physical block size.

This is identical to commit 3da023b5827543ee4c022986ea2ad9d1274410b2 but applied to virtio-blk (instead of virtio-scsi).

Signed-off-by Mark Kanda <[email protected]>
Reviewed-by Konrad Rzeszutek Wilk <[email protected]>
Reviewed-by Ameya More <[email protected]>
Reviewed-by Martin K. Petersen <[email protected]>
Reviewed-by Stefan Hajnoczi <[email protected]>
---
 hw/block/virtio-blk.c | 7 +++++++
 1 file changed, 7 insertions(+)
hw/block/virtio-blk.c +7 -0
index fb59f57..36c7608 100644
@@ -952,6 +952,13 @@ static void virtio_blk_device_realize(DeviceState *dev, Error **errp)
     }
     blkconf_blocksizes(&conf->conf);
 
+    if (conf->conf.logical_block_size >
+        conf->conf.physical_block_size) {
+        error_setg(errp,
+                   "logical_block_size > physical_block_size not supported");
+        return;
+    }
+
     virtio_init(vdev, "virtio-blk", VIRTIO_ID_BLOCK,
                 sizeof(struct virtio_blk_config));
Signature · 1 line
1.8.3.1
Applied
#3 lore raw
In reply to Mark Kanda
Quoted 7 lines · Mark Kanda

v2: add check for maximum queue size [Stefan]

This series is for two minor virtio-blk changes. The first patch makes the virtio-blk queue size user configurable. The second patch rejects logical block size > physical block configurations (similar to a recent change in virtio-scsi).

Quoted 12 lines

Mark Kanda (2):
virtio-blk: make queue size configurable
virtio-blk: reject configs with logical block size > physical block
size

 hw/block/virtio-blk.c          | 17 ++++++++++++++++-
 include/hw/virtio/virtio-blk.h |  1 +
 2 files changed, 17 insertions(+), 1 deletion(-)
Signature · 1 line
1.8.3.1

Thanks, applied to my block-next tree:
https://github.com/stefanha/qemu/commits/block-next

Stefan