From: Igor Mammedov <imammedo@redhat.com>
To: "Daniel P. Berrangé" <berrange@redhat.com>
Cc: qemu-devel@nongnu.org, pkrempa@redhat.com, ehabkost@redhat.com,
armbru@redhat.com
Subject: Re: [Qemu-devel] [PATCH v6 03/11] cli: add --preconfig option
Date: Thu, 3 May 2018 13:35:33 +0200 [thread overview]
Message-ID: <20180503133533.15b8a677@redhat.com> (raw)
In-Reply-To: <20180501150723.GF32129@redhat.com>
On Tue, 1 May 2018 16:07:23 +0100
Daniel P. Berrangé <berrange@redhat.com> wrote:
> On Fri, Apr 27, 2018 at 05:05:15PM +0200, Igor Mammedov wrote:
> > This option allows pausing QEMU in the new RUN_STATE_PRECONFIG state,
> > allowing the configuration of QEMU from QMP before the machine jumps
> > into board initialization code of machine_run_board_init()
> >
> > The intent is to allow management to query machine state and additionally
> > configure it using previous query results within one QEMU instance
> > (i.e. eliminate the need to start QEMU twice, 1st to query board specific
> > parameters and 2nd for actual VM start using query results for
> > additional parameters).
> >
> > The new option complements -S option and could be used with or without
> > it. The difference is that -S pauses QEMU when the machine is completely
> > initialized with all devices wired up and ready to execute guest code
> > (QEMU needs only to unpause VCPUs to let guest execute its code),
> > while the "preconfig" option pauses QEMU early before board specific init
> > callback (machine_run_board_init) is executed and allows the configuration
> > of machine parameters which will be used by board init code.
>
> > When early introspection/configuration is done, command 'exit-preconfig'
> > should be used to exit RUN_STATE_PRECONFIG and transition to the next
> > requested state (i.e. if -S is used then QEMU will pause the second
> > time when board/device initialization is completed or start guest
> > execution if -S isn't provided on CLI)
>
> If we're going to introduce this --preconfig facility I would very much
> like us to ensure that we do *not* need to continue to also use -S. IOW
> it would be nice to essentially consider --preconfig a better impl of
> -S and stop using -S, perhaps even deprecating it for removal.
That's how I view this as well,
modulo removal of -S option as it still might be used for hotplugging and
for early VM startup dubugging and other thing that really need ready
to run machine.
> What logical things does QEMU do after exiting pre-config stage, but
> before starting CPUs that might be dependency for mgmt apps that would
> neccessitate using both --preconfig and -S ?
I've replied to your other email listing them, so lets figure
out the plan of what needs to be fixed so that libvirt could
switch from -S to --preconfig only.
PS:
(Since I've started this whole thing, I'm willing to work on
gradually improving it)
> Regards,
> Daniel
next prev parent reply other threads:[~2018-05-03 11:35 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-27 15:05 [Qemu-devel] [PATCH v6 00/11] enable numa configuration before machine_init() from QMP Igor Mammedov
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 01/11] numa: postpone options post-processing till machine_run_board_init() Igor Mammedov
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 02/11] numa: split out NumaOptions parsing into set_numa_options() Igor Mammedov
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 03/11] cli: add --preconfig option Igor Mammedov
2018-04-27 21:44 ` Eric Blake
2018-05-01 15:07 ` Daniel P. Berrangé
2018-05-03 11:35 ` Igor Mammedov [this message]
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 04/11] hmp: disable monitor in preconfig state Igor Mammedov
2018-04-27 21:48 ` Eric Blake
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 05/11] qapi: introduce new cmd option "allowed-in-preconfig" Igor Mammedov
2018-04-27 22:05 ` Eric Blake
2018-04-28 3:51 ` Peter Xu
2018-04-30 14:49 ` Eric Blake
2018-05-01 14:57 ` Igor Mammedov
2018-05-01 15:29 ` Daniel P. Berrangé
2018-05-03 11:22 ` Igor Mammedov
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 06/11] tests: let qapi-schema tests detect allowed-in-preconfig Igor Mammedov
2018-04-27 22:08 ` Eric Blake
2018-04-27 22:12 ` Eric Blake
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 07/11] tests: add allowed-in-preconfig-test for qapi-schema Igor Mammedov
2018-04-27 22:11 ` Eric Blake
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 08/11] tests: extend qmp test with preconfig checks Igor Mammedov
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 09/11] qmp: permit query-hotpluggable-cpus in preconfig state Igor Mammedov
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 10/11] qmp: add set-numa-node command Igor Mammedov
2018-04-27 15:05 ` [Qemu-devel] [PATCH v6 11/11] tests: functional tests for QMP command set-numa-node Igor Mammedov
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=20180503133533.15b8a677@redhat.com \
--to=imammedo@redhat.com \
--cc=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=ehabkost@redhat.com \
--cc=pkrempa@redhat.com \
--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).