[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] live-block-ops.txt: Rename, rewrite, and imp
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH v3] live-block-ops.txt: Rename, rewrite, and improve it |
Date: |
Wed, 21 Jun 2017 18:49:02 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
On 06/21/2017 06:19 AM, Kashyap Chamarthy wrote:
> This edition documents (including their QMP invocations) all four
> operations:
>
> - `block-stream`
> - `block-commit`
> - `drive-mirror` (& `blockdev-mirror`)
> - `drive-backup` (& `blockdev-backup`)
>
> Things considered while writing this document:
>
> - Use reStructuredText as markup language (with the goal of generating
> the HTML output using the Sphinx Documentation Generator). It is
> gentler on the eye, and can be trivially converted to different
> formats. (Another reason: upstream QEMU is considering to switch to
> Sphinx, which uses reStructuredText as its markup language.)
>
> - Raw QMP JSON output vs. 'qmp-shell'. I debated with myself whether
> to only show raw QMP JSON output (as that is the canonical
> representation), or use 'qmp-shell', which takes key-value pairs. I
> settled on the approach of: for the first occurence of a command,
> use raw JSON; for subsequent occurences, use 'qmp-shell', with an
> occasional exception.
>
> - Usage of `-blockdev` command-line.
>
> - Usage of 'node-name' vs. file path to refer to disks. While we have
> `blockdev-{mirror, backup}` as 'node-name'-alternatives for
> `drive-{mirror, backup}`, the `block-commit` command still operate
> on file names for parameters 'base' and 'top'. So I added a caveat
> at the beginning to that effect.
>
> Refer this related thread that I started (where I learnt
> `block-stream` was recently reworked to accept 'node-name' for 'top'
> and 'base' parameters):
> https://lists.nongnu.org/archive/html/qemu-devel/2017-05/msg06466.html
> "[RFC] Making 'block-stream', and 'block-commit' accept node-name"
>
> All commands showed in this document were tested while documenting.
>
> Thanks: Eric Blake for the section: "A note on points-in-time vs file
> names". This useful bit was originally articulated by Eric in his
> KVMForum 2015 presentation, so I included that specific bit in this
> document.
>
> Signed-off-by: Kashyap Chamarthy <address@hidden>
> ---
> * A Sphinx-rendered HTML version is here:
>
> https://kashyapc.fedorapeople.org/v3-QEMU-Docs/_build/html/docs/live-block-operations.html
>
>
[snip]
>
> * TODO (after feedback from John Snow):
> - Eric Blake suggested to consider documenting incremental backup
> policies as part of the section: "Live disk backup ---
> `drive-backup` and `blockdev-backup`"
Perhaps it could be mentioned, but hopefully I've covered it in some
sufficient detail in the (now) docs/devel/bitmaps.md file; I'm a little
wary of duplicating efforts in this area, but you've covered everything
*else* in good detail here, so now my file is the odd one out.
I will leave this up to you, really. Perhaps it could be paid some lip
service with a link to the other document? The detail in bitmaps.md is a
little more verbose than the rest of this file, so if you include it
wholesale it'd dwarf the rest of this document.
What do you think?
- [Qemu-devel] [PATCH v3] live-block-ops.txt: Rename, rewrite, and improve it, Kashyap Chamarthy, 2017/06/21
- Re: [Qemu-devel] [PATCH v3] live-block-ops.txt: Rename, rewrite, and improve it,
John Snow <=
- Re: [Qemu-devel] [PATCH v3] live-block-ops.txt: Rename, rewrite, and improve it, Kashyap Chamarthy, 2017/06/22
- Re: [Qemu-devel] [PATCH v3] live-block-ops.txt: Rename, rewrite, and improve it, John Snow, 2017/06/22
- Re: [Qemu-devel] [PATCH v3] live-block-ops.txt: Rename, rewrite, and improve it, Kashyap Chamarthy, 2017/06/23
- Re: [Qemu-devel] [PATCH v3] live-block-ops.txt: Rename, rewrite, and improve it, John Snow, 2017/06/23