From: Eric Blake <eblake@redhat.com>
To: Qiao Nuohan <qiaonuohan@cn.fujitsu.com>,
lcapitulino@redhat.com, zhangxh@cn.fujitsu.com,
Amos <akong@redhat.com>
Cc: stefanha@gmail.com, kumagai-atsushi@mxc.nes.nec.co.jp,
anderson@redhat.com, afaerber@suse.de, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v5 0/9] Make 'dump-guest-memory' dump in kdump-compressed format
Date: Thu, 12 Dec 2013 19:25:44 -0700 [thread overview]
Message-ID: <52AA7028.2090008@redhat.com> (raw)
In-Reply-To: <52AA6BD0.5060302@cn.fujitsu.com>
[-- Attachment #1: Type: text/plain, Size: 1775 bytes --]
On 12/12/2013 07:07 PM, Qiao Nuohan wrote:
> Hi, all
>
> Would you please give some advice about how to continue my series?
>
> My patches have stuck for several months, because I was not allowed to
> add an
> option before introspection is implemented. After observing mails about
> introspection, I find it is still not confirmed its implement way.
I can see several options:
1. Take over the introspection patches and push them through to completion
2. Come up with some alternative witness when your feature is available.
Full-blown introspection of the entire QAPI schema is a heavy hammer,
and if we can have something simpler to use for just this case, but
still discoverable, then management can use that simpler method to learn
if this feature is present. See for example how we added
'query-migrate-capabilities' as a way to add migration capabilities
without needing fullblown introspection.
3. Rebase your patches and post without waiting for any means of
discovery - maybe someone will still be interested enough in using the
patches even without management being able to programmatically determine
the availability.
This is a busy list; it doesn't mean your patch will be outright
rejected, but if you aren't actively pinging and rebasing it, the patch
gets less attention, so people assume no one wanted it. Thanks for
reviving the discussion, and good luck with what you try next.
Personally, I think that both option 1 and 2 at the same time is worth
pursuing (I'd really like introspection, but introspection is a heavy
hammer so having something lighter weight for just your feature is also
worth having).
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 621 bytes --]
prev parent reply other threads:[~2013-12-13 2:26 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-09 7:30 [Qemu-devel] [PATCH v5 0/9] Make 'dump-guest-memory' dump in kdump-compressed format Qiao Nuohan
2013-07-09 7:30 ` [Qemu-devel] [PATCH v5 1/9] dump: Add argument to write_elfxx_notes Qiao Nuohan
2013-07-09 7:30 ` [Qemu-devel] [PATCH v5 2/9] dump: Add API to write header of flatten format Qiao Nuohan
2013-07-16 2:06 ` Stefan Hajnoczi
2013-07-16 3:50 ` Qiao Nuohan
2013-07-09 7:30 ` [Qemu-devel] [PATCH v5 3/9] dump: Add API to write vmcore Qiao Nuohan
2013-07-09 7:30 ` [Qemu-devel] [PATCH v5 4/9] dump: Add API to write elf notes to buffer Qiao Nuohan
2013-07-09 7:30 ` [Qemu-devel] [PATCH v5 5/9] dump: add API to write dump header Qiao Nuohan
2013-07-16 2:18 ` Stefan Hajnoczi
2013-07-09 7:30 ` [Qemu-devel] [PATCH v5 6/9] dump: Add API to write dump_bitmap Qiao Nuohan
2013-07-16 2:25 ` Stefan Hajnoczi
2013-07-09 7:30 ` [Qemu-devel] [PATCH v5 7/9] dump: Add APIs to operate DataCache Qiao Nuohan
2013-07-09 7:30 ` [Qemu-devel] [PATCH v5 8/9] dump: Add API to write dump pages Qiao Nuohan
2013-07-16 2:43 ` Stefan Hajnoczi
2013-07-16 3:40 ` Qiao Nuohan
2013-07-16 16:28 ` Eric Blake
2013-07-09 7:30 ` [Qemu-devel] [PATCH v5 9/9] dump: Make kdump-compressed format available for 'dump-guest-memory' Qiao Nuohan
2013-07-09 7:36 ` [Qemu-devel] [PATCH v5 0/9] Make 'dump-guest-memory' dump in kdump-compressed format Qiao Nuohan
2013-07-15 9:56 ` Qiao Nuohan
2013-07-16 2:45 ` Stefan Hajnoczi
2013-07-31 9:26 ` Qiao Nuohan
2013-07-31 12:23 ` Laszlo Ersek
2013-07-31 13:10 ` Luiz Capitulino
2013-08-02 3:25 ` Qiao Nuohan
2013-08-02 3:41 ` Amos Kong
2013-08-02 3:43 ` Qiao Nuohan
2013-08-02 3:37 ` Qiao Nuohan
2013-12-13 2:07 ` Qiao Nuohan
2013-12-13 2:25 ` Eric Blake [this message]
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=52AA7028.2090008@redhat.com \
--to=eblake@redhat.com \
--cc=afaerber@suse.de \
--cc=akong@redhat.com \
--cc=anderson@redhat.com \
--cc=kumagai-atsushi@mxc.nes.nec.co.jp \
--cc=lcapitulino@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=qiaonuohan@cn.fujitsu.com \
--cc=stefanha@gmail.com \
--cc=zhangxh@cn.fujitsu.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).