[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 1/3] block/blkdebug: add blocksize parameter
From: |
Kit Westneat |
Subject: |
[PATCH v3 1/3] block/blkdebug: add blocksize parameter |
Date: |
Tue, 25 May 2021 19:46:05 +0000 |
Allow users to specify the block size of the qdev for testing purposes.
Signed-off-by: Kit Westneat <kit.westneat@gmail.com>
---
block/blkdebug.c | 29 +++++++++++++++++++++++++++++
qapi/block-core.json | 4 ++++
2 files changed, 33 insertions(+)
diff --git a/block/blkdebug.c b/block/blkdebug.c
index 2c0b9b0ee8..d5f589920c 100644
--- a/block/blkdebug.c
+++ b/block/blkdebug.c
@@ -47,6 +47,7 @@ typedef struct BDRVBlkdebugState {
uint64_t max_write_zero;
uint64_t opt_discard;
uint64_t max_discard;
+ uint64_t blocksize;
uint64_t take_child_perms;
uint64_t unshare_child_perms;
@@ -455,6 +456,11 @@ static QemuOptsList runtime_opts = {
.type = QEMU_OPT_SIZE,
.help = "Maximum discard size in bytes",
},
+ {
+ .name = "blocksize",
+ .type = QEMU_OPT_SIZE,
+ .help = "Blocksize of device",
+ },
{ /* end of list */ }
},
};
@@ -562,6 +568,14 @@ static int blkdebug_open(BlockDriverState *bs, QDict
*options, int flags,
goto out;
}
+ s->blocksize = qemu_opt_get_size(opts, "blocksize", 512);
+ if (s->blocksize && (s->blocksize >= INT_MAX ||
+ !is_power_of_2(s->blocksize))) {
+ error_setg(errp, "Cannot meet constraints with blocksize %" PRIu64,
+ s->blocksize);
+ goto out;
+ }
+
bdrv_debug_event(bs, BLKDBG_NONE);
ret = 0;
@@ -984,6 +998,19 @@ static void blkdebug_refresh_limits(BlockDriverState *bs,
Error **errp)
}
}
+static int blkdebug_probe_blocksizes(BlockDriverState *bs, BlockSizes *bsz)
+{
+ BDRVBlkdebugState *s = bs->opaque;
+
+ if (!s->blocksize) {
+ return 0;
+ }
+
+ bsz->phys = s->blocksize;
+ bsz->log = s->blocksize;
+ return 0;
+}
+
static int blkdebug_reopen_prepare(BDRVReopenState *reopen_state,
BlockReopenQueue *queue, Error **errp)
{
@@ -1010,6 +1037,7 @@ static const char *const blkdebug_strong_runtime_opts[] =
{
"inject-error.",
"set-state.",
"align",
+ "blocksize",
"max-transfer",
"opt-write-zero",
"max-write-zero",
@@ -1034,6 +1062,7 @@ static BlockDriver bdrv_blkdebug = {
.bdrv_getlength = blkdebug_getlength,
.bdrv_refresh_filename = blkdebug_refresh_filename,
.bdrv_refresh_limits = blkdebug_refresh_limits,
+ .bdrv_probe_blocksizes = blkdebug_probe_blocksizes,
.bdrv_co_preadv = blkdebug_co_preadv,
.bdrv_co_pwritev = blkdebug_co_pwritev,
diff --git a/qapi/block-core.json b/qapi/block-core.json
index 2ea294129e..1b1f2692ef 100644
--- a/qapi/block-core.json
+++ b/qapi/block-core.json
@@ -3404,6 +3404,9 @@
# on how the blkdebug node is used). Defaults
# to none. (since 5.0)
#
+# @blocksize: blocksize of device in bytes, must be
+# positive power of 2 (since 6.1)
+#
# Since: 2.9
##
{ 'struct': 'BlockdevOptionsBlkdebug',
@@ -3412,6 +3415,7 @@
'*align': 'int', '*max-transfer': 'int32',
'*opt-write-zero': 'int32', '*max-write-zero': 'int32',
'*opt-discard': 'int32', '*max-discard': 'int32',
+ '*blocksize': 'int',
'*inject-error': ['BlkdebugInjectErrorOptions'],
'*set-state': ['BlkdebugSetStateOptions'],
'*take-child-perms': ['BlockPermission'],
--
2.26.3