[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 2/6] replication: add shared-disk and shared-
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v3 2/6] replication: add shared-disk and shared-disk-id options |
Date: |
Mon, 27 Feb 2017 17:10:52 +0000 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Fri, Jan 20, 2017 at 11:47:56AM +0800, zhanghailiang wrote:
> @@ -119,12 +136,31 @@ static int replication_open(BlockDriverState *bs, QDict
> *options,
> "The option mode's value should be primary or secondary");
> goto fail;
> }
> + s->is_shared_disk = qemu_opt_get_bool(opts, REPLICATION_SHARED_DISK,
> + false);
> + if (s->is_shared_disk && (s->mode == REPLICATION_MODE_PRIMARY)) {
> + shared_disk_id = qemu_opt_get(opts, REPLICATION_SHARED_DISK_ID);
> + if (!shared_disk_id) {
> + error_setg(&local_err, "Missing shared disk blk option");
> + goto fail;
> + }
> + s->shared_disk_id = g_strdup(shared_disk_id);
> + blk = blk_by_name(s->shared_disk_id);
> + if (!blk) {
> + error_setg(&local_err, "There is no %s block",
> s->shared_disk_id);
> + goto fail;
> + }
> + /* We can't access root member of BlockBackend directly */
> + tmp_bs = blk_bs(blk);
> + s->primary_disk = QLIST_FIRST(&tmp_bs->parents);
Why is this necessary?
We already have the BlockBackend for the primary disk. I'm not sure why
the BdrvChild is needed.
Stefan
signature.asc
Description: PGP signature
- Re: [Qemu-devel] [PATCH v3 2/6] replication: add shared-disk and shared-disk-id options,
Stefan Hajnoczi <=