[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 1/3] qemu-io: add aio_discard
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
[PATCH v4 1/3] qemu-io: add aio_discard |
Date: |
Fri, 19 Mar 2021 13:08:09 +0300 |
Add aio_discard command like existing aio_write. It will be used in
further test.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
---
qemu-io-cmds.c | 117 +++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 117 insertions(+)
diff --git a/qemu-io-cmds.c b/qemu-io-cmds.c
index 97611969cb..28b5c3c092 100644
--- a/qemu-io-cmds.c
+++ b/qemu-io-cmds.c
@@ -1332,6 +1332,7 @@ struct aio_ctx {
BlockBackend *blk;
QEMUIOVector qiov;
int64_t offset;
+ int64_t discard_bytes;
char *buf;
bool qflag;
bool vflag;
@@ -1343,6 +1344,34 @@ struct aio_ctx {
struct timespec t1;
};
+static void aio_discard_done(void *opaque, int ret)
+{
+ struct aio_ctx *ctx = opaque;
+ struct timespec t2;
+
+ clock_gettime(CLOCK_MONOTONIC, &t2);
+
+
+ if (ret < 0) {
+ printf("aio_discard failed: %s\n", strerror(-ret));
+ block_acct_failed(blk_get_stats(ctx->blk), &ctx->acct);
+ goto out;
+ }
+
+ block_acct_done(blk_get_stats(ctx->blk), &ctx->acct);
+
+ if (ctx->qflag) {
+ goto out;
+ }
+
+ /* Finally, report back -- -C gives a parsable format */
+ t2 = tsub(t2, ctx->t1);
+ print_report("discarded", &t2, ctx->offset, ctx->discard_bytes,
+ ctx->discard_bytes, 1, ctx->Cflag);
+out:
+ g_free(ctx);
+}
+
static void aio_write_done(void *opaque, int ret)
{
struct aio_ctx *ctx = opaque;
@@ -1671,6 +1700,93 @@ static int aio_write_f(BlockBackend *blk, int argc, char
**argv)
return 0;
}
+static void aio_discard_help(void)
+{
+ printf(
+"\n"
+" asynchronously discards a range of bytes from the given offset\n"
+"\n"
+" Example:\n"
+" 'aio_discard 0 64k' - discards 64K at start of a disk\n"
+"\n"
+" Note that due to its asynchronous nature, this command will be\n"
+" considered successful once the request is submitted, independently\n"
+" of potential I/O errors or pattern mismatches.\n"
+" -C, -- report statistics in a machine parsable format\n"
+" -i, -- treat request as invalid, for exercising stats\n"
+" -q, -- quiet mode, do not show I/O statistics\n"
+"\n");
+}
+
+static int aio_discard_f(BlockBackend *blk, int argc, char **argv);
+
+static const cmdinfo_t aio_discard_cmd = {
+ .name = "aio_discard",
+ .cfunc = aio_discard_f,
+ .perm = BLK_PERM_WRITE,
+ .argmin = 2,
+ .argmax = -1,
+ .args = "[-Ciq] off len",
+ .oneline = "asynchronously discards a number of bytes",
+ .help = aio_discard_help,
+};
+
+static int aio_discard_f(BlockBackend *blk, int argc, char **argv)
+{
+ int ret;
+ int c;
+ struct aio_ctx *ctx = g_new0(struct aio_ctx, 1);
+
+ ctx->blk = blk;
+ while ((c = getopt(argc, argv, "Ciq")) != -1) {
+ switch (c) {
+ case 'C':
+ ctx->Cflag = true;
+ break;
+ case 'q':
+ ctx->qflag = true;
+ break;
+ case 'i':
+ printf("injecting invalid discard request\n");
+ block_acct_invalid(blk_get_stats(blk), BLOCK_ACCT_UNMAP);
+ g_free(ctx);
+ return 0;
+ default:
+ g_free(ctx);
+ qemuio_command_usage(&aio_write_cmd);
+ return -EINVAL;
+ }
+ }
+
+ if (optind != argc - 2) {
+ g_free(ctx);
+ qemuio_command_usage(&aio_write_cmd);
+ return -EINVAL;
+ }
+
+ ctx->offset = cvtnum(argv[optind]);
+ if (ctx->offset < 0) {
+ ret = ctx->offset;
+ print_cvtnum_err(ret, argv[optind]);
+ g_free(ctx);
+ return ret;
+ }
+ optind++;
+
+ ctx->discard_bytes = cvtnum(argv[optind]);
+ if (ctx->discard_bytes < 0) {
+ ret = ctx->discard_bytes;
+ print_cvtnum_err(ret, argv[optind]);
+ g_free(ctx);
+ return ret;
+ }
+
+ blk_aio_pdiscard(blk, ctx->offset, ctx->discard_bytes,
+ aio_discard_done, ctx);
+
+ return 0;
+}
+
static int aio_flush_f(BlockBackend *blk, int argc, char **argv)
{
BlockAcctCookie cookie;
@@ -2494,6 +2610,7 @@ static void __attribute((constructor))
init_qemuio_commands(void)
qemuio_add_command(&readv_cmd);
qemuio_add_command(&write_cmd);
qemuio_add_command(&writev_cmd);
+ qemuio_add_command(&aio_discard_cmd);
qemuio_add_command(&aio_read_cmd);
qemuio_add_command(&aio_write_cmd);
qemuio_add_command(&aio_flush_cmd);
--
2.29.2
- [PATCH v4 0/3] qcow2: fix parallel rewrite and discard (rw-lock), Vladimir Sementsov-Ogievskiy, 2021/03/19
- [PATCH v4 1/3] qemu-io: add aio_discard,
Vladimir Sementsov-Ogievskiy <=
- [PATCH v4 2/3] iotests: add qcow2-discard-during-rewrite, Vladimir Sementsov-Ogievskiy, 2021/03/19
- [PATCH v4 3/3] block/qcow2: introduce discard_rw_lock: fix discarding host clusters, Vladimir Sementsov-Ogievskiy, 2021/03/19
- Re: [PATCH v4 for-6.0? 0/3] qcow2: fix parallel rewrite and discard (rw-lock), Vladimir Sementsov-Ogievskiy, 2021/03/25
- Re: [PATCH v4 for-6.0? 0/3] qcow2: fix parallel rewrite and discard (rw-lock), Max Reitz, 2021/03/30
- Re: [PATCH v4 for-6.0? 0/3] qcow2: fix parallel rewrite and discard (rw-lock), Vladimir Sementsov-Ogievskiy, 2021/03/30
- Re: [PATCH v4 for-6.0? 0/3] qcow2: fix parallel rewrite and discard (rw-lock), Max Reitz, 2021/03/30
- Re: [PATCH v4 for-6.0? 0/3] qcow2: fix parallel rewrite and discard (rw-lock), Vladimir Sementsov-Ogievskiy, 2021/03/30
- Re: [PATCH v4 for-6.0? 0/3] qcow2: fix parallel rewrite and discard (rw-lock), Max Reitz, 2021/03/30
- Re: [PATCH v4 for-6.0? 0/3] qcow2: fix parallel rewrite and discard (rw-lock), Vladimir Sementsov-Ogievskiy, 2021/03/30
- Re: [PATCH v4 for-6.0? 0/3] qcow2: fix parallel rewrite and discard (rw-lock), Vladimir Sementsov-Ogievskiy, 2021/03/31