xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Ian Campbell <Ian.Campbell@citrix.com>
To: Stefano Stabellini <Stefano.Stabellini@eu.citrix.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	"keir@xen.org" <keir@xen.org>
Subject: Re: [PATCH 1/2] Introduce support for upstream qemu in the xen-unstable build system
Date: Wed, 7 Sep 2011 11:17:43 -0700	[thread overview]
Message-ID: <1315419466.3180.34.camel@cthulhu.hellion.org.uk> (raw)
In-Reply-To: <alpine.DEB.2.00.1109071744340.12963@kaball-desktop>

On Wed, 2011-09-07 at 12:54 -0400, Stefano Stabellini wrote:
> On Wed, 7 Sep 2011, Ian Campbell wrote:
> > On Wed, 2011-09-07 at 10:39 -0400, stefano.stabellini@eu.citrix.com
> > wrote:
> > > In order to distinguish between upstream qemu and qemu-xen I am
> > > introducing a new variable named "QEMU" that only if is equal to
> > > "upstream" switches the build system to the new qemu.
> > 
> > Thanks Stefano, this integrated support is overdue.
> > 
> > Ultimately though I expect we will need a "both" mode since people will
> > want old qemu for compatibility with their existing installed guests and
> > new qemu for new ones. The allegation (and I don't really know how true
> > it is or if it is pessimism or realism) is that some OSes don't cope
> > with having the platform components etc changed under them.
> 
> Yes. At that point we'll probably have to always build them both,

Why wait? Why not just do that now.

> including two hvmloader...

Only one hvmloader will be needed, it will include both rombios and
seabios support and it selects the right one at boot time, since the
toolstack pushes down the choice. This is what happens today if you turn
on the SEABIOS option.

> > > Users that want to try the new qemu just have to export QEMU=upstream
> > > before calling make in the xen-unstable top level directory.
> > 
> > In xl/libxl we call them "qemu-xen-traditional" and "qemu-xen". Perhaps
> > we should mirror that nomenclature here?
> 
> I looked at libxl but I thought that QEMU=qemu-xen would be confusing for
> users/developers.
> Like you said, considering that we already have
> device_model_version=qemu-xen in VM config files, maybe we should go for
> DEVICE_MODEL=qemu-xen?

With the (current) default being DEVICE_MODEL=qemu-xen-traditional? That
would makes sense to me (excepting that, as above, I think we shouldn't
offer it as a choice in the first place).

Ian.

  reply	other threads:[~2011-09-07 18:17 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-07 14:39 [PATCH 1/2] Introduce support for upstream qemu in the xen-unstable build system stefano.stabellini
2011-09-07 14:39 ` [PATCH 2/2] Automatically select seabios when we are building upstream qemu stefano.stabellini
2011-09-07 16:21 ` [PATCH 1/2] Introduce support for upstream qemu in the xen-unstable build system Ian Campbell
2011-09-07 16:54   ` Stefano Stabellini
2011-09-07 18:17     ` Ian Campbell [this message]
2011-09-07 16:50 ` Keir Fraser

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=1315419466.3180.34.camel@cthulhu.hellion.org.uk \
    --to=ian.campbell@citrix.com \
    --cc=Stefano.Stabellini@eu.citrix.com \
    --cc=keir@xen.org \
    --cc=xen-devel@lists.xensource.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).