2012-01-18 18:40:44 +04:00
|
|
|
/*
|
|
|
|
* Image streaming
|
|
|
|
*
|
|
|
|
* Copyright IBM, Corp. 2011
|
|
|
|
*
|
|
|
|
* Authors:
|
|
|
|
* Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
|
|
|
|
*
|
|
|
|
* 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"
|
2012-01-18 18:40:44 +04:00
|
|
|
#include "trace.h"
|
2012-12-17 21:19:44 +04:00
|
|
|
#include "block/block_int.h"
|
|
|
|
#include "block/blockjob.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-05-09 18:09:46 +04:00
|
|
|
#include "qemu/ratelimit.h"
|
2015-10-19 18:53:22 +03:00
|
|
|
#include "sysemu/block-backend.h"
|
2012-01-18 18:40:44 +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.
|
|
|
|
*/
|
|
|
|
STREAM_BUFFER_SIZE = 512 * 1024, /* in bytes */
|
|
|
|
};
|
|
|
|
|
2012-01-18 18:40:45 +04:00
|
|
|
#define SLICE_TIME 100000000ULL /* ns */
|
|
|
|
|
2012-01-18 18:40:44 +04:00
|
|
|
typedef struct StreamBlockJob {
|
|
|
|
BlockJob common;
|
2012-01-18 18:40:45 +04:00
|
|
|
RateLimit limit;
|
2012-01-18 18:40:44 +04:00
|
|
|
BlockDriverState *base;
|
2012-09-28 19:22:59 +04:00
|
|
|
BlockdevOnError on_error;
|
block: add backing-file option to block-stream
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 job.
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-stream api, the user is able to change
the backing file of the active layer as part of the block-stream
operation.
This allows the change to be 'safe', in the sense that if the attempt
to write the active image metadata fails, then the block-stream
operation returns failure, without disrupting the guest.
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>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
2014-06-25 23:40:11 +04:00
|
|
|
char *backing_file_str;
|
2016-10-28 10:08:10 +03:00
|
|
|
int bs_flags;
|
2012-01-18 18:40:44 +04:00
|
|
|
} StreamBlockJob;
|
|
|
|
|
2016-04-12 16:15:49 +03:00
|
|
|
static int coroutine_fn stream_populate(BlockBackend *blk,
|
2012-01-18 18:40:44 +04:00
|
|
|
int64_t sector_num, int nb_sectors,
|
|
|
|
void *buf)
|
|
|
|
{
|
|
|
|
struct iovec iov = {
|
|
|
|
.iov_base = buf,
|
|
|
|
.iov_len = nb_sectors * BDRV_SECTOR_SIZE,
|
|
|
|
};
|
|
|
|
QEMUIOVector qiov;
|
|
|
|
|
|
|
|
qemu_iovec_init_external(&qiov, &iov, 1);
|
|
|
|
|
|
|
|
/* Copy-on-read the unallocated clusters */
|
2016-04-12 16:15:49 +03:00
|
|
|
return blk_co_preadv(blk, sector_num * BDRV_SECTOR_SIZE, qiov.size, &qiov,
|
|
|
|
BDRV_REQ_COPY_ON_READ);
|
2012-01-18 18:40:44 +04:00
|
|
|
}
|
|
|
|
|
2014-10-21 15:03:57 +04:00
|
|
|
typedef struct {
|
|
|
|
int ret;
|
|
|
|
bool reached_end;
|
|
|
|
} StreamCompleteData;
|
|
|
|
|
|
|
|
static void stream_complete(BlockJob *job, void *opaque)
|
|
|
|
{
|
|
|
|
StreamBlockJob *s = container_of(job, StreamBlockJob, common);
|
|
|
|
StreamCompleteData *data = opaque;
|
2016-04-12 16:15:49 +03:00
|
|
|
BlockDriverState *bs = blk_bs(job->blk);
|
2014-10-21 15:03:57 +04:00
|
|
|
BlockDriverState *base = s->base;
|
|
|
|
|
|
|
|
if (!block_job_is_cancelled(&s->common) && data->reached_end &&
|
|
|
|
data->ret == 0) {
|
|
|
|
const char *base_id = NULL, *base_fmt = NULL;
|
|
|
|
if (base) {
|
|
|
|
base_id = s->backing_file_str;
|
|
|
|
if (base->drv) {
|
|
|
|
base_fmt = base->drv->format_name;
|
|
|
|
}
|
|
|
|
}
|
2016-04-12 16:15:49 +03:00
|
|
|
data->ret = bdrv_change_backing_file(bs, base_id, base_fmt);
|
|
|
|
bdrv_set_backing_hd(bs, base);
|
2014-10-21 15:03:57 +04:00
|
|
|
}
|
|
|
|
|
2016-10-28 10:08:10 +03:00
|
|
|
/* Reopen the image back in read-only mode if necessary */
|
|
|
|
if (s->bs_flags != bdrv_get_flags(bs)) {
|
|
|
|
bdrv_reopen(bs, s->bs_flags, NULL);
|
|
|
|
}
|
|
|
|
|
2014-10-21 15:03:57 +04:00
|
|
|
g_free(s->backing_file_str);
|
|
|
|
block_job_completed(&s->common, data->ret);
|
|
|
|
g_free(data);
|
|
|
|
}
|
|
|
|
|
2012-01-18 18:40:44 +04:00
|
|
|
static void coroutine_fn stream_run(void *opaque)
|
|
|
|
{
|
|
|
|
StreamBlockJob *s = opaque;
|
2014-10-21 15:03:57 +04:00
|
|
|
StreamCompleteData *data;
|
2016-04-12 16:15:49 +03:00
|
|
|
BlockBackend *blk = s->common.blk;
|
|
|
|
BlockDriverState *bs = blk_bs(blk);
|
2012-01-18 18:40:53 +04:00
|
|
|
BlockDriverState *base = s->base;
|
2016-03-21 16:47:25 +03:00
|
|
|
int64_t sector_num = 0;
|
|
|
|
int64_t end = -1;
|
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-28 19:22:59 +04:00
|
|
|
int error = 0;
|
2012-01-18 18:40:44 +04:00
|
|
|
int ret = 0;
|
2012-05-10 18:10:42 +04:00
|
|
|
int n = 0;
|
2012-01-18 18:40:44 +04:00
|
|
|
void *buf;
|
|
|
|
|
2015-06-17 15:55:21 +03:00
|
|
|
if (!bs->backing) {
|
2016-03-21 16:47:25 +03:00
|
|
|
goto out;
|
2013-11-13 23:37:58 +04:00
|
|
|
}
|
|
|
|
|
2012-01-18 18:40:44 +04:00
|
|
|
s->common.len = bdrv_getlength(bs);
|
|
|
|
if (s->common.len < 0) {
|
2016-03-21 16:47:25 +03:00
|
|
|
ret = s->common.len;
|
|
|
|
goto out;
|
2012-01-18 18:40:44 +04:00
|
|
|
}
|
|
|
|
|
|
|
|
end = s->common.len >> BDRV_SECTOR_BITS;
|
|
|
|
buf = qemu_blockalign(bs, STREAM_BUFFER_SIZE);
|
|
|
|
|
|
|
|
/* Turn on copy-on-read for the whole block device so that guest read
|
|
|
|
* requests help us make progress. Only do this when copying the entire
|
|
|
|
* backing chain since the copy-on-read operation does not take base into
|
|
|
|
* account.
|
|
|
|
*/
|
|
|
|
if (!base) {
|
|
|
|
bdrv_enable_copy_on_read(bs);
|
|
|
|
}
|
|
|
|
|
|
|
|
for (sector_num = 0; sector_num < end; sector_num += n) {
|
2012-05-08 18:52:00 +04:00
|
|
|
bool copy;
|
2012-05-08 18:51:45 +04:00
|
|
|
|
|
|
|
/* 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-05-08 18:51:45 +04:00
|
|
|
*/
|
2013-08-21 19:03:05 +04:00
|
|
|
block_job_sleep_ns(&s->common, QEMU_CLOCK_REALTIME, delay_ns);
|
2012-01-18 18:40:44 +04:00
|
|
|
if (block_job_is_cancelled(&s->common)) {
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
2013-09-22 10:19:10 +04:00
|
|
|
copy = false;
|
|
|
|
|
2013-09-04 21:00:22 +04:00
|
|
|
ret = bdrv_is_allocated(bs, sector_num,
|
|
|
|
STREAM_BUFFER_SIZE / BDRV_SECTOR_SIZE, &n);
|
2012-05-08 18:52:00 +04:00
|
|
|
if (ret == 1) {
|
|
|
|
/* Allocated in the top, no need to copy. */
|
2013-09-04 21:00:25 +04:00
|
|
|
} else if (ret >= 0) {
|
2012-05-08 18:52:00 +04:00
|
|
|
/* Copy if allocated in the intermediate images. Limit to the
|
|
|
|
* known-unallocated area [sector_num, sector_num+n). */
|
2015-06-17 15:55:21 +03:00
|
|
|
ret = bdrv_is_allocated_above(backing_bs(bs), base,
|
2013-09-04 21:00:24 +04:00
|
|
|
sector_num, n, &n);
|
2012-08-28 18:26:48 +04:00
|
|
|
|
|
|
|
/* Finish early if end of backing file has been reached */
|
|
|
|
if (ret == 0 && n == 0) {
|
|
|
|
n = end - sector_num;
|
|
|
|
}
|
|
|
|
|
2012-05-08 18:52:00 +04:00
|
|
|
copy = (ret == 1);
|
|
|
|
}
|
2012-01-18 18:40:44 +04:00
|
|
|
trace_stream_one_iteration(s, sector_num, n, ret);
|
2013-09-22 10:19:10 +04:00
|
|
|
if (copy) {
|
2016-04-12 16:15:49 +03:00
|
|
|
ret = stream_populate(blk, sector_num, n, buf);
|
2012-01-18 18:40:44 +04:00
|
|
|
}
|
|
|
|
if (ret < 0) {
|
2012-09-28 19:22:59 +04:00
|
|
|
BlockErrorAction action =
|
2016-04-18 12:36:38 +03:00
|
|
|
block_job_error_action(&s->common, s->on_error, true, -ret);
|
2014-06-18 10:43:30 +04:00
|
|
|
if (action == BLOCK_ERROR_ACTION_STOP) {
|
2012-09-28 19:22:59 +04:00
|
|
|
n = 0;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if (error == 0) {
|
|
|
|
error = ret;
|
|
|
|
}
|
2014-06-18 10:43:30 +04:00
|
|
|
if (action == BLOCK_ERROR_ACTION_REPORT) {
|
2012-09-28 19:22:59 +04:00
|
|
|
break;
|
|
|
|
}
|
2012-01-18 18:40:44 +04:00
|
|
|
}
|
2012-01-18 18:40:53 +04:00
|
|
|
ret = 0;
|
2012-01-18 18:40:44 +04:00
|
|
|
|
|
|
|
/* Publish progress */
|
|
|
|
s->common.offset += n * BDRV_SECTOR_SIZE;
|
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) {
|
|
|
|
delay_ns = ratelimit_calculate_delay(&s->limit, n);
|
|
|
|
}
|
2012-01-18 18:40:44 +04:00
|
|
|
}
|
|
|
|
|
|
|
|
if (!base) {
|
|
|
|
bdrv_disable_copy_on_read(bs);
|
|
|
|
}
|
|
|
|
|
2012-09-28 19:22:59 +04:00
|
|
|
/* Do not remove the backing file if an error was there but ignored. */
|
|
|
|
ret = error;
|
|
|
|
|
2012-01-18 18:40:44 +04:00
|
|
|
qemu_vfree(buf);
|
2014-10-21 15:03:57 +04:00
|
|
|
|
2016-03-21 16:47:25 +03:00
|
|
|
out:
|
2014-10-21 15:03:57 +04:00
|
|
|
/* Modify backing chain and close BDSes in main loop */
|
|
|
|
data = g_malloc(sizeof(*data));
|
|
|
|
data->ret = ret;
|
|
|
|
data->reached_end = sector_num == end;
|
|
|
|
block_job_defer_to_main_loop(&s->common, stream_complete, data);
|
2012-01-18 18:40:44 +04:00
|
|
|
}
|
|
|
|
|
2012-04-25 19:51:02 +04:00
|
|
|
static void stream_set_speed(BlockJob *job, int64_t speed, Error **errp)
|
2012-01-18 18:40:45 +04:00
|
|
|
{
|
|
|
|
StreamBlockJob *s = container_of(job, StreamBlockJob, common);
|
|
|
|
|
2012-04-25 19:51:02 +04:00
|
|
|
if (speed < 0) {
|
2015-03-17 13:54:50 +03:00
|
|
|
error_setg(errp, QERR_INVALID_PARAMETER, "speed");
|
2012-04-25 19:51:01 +04:00
|
|
|
return;
|
2012-01-18 18:40:45 +04:00
|
|
|
}
|
2012-05-09 18:09:46 +04:00
|
|
|
ratelimit_set_speed(&s->limit, speed / BDRV_SECTOR_SIZE, SLICE_TIME);
|
2012-01-18 18:40:45 +04:00
|
|
|
}
|
|
|
|
|
2013-10-08 13:29:38 +04:00
|
|
|
static const BlockJobDriver stream_job_driver = {
|
2012-01-18 18:40:44 +04:00
|
|
|
.instance_size = sizeof(StreamBlockJob),
|
2013-10-08 13:29:40 +04:00
|
|
|
.job_type = BLOCK_JOB_TYPE_STREAM,
|
2012-01-18 18:40:45 +04:00
|
|
|
.set_speed = stream_set_speed,
|
2012-01-18 18:40:44 +04:00
|
|
|
};
|
|
|
|
|
2016-07-05 17:28:59 +03:00
|
|
|
void stream_start(const char *job_id, BlockDriverState *bs,
|
|
|
|
BlockDriverState *base, const char *backing_file_str,
|
|
|
|
int64_t speed, BlockdevOnError on_error,
|
|
|
|
BlockCompletionFunc *cb, void *opaque, Error **errp)
|
2012-01-18 18:40:44 +04:00
|
|
|
{
|
|
|
|
StreamBlockJob *s;
|
2016-10-28 10:08:10 +03:00
|
|
|
BlockDriverState *iter;
|
|
|
|
int orig_bs_flags;
|
2012-01-18 18:40:44 +04:00
|
|
|
|
2016-07-05 17:28:59 +03:00
|
|
|
s = block_job_create(job_id, &stream_job_driver, bs, speed,
|
2016-10-27 19:06:56 +03:00
|
|
|
BLOCK_JOB_DEFAULT, cb, opaque, errp);
|
2012-01-18 18:40:44 +04:00
|
|
|
if (!s) {
|
2012-04-25 19:51:00 +04:00
|
|
|
return;
|
2012-01-18 18:40:44 +04:00
|
|
|
}
|
|
|
|
|
2016-10-28 10:08:10 +03:00
|
|
|
/* Make sure that the image is opened in read-write mode */
|
|
|
|
orig_bs_flags = bdrv_get_flags(bs);
|
|
|
|
if (!(orig_bs_flags & BDRV_O_RDWR)) {
|
|
|
|
if (bdrv_reopen(bs, orig_bs_flags | BDRV_O_RDWR, errp) != 0) {
|
|
|
|
block_job_unref(&s->common);
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Block all intermediate nodes between bs and base, because they
|
|
|
|
* will disappear from the chain after this operation */
|
|
|
|
for (iter = backing_bs(bs); iter && iter != base; iter = backing_bs(iter)) {
|
|
|
|
block_job_add_bdrv(&s->common, iter);
|
|
|
|
}
|
|
|
|
|
2012-01-18 18:40:44 +04:00
|
|
|
s->base = base;
|
block: add backing-file option to block-stream
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 job.
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-stream api, the user is able to change
the backing file of the active layer as part of the block-stream
operation.
This allows the change to be 'safe', in the sense that if the attempt
to write the active image metadata fails, then the block-stream
operation returns failure, without disrupting the guest.
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>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
2014-06-25 23:40:11 +04:00
|
|
|
s->backing_file_str = g_strdup(backing_file_str);
|
2016-10-28 10:08:10 +03:00
|
|
|
s->bs_flags = orig_bs_flags;
|
2012-01-18 18:40:44 +04:00
|
|
|
|
2012-09-28 19:22:59 +04:00
|
|
|
s->on_error = on_error;
|
coroutine: move entry argument to qemu_coroutine_create
In practice the entry argument is always known at creation time, and
it is confusing that sometimes qemu_coroutine_enter is used with a
non-NULL argument to re-enter a coroutine (this happens in
block/sheepdog.c and tests/test-coroutine.c). So pass the opaque value
at creation time, for consistency with e.g. aio_bh_new.
Mostly done with the following semantic patch:
@ entry1 @
expression entry, arg, co;
@@
- co = qemu_coroutine_create(entry);
+ co = qemu_coroutine_create(entry, arg);
...
- qemu_coroutine_enter(co, arg);
+ qemu_coroutine_enter(co);
@ entry2 @
expression entry, arg;
identifier co;
@@
- Coroutine *co = qemu_coroutine_create(entry);
+ Coroutine *co = qemu_coroutine_create(entry, arg);
...
- qemu_coroutine_enter(co, arg);
+ qemu_coroutine_enter(co);
@ entry3 @
expression entry, arg;
@@
- qemu_coroutine_enter(qemu_coroutine_create(entry), arg);
+ qemu_coroutine_enter(qemu_coroutine_create(entry, arg));
@ reentry @
expression co;
@@
- qemu_coroutine_enter(co, NULL);
+ qemu_coroutine_enter(co);
except for the aforementioned few places where the semantic patch
stumbled (as expected) and for test_co_queue, which would otherwise
produce an uninitialized variable warning.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2016-07-04 20:10:01 +03:00
|
|
|
s->common.co = qemu_coroutine_create(stream_run, s);
|
2012-05-08 18:51:46 +04:00
|
|
|
trace_stream_start(bs, base, s, s->common.co, opaque);
|
coroutine: move entry argument to qemu_coroutine_create
In practice the entry argument is always known at creation time, and
it is confusing that sometimes qemu_coroutine_enter is used with a
non-NULL argument to re-enter a coroutine (this happens in
block/sheepdog.c and tests/test-coroutine.c). So pass the opaque value
at creation time, for consistency with e.g. aio_bh_new.
Mostly done with the following semantic patch:
@ entry1 @
expression entry, arg, co;
@@
- co = qemu_coroutine_create(entry);
+ co = qemu_coroutine_create(entry, arg);
...
- qemu_coroutine_enter(co, arg);
+ qemu_coroutine_enter(co);
@ entry2 @
expression entry, arg;
identifier co;
@@
- Coroutine *co = qemu_coroutine_create(entry);
+ Coroutine *co = qemu_coroutine_create(entry, arg);
...
- qemu_coroutine_enter(co, arg);
+ qemu_coroutine_enter(co);
@ entry3 @
expression entry, arg;
@@
- qemu_coroutine_enter(qemu_coroutine_create(entry), arg);
+ qemu_coroutine_enter(qemu_coroutine_create(entry, arg));
@ reentry @
expression co;
@@
- qemu_coroutine_enter(co, NULL);
+ qemu_coroutine_enter(co);
except for the aforementioned few places where the semantic patch
stumbled (as expected) and for test_co_queue, which would otherwise
produce an uninitialized variable warning.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2016-07-04 20:10:01 +03:00
|
|
|
qemu_coroutine_enter(s->common.co);
|
2012-01-18 18:40:44 +04:00
|
|
|
}
|