[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 00/24] block: Command line option -blockdev
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-block] [PATCH v2 00/24] block: Command line option -blockdev |
Date: |
Tue, 28 Feb 2017 22:55:57 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Markus Armbruster <address@hidden> writes:
> Actually, the command line option is the least part of this series.
> Its bulk is about building infrastructure and getting errors out of
> the JSON parser[*]. The latter part could be spun out in its own
> series, if that helps. We'll see.
>
> The design of the command line interface was discussed here:
> Subject: Non-flat command line option argument syntax
> Message-ID: <address@hidden>
> https://lists.gnu.org/archive/html/qemu-devel/2017-02/msg00555.html
>
> This series is based on "[PULL 00/26] QAPI patches for 2017-02-28".
> Available as branch blockdev-cmdline in my public repo
> <http://repo.or.cz/w/qemu/armbru.git>.
>
> [*] Who'd have thunk anyone actually wants to know what's wrong with
> his JSON!
>
> v2: Address review by Eric and Kevin
Forgot to mention: patches reordered, PATCH 21-23 are now PATCH 17-19.
> PATCH 03: Comment fixes, memory leak plugged
> PATCH 05: Comment fix
> PATCH 08+12: Announce followup work in commit message
> PATCH 18: Comment fix
> PATCH 20: A missing test for failure fixed
> PATCH 21: Option help text improved, loop to call qmp_blockdev_add()
> moved to the right place
> PATCH 22: Commit message fix
> PATCH 23: Commit message fix, qobject_input_get_keyval() fixed to set
> v->keyval, memory leaks plugged, test case tweaked to cover
> leading zero, keyval_listify() thinko fixed (this is the
> only non-trivial change in v2)
- [Qemu-block] [PATCH v2 18/24] qapi: New parse_qapi_name(), (continued)
- [Qemu-block] [PATCH v2 18/24] qapi: New parse_qapi_name(), Markus Armbruster, 2017/02/28
- [Qemu-block] [PATCH v2 17/24] test-qapi-util: New, covering qapi/qapi-util.c, Markus Armbruster, 2017/02/28
- [Qemu-block] [PATCH v2 21/24] block: Initial implementation of -blockdev, Markus Armbruster, 2017/02/28
- [Qemu-block] [PATCH v2 24/24] keyval: Support lists, Markus Armbruster, 2017/02/28
- Re: [Qemu-block] [PATCH v2 24/24] keyval: Support lists, Eric Blake, 2017/02/28
- Re: [Qemu-block] [PATCH v2 00/24] block: Command line option -blockdev,
Markus Armbruster <=