qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Nadav Har'El <nyh@math.technion.ac.il>
Cc: Alexander Graf <agraf@suse.de>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] Add VMX cpuid feature to qemu64
Date: Wed, 05 Jan 2011 11:06:09 +0200	[thread overview]
Message-ID: <4D243481.2070005@redhat.com> (raw)
In-Reply-To: <20110105085047.GA24818@fermat.math.technion.ac.il>

On 01/05/2011 10:50 AM, Nadav Har'El wrote:
> On Wed, Jan 05, 2011, Avi Kivity wrote about "Re: [Qemu-devel] [PATCH] Add VMX cpuid feature to qemu64":
> >  The intent of kvm64/qemu64 is to provide some feature stability, so that
> >  when you run a guest with those cpu types, you get something expected.
> >  So adding to them isn't a good idea.
> >
> >  I think we'd be fine with -cpu host and -cpu qemu64,+vmx enabling vmx
> >  support.
>
> Thanks. While this is ugly, I can certainly live with that (I've been telling
> people to use "-cpu host" to run nested VMX, until you asked me why I don't
> send a patch for qemu to fix that ;-)).

Sorry about the confusion.

> Though I wonder why there shouldn't be some sort of "default" CPU type which
> just provides the maximum number of supported features, without attempting
> to provide stable features. If someone wants a specific stable CPU he can
> always use "-cpu core2duo" or even "-cpu qemu64", but shouldn't the default
> (when KVM is enabled) be to just provide all the features that KVM can provide?

That's -cpu host.  People have been talking that it should be the 
default, and to a certain extent I agree.  But changing defaults is 
dangerous business, it can break setups that rely on them.

> Anyway, what is your opinion on which should be the default in KVM - qemu64
> or kvm64? If it's qemu64, does it make sense (as far as KVM is concerned)
> that it lists the SVM capability, but not the VMX capability?

IMO the emphasis on defaults is misplaced.  Most people run virtual 
machines via a management tool, and we should focus on making it easy 
for management tool maintainers to do the right thing, by documenting 
our APIs and recommendations in the QEMU Management Tool Writer Guide.

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

  reply	other threads:[~2011-01-05  9:06 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-04 15:06 [Qemu-devel] [PATCH] Add VMX cpuid feature to qemu64 Nadav Har'El
2011-01-04 15:53 ` Alexander Graf
2011-01-04 21:39   ` Nadav Har'El
2011-01-04 21:55     ` Alexander Graf
2011-01-05  8:17       ` Nadav Har'El
2011-01-05  8:32         ` Avi Kivity
2011-01-05  8:50           ` Nadav Har'El
2011-01-05  9:06             ` Avi Kivity [this message]
2011-01-05  9:22               ` Nadav Har'El

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=4D243481.2070005@redhat.com \
    --to=avi@redhat.com \
    --cc=agraf@suse.de \
    --cc=nyh@math.technion.ac.il \
    --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).