All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: qemu-devel@nongnu.org, quintela@redhat.com
Subject: Re: [Qemu-devel] Re: [PATCH] Split machine creation from the main loop
Date: Thu, 24 Feb 2011 18:01:39 +0200	[thread overview]
Message-ID: <4D6680E3.8010802@redhat.com> (raw)
In-Reply-To: <4D65945A.3090106@codemonkey.ws>

On 02/24/2011 01:12 AM, Anthony Liguori wrote:
>> What is the plan from here?
>
>
> 1) Decouple QMP from qemu_machine_init().  This really requires the 
> introduction of the new QAPI server that exists outside of the chardev 
> infrastructure since chardevs are currently initialized in 
> qemu_machine_init().

Is it really necessary?  What's blocking us from initializing chardevs 
early?

It would be a pity to divorce the monitor from chardevs, they're really 
flexible.


> 2) Make qemu_machine_init() take no parameters and just reference 
> global state.
>
> 3) Teach all QMP functions to behave themselves if called before 
> qemu_machine_init()
>
> 4) Introduce QMP function to call qemu_machine_init()

An alternative is to remove all guest-visible content from 
qemu_machine_init().  So machine->init() would take no parameters and 
only build the static devices (power supply?).  Everything else would be 
hot-plugged (perhaps some would fail if the machine was started - 
cold-plug only).

>
> 5) Introduce new command line flag to not automatically call 
> qemu_machine_init()
>
> 6) Convert all command line options to just be QMP function calls
>
> (6) can be started right now.  (1) comes with the QAPI merge.  (2) is 
> pretty easy to do after applying this patch.  (3) is probably 
> something that can be done shortly after (1).  (4) and (5) really 
> require everything but (6) to be in place before we can meaningful do it.
>
> I think we can lay out much of the ground work for this in 0.15 and I 
> think we can have a total conversion realistically for 0.16.  That 
> means that by EOY, we could invoke QEMU with no options and do 
> everything through QMP.

It's something that I've agitated for a long while, but when I see all 
the work needed, I'm not sure it's cost effective.

-- 
error compiling committee.c: too many arguments to function

  parent reply	other threads:[~2011-02-24 16:01 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-23 21:38 [Qemu-devel] [PATCH] Split machine creation from the main loop Anthony Liguori
2011-02-23 23:00 ` [Qemu-devel] " Juan Quintela
2011-02-23 23:12   ` Anthony Liguori
2011-02-23 23:38     ` Juan Quintela
2011-02-24  0:36       ` Anthony Liguori
2011-02-24 10:19         ` Stefan Hajnoczi
2011-02-24 14:47           ` Anthony Liguori
2011-02-24 16:01     ` Avi Kivity [this message]
2011-02-24 17:25       ` Anthony Liguori
2011-02-27 11:33         ` Avi Kivity
2011-02-28  4:01           ` Anthony Liguori
2011-02-28  8:20             ` Avi Kivity
2011-02-28  8:57               ` Paolo Bonzini
2011-02-28  9:13                 ` Avi Kivity
2011-02-28 10:08                   ` Paolo Bonzini
2011-02-28 12:08                   ` Anthony Liguori
2011-02-25 17:02 ` [Qemu-devel] " Blue Swirl

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=4D6680E3.8010802@redhat.com \
    --to=avi@redhat.com \
    --cc=anthony@codemonkey.ws \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.