[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 00/15] qapi netdev_add introspection (post-in
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v7 00/15] qapi netdev_add introspection (post-introspection cleanups subset F) |
Date: |
Thu, 16 Jun 2016 16:57:40 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Eric Blake <address@hidden> writes:
> The end is nearly in sight! Last August, I started a process of
> cleaning up qapi on top of Markus' introspection work. Several
> hundred emails later, and numerous intermediate series out of
> the way, I'm finally at the end of my queue of patches that were
> originally sent in v5:
> https://lists.gnu.org/archive/html/qemu-devel/2015-09/msg05410.html
So your QAPI queue is not in fact infinite? ;-)
The series looks pretty close. I'm confident we can get it into 2.7.
> v6 was here (yes, 5 months ago):
> https://www.mail-archive.com/address@hidden/msg343192.html
>
> Prerequisites:
> My clone/JSON visitor (weak; this series is mostly orthogonal to
> that one, if we want to swap the order of the two based on review
> complexity):
> https://www.mail-archive.com/qemu-devel%40nongnu.org/msg371555.html
>
> Once my series are in, we should focus on Marc-Andre's cleanups.
Yes.
> Ideas for further qapi work - which I have NOT coded yet:
> - convert more clients to use boxed commands
> - get rid of has_FOO when FOO is pointer
> - support default values (and get rid of has_FOO when FOO has default)
> - finally QAPI-fy device_add
> - consider swapping blockdev-add to use QAPI structs throughout the
> code-base, rather than converting back into flat QDict and using QemuOpts
> - introspection of command line options
Worthy projects.
I appreciate your patches very much, but if you keep making them faster
than I can review them, I'll get tired of QAPI. I'd love to get around
to some serious development once in a while...
- Re: [Qemu-devel] [PATCH v7 00/15] qapi netdev_add introspection (post-introspection cleanups subset F),
Markus Armbruster <=