[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-cap
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-capability' command |
Date: |
Mon, 10 Feb 2014 21:47:21 -0500 |
On Tue, 11 Feb 2014 00:34:37 +0100
Paolo Bonzini <address@hidden> wrote:
> Il 11/02/2014 00:30, Laszlo Ersek ha scritto:
> >> > However, I don't see the point in having the "format-option" field. What
> >> > about:
> >> >
> >> > -> { "execute": "query-dump-guest-memory-capabilities" }
> >> > <- { "return": { "formats":
> >> > ["elf", "kdump-zlib", "kdump-lzo", "kdump-snappy"] }
> > Technically you might be right. However, this partial introspection
> > feature is entirely a wart
>
> I don't see it entirely like that. For example, whether paging is
> supported could also be part of the capabilities and not part of the
> regular QAPI introspection. Of course, quiaonuohan need not add
> anything like that.
>
> > whose existence is exclusively justified by
> > non-technical reasons, such as deadlines, and not wanting to be blocked
> > indefinitely by architecture astronautics around full introspection. I
> > don't see the point of polishing it beyond bare usability, at least not
> > after I've reviewed three versions of the patchset.
>
> Luiz, can you apply patches 1-12 for now? I agree with Laszlo that, no
> matter how unfortunate this is, the "unused" warnings are a necessary
> evil and one that can be worked around easily when bisecting.
Yes, I can. But what's the problem with patch 13? For me having Eric's
ACK is enough for applying it. Anything else will be done by QMP
introspection.
- Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-capability' command, Luiz Capitulino, 2014/02/10
- Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-capability' command, Laszlo Ersek, 2014/02/10
- Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-capability' command, Paolo Bonzini, 2014/02/10
- Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-capability' command, Paolo Bonzini, 2014/02/10
- Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-capability' command,
Luiz Capitulino <=
- Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-capability' command, Qiao Nuohan, 2014/02/10
- Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-capability' command, Paolo Bonzini, 2014/02/11
- Re: [Qemu-devel] [PATCH v8 13/13] dump: add 'query-dump-guest-memory-capability' command, Eric Blake, 2014/02/11
- [Qemu-devel] [PATCH v9 13/13] dump: add 'query-dump-guest-memory-capability' command, Qiao Nuohan, 2014/02/11
- Re: [Qemu-devel] [PATCH v9 13/13] dump: add 'query-dump-guest-memory-capability' command, Eric Blake, 2014/02/11
- [Qemu-devel] [PATCH v10 13/13] dump: add 'query-dump-guest-memory-capability' command, Qiao Nuohan, 2014/02/12
- Re: [Qemu-devel] [PATCH v10 13/13] dump: add 'query-dump-guest-memory-capability' command, Luiz Capitulino, 2014/02/12
- Re: [Qemu-devel] [PATCH v10 13/13] dump: add 'query-dump-guest-memory-capability' command, Qiao Nuohan, 2014/02/12
- Re: [Qemu-devel] [PATCH v10 13/13] dump: add 'query-dump-guest-memory-capability' command, Luiz Capitulino, 2014/02/12
- Re: [Qemu-devel] [PATCH v10 13/13] dump: add 'query-dump-guest-memory-capability' command, Eric Blake, 2014/02/13