From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:50655)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <lersek@redhat.com>) id 1WCywO-0008Hw-MH
	for qemu-devel@nongnu.org; Mon, 10 Feb 2014 17:03:04 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <lersek@redhat.com>) id 1WCywK-0007lA-5w
	for qemu-devel@nongnu.org; Mon, 10 Feb 2014 17:03:00 -0500
Received: from mx1.redhat.com ([209.132.183.28]:44868)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <lersek@redhat.com>) id 1WCywJ-0007kv-U1
	for qemu-devel@nongnu.org; Mon, 10 Feb 2014 17:02:56 -0500
Message-ID: <52F94C84.20105@redhat.com>
Date: Mon, 10 Feb 2014 23:02:44 +0100
From: Laszlo Ersek <lersek@redhat.com>
MIME-Version: 1.0
References: <1390890126-17377-1-git-send-email-qiaonuohan@cn.fujitsu.com>
	<1390890126-17377-14-git-send-email-qiaonuohan@cn.fujitsu.com>
	<20140210141048.796e9c91@redhat.com>
In-Reply-To: <20140210141048.796e9c91@redhat.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH v8 13/13] dump: add
 'query-dump-guest-memory-capability' command
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Luiz Capitulino <lcapitulino@redhat.com>
Cc: stefanha@gmail.com, qemu-devel@nongnu.org, qiaonuohan <qiaonuohan@cn.fujitsu.com>, kumagai-atsushi@mxc.nes.nec.co.jp, anderson@redhat.com, afaerber@suse.de

On 02/10/14 20:10, Luiz Capitulino wrote:
> On Tue, 28 Jan 2014 14:22:06 +0800
> qiaonuohan <qiaonuohan@cn.fujitsu.com> wrote:
> 
>> 'query-dump-guest-memory-capability' is used to query whether option 'format'
>> is available for 'dump-guest-memory' and the available format. The output
>> of the command will be like:
>>
>> -> { "execute": "query-dump-guest-memory-capability" }
>> <- { "return": {
>>         "format-option": "optional",
>>         "capabilities": [
>>             {"available": true, "format": "elf"},
>>             {"available": true, "format": "kdump-zlib"},
>>             {"available": true, "format": "kdump-lzo"},
>>             {"available": true, "format": "kdump-snappy"}
>>         ]
>>     }
> 
> I don't want to hold this series anymore, this series is long and I know it
> took you and Laszlo's a long time to get it right. On the other hand we can't
> allow every single command to have its own introspection protocol.
> 
> I think I'm fine accepting this one now, as long as it's fine for libvirt
> too. Eric, can you confirm this please?

We discussed this before, and Eric participated. In fact the custom
introspection was one of his recommendations.

http://thread.gmane.org/gmane.comp.emulators.qemu/221270/focus=246650

(Which I agreed with because it would give us the most independence.)

Of course I'm not trying to imply that this one specific interface will
doubtlessly serve all of libvirt's needs wrt. the kdump feature. We
certainly need Eric to sign off on that.

Thanks,
Laszlo