2012-09-27 21:29:13 +04:00
|
|
|
/*
|
|
|
|
* Live block commit
|
|
|
|
*
|
|
|
|
* Copyright Red Hat, Inc. 2012
|
|
|
|
*
|
|
|
|
* Authors:
|
|
|
|
* Jeff Cody <jcody@redhat.com>
|
|
|
|
* Based on stream.c by Stefan Hajnoczi
|
|
|
|
*
|
|
|
|
* This work is licensed under the terms of the GNU LGPL, version 2 or later.
|
|
|
|
* See the COPYING.LIB file in the top-level directory.
|
|
|
|
*
|
|
|
|
*/
|
|
|
|
|
2016-01-18 21:01:42 +03:00
|
|
|
#include "qemu/osdep.h"
|
2017-03-09 13:49:16 +03:00
|
|
|
#include "qemu/cutils.h"
|
2012-09-27 21:29:13 +04:00
|
|
|
#include "trace.h"
|
2012-12-17 21:19:44 +04:00
|
|
|
#include "block/block_int.h"
|
2016-10-27 19:07:00 +03:00
|
|
|
#include "block/blockjob_int.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 11:01:28 +03:00
|
|
|
#include "qapi/error.h"
|
2015-03-17 19:22:46 +03:00
|
|
|
#include "qapi/qmp/qerror.h"
|
2012-09-27 21:29:13 +04:00
|
|
|
#include "qemu/ratelimit.h"
|
2015-10-19 18:53:22 +03:00
|
|
|
#include "sysemu/block-backend.h"
|
2012-09-27 21:29:13 +04:00
|
|
|
|
|
|
|
enum {
|
|
|
|
/*
|
|
|
|
* Size of data buffer for populating the image file. This should be large
|
|
|
|
* enough to process multiple clusters in a single call, so that populating
|
|
|
|
* contiguous regions of the image is efficient.
|
|
|
|
*/
|
|
|
|
COMMIT_BUFFER_SIZE = 512 * 1024, /* in bytes */
|
|
|
|
};
|
|
|
|
|
|
|
|
#define SLICE_TIME 100000000ULL /* ns */
|
|
|
|
|
|
|
|
typedef struct CommitBlockJob {
|
|
|
|
BlockJob common;
|
|
|
|
RateLimit limit;
|
2017-01-16 18:22:34 +03:00
|
|
|
BlockDriverState *commit_top_bs;
|
2016-04-14 14:09:53 +03:00
|
|
|
BlockBackend *top;
|
|
|
|
BlockBackend *base;
|
2012-09-28 19:22:55 +04:00
|
|
|
BlockdevOnError on_error;
|
2012-09-27 21:29:13 +04:00
|
|
|
int base_flags;
|
block: extend block-commit to accept a string for the backing file
On some image chains, QEMU may not always be able to resolve the
filenames properly, when updating the backing file of an image
after a block commit.
For instance, certain relative pathnames may fail, or drives may
have been specified originally by file descriptor (e.g. /dev/fd/???),
or a relative protocol pathname may have been used.
In these instances, QEMU may lack the information to be able to make
the correct choice, but the user or management layer most likely does
have that knowledge.
With this extension to the block-commit api, the user is able to change
the backing file of the overlay image as part of the block-commit
operation.
This allows the change to be 'safe', in the sense that if the attempt
to write the overlay image metadata fails, then the block-commit
operation returns failure, without disrupting the guest.
If the commit top is the active layer, then specifying the backing
file string will be treated as an error (there is no overlay image
to modify in that case).
If a backing file string is not specified in the command, the backing
file string to use is determined in the same manner as it was
previously.
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Jeff Cody <jcody@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
2014-06-25 23:40:10 +04:00
|
|
|
char *backing_file_str;
|
2012-09-27 21:29:13 +04:00
|
|
|
} CommitBlockJob;
|
|
|
|
|
2016-04-14 14:09:53 +03:00
|
|
|
static int coroutine_fn commit_populate(BlockBackend *bs, BlockBackend *base,
|
2017-07-07 15:44:44 +03:00
|
|
|
int64_t offset, uint64_t bytes,
|
2012-09-27 21:29:13 +04:00
|
|
|
void *buf)
|
|
|
|
{
|
|
|
|
int ret = 0;
|
2016-04-14 14:09:53 +03:00
|
|
|
QEMUIOVector qiov;
|
|
|
|
struct iovec iov = {
|
|
|
|
.iov_base = buf,
|
2017-07-07 15:44:44 +03:00
|
|
|
.iov_len = bytes,
|
2016-04-14 14:09:53 +03:00
|
|
|
};
|
2012-09-27 21:29:13 +04:00
|
|
|
|
2017-07-07 15:44:44 +03:00
|
|
|
assert(bytes < SIZE_MAX);
|
2016-04-14 14:09:53 +03:00
|
|
|
qemu_iovec_init_external(&qiov, &iov, 1);
|
|
|
|
|
2017-07-07 15:44:44 +03:00
|
|
|
ret = blk_co_preadv(bs, offset, qiov.size, &qiov, 0);
|
2016-04-14 14:09:53 +03:00
|
|
|
if (ret < 0) {
|
2012-09-27 21:29:13 +04:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2017-07-07 15:44:44 +03:00
|
|
|
ret = blk_co_pwritev(base, offset, qiov.size, &qiov, 0);
|
2016-04-14 14:09:53 +03:00
|
|
|
if (ret < 0) {
|
2012-09-27 21:29:13 +04:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2014-10-21 15:03:59 +04:00
|
|
|
typedef struct {
|
|
|
|
int ret;
|
|
|
|
} CommitCompleteData;
|
|
|
|
|
|
|
|
static void commit_complete(BlockJob *job, void *opaque)
|
2012-09-27 21:29:13 +04:00
|
|
|
{
|
2014-10-21 15:03:59 +04:00
|
|
|
CommitBlockJob *s = container_of(job, CommitBlockJob, common);
|
|
|
|
CommitCompleteData *data = opaque;
|
2016-04-14 14:09:53 +03:00
|
|
|
BlockDriverState *top = blk_bs(s->top);
|
|
|
|
BlockDriverState *base = blk_bs(s->base);
|
2017-06-27 21:36:18 +03:00
|
|
|
BlockDriverState *commit_top_bs = s->commit_top_bs;
|
2014-10-21 15:03:59 +04:00
|
|
|
int ret = data->ret;
|
2017-01-16 18:22:34 +03:00
|
|
|
bool remove_commit_top_bs = false;
|
|
|
|
|
2017-06-27 21:36:18 +03:00
|
|
|
/* Make sure commit_top_bs and top stay around until bdrv_replace_node() */
|
2017-06-03 00:04:55 +03:00
|
|
|
bdrv_ref(top);
|
2017-06-27 21:36:18 +03:00
|
|
|
bdrv_ref(commit_top_bs);
|
2017-06-03 00:04:55 +03:00
|
|
|
|
2017-01-16 18:22:34 +03:00
|
|
|
/* Remove base node parent that still uses BLK_PERM_WRITE/RESIZE before
|
|
|
|
* the normal backing chain can be restored. */
|
|
|
|
blk_unref(s->base);
|
2014-10-21 15:03:59 +04:00
|
|
|
|
|
|
|
if (!block_job_is_cancelled(&s->common) && ret == 0) {
|
|
|
|
/* success */
|
2017-06-27 21:36:18 +03:00
|
|
|
ret = bdrv_drop_intermediate(s->commit_top_bs, base,
|
2017-01-16 18:22:34 +03:00
|
|
|
s->backing_file_str);
|
2017-06-27 21:36:18 +03:00
|
|
|
} else {
|
2017-01-16 18:22:34 +03:00
|
|
|
/* XXX Can (or should) we somehow keep 'consistent read' blocked even
|
|
|
|
* after the failed/cancelled commit job is gone? If we already wrote
|
|
|
|
* something to base, the intermediate images aren't valid any more. */
|
|
|
|
remove_commit_top_bs = true;
|
2014-10-21 15:03:59 +04:00
|
|
|
}
|
|
|
|
|
|
|
|
/* restore base open flags here if appropriate (e.g., change the base back
|
|
|
|
* to r/o). These reopens do not need to be atomic, since we won't abort
|
|
|
|
* even on failure here */
|
|
|
|
if (s->base_flags != bdrv_get_flags(base)) {
|
|
|
|
bdrv_reopen(base, s->base_flags, NULL);
|
|
|
|
}
|
|
|
|
g_free(s->backing_file_str);
|
2016-04-14 14:09:53 +03:00
|
|
|
blk_unref(s->top);
|
2017-06-09 14:29:36 +03:00
|
|
|
|
|
|
|
/* If there is more than one reference to the job (e.g. if called from
|
|
|
|
* block_job_finish_sync()), block_job_completed() won't free it and
|
|
|
|
* therefore the blockers on the intermediate nodes remain. This would
|
|
|
|
* cause bdrv_set_backing_hd() to fail. */
|
|
|
|
block_job_remove_all_bdrv(job);
|
|
|
|
|
2014-10-21 15:03:59 +04:00
|
|
|
block_job_completed(&s->common, ret);
|
|
|
|
g_free(data);
|
2017-01-16 18:22:34 +03:00
|
|
|
|
|
|
|
/* If bdrv_drop_intermediate() didn't already do that, remove the commit
|
|
|
|
* filter driver from the backing chain. Do this as the final step so that
|
|
|
|
* the 'consistent read' permission can be granted. */
|
|
|
|
if (remove_commit_top_bs) {
|
2017-06-27 21:36:18 +03:00
|
|
|
bdrv_child_try_set_perm(commit_top_bs->backing, 0, BLK_PERM_ALL,
|
|
|
|
&error_abort);
|
|
|
|
bdrv_replace_node(commit_top_bs, backing_bs(commit_top_bs),
|
|
|
|
&error_abort);
|
2017-01-16 18:22:34 +03:00
|
|
|
}
|
2017-06-03 00:04:55 +03:00
|
|
|
|
2017-06-27 21:36:18 +03:00
|
|
|
bdrv_unref(commit_top_bs);
|
2017-06-03 00:04:55 +03:00
|
|
|
bdrv_unref(top);
|
2014-10-21 15:03:59 +04:00
|
|
|
}
|
|
|
|
|
|
|
|
static void coroutine_fn commit_run(void *opaque)
|
|
|
|
{
|
|
|
|
CommitBlockJob *s = opaque;
|
|
|
|
CommitCompleteData *data;
|
2017-07-07 15:44:45 +03:00
|
|
|
int64_t offset;
|
Improve block job rate limiting for small bandwidth values
ratelimit_calculate_delay() previously reset the accounting every time
slice, no matter how much data had been processed before. This had (at
least) two consequences:
1. The minimum speed is rather large, e.g. 5 MiB/s for commit and stream.
Not sure if there are real-world use cases where this would be a
problem. Mirroring and backup over a slow link (e.g. DSL) would
come to mind, though.
2. Tests for block job operations (e.g. cancel) were rather racy
All block jobs currently use a time slice of 100ms. That's a
reasonable value to get smooth output during regular
operation. However this also meant that the state of block jobs
changed every 100ms, no matter how low the configured limit was. On
busy hosts, qemu often transferred additional chunks until the test
case had a chance to cancel the job.
Fix the block job rate limit code to delay for more than one time
slice to address the above issues. To make it easier to handle
oversized chunks we switch the semantics from returning a delay
_before_ the current request to a delay _after_ the current
request. If necessary, this delay consists of multiple time slice
units.
Since the mirror job sends multiple chunks in one go even if the rate
limit was exceeded in between, we need to keep track of the start of
the current time slice so we can correctly re-compute the delay for
the updated amount of data.
The minimum bandwidth now is 1 data unit per time slice. The block
jobs are currently passing the amount of data transferred in sectors
and using 100ms time slices, so this translates to 5120
bytes/second. With chunk sizes usually being O(512KiB), tests have
plenty of time (O(100s)) to operate on block jobs. The chance of a
race condition now is fairly remote, except possibly on insanely
loaded systems.
Signed-off-by: Sascha Silbe <silbe@linux.vnet.ibm.com>
Message-id: 1467127721-9564-2-git-send-email-silbe@linux.vnet.ibm.com
Reviewed-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
2016-06-28 18:28:41 +03:00
|
|
|
uint64_t delay_ns = 0;
|
2012-09-27 21:29:13 +04:00
|
|
|
int ret = 0;
|
block: Make bdrv_is_allocated_above() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, for the most part this patch is just the
addition of scaling at the callers followed by inverse scaling at
bdrv_is_allocated(). But some code, particularly stream_run(),
gets a lot simpler because it no longer has to mess with sectors.
Leave comments where we can further simplify by switching to
byte-based iterations, once later patches eliminate the need for
sector-aligned operations.
For ease of review, bdrv_is_allocated() was tackled separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:59 +03:00
|
|
|
int64_t n = 0; /* bytes */
|
2014-10-21 15:03:59 +04:00
|
|
|
void *buf = NULL;
|
2012-09-27 21:29:13 +04:00
|
|
|
int bytes_written = 0;
|
|
|
|
int64_t base_len;
|
|
|
|
|
2016-04-14 14:09:53 +03:00
|
|
|
ret = s->common.len = blk_getlength(s->top);
|
2012-09-27 21:29:13 +04:00
|
|
|
|
|
|
|
if (s->common.len < 0) {
|
2014-10-21 15:03:59 +04:00
|
|
|
goto out;
|
2012-09-27 21:29:13 +04:00
|
|
|
}
|
|
|
|
|
2016-04-14 14:09:53 +03:00
|
|
|
ret = base_len = blk_getlength(s->base);
|
2012-09-27 21:29:13 +04:00
|
|
|
if (base_len < 0) {
|
2014-10-21 15:03:59 +04:00
|
|
|
goto out;
|
2012-09-27 21:29:13 +04:00
|
|
|
}
|
|
|
|
|
|
|
|
if (base_len < s->common.len) {
|
2017-06-13 23:20:54 +03:00
|
|
|
ret = blk_truncate(s->base, s->common.len, PREALLOC_MODE_OFF, NULL);
|
2012-09-27 21:29:13 +04:00
|
|
|
if (ret) {
|
2014-10-21 15:03:59 +04:00
|
|
|
goto out;
|
2012-09-27 21:29:13 +04:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-04-14 14:09:53 +03:00
|
|
|
buf = blk_blockalign(s->top, COMMIT_BUFFER_SIZE);
|
2012-09-27 21:29:13 +04:00
|
|
|
|
block: Make bdrv_is_allocated_above() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, for the most part this patch is just the
addition of scaling at the callers followed by inverse scaling at
bdrv_is_allocated(). But some code, particularly stream_run(),
gets a lot simpler because it no longer has to mess with sectors.
Leave comments where we can further simplify by switching to
byte-based iterations, once later patches eliminate the need for
sector-aligned operations.
For ease of review, bdrv_is_allocated() was tackled separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:59 +03:00
|
|
|
for (offset = 0; offset < s->common.len; offset += n) {
|
2012-09-27 21:29:13 +04:00
|
|
|
bool copy;
|
|
|
|
|
|
|
|
/* Note that even when no rate limit is applied we need to yield
|
2012-11-13 19:35:13 +04:00
|
|
|
* with no pending I/O here so that bdrv_drain_all() returns.
|
2012-09-27 21:29:13 +04:00
|
|
|
*/
|
2017-11-29 13:25:11 +03:00
|
|
|
block_job_sleep_ns(&s->common, delay_ns);
|
2012-09-27 21:29:13 +04:00
|
|
|
if (block_job_is_cancelled(&s->common)) {
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
/* Copy if allocated above the base */
|
2016-04-14 14:09:53 +03:00
|
|
|
ret = bdrv_is_allocated_above(blk_bs(s->top), blk_bs(s->base),
|
block: Make bdrv_is_allocated_above() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, for the most part this patch is just the
addition of scaling at the callers followed by inverse scaling at
bdrv_is_allocated(). But some code, particularly stream_run(),
gets a lot simpler because it no longer has to mess with sectors.
Leave comments where we can further simplify by switching to
byte-based iterations, once later patches eliminate the need for
sector-aligned operations.
For ease of review, bdrv_is_allocated() was tackled separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:59 +03:00
|
|
|
offset, COMMIT_BUFFER_SIZE, &n);
|
2012-09-27 21:29:13 +04:00
|
|
|
copy = (ret == 1);
|
block: Make bdrv_is_allocated_above() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, for the most part this patch is just the
addition of scaling at the callers followed by inverse scaling at
bdrv_is_allocated(). But some code, particularly stream_run(),
gets a lot simpler because it no longer has to mess with sectors.
Leave comments where we can further simplify by switching to
byte-based iterations, once later patches eliminate the need for
sector-aligned operations.
For ease of review, bdrv_is_allocated() was tackled separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:59 +03:00
|
|
|
trace_commit_one_iteration(s, offset, n, ret);
|
2012-09-27 21:29:13 +04:00
|
|
|
if (copy) {
|
block: Make bdrv_is_allocated_above() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, for the most part this patch is just the
addition of scaling at the callers followed by inverse scaling at
bdrv_is_allocated(). But some code, particularly stream_run(),
gets a lot simpler because it no longer has to mess with sectors.
Leave comments where we can further simplify by switching to
byte-based iterations, once later patches eliminate the need for
sector-aligned operations.
For ease of review, bdrv_is_allocated() was tackled separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:59 +03:00
|
|
|
ret = commit_populate(s->top, s->base, offset, n, buf);
|
|
|
|
bytes_written += n;
|
2012-09-27 21:29:13 +04:00
|
|
|
}
|
|
|
|
if (ret < 0) {
|
2016-06-29 18:38:57 +03:00
|
|
|
BlockErrorAction action =
|
|
|
|
block_job_error_action(&s->common, false, s->on_error, -ret);
|
|
|
|
if (action == BLOCK_ERROR_ACTION_REPORT) {
|
2014-10-21 15:03:59 +04:00
|
|
|
goto out;
|
2012-09-27 21:29:13 +04:00
|
|
|
} else {
|
|
|
|
n = 0;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
/* Publish progress */
|
block: Make bdrv_is_allocated_above() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, for the most part this patch is just the
addition of scaling at the callers followed by inverse scaling at
bdrv_is_allocated(). But some code, particularly stream_run(),
gets a lot simpler because it no longer has to mess with sectors.
Leave comments where we can further simplify by switching to
byte-based iterations, once later patches eliminate the need for
sector-aligned operations.
For ease of review, bdrv_is_allocated() was tackled separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:59 +03:00
|
|
|
s->common.offset += n;
|
Improve block job rate limiting for small bandwidth values
ratelimit_calculate_delay() previously reset the accounting every time
slice, no matter how much data had been processed before. This had (at
least) two consequences:
1. The minimum speed is rather large, e.g. 5 MiB/s for commit and stream.
Not sure if there are real-world use cases where this would be a
problem. Mirroring and backup over a slow link (e.g. DSL) would
come to mind, though.
2. Tests for block job operations (e.g. cancel) were rather racy
All block jobs currently use a time slice of 100ms. That's a
reasonable value to get smooth output during regular
operation. However this also meant that the state of block jobs
changed every 100ms, no matter how low the configured limit was. On
busy hosts, qemu often transferred additional chunks until the test
case had a chance to cancel the job.
Fix the block job rate limit code to delay for more than one time
slice to address the above issues. To make it easier to handle
oversized chunks we switch the semantics from returning a delay
_before_ the current request to a delay _after_ the current
request. If necessary, this delay consists of multiple time slice
units.
Since the mirror job sends multiple chunks in one go even if the rate
limit was exceeded in between, we need to keep track of the start of
the current time slice so we can correctly re-compute the delay for
the updated amount of data.
The minimum bandwidth now is 1 data unit per time slice. The block
jobs are currently passing the amount of data transferred in sectors
and using 100ms time slices, so this translates to 5120
bytes/second. With chunk sizes usually being O(512KiB), tests have
plenty of time (O(100s)) to operate on block jobs. The chance of a
race condition now is fairly remote, except possibly on insanely
loaded systems.
Signed-off-by: Sascha Silbe <silbe@linux.vnet.ibm.com>
Message-id: 1467127721-9564-2-git-send-email-silbe@linux.vnet.ibm.com
Reviewed-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
2016-06-28 18:28:41 +03:00
|
|
|
|
|
|
|
if (copy && s->common.speed) {
|
block: Make bdrv_is_allocated_above() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, for the most part this patch is just the
addition of scaling at the callers followed by inverse scaling at
bdrv_is_allocated(). But some code, particularly stream_run(),
gets a lot simpler because it no longer has to mess with sectors.
Leave comments where we can further simplify by switching to
byte-based iterations, once later patches eliminate the need for
sector-aligned operations.
For ease of review, bdrv_is_allocated() was tackled separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:59 +03:00
|
|
|
delay_ns = ratelimit_calculate_delay(&s->limit, n);
|
Improve block job rate limiting for small bandwidth values
ratelimit_calculate_delay() previously reset the accounting every time
slice, no matter how much data had been processed before. This had (at
least) two consequences:
1. The minimum speed is rather large, e.g. 5 MiB/s for commit and stream.
Not sure if there are real-world use cases where this would be a
problem. Mirroring and backup over a slow link (e.g. DSL) would
come to mind, though.
2. Tests for block job operations (e.g. cancel) were rather racy
All block jobs currently use a time slice of 100ms. That's a
reasonable value to get smooth output during regular
operation. However this also meant that the state of block jobs
changed every 100ms, no matter how low the configured limit was. On
busy hosts, qemu often transferred additional chunks until the test
case had a chance to cancel the job.
Fix the block job rate limit code to delay for more than one time
slice to address the above issues. To make it easier to handle
oversized chunks we switch the semantics from returning a delay
_before_ the current request to a delay _after_ the current
request. If necessary, this delay consists of multiple time slice
units.
Since the mirror job sends multiple chunks in one go even if the rate
limit was exceeded in between, we need to keep track of the start of
the current time slice so we can correctly re-compute the delay for
the updated amount of data.
The minimum bandwidth now is 1 data unit per time slice. The block
jobs are currently passing the amount of data transferred in sectors
and using 100ms time slices, so this translates to 5120
bytes/second. With chunk sizes usually being O(512KiB), tests have
plenty of time (O(100s)) to operate on block jobs. The chance of a
race condition now is fairly remote, except possibly on insanely
loaded systems.
Signed-off-by: Sascha Silbe <silbe@linux.vnet.ibm.com>
Message-id: 1467127721-9564-2-git-send-email-silbe@linux.vnet.ibm.com
Reviewed-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
2016-06-28 18:28:41 +03:00
|
|
|
}
|
2012-09-27 21:29:13 +04:00
|
|
|
}
|
|
|
|
|
|
|
|
ret = 0;
|
|
|
|
|
2014-10-21 15:03:59 +04:00
|
|
|
out:
|
2012-09-27 21:29:13 +04:00
|
|
|
qemu_vfree(buf);
|
|
|
|
|
2014-10-21 15:03:59 +04:00
|
|
|
data = g_malloc(sizeof(*data));
|
|
|
|
data->ret = ret;
|
|
|
|
block_job_defer_to_main_loop(&s->common, commit_complete, data);
|
2012-09-27 21:29:13 +04:00
|
|
|
}
|
|
|
|
|
|
|
|
static void commit_set_speed(BlockJob *job, int64_t speed, Error **errp)
|
|
|
|
{
|
|
|
|
CommitBlockJob *s = container_of(job, CommitBlockJob, common);
|
|
|
|
|
|
|
|
if (speed < 0) {
|
2015-03-17 13:54:50 +03:00
|
|
|
error_setg(errp, QERR_INVALID_PARAMETER, "speed");
|
2012-09-27 21:29:13 +04:00
|
|
|
return;
|
|
|
|
}
|
2017-07-07 15:44:39 +03:00
|
|
|
ratelimit_set_speed(&s->limit, speed, SLICE_TIME);
|
2012-09-27 21:29:13 +04:00
|
|
|
}
|
|
|
|
|
2013-10-08 13:29:38 +04:00
|
|
|
static const BlockJobDriver commit_job_driver = {
|
2012-09-27 21:29:13 +04:00
|
|
|
.instance_size = sizeof(CommitBlockJob),
|
2013-10-08 13:29:40 +04:00
|
|
|
.job_type = BLOCK_JOB_TYPE_COMMIT,
|
2012-09-27 21:29:13 +04:00
|
|
|
.set_speed = commit_set_speed,
|
2016-11-08 09:50:36 +03:00
|
|
|
.start = commit_run,
|
2012-09-27 21:29:13 +04:00
|
|
|
};
|
|
|
|
|
2017-01-16 18:22:34 +03:00
|
|
|
static int coroutine_fn bdrv_commit_top_preadv(BlockDriverState *bs,
|
|
|
|
uint64_t offset, uint64_t bytes, QEMUIOVector *qiov, int flags)
|
|
|
|
{
|
|
|
|
return bdrv_co_preadv(bs->backing, offset, bytes, qiov, flags);
|
|
|
|
}
|
|
|
|
|
2017-03-09 13:49:16 +03:00
|
|
|
static void bdrv_commit_top_refresh_filename(BlockDriverState *bs, QDict *opts)
|
|
|
|
{
|
|
|
|
bdrv_refresh_filename(bs->backing->bs);
|
|
|
|
pstrcpy(bs->exact_filename, sizeof(bs->exact_filename),
|
|
|
|
bs->backing->bs->filename);
|
|
|
|
}
|
2017-03-08 17:07:12 +03:00
|
|
|
|
2017-01-16 18:22:34 +03:00
|
|
|
static void bdrv_commit_top_close(BlockDriverState *bs)
|
|
|
|
{
|
|
|
|
}
|
|
|
|
|
|
|
|
static void bdrv_commit_top_child_perm(BlockDriverState *bs, BdrvChild *c,
|
|
|
|
const BdrvChildRole *role,
|
2017-09-14 13:47:11 +03:00
|
|
|
BlockReopenQueue *reopen_queue,
|
2017-01-16 18:22:34 +03:00
|
|
|
uint64_t perm, uint64_t shared,
|
|
|
|
uint64_t *nperm, uint64_t *nshared)
|
|
|
|
{
|
|
|
|
*nperm = 0;
|
|
|
|
*nshared = BLK_PERM_ALL;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Dummy node that provides consistent read to its users without requiring it
|
|
|
|
* from its backing file and that allows writes on the backing file chain. */
|
|
|
|
static BlockDriver bdrv_commit_top = {
|
2017-03-08 17:07:12 +03:00
|
|
|
.format_name = "commit_top",
|
|
|
|
.bdrv_co_preadv = bdrv_commit_top_preadv,
|
2018-02-13 23:26:43 +03:00
|
|
|
.bdrv_co_block_status = bdrv_co_block_status_from_backing,
|
2017-03-09 13:49:16 +03:00
|
|
|
.bdrv_refresh_filename = bdrv_commit_top_refresh_filename,
|
2017-03-08 17:07:12 +03:00
|
|
|
.bdrv_close = bdrv_commit_top_close,
|
|
|
|
.bdrv_child_perm = bdrv_commit_top_child_perm,
|
2017-01-16 18:22:34 +03:00
|
|
|
};
|
|
|
|
|
2016-07-05 17:29:00 +03:00
|
|
|
void commit_start(const char *job_id, BlockDriverState *bs,
|
|
|
|
BlockDriverState *base, BlockDriverState *top, int64_t speed,
|
2016-10-27 19:06:58 +03:00
|
|
|
BlockdevOnError on_error, const char *backing_file_str,
|
2017-02-20 20:10:05 +03:00
|
|
|
const char *filter_node_name, Error **errp)
|
2012-09-27 21:29:13 +04:00
|
|
|
{
|
|
|
|
CommitBlockJob *s;
|
|
|
|
int orig_base_flags;
|
2016-10-28 10:08:08 +03:00
|
|
|
BlockDriverState *iter;
|
2017-01-16 18:22:34 +03:00
|
|
|
BlockDriverState *commit_top_bs = NULL;
|
2012-09-27 21:29:13 +04:00
|
|
|
Error *local_err = NULL;
|
2017-01-13 21:02:32 +03:00
|
|
|
int ret;
|
2012-09-27 21:29:13 +04:00
|
|
|
|
2013-12-16 10:45:33 +04:00
|
|
|
assert(top != bs);
|
2012-09-27 21:29:13 +04:00
|
|
|
if (top == base) {
|
|
|
|
error_setg(errp, "Invalid files for merge: top and base are the same");
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2018-03-10 11:27:27 +03:00
|
|
|
s = block_job_create(job_id, &commit_job_driver, NULL, bs, 0, BLK_PERM_ALL,
|
2017-01-16 19:18:09 +03:00
|
|
|
speed, BLOCK_JOB_DEFAULT, NULL, NULL, errp);
|
2016-05-27 13:53:39 +03:00
|
|
|
if (!s) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2017-06-27 21:36:18 +03:00
|
|
|
/* convert base to r/w, if necessary */
|
|
|
|
orig_base_flags = bdrv_get_flags(base);
|
2015-10-28 16:43:49 +03:00
|
|
|
if (!(orig_base_flags & BDRV_O_RDWR)) {
|
2017-12-06 15:53:36 +03:00
|
|
|
bdrv_reopen(base, orig_base_flags | BDRV_O_RDWR, &local_err);
|
2012-09-27 21:29:13 +04:00
|
|
|
if (local_err != NULL) {
|
|
|
|
error_propagate(errp, local_err);
|
2017-01-13 21:02:32 +03:00
|
|
|
goto fail;
|
2012-09-27 21:29:13 +04:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2017-01-16 18:22:34 +03:00
|
|
|
/* Insert commit_top block node above top, so we can block consistent read
|
|
|
|
* on the backing chain below it */
|
2017-02-20 20:10:05 +03:00
|
|
|
commit_top_bs = bdrv_new_open_driver(&bdrv_commit_top, filter_node_name, 0,
|
|
|
|
errp);
|
2017-01-16 18:22:34 +03:00
|
|
|
if (commit_top_bs == NULL) {
|
|
|
|
goto fail;
|
|
|
|
}
|
2017-07-18 18:24:05 +03:00
|
|
|
if (!filter_node_name) {
|
|
|
|
commit_top_bs->implicit = true;
|
|
|
|
}
|
2017-04-06 20:07:14 +03:00
|
|
|
commit_top_bs->total_sectors = top->total_sectors;
|
2017-04-06 20:05:07 +03:00
|
|
|
bdrv_set_aio_context(commit_top_bs, bdrv_get_aio_context(top));
|
2017-01-16 18:22:34 +03:00
|
|
|
|
2017-03-07 14:07:22 +03:00
|
|
|
bdrv_set_backing_hd(commit_top_bs, top, &local_err);
|
|
|
|
if (local_err) {
|
|
|
|
bdrv_unref(commit_top_bs);
|
|
|
|
commit_top_bs = NULL;
|
|
|
|
error_propagate(errp, local_err);
|
|
|
|
goto fail;
|
|
|
|
}
|
2017-09-19 17:22:54 +03:00
|
|
|
bdrv_replace_node(top, commit_top_bs, &local_err);
|
2017-03-07 14:07:22 +03:00
|
|
|
if (local_err) {
|
|
|
|
bdrv_unref(commit_top_bs);
|
|
|
|
commit_top_bs = NULL;
|
|
|
|
error_propagate(errp, local_err);
|
|
|
|
goto fail;
|
|
|
|
}
|
2017-01-16 18:22:34 +03:00
|
|
|
|
|
|
|
s->commit_top_bs = commit_top_bs;
|
|
|
|
bdrv_unref(commit_top_bs);
|
2012-09-27 21:29:13 +04:00
|
|
|
|
2016-10-28 10:08:08 +03:00
|
|
|
/* Block all nodes between top and base, because they will
|
|
|
|
* disappear from the chain after this operation. */
|
|
|
|
assert(bdrv_chain_contains(top, base));
|
2017-01-16 18:22:34 +03:00
|
|
|
for (iter = top; iter != base; iter = backing_bs(iter)) {
|
|
|
|
/* XXX BLK_PERM_WRITE needs to be allowed so we don't block ourselves
|
|
|
|
* at s->base (if writes are blocked for a node, they are also blocked
|
|
|
|
* for its backing file). The other options would be a second filter
|
|
|
|
* driver above s->base. */
|
|
|
|
ret = block_job_add_bdrv(&s->common, "intermediate node", iter, 0,
|
|
|
|
BLK_PERM_WRITE_UNCHANGED | BLK_PERM_WRITE,
|
|
|
|
errp);
|
|
|
|
if (ret < 0) {
|
|
|
|
goto fail;
|
|
|
|
}
|
2016-10-28 10:08:08 +03:00
|
|
|
}
|
2017-01-16 18:22:34 +03:00
|
|
|
|
|
|
|
ret = block_job_add_bdrv(&s->common, "base", base, 0, BLK_PERM_ALL, errp);
|
|
|
|
if (ret < 0) {
|
|
|
|
goto fail;
|
|
|
|
}
|
|
|
|
|
|
|
|
s->base = blk_new(BLK_PERM_CONSISTENT_READ
|
|
|
|
| BLK_PERM_WRITE
|
|
|
|
| BLK_PERM_RESIZE,
|
|
|
|
BLK_PERM_CONSISTENT_READ
|
|
|
|
| BLK_PERM_GRAPH_MOD
|
|
|
|
| BLK_PERM_WRITE_UNCHANGED);
|
2017-01-13 21:02:32 +03:00
|
|
|
ret = blk_insert_bs(s->base, base, errp);
|
|
|
|
if (ret < 0) {
|
|
|
|
goto fail;
|
|
|
|
}
|
2016-04-14 14:09:53 +03:00
|
|
|
|
2017-01-16 18:22:34 +03:00
|
|
|
/* Required permissions are already taken with block_job_add_bdrv() */
|
2017-01-20 19:07:26 +03:00
|
|
|
s->top = blk_new(0, BLK_PERM_ALL);
|
2017-03-03 18:54:21 +03:00
|
|
|
ret = blk_insert_bs(s->top, top, errp);
|
2017-01-13 21:02:32 +03:00
|
|
|
if (ret < 0) {
|
|
|
|
goto fail;
|
|
|
|
}
|
2016-04-14 14:09:53 +03:00
|
|
|
|
2017-06-27 21:36:18 +03:00
|
|
|
s->base_flags = orig_base_flags;
|
block: extend block-commit to accept a string for the backing file
On some image chains, QEMU may not always be able to resolve the
filenames properly, when updating the backing file of an image
after a block commit.
For instance, certain relative pathnames may fail, or drives may
have been specified originally by file descriptor (e.g. /dev/fd/???),
or a relative protocol pathname may have been used.
In these instances, QEMU may lack the information to be able to make
the correct choice, but the user or management layer most likely does
have that knowledge.
With this extension to the block-commit api, the user is able to change
the backing file of the overlay image as part of the block-commit
operation.
This allows the change to be 'safe', in the sense that if the attempt
to write the overlay image metadata fails, then the block-commit
operation returns failure, without disrupting the guest.
If the commit top is the active layer, then specifying the backing
file string will be treated as an error (there is no overlay image
to modify in that case).
If a backing file string is not specified in the command, the backing
file string to use is determined in the same manner as it was
previously.
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Jeff Cody <jcody@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
2014-06-25 23:40:10 +04:00
|
|
|
s->backing_file_str = g_strdup(backing_file_str);
|
2012-09-27 21:29:13 +04:00
|
|
|
s->on_error = on_error;
|
|
|
|
|
2016-11-08 09:50:37 +03:00
|
|
|
trace_commit_start(bs, base, top, s);
|
|
|
|
block_job_start(&s->common);
|
2017-01-13 21:02:32 +03:00
|
|
|
return;
|
|
|
|
|
|
|
|
fail:
|
|
|
|
if (s->base) {
|
|
|
|
blk_unref(s->base);
|
|
|
|
}
|
|
|
|
if (s->top) {
|
|
|
|
blk_unref(s->top);
|
|
|
|
}
|
2017-01-16 18:22:34 +03:00
|
|
|
if (commit_top_bs) {
|
2017-06-27 21:36:18 +03:00
|
|
|
bdrv_replace_node(commit_top_bs, top, &error_abort);
|
2017-01-16 18:22:34 +03:00
|
|
|
}
|
2017-05-08 17:13:02 +03:00
|
|
|
block_job_early_fail(&s->common);
|
2012-09-27 21:29:13 +04:00
|
|
|
}
|
2016-05-30 16:53:15 +03:00
|
|
|
|
|
|
|
|
block: Make bdrv_is_allocated() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned
on input and that *pnum is sector-aligned on return to the caller,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, this code adds usages like
DIV_ROUND_UP(,BDRV_SECTOR_SIZE) to callers that still want aligned
values, where the call might reasonbly give non-aligned results
in the future; on the other hand, no rounding is needed for callers
that should just continue to work with byte alignment.
For the most part this patch is just the addition of scaling at the
callers followed by inverse scaling at bdrv_is_allocated(). But
some code, particularly bdrv_commit(), gets a lot simpler because it
no longer has to mess with sectors; also, it is now possible to pass
NULL if the caller does not care how much of the image is allocated
beyond the initial offset. Leave comments where we can further
simplify once a later patch eliminates the need for sector-aligned
requests through bdrv_is_allocated().
For ease of review, bdrv_is_allocated_above() will be tackled
separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:57 +03:00
|
|
|
#define COMMIT_BUF_SIZE (2048 * BDRV_SECTOR_SIZE)
|
2016-05-30 16:53:15 +03:00
|
|
|
|
|
|
|
/* commit COW file into the raw image */
|
|
|
|
int bdrv_commit(BlockDriverState *bs)
|
|
|
|
{
|
2016-05-30 17:29:47 +03:00
|
|
|
BlockBackend *src, *backing;
|
2017-01-19 20:16:03 +03:00
|
|
|
BlockDriverState *backing_file_bs = NULL;
|
|
|
|
BlockDriverState *commit_top_bs = NULL;
|
2016-05-30 16:53:15 +03:00
|
|
|
BlockDriver *drv = bs->drv;
|
block: Make bdrv_is_allocated() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned
on input and that *pnum is sector-aligned on return to the caller,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, this code adds usages like
DIV_ROUND_UP(,BDRV_SECTOR_SIZE) to callers that still want aligned
values, where the call might reasonbly give non-aligned results
in the future; on the other hand, no rounding is needed for callers
that should just continue to work with byte alignment.
For the most part this patch is just the addition of scaling at the
callers followed by inverse scaling at bdrv_is_allocated(). But
some code, particularly bdrv_commit(), gets a lot simpler because it
no longer has to mess with sectors; also, it is now possible to pass
NULL if the caller does not care how much of the image is allocated
beyond the initial offset. Leave comments where we can further
simplify once a later patch eliminates the need for sector-aligned
requests through bdrv_is_allocated().
For ease of review, bdrv_is_allocated_above() will be tackled
separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:57 +03:00
|
|
|
int64_t offset, length, backing_length;
|
|
|
|
int ro, open_flags;
|
|
|
|
int64_t n;
|
2016-05-30 16:53:15 +03:00
|
|
|
int ret = 0;
|
|
|
|
uint8_t *buf = NULL;
|
2017-01-19 20:16:03 +03:00
|
|
|
Error *local_err = NULL;
|
2016-05-30 16:53:15 +03:00
|
|
|
|
|
|
|
if (!drv)
|
|
|
|
return -ENOMEDIUM;
|
|
|
|
|
|
|
|
if (!bs->backing) {
|
|
|
|
return -ENOTSUP;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (bdrv_op_is_blocked(bs, BLOCK_OP_TYPE_COMMIT_SOURCE, NULL) ||
|
|
|
|
bdrv_op_is_blocked(bs->backing->bs, BLOCK_OP_TYPE_COMMIT_TARGET, NULL)) {
|
|
|
|
return -EBUSY;
|
|
|
|
}
|
|
|
|
|
|
|
|
ro = bs->backing->bs->read_only;
|
|
|
|
open_flags = bs->backing->bs->open_flags;
|
|
|
|
|
|
|
|
if (ro) {
|
|
|
|
if (bdrv_reopen(bs->backing->bs, open_flags | BDRV_O_RDWR, NULL)) {
|
|
|
|
return -EACCES;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2017-01-19 20:16:03 +03:00
|
|
|
src = blk_new(BLK_PERM_CONSISTENT_READ, BLK_PERM_ALL);
|
|
|
|
backing = blk_new(BLK_PERM_WRITE | BLK_PERM_RESIZE, BLK_PERM_ALL);
|
2017-01-13 21:02:32 +03:00
|
|
|
|
2017-01-19 20:16:03 +03:00
|
|
|
ret = blk_insert_bs(src, bs, &local_err);
|
2017-01-13 21:02:32 +03:00
|
|
|
if (ret < 0) {
|
2017-01-19 20:16:03 +03:00
|
|
|
error_report_err(local_err);
|
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Insert commit_top block node above backing, so we can write to it */
|
|
|
|
backing_file_bs = backing_bs(bs);
|
|
|
|
|
|
|
|
commit_top_bs = bdrv_new_open_driver(&bdrv_commit_top, NULL, BDRV_O_RDWR,
|
|
|
|
&local_err);
|
|
|
|
if (commit_top_bs == NULL) {
|
|
|
|
error_report_err(local_err);
|
2017-01-13 21:02:32 +03:00
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
2017-04-06 20:05:07 +03:00
|
|
|
bdrv_set_aio_context(commit_top_bs, bdrv_get_aio_context(backing_file_bs));
|
2017-01-13 21:02:32 +03:00
|
|
|
|
2017-02-17 22:42:32 +03:00
|
|
|
bdrv_set_backing_hd(commit_top_bs, backing_file_bs, &error_abort);
|
|
|
|
bdrv_set_backing_hd(bs, commit_top_bs, &error_abort);
|
2017-01-19 20:16:03 +03:00
|
|
|
|
|
|
|
ret = blk_insert_bs(backing, backing_file_bs, &local_err);
|
2017-01-13 21:02:32 +03:00
|
|
|
if (ret < 0) {
|
2017-01-19 20:16:03 +03:00
|
|
|
error_report_err(local_err);
|
2017-01-13 21:02:32 +03:00
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
2016-05-30 17:29:47 +03:00
|
|
|
|
|
|
|
length = blk_getlength(src);
|
2016-05-30 16:53:15 +03:00
|
|
|
if (length < 0) {
|
|
|
|
ret = length;
|
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
|
|
|
|
2016-05-30 17:29:47 +03:00
|
|
|
backing_length = blk_getlength(backing);
|
2016-05-30 16:53:15 +03:00
|
|
|
if (backing_length < 0) {
|
|
|
|
ret = backing_length;
|
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* If our top snapshot is larger than the backing file image,
|
|
|
|
* grow the backing file image if possible. If not possible,
|
|
|
|
* we must return an error */
|
|
|
|
if (length > backing_length) {
|
2017-06-13 23:20:54 +03:00
|
|
|
ret = blk_truncate(backing, length, PREALLOC_MODE_OFF, &local_err);
|
2016-05-30 16:53:15 +03:00
|
|
|
if (ret < 0) {
|
2017-03-28 23:51:27 +03:00
|
|
|
error_report_err(local_err);
|
2016-05-30 16:53:15 +03:00
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-05-30 17:29:47 +03:00
|
|
|
/* blk_try_blockalign() for src will choose an alignment that works for
|
|
|
|
* backing as well, so no need to compare the alignment manually. */
|
block: Make bdrv_is_allocated() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned
on input and that *pnum is sector-aligned on return to the caller,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, this code adds usages like
DIV_ROUND_UP(,BDRV_SECTOR_SIZE) to callers that still want aligned
values, where the call might reasonbly give non-aligned results
in the future; on the other hand, no rounding is needed for callers
that should just continue to work with byte alignment.
For the most part this patch is just the addition of scaling at the
callers followed by inverse scaling at bdrv_is_allocated(). But
some code, particularly bdrv_commit(), gets a lot simpler because it
no longer has to mess with sectors; also, it is now possible to pass
NULL if the caller does not care how much of the image is allocated
beyond the initial offset. Leave comments where we can further
simplify once a later patch eliminates the need for sector-aligned
requests through bdrv_is_allocated().
For ease of review, bdrv_is_allocated_above() will be tackled
separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:57 +03:00
|
|
|
buf = blk_try_blockalign(src, COMMIT_BUF_SIZE);
|
2016-05-30 16:53:15 +03:00
|
|
|
if (buf == NULL) {
|
|
|
|
ret = -ENOMEM;
|
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
|
|
|
|
block: Make bdrv_is_allocated() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned
on input and that *pnum is sector-aligned on return to the caller,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, this code adds usages like
DIV_ROUND_UP(,BDRV_SECTOR_SIZE) to callers that still want aligned
values, where the call might reasonbly give non-aligned results
in the future; on the other hand, no rounding is needed for callers
that should just continue to work with byte alignment.
For the most part this patch is just the addition of scaling at the
callers followed by inverse scaling at bdrv_is_allocated(). But
some code, particularly bdrv_commit(), gets a lot simpler because it
no longer has to mess with sectors; also, it is now possible to pass
NULL if the caller does not care how much of the image is allocated
beyond the initial offset. Leave comments where we can further
simplify once a later patch eliminates the need for sector-aligned
requests through bdrv_is_allocated().
For ease of review, bdrv_is_allocated_above() will be tackled
separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:57 +03:00
|
|
|
for (offset = 0; offset < length; offset += n) {
|
|
|
|
ret = bdrv_is_allocated(bs, offset, COMMIT_BUF_SIZE, &n);
|
2016-05-30 16:53:15 +03:00
|
|
|
if (ret < 0) {
|
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
|
|
|
if (ret) {
|
block: Make bdrv_is_allocated() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned
on input and that *pnum is sector-aligned on return to the caller,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, this code adds usages like
DIV_ROUND_UP(,BDRV_SECTOR_SIZE) to callers that still want aligned
values, where the call might reasonbly give non-aligned results
in the future; on the other hand, no rounding is needed for callers
that should just continue to work with byte alignment.
For the most part this patch is just the addition of scaling at the
callers followed by inverse scaling at bdrv_is_allocated(). But
some code, particularly bdrv_commit(), gets a lot simpler because it
no longer has to mess with sectors; also, it is now possible to pass
NULL if the caller does not care how much of the image is allocated
beyond the initial offset. Leave comments where we can further
simplify once a later patch eliminates the need for sector-aligned
requests through bdrv_is_allocated().
For ease of review, bdrv_is_allocated_above() will be tackled
separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:57 +03:00
|
|
|
ret = blk_pread(src, offset, buf, n);
|
2016-05-30 16:53:15 +03:00
|
|
|
if (ret < 0) {
|
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
|
|
|
|
block: Make bdrv_is_allocated() byte-based
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file that can do byte-based access.
Changing the signature of the function to use int64_t *pnum ensures
that the compiler enforces that all callers are updated. For now,
the io.c layer still assert()s that all callers are sector-aligned
on input and that *pnum is sector-aligned on return to the caller,
but that can be relaxed when a later patch implements byte-based
block status. Therefore, this code adds usages like
DIV_ROUND_UP(,BDRV_SECTOR_SIZE) to callers that still want aligned
values, where the call might reasonbly give non-aligned results
in the future; on the other hand, no rounding is needed for callers
that should just continue to work with byte alignment.
For the most part this patch is just the addition of scaling at the
callers followed by inverse scaling at bdrv_is_allocated(). But
some code, particularly bdrv_commit(), gets a lot simpler because it
no longer has to mess with sectors; also, it is now possible to pass
NULL if the caller does not care how much of the image is allocated
beyond the initial offset. Leave comments where we can further
simplify once a later patch eliminates the need for sector-aligned
requests through bdrv_is_allocated().
For ease of review, bdrv_is_allocated_above() will be tackled
separately.
Signed-off-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2017-07-07 15:44:57 +03:00
|
|
|
ret = blk_pwrite(backing, offset, buf, n, 0);
|
2016-05-30 16:53:15 +03:00
|
|
|
if (ret < 0) {
|
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if (drv->bdrv_make_empty) {
|
|
|
|
ret = drv->bdrv_make_empty(bs);
|
|
|
|
if (ret < 0) {
|
|
|
|
goto ro_cleanup;
|
|
|
|
}
|
2016-05-30 17:29:47 +03:00
|
|
|
blk_flush(src);
|
2016-05-30 16:53:15 +03:00
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Make sure all data we wrote to the backing device is actually
|
|
|
|
* stable on disk.
|
|
|
|
*/
|
2016-05-30 17:29:47 +03:00
|
|
|
blk_flush(backing);
|
2016-05-30 16:53:15 +03:00
|
|
|
|
|
|
|
ret = 0;
|
|
|
|
ro_cleanup:
|
|
|
|
qemu_vfree(buf);
|
|
|
|
|
2016-05-30 17:29:47 +03:00
|
|
|
blk_unref(backing);
|
2017-01-19 20:16:03 +03:00
|
|
|
if (backing_file_bs) {
|
2017-02-17 22:42:32 +03:00
|
|
|
bdrv_set_backing_hd(bs, backing_file_bs, &error_abort);
|
2017-01-19 20:16:03 +03:00
|
|
|
}
|
|
|
|
bdrv_unref(commit_top_bs);
|
|
|
|
blk_unref(src);
|
2016-05-30 17:29:47 +03:00
|
|
|
|
2016-05-30 16:53:15 +03:00
|
|
|
if (ro) {
|
|
|
|
/* ignoring error return here */
|
|
|
|
bdrv_reopen(bs->backing->bs, open_flags & ~BDRV_O_RDWR, NULL);
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|