2010-09-21 16:43:03 +02:00
|
|
|
/*
|
|
|
|
* Block protocol for block driver correctness testing
|
|
|
|
*
|
|
|
|
* Copyright (C) 2010 IBM, Corp.
|
|
|
|
*
|
|
|
|
* This work is licensed under the terms of the GNU GPL, version 2 or later.
|
|
|
|
* See the COPYING file in the top-level directory.
|
|
|
|
*/
|
|
|
|
|
2016-01-18 19:01:42 +01:00
|
|
|
#include "qemu/osdep.h"
|
include/qemu/osdep.h: Don't include qapi/error.h
Commit 57cb38b included qapi/error.h into qemu/osdep.h to get the
Error typedef. Since then, we've moved to include qemu/osdep.h
everywhere. Its file comment explains: "To avoid getting into
possible circular include dependencies, this file should not include
any other QEMU headers, with the exceptions of config-host.h,
compiler.h, os-posix.h and os-win32.h, all of which are doing a
similar job to this file and are under similar constraints."
qapi/error.h doesn't do a similar job, and it doesn't adhere to
similar constraints: it includes qapi-types.h. That's in excess of
100KiB of crap most .c files don't actually need.
Add the typedef to qemu/typedefs.h, and include that instead of
qapi/error.h. Include qapi/error.h in .c files that need it and don't
get it now. Include qapi-types.h in qom/object.h for uint16List.
Update scripts/clean-includes accordingly. Update it further to match
reality: replace config.h by config-target.h, add sysemu/os-posix.h,
sysemu/os-win32.h. Update the list of includes in the qemu/osdep.h
comment quoted above similarly.
This reduces the number of objects depending on qapi/error.h from "all
of them" to less than a third. Unfortunately, the number depending on
qapi-types.h shrinks only a little. More work is needed for that one.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
[Fix compilation without the spice devel packages. - Paolo]
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2016-03-14 09:01:28 +01:00
|
|
|
#include "qapi/error.h"
|
2012-12-17 18:20:00 +01:00
|
|
|
#include "qemu/sockets.h" /* for EINPROGRESS on Windows */
|
2022-12-21 14:35:49 +01:00
|
|
|
#include "block/block-io.h"
|
2012-12-17 18:19:44 +01:00
|
|
|
#include "block/block_int.h"
|
2014-07-18 20:24:58 +02:00
|
|
|
#include "qapi/qmp/qdict.h"
|
|
|
|
#include "qapi/qmp/qstring.h"
|
2016-03-20 18:16:19 +01:00
|
|
|
#include "qemu/cutils.h"
|
2019-05-23 16:35:07 +02:00
|
|
|
#include "qemu/module.h"
|
2018-02-01 12:18:46 +01:00
|
|
|
#include "qemu/option.h"
|
2022-02-26 19:07:23 +01:00
|
|
|
#include "qemu/memalign.h"
|
2010-09-21 16:43:03 +02:00
|
|
|
|
|
|
|
typedef struct {
|
2015-06-16 11:13:47 +02:00
|
|
|
BdrvChild *test_file;
|
2010-09-21 16:43:03 +02:00
|
|
|
} BDRVBlkverifyState;
|
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
typedef struct BlkverifyRequest {
|
|
|
|
Coroutine *co;
|
|
|
|
BlockDriverState *bs;
|
2010-09-21 16:43:03 +02:00
|
|
|
|
|
|
|
/* Request metadata */
|
|
|
|
bool is_write;
|
2016-11-04 21:13:45 +01:00
|
|
|
uint64_t offset;
|
|
|
|
uint64_t bytes;
|
|
|
|
int flags;
|
2010-09-21 16:43:03 +02:00
|
|
|
|
2020-12-11 19:39:33 +01:00
|
|
|
int (*request_fn)(BdrvChild *, int64_t, int64_t, QEMUIOVector *,
|
2016-11-04 21:13:45 +01:00
|
|
|
BdrvRequestFlags);
|
2010-09-21 16:43:03 +02:00
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
int ret; /* test image result */
|
|
|
|
int raw_ret; /* raw image result */
|
2010-09-21 16:43:03 +02:00
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
unsigned int done; /* completion counter */
|
2010-09-21 16:43:03 +02:00
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
QEMUIOVector *qiov; /* user I/O vector */
|
|
|
|
QEMUIOVector *raw_qiov; /* cloned I/O vector for raw file */
|
|
|
|
} BlkverifyRequest;
|
2010-09-21 16:43:03 +02:00
|
|
|
|
2022-02-20 17:39:25 +01:00
|
|
|
static void G_GNUC_PRINTF(2, 3) blkverify_err(BlkverifyRequest *r,
|
2010-09-24 21:02:05 +02:00
|
|
|
const char *fmt, ...)
|
2010-09-21 16:43:03 +02:00
|
|
|
{
|
|
|
|
va_list ap;
|
|
|
|
|
|
|
|
va_start(ap, fmt);
|
2016-11-04 21:13:45 +01:00
|
|
|
fprintf(stderr, "blkverify: %s offset=%" PRId64 " bytes=%" PRId64 " ",
|
|
|
|
r->is_write ? "write" : "read", r->offset, r->bytes);
|
2010-09-21 16:43:03 +02:00
|
|
|
vfprintf(stderr, fmt, ap);
|
|
|
|
fprintf(stderr, "\n");
|
|
|
|
va_end(ap);
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Valid blkverify filenames look like blkverify:path/to/raw_image:path/to/image */
|
2013-04-10 14:40:28 +02:00
|
|
|
static void blkverify_parse_filename(const char *filename, QDict *options,
|
|
|
|
Error **errp)
|
2010-09-21 16:43:03 +02:00
|
|
|
{
|
2013-04-10 14:40:28 +02:00
|
|
|
const char *c;
|
|
|
|
QString *raw_path;
|
|
|
|
|
2010-09-21 16:43:03 +02:00
|
|
|
|
|
|
|
/* Parse the blkverify: prefix */
|
2013-04-10 14:40:28 +02:00
|
|
|
if (!strstart(filename, "blkverify:", &filename)) {
|
2013-12-20 19:28:17 +01:00
|
|
|
/* There was no prefix; therefore, all options have to be already
|
|
|
|
present in the QDict (except for the filename) */
|
2017-04-27 23:58:17 +02:00
|
|
|
qdict_put_str(options, "x-image", filename);
|
2013-04-10 14:40:28 +02:00
|
|
|
return;
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
/* Parse the raw image filename */
|
|
|
|
c = strchr(filename, ':');
|
|
|
|
if (c == NULL) {
|
2013-04-10 14:40:28 +02:00
|
|
|
error_setg(errp, "blkverify requires raw copy and original image path");
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* TODO Implement option pass-through and set raw.filename here */
|
2018-07-27 08:22:04 +02:00
|
|
|
raw_path = qstring_from_substr(filename, 0, c - filename);
|
2013-04-10 14:40:28 +02:00
|
|
|
qdict_put(options, "x-raw", raw_path);
|
|
|
|
|
|
|
|
/* TODO Allow multi-level nesting and set file.filename here */
|
|
|
|
filename = c + 1;
|
2017-04-27 23:58:17 +02:00
|
|
|
qdict_put_str(options, "x-image", filename);
|
2013-04-10 14:40:28 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
static QemuOptsList runtime_opts = {
|
|
|
|
.name = "blkverify",
|
|
|
|
.head = QTAILQ_HEAD_INITIALIZER(runtime_opts.head),
|
|
|
|
.desc = {
|
|
|
|
{
|
|
|
|
.name = "x-raw",
|
|
|
|
.type = QEMU_OPT_STRING,
|
|
|
|
.help = "[internal use only, will be removed]",
|
|
|
|
},
|
|
|
|
{
|
|
|
|
.name = "x-image",
|
|
|
|
.type = QEMU_OPT_STRING,
|
|
|
|
.help = "[internal use only, will be removed]",
|
|
|
|
},
|
|
|
|
{ /* end of list */ }
|
|
|
|
},
|
|
|
|
};
|
|
|
|
|
2013-09-05 14:22:29 +02:00
|
|
|
static int blkverify_open(BlockDriverState *bs, QDict *options, int flags,
|
|
|
|
Error **errp)
|
2013-04-10 14:40:28 +02:00
|
|
|
{
|
|
|
|
BDRVBlkverifyState *s = bs->opaque;
|
|
|
|
QemuOpts *opts;
|
|
|
|
int ret;
|
|
|
|
|
2014-01-02 03:49:17 +01:00
|
|
|
opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
|
2020-07-07 18:06:03 +02:00
|
|
|
if (!qemu_opts_absorb_qdict(opts, options, errp)) {
|
2013-04-10 14:40:28 +02:00
|
|
|
ret = -EINVAL;
|
|
|
|
goto fail;
|
|
|
|
}
|
|
|
|
|
2013-12-20 19:28:16 +01:00
|
|
|
/* Open the raw file */
|
2022-07-26 22:11:21 +02:00
|
|
|
ret = bdrv_open_file_child(qemu_opt_get(opts, "x-raw"), options, "raw",
|
|
|
|
bs, errp);
|
|
|
|
if (ret < 0) {
|
2013-04-10 14:40:28 +02:00
|
|
|
goto fail;
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
/* Open the test file */
|
2015-06-16 11:13:47 +02:00
|
|
|
s->test_file = bdrv_open_child(qemu_opt_get(opts, "x-image"), options,
|
2020-05-13 13:05:31 +02:00
|
|
|
"test", bs, &child_of_bds, BDRV_CHILD_DATA,
|
2021-02-02 13:49:45 +01:00
|
|
|
false, errp);
|
|
|
|
if (!s->test_file) {
|
2015-06-16 11:13:47 +02:00
|
|
|
ret = -EINVAL;
|
2013-04-10 14:40:28 +02:00
|
|
|
goto fail;
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
2018-04-21 15:29:26 +02:00
|
|
|
bs->supported_write_flags = BDRV_REQ_WRITE_UNCHANGED;
|
|
|
|
bs->supported_zero_flags = BDRV_REQ_WRITE_UNCHANGED;
|
|
|
|
|
2013-04-10 14:40:28 +02:00
|
|
|
ret = 0;
|
|
|
|
fail:
|
2014-08-28 07:56:11 +02:00
|
|
|
qemu_opts_del(opts);
|
2013-04-10 14:40:28 +02:00
|
|
|
return ret;
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
static void blkverify_close(BlockDriverState *bs)
|
|
|
|
{
|
|
|
|
BDRVBlkverifyState *s = bs->opaque;
|
|
|
|
|
2015-06-16 11:13:47 +02:00
|
|
|
bdrv_unref_child(bs, s->test_file);
|
2010-09-21 16:43:03 +02:00
|
|
|
s->test_file = NULL;
|
|
|
|
}
|
|
|
|
|
2023-01-13 21:42:04 +01:00
|
|
|
static int64_t coroutine_fn blkverify_co_getlength(BlockDriverState *bs)
|
2010-09-21 16:43:03 +02:00
|
|
|
{
|
|
|
|
BDRVBlkverifyState *s = bs->opaque;
|
|
|
|
|
2023-01-13 21:42:04 +01:00
|
|
|
return bdrv_co_getlength(s->test_file->bs);
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
static void coroutine_fn blkverify_do_test_req(void *opaque)
|
2010-09-21 16:43:03 +02:00
|
|
|
{
|
2016-11-04 21:13:45 +01:00
|
|
|
BlkverifyRequest *r = opaque;
|
|
|
|
BDRVBlkverifyState *s = r->bs->opaque;
|
|
|
|
|
|
|
|
r->ret = r->request_fn(s->test_file, r->offset, r->bytes, r->qiov,
|
|
|
|
r->flags);
|
|
|
|
r->done++;
|
|
|
|
qemu_coroutine_enter_if_inactive(r->co);
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
static void coroutine_fn blkverify_do_raw_req(void *opaque)
|
2010-09-21 16:43:03 +02:00
|
|
|
{
|
2016-11-04 21:13:45 +01:00
|
|
|
BlkverifyRequest *r = opaque;
|
2010-09-21 16:43:03 +02:00
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
r->raw_ret = r->request_fn(r->bs->file, r->offset, r->bytes, r->raw_qiov,
|
|
|
|
r->flags);
|
|
|
|
r->done++;
|
|
|
|
qemu_coroutine_enter_if_inactive(r->co);
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
static int coroutine_fn
|
|
|
|
blkverify_co_prwv(BlockDriverState *bs, BlkverifyRequest *r, uint64_t offset,
|
|
|
|
uint64_t bytes, QEMUIOVector *qiov, QEMUIOVector *raw_qiov,
|
|
|
|
int flags, bool is_write)
|
2010-09-21 16:43:03 +02:00
|
|
|
{
|
2016-11-04 21:13:45 +01:00
|
|
|
Coroutine *co_a, *co_b;
|
|
|
|
|
|
|
|
*r = (BlkverifyRequest) {
|
|
|
|
.co = qemu_coroutine_self(),
|
|
|
|
.bs = bs,
|
|
|
|
.offset = offset,
|
|
|
|
.bytes = bytes,
|
|
|
|
.qiov = qiov,
|
|
|
|
.raw_qiov = raw_qiov,
|
|
|
|
.flags = flags,
|
|
|
|
.is_write = is_write,
|
|
|
|
.request_fn = is_write ? bdrv_co_pwritev : bdrv_co_preadv,
|
|
|
|
};
|
|
|
|
|
|
|
|
co_a = qemu_coroutine_create(blkverify_do_test_req, r);
|
|
|
|
co_b = qemu_coroutine_create(blkverify_do_raw_req, r);
|
|
|
|
|
|
|
|
qemu_coroutine_enter(co_a);
|
|
|
|
qemu_coroutine_enter(co_b);
|
|
|
|
|
|
|
|
while (r->done < 2) {
|
|
|
|
qemu_coroutine_yield();
|
|
|
|
}
|
2010-09-21 16:43:03 +02:00
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
if (r->ret != r->raw_ret) {
|
|
|
|
blkverify_err(r, "return value mismatch %d != %d", r->ret, r->raw_ret);
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
2016-11-04 21:13:45 +01:00
|
|
|
|
|
|
|
return r->ret;
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
static int coroutine_fn
|
block: use int64_t instead of uint64_t in driver read handlers
We are generally moving to int64_t for both offset and bytes parameters
on all io paths.
Main motivation is realization of 64-bit write_zeroes operation for
fast zeroing large disk chunks, up to the whole disk.
We chose signed type, to be consistent with off_t (which is signed) and
with possibility for signed return type (where negative value means
error).
So, convert driver read handlers parameters which are already 64bit to
signed type.
While being here, convert also flags parameter to be BdrvRequestFlags.
Now let's consider all callers. Simple
git grep '\->bdrv_\(aio\|co\)_preadv\(_part\)\?'
shows that's there three callers of driver function:
bdrv_driver_preadv() in block/io.c, passes int64_t, checked by
bdrv_check_qiov_request() to be non-negative.
qcow2_load_vmstate() does bdrv_check_qiov_request().
do_perform_cow_read() has uint64_t argument. And a lot of things in
qcow2 driver are uint64_t, so converting it is big job. But we must
not work with requests that don't satisfy bdrv_check_qiov_request(),
so let's just assert it here.
Still, the functions may be called directly, not only by drv->...
Let's check:
git grep '\.bdrv_\(aio\|co\)_preadv\(_part\)\?\s*=' | \
awk '{print $4}' | sed 's/,//' | sed 's/&//' | sort | uniq | \
while read func; do git grep "$func(" | \
grep -v "$func(BlockDriverState"; done
The only one such caller:
QEMUIOVector qiov = QEMU_IOVEC_INIT_BUF(qiov, &data, 1);
...
ret = bdrv_replace_test_co_preadv(bs, 0, 1, &qiov, 0);
in tests/unit/test-bdrv-drain.c, and it's OK obviously.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Message-Id: <20210903102807.27127-4-vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
[eblake: fix typos]
Signed-off-by: Eric Blake <eblake@redhat.com>
2021-09-03 12:27:59 +02:00
|
|
|
blkverify_co_preadv(BlockDriverState *bs, int64_t offset, int64_t bytes,
|
|
|
|
QEMUIOVector *qiov, BdrvRequestFlags flags)
|
2010-09-21 16:43:03 +02:00
|
|
|
{
|
2016-11-04 21:13:45 +01:00
|
|
|
BlkverifyRequest r;
|
|
|
|
QEMUIOVector raw_qiov;
|
|
|
|
void *buf;
|
|
|
|
ssize_t cmp_offset;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
buf = qemu_blockalign(bs->file->bs, qiov->size);
|
|
|
|
qemu_iovec_init(&raw_qiov, qiov->niov);
|
|
|
|
qemu_iovec_clone(&raw_qiov, qiov, buf);
|
|
|
|
|
2022-10-13 20:59:01 +02:00
|
|
|
ret = blkverify_co_prwv(bs, &r, offset, bytes, qiov, &raw_qiov,
|
|
|
|
flags & ~BDRV_REQ_REGISTERED_BUF, false);
|
2016-11-04 21:13:45 +01:00
|
|
|
|
|
|
|
cmp_offset = qemu_iovec_compare(qiov, &raw_qiov);
|
|
|
|
if (cmp_offset != -1) {
|
|
|
|
blkverify_err(&r, "contents mismatch at offset %" PRId64,
|
|
|
|
offset + cmp_offset);
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
qemu_iovec_destroy(&raw_qiov);
|
|
|
|
qemu_vfree(buf);
|
|
|
|
|
|
|
|
return ret;
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
static int coroutine_fn
|
block: use int64_t instead of uint64_t in driver write handlers
We are generally moving to int64_t for both offset and bytes parameters
on all io paths.
Main motivation is realization of 64-bit write_zeroes operation for
fast zeroing large disk chunks, up to the whole disk.
We chose signed type, to be consistent with off_t (which is signed) and
with possibility for signed return type (where negative value means
error).
So, convert driver write handlers parameters which are already 64bit to
signed type.
While being here, convert also flags parameter to be BdrvRequestFlags.
Now let's consider all callers. Simple
git grep '\->bdrv_\(aio\|co\)_pwritev\(_part\)\?'
shows that's there three callers of driver function:
bdrv_driver_pwritev() and bdrv_driver_pwritev_compressed() in
block/io.c, both pass int64_t, checked by bdrv_check_qiov_request() to
be non-negative.
qcow2_save_vmstate() does bdrv_check_qiov_request().
Still, the functions may be called directly, not only by drv->...
Let's check:
git grep '\.bdrv_\(aio\|co\)_pwritev\(_part\)\?\s*=' | \
awk '{print $4}' | sed 's/,//' | sed 's/&//' | sort | uniq | \
while read func; do git grep "$func(" | \
grep -v "$func(BlockDriverState"; done
shows several callers:
qcow2:
qcow2_co_truncate() write at most up to @offset, which is checked in
generic qcow2_co_truncate() by bdrv_check_request().
qcow2_co_pwritev_compressed_task() pass the request (or part of the
request) that already went through normal write path, so it should
be OK
qcow:
qcow_co_pwritev_compressed() pass int64_t, it's updated by this patch
quorum:
quorum_co_pwrite_zeroes() pass int64_t and int - OK
throttle:
throttle_co_pwritev_compressed() pass int64_t, it's updated by this
patch
vmdk:
vmdk_co_pwritev_compressed() pass int64_t, it's updated by this
patch
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Message-Id: <20210903102807.27127-5-vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Eric Blake <eblake@redhat.com>
2021-09-03 12:28:00 +02:00
|
|
|
blkverify_co_pwritev(BlockDriverState *bs, int64_t offset, int64_t bytes,
|
|
|
|
QEMUIOVector *qiov, BdrvRequestFlags flags)
|
2010-09-21 16:43:03 +02:00
|
|
|
{
|
2016-11-04 21:13:45 +01:00
|
|
|
BlkverifyRequest r;
|
|
|
|
return blkverify_co_prwv(bs, &r, offset, bytes, qiov, qiov, flags, true);
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
2022-09-22 10:49:05 +02:00
|
|
|
static int coroutine_fn blkverify_co_flush(BlockDriverState *bs)
|
2010-09-21 16:43:03 +02:00
|
|
|
{
|
|
|
|
BDRVBlkverifyState *s = bs->opaque;
|
|
|
|
|
|
|
|
/* Only flush test file, the raw file is not important */
|
2016-11-04 21:13:45 +01:00
|
|
|
return bdrv_co_flush(s->test_file->bs);
|
2010-09-21 16:43:03 +02:00
|
|
|
}
|
|
|
|
|
2020-02-18 11:34:42 +01:00
|
|
|
static bool blkverify_recurse_can_replace(BlockDriverState *bs,
|
|
|
|
BlockDriverState *to_replace)
|
|
|
|
{
|
|
|
|
BDRVBlkverifyState *s = bs->opaque;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* blkverify quits the whole qemu process if there is a mismatch
|
|
|
|
* between bs->file->bs and s->test_file->bs. Therefore, we know
|
|
|
|
* know that both must match bs and we can recurse down to either.
|
|
|
|
*/
|
|
|
|
return bdrv_recurse_can_replace(bs->file->bs, to_replace) ||
|
|
|
|
bdrv_recurse_can_replace(s->test_file->bs, to_replace);
|
|
|
|
}
|
|
|
|
|
2019-02-01 20:29:28 +01:00
|
|
|
static void blkverify_refresh_filename(BlockDriverState *bs)
|
2014-07-18 20:24:58 +02:00
|
|
|
{
|
|
|
|
BDRVBlkverifyState *s = bs->opaque;
|
|
|
|
|
2015-06-16 14:19:22 +02:00
|
|
|
if (bs->file->bs->exact_filename[0]
|
|
|
|
&& s->test_file->bs->exact_filename[0])
|
|
|
|
{
|
2017-06-13 19:20:06 +02:00
|
|
|
int ret = snprintf(bs->exact_filename, sizeof(bs->exact_filename),
|
|
|
|
"blkverify:%s:%s",
|
|
|
|
bs->file->bs->exact_filename,
|
|
|
|
s->test_file->bs->exact_filename);
|
|
|
|
if (ret >= sizeof(bs->exact_filename)) {
|
|
|
|
/* An overflow makes the filename unusable, so do not report any */
|
|
|
|
bs->exact_filename[0] = 0;
|
|
|
|
}
|
2014-07-18 20:24:58 +02:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2019-02-01 20:29:19 +01:00
|
|
|
static char *blkverify_dirname(BlockDriverState *bs, Error **errp)
|
|
|
|
{
|
|
|
|
/* In general, there are two BDSs with different dirnames below this one;
|
|
|
|
* so there is no unique dirname we could return (unless both are equal by
|
|
|
|
* chance). Therefore, to be consistent, just always return NULL. */
|
|
|
|
error_setg(errp, "Cannot generate a base directory for blkverify nodes");
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2010-09-21 16:43:03 +02:00
|
|
|
static BlockDriver bdrv_blkverify = {
|
2014-05-08 16:34:39 +02:00
|
|
|
.format_name = "blkverify",
|
|
|
|
.protocol_name = "blkverify",
|
|
|
|
.instance_size = sizeof(BDRVBlkverifyState),
|
|
|
|
|
|
|
|
.bdrv_parse_filename = blkverify_parse_filename,
|
|
|
|
.bdrv_file_open = blkverify_open,
|
|
|
|
.bdrv_close = blkverify_close,
|
2020-05-13 13:05:39 +02:00
|
|
|
.bdrv_child_perm = bdrv_default_perms,
|
2023-01-13 21:42:04 +01:00
|
|
|
.bdrv_co_getlength = blkverify_co_getlength,
|
2014-07-18 20:24:58 +02:00
|
|
|
.bdrv_refresh_filename = blkverify_refresh_filename,
|
2019-02-01 20:29:19 +01:00
|
|
|
.bdrv_dirname = blkverify_dirname,
|
2013-04-10 14:40:28 +02:00
|
|
|
|
2016-11-04 21:13:45 +01:00
|
|
|
.bdrv_co_preadv = blkverify_co_preadv,
|
|
|
|
.bdrv_co_pwritev = blkverify_co_pwritev,
|
|
|
|
.bdrv_co_flush = blkverify_co_flush,
|
2013-04-10 14:40:28 +02:00
|
|
|
|
2014-05-08 16:34:39 +02:00
|
|
|
.is_filter = true,
|
2020-02-18 11:34:42 +01:00
|
|
|
.bdrv_recurse_can_replace = blkverify_recurse_can_replace,
|
2010-09-21 16:43:03 +02:00
|
|
|
};
|
|
|
|
|
|
|
|
static void bdrv_blkverify_init(void)
|
|
|
|
{
|
|
|
|
bdrv_register(&bdrv_blkverify);
|
|
|
|
}
|
|
|
|
|
|
|
|
block_init(bdrv_blkverify_init);
|