From: Paul Brook <paul@codesourcery.com>
To: qemu-devel@nongnu.org
Cc: kvm-devel@lists.sourceforge.net,
Fabrice Bellard <fabrice@bellard.org>,
Avi Kivity <avi@qumranet.com>
Subject: Re: [Qemu-devel] Re: [PATCH] Add support for a configuration file
Date: Wed, 14 May 2008 15:26:33 +0100 [thread overview]
Message-ID: <200805141526.34293.paul@codesourcery.com> (raw)
In-Reply-To: <482ADA80.3000309@bellard.org>
> I suggested it because my original plan for the configuration file was
> based on this syntax with a strong inspiration from the OpenFirmware
> device tree. The idea was that the object name ("drive" here) had no
> hardcoded meaning, except for some predefined object names in order to
> keep a kind of backward compatibility with the current QEMU options. In
> order to create a new drive for example, you just have to do:
>
> mydrive.class=drive
> mydrive.if=scsi
> mydrive.file=abc.img
>
> the "class" field is used to select the device model. Then all the other
> parameters are used to initialize the device model. That way it is
> possible to keep the compatibility with the existing options and add a
> provision to instanciate arbitrary new device models, such as:
I like the idea, but I'm not so keen on the automatic allocation. I generally
prefer explicit declaration over implicit things. The latter makes it very
easy to not notice when you make a typo.
It sounds like what you really want is something similar to an OF device tree.
So you have something like:
# pciide0 may be an alias (possibly provided by qemu)
# e.g. pci0.slot1.func1.ide
alias hda ide0.primary.master
hda.type=disk
hda.file=foo.img
You can then define some form of magic aliases that select the next unused
device. e.g.
alias mydrive $next_ide_disk
IMHO This provides the flexibility and structure that Fabrice is talking
about, and with suitable aliases can be made to look a lot like the existing
options.
This may require some internal restructuring to allow the machine descriptions
to feed into the user config file.
Thoughts?
Paul
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
next prev parent reply other threads:[~2008-05-14 14:26 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-13 21:19 [PATCH] Add support for a configuration file Anthony Liguori
2008-05-13 23:07 ` Anthony Liguori
2008-05-13 23:20 ` Daniel P. Berrange
2008-05-14 6:35 ` Colin Adams
2008-05-14 14:41 ` [Qemu-devel] " Avi Kivity
2008-05-14 14:52 ` Dor Laor
2008-05-14 15:02 ` Daniel P. Berrange
2008-05-14 15:18 ` Anthony Liguori
2008-05-14 14:59 ` Anthony Liguori
2008-05-14 8:27 ` Fabrice Bellard
2008-05-14 10:31 ` Avi Kivity
2008-05-14 12:26 ` Fabrice Bellard
2008-05-14 13:31 ` Daniel P. Berrange
2008-05-15 8:04 ` Avi Kivity
2008-05-15 11:52 ` Daniel P. Berrange
2008-05-15 12:04 ` Avi Kivity
2008-05-15 12:20 ` Laurent Vivier
2008-05-14 14:06 ` Anthony Liguori
2008-05-14 14:26 ` Paul Brook [this message]
2008-05-14 14:45 ` Javier Guerra
2008-05-14 15:13 ` Johannes Schindelin
2008-05-14 15:30 ` [kvm-devel] " Javier Guerra
2008-05-14 15:37 ` Johannes Schindelin
2008-05-14 15:42 ` Javier Guerra
2008-05-14 16:07 ` [Qemu-devel] " Kelly French
2008-05-15 14:59 ` Paul Brook
2008-05-14 15:22 ` [kvm-devel] " Daniel P. Berrange
2008-05-14 15:09 ` [Qemu-devel] " Anthony Liguori
2008-05-14 15:18 ` Paul Brook
2008-05-14 16:25 ` Fabrice Bellard
2008-05-14 14:36 ` [Qemu-devel] " andrzej zaborowski
2008-05-14 15:10 ` Johannes Schindelin
2008-05-15 14:58 ` Ian Jackson
-- strict thread matches above, loose matches on Subject: below --
2008-05-14 10:03 Balazs Attila-Mihaly (Cd-MaN)
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=200805141526.34293.paul@codesourcery.com \
--to=paul@codesourcery.com \
--cc=avi@qumranet.com \
--cc=fabrice@bellard.org \
--cc=kvm-devel@lists.sourceforge.net \
--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