qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: "Jorge Lucángeli Obes" <t4m5yn@gmail.com>
Cc: kvm-devel@lists.sourceforge.net, qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [kvm-devel] Storing command line options in images
Date: Fri, 10 Aug 2007 11:02:32 -0500	[thread overview]
Message-ID: <46BC8C18.6020108@codemonkey.ws> (raw)
In-Reply-To: <59abf66e0708092155t2e3cd5o32f23c018bed65af@mail.gmail.com>

Jorge Lucángeli Obes wrote:
> Hi all,
>
> >From what I've gathered, it seems that we have basically four options
> at hand. I think it's important to notice, however, that whatever
> comes out of this will probably be, as Avi said, a "low-end" solution.
> IMHO there's room to have both the high-end libvirt-like approach, and
> the "shell replacer" approach.
>
> So let's see:
>
> 1- We could go the way of the patches I've posted, adding the comments
> everyone's made.
> 1a- It's a good idea to actively signal QEMU to read command line
> options from the image file, and not have it on by default.
>   

I think if you have to add a new option, the base implementation is 
wrong.  The whole point of this is to simplify things for the user and 
adding more weird options just makes things more complicated.

> 1b- As Avi said, there are many guest-related options that would be
> good to store _somewhere_. The user always has the option of using
> qemu-img to review the options that the VM is going to use. I think
> that having a "valid" set of options that the user can store will
> complicate things too much.
>
> 2- We could bump qcow's version number and add command line options as
> "first-class citizens" of the image world.
> 2a- As Anthony suggested, it could be a good starting point to make
> sure these version transition happen in a smoother way.
>
> 3- We could add command line options as plain text in the image
> itself. Sounds (to me) risky and not very user friendly.
>   

Why is it risky or user unfriendly?  From the user's perspective, it's 
no different from storing it in a snapshot.  The user still needs a 
separate tool (qemu-img) to view and manipulate the command line.  The 
only real user facing difference is that the image itself is now 
directly executable.  This turns out to be a Good Thing in that the user 
is now aware that he must trust that image which addresses the concern 
in 1a.

> 4- We could have configuration files associated with the host and guests.
> 4a- Embedded XML.
> 4b- Separate files.
>   

This is a big effort but a config file is the right long term solution.

Regards,

Anthony Liguori

> What do you guys think? I'm partial to 1 or 2. Since the beginning my
> approach was that of a simple solution for a simple feature.
>
> Cheers,
> Jorge
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc.
> Still grepping through log files to find problems?  Stop.
> Now Search log events and configuration files using AJAX and a browser.
> Download your FREE copy of Splunk now >>  http://get.splunk.com/
> _______________________________________________
> kvm-devel mailing list
> kvm-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/kvm-devel
>
>   

  reply	other threads:[~2007-08-10 16:02 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-10  4:55 [Qemu-devel] Storing command line options in images Jorge Lucángeli Obes
2007-08-10 16:02 ` Anthony Liguori [this message]
2007-08-10 17:14   ` [Qemu-devel] Re: [kvm-devel] " Avi Kivity
2007-08-10 18:43     ` [kvm-devel] [Qemu-devel] " Anthony Liguori
2007-08-10 19:41       ` Avi Kivity
2007-08-10 20:11         ` Anthony Liguori
2007-08-11  1:41           ` Jorge Lucángeli Obes
2007-08-13  8:55             ` Avi Kivity
2007-08-13  9:19               ` Laurent Vivier
2007-08-13  9:27                 ` Christian MICHON
2007-08-14 14:18                 ` Markus Hitter
2007-08-14 14:31                   ` Laurent Vivier
2007-08-14 19:02                     ` Jorge Lucángeli Obes
2007-08-14 20:07                       ` Thiemo Seufer
2007-08-14 20:46                         ` Christian MICHON
2007-08-13 19:39             ` Thiemo Seufer
2007-08-13 20:26               ` Christian MICHON
2007-08-13 22:21               ` Philip Boulain
2007-08-13 22:27                 ` [Qemu-devel] Re: [kvm-devel] " Jernej Simončič
2007-08-13 23:31                 ` [kvm-devel] [Qemu-devel] " Thiemo Seufer
2007-08-14 14:26                   ` Philip Boulain
2007-08-14  3:17               ` Anthony Liguori
2007-08-14  4:39                 ` Jorge Lucángeli Obes
2007-08-14  7:44                   ` Kevin Wolf
2007-08-14  7:46               ` Laurent Vivier
2007-08-15 21:26                 ` Segher Boessenkool

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=46BC8C18.6020108@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=kvm-devel@lists.sourceforge.net \
    --cc=qemu-devel@nongnu.org \
    --cc=t4m5yn@gmail.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 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).