From: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
To: qemu-block@nongnu.org
Cc: qemu-devel@nongnu.org, eblake@redhat.com, armbru@redhat.com,
hreitz@redhat.com, kwolf@redhat.com, vsementsov@virtuozzo.com,
den@openvz.org, nshirokovskiy@virtuozzo.com, yur@virtuozzo.com,
dim@virtuozzo.com, igor@virtuozzo.com, pkrempa@redhat.com,
libvir-list@redhat.com, stefanha@redhat.com
Subject: [PATCH RFC v2 0/4] blockdev-replace
Date: Wed, 22 Dec 2021 17:02:50 +0100 [thread overview]
Message-ID: <20211222160254.226909-1-vsementsov@virtuozzo.com> (raw)
Hi all!
v2:
1. Let's start with minimal functionality: replace only one specific
child.
2. Explicit type of parent: qdev / export / driver
Hmm, 'driver' is for BlockDriverState.. Any better idea? I don't want to
call it "node" or even "block-node", as finally, devices and exports are
kind of nodes too in the whole block graph..
3. I think it's better to start with x- prefix
Plans for v3:
- add tests of filter insertion and removing
- add qmp transaction support
Vladimir Sementsov-Ogievskiy (4):
block-backend: blk_root(): drop const specifier on return type
block/export: add blk_by_export_id()
block: make bdrv_find_child() function public
qapi: add blockdev-replace command
qapi/block.json | 61 ++++++++++++++++++++++++++++++++++
include/block/block_int.h | 1 +
include/sysemu/block-backend.h | 3 +-
block.c | 13 ++++++++
block/block-backend.c | 2 +-
block/export/export.c | 18 ++++++++++
block/qapi-sysemu.c | 56 +++++++++++++++++++++++++++++++
blockdev.c | 14 --------
8 files changed, 152 insertions(+), 16 deletions(-)
--
2.31.1
next reply other threads:[~2021-12-22 16:07 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-22 16:02 Vladimir Sementsov-Ogievskiy [this message]
2021-12-22 16:02 ` [PATCH v2 1/4] block-backend: blk_root(): drop const specifier on return type Vladimir Sementsov-Ogievskiy
2021-12-22 16:02 ` [PATCH v2 2/4] block/export: add blk_by_export_id() Vladimir Sementsov-Ogievskiy
2021-12-22 16:02 ` [PATCH v2 3/4] block: make bdrv_find_child() function public Vladimir Sementsov-Ogievskiy
2021-12-22 16:02 ` [PATCH v2 4/4] qapi: add blockdev-replace command Vladimir Sementsov-Ogievskiy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211222160254.226909-1-vsementsov@virtuozzo.com \
--to=vsementsov@virtuozzo.com \
--cc=armbru@redhat.com \
--cc=den@openvz.org \
--cc=dim@virtuozzo.com \
--cc=eblake@redhat.com \
--cc=hreitz@redhat.com \
--cc=igor@virtuozzo.com \
--cc=kwolf@redhat.com \
--cc=libvir-list@redhat.com \
--cc=nshirokovskiy@virtuozzo.com \
--cc=pkrempa@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
--cc=yur@virtuozzo.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).