qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Apfelbaum <marcel@redhat.com>
To: "Daniel P. Berrange" <berrange@redhat.com>,
	Alexander Graf <agraf@suse.de>
Cc: Marcel Apfelbaum <marcel@redhat.com>,
	peter.maydell@linaro.org, greg.bellows@linaro.org,
	qemu-devel@nongnu.org, mst@redhat.com
Subject: Re: [Qemu-devel] [PATCH 3/3] vl.c: add HMP help to machine
Date: Mon, 15 Dec 2014 21:24:21 +0200	[thread overview]
Message-ID: <548F3565.8020908@redhat.com> (raw)
In-Reply-To: <20141215152050.GF11566@redhat.com>

On 12/15/2014 05:20 PM, Daniel P. Berrange wrote:

> On Mon, Dec 15, 2014 at 04:16:02PM +0100, Alexander Graf wrote:
>>
>> On 10.12.14 14:19, Marcel Apfelbaum wrote:
>>> The help is based on the actual machine properties
>>> exposing only the relevant options.
>>>
>>> Signed-off-by: Marcel Apfelbaum <marcel.a@redhat.com>
>> Can libvirt make use of this or would it need QMP based exposure?
> Libvirt *never* looks at any -help option anymore. We exclusively use
> the QMP  query-XXXX commands to discover information about QEMU. So
> if its not already possible to query existance of these new machine
> properties via QMP, it'll need work.

Hi Daniel,
I thought we already have something generic that creates temp QOM object,
query its properties and then deletes it.
Who can answer this question? (I didn't get the chance to work with qmp yet.)

Thanks,
Marcel

>
> Regards,
> Daniel
>

  reply	other threads:[~2014-12-15 19:24 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-10 13:19 [Qemu-devel] [PATCH 0/3] machine: dynamic options per machine type Marcel Apfelbaum
2014-12-10 13:19 ` [Qemu-devel] [PATCH 1/3] machine: remove qemu_machine_opts global list Marcel Apfelbaum
2014-12-10 22:59   ` Greg Bellows
2014-12-11  5:05     ` Marcel Apfelbaum
2014-12-11 14:57       ` Greg Bellows
2014-12-15 15:13       ` Alexander Graf
2014-12-15 15:20         ` Paolo Bonzini
2014-12-10 13:19 ` [Qemu-devel] [PATCH 2/3] vl.c: simplified machine_set_property Marcel Apfelbaum
2014-12-11 17:49   ` Greg Bellows
2014-12-10 13:19 ` [Qemu-devel] [PATCH 3/3] vl.c: add HMP help to machine Marcel Apfelbaum
2014-12-15 15:16   ` Alexander Graf
2014-12-15 15:20     ` Daniel P. Berrange
2014-12-15 19:24       ` Marcel Apfelbaum [this message]
2014-12-15 15:14 ` [Qemu-devel] [PATCH 0/3] machine: dynamic options per machine type Alexander Graf
2014-12-15 17:24 ` Peter Maydell
2014-12-15 17:33   ` Alexander Graf
2014-12-15 17:55     ` Greg Bellows

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=548F3565.8020908@redhat.com \
    --to=marcel@redhat.com \
    --cc=agraf@suse.de \
    --cc=berrange@redhat.com \
    --cc=greg.bellows@linaro.org \
    --cc=mst@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /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).