From: Keir Fraser <keir.fraser@eu.citrix.com>
To: Christoph Egger <Christoph.Egger@amd.com>,
"Dong, Eddie" <eddie.dong@intel.com>
Cc: Deegan <Tim.Deegan@eu.citrix.com>Tim,
"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: [PATCH 0/14] Nested Virtualization: Overview
Date: Tue, 17 Aug 2010 11:41:58 +0100 [thread overview]
Message-ID: <C8902606.1E12D%keir.fraser@eu.citrix.com> (raw)
In-Reply-To: <201008171233.30154.Christoph.Egger@amd.com>
On 17/08/2010 11:33, "Christoph Egger" <Christoph.Egger@amd.com> wrote:
>>> - implement HVM-on-HVM (instead of SVM-on-HVM)
>>
>> Given that we don't have consensus on cross architecture nested
>> virtualization support, I am doubting why this is urgent for now.
>
> The reason to be "urgent" is not the time. This is the best way from
> the software engineering side.
>
>> I would prefer we make SVM-on-SVM and VMX-on-VMX work first. After that,
>> if you prove SVM-on-VMX has real performance gain (which I doubt), we can
>> see how to make a much generic effort to accomodate both natively nested
>> virtualization and cross architecture nested virtualization.
>
> Tim and Keir made clear they don't want to have two implementations after
> I submitted my patch series the *first* time.
I think maybe this is an argument over two different things. To be clear we
want to support VMX-on-VMX and SVM-on-SVM. I assume this is what Christoph
means by HVM-on-HVM: any-like-on-like. In which case there is no
disagreement here.
Now, separately there is a debate to be had on how much code can be shared
in HVM-on-HVM, given the big differences between VMX and SVM. I would guess
that there will be at least things in the area of nested shadow and nested
HAP that can be shared, for example. Probably there is other stuff too. The
question is to what degree do we pursue that now rather than get divergent
stuff in tree and then go after it later. My mind isn't totally made up on
that; I don't know about Tim's.
-- Keir
next prev parent reply other threads:[~2010-08-17 10:41 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-05 14:59 [PATCH 0/14] Nested Virtualization: Overview Christoph Egger
2010-08-17 6:04 ` Dong, Eddie
2010-08-17 10:33 ` Christoph Egger
2010-08-17 10:41 ` Keir Fraser [this message]
2010-08-17 12:30 ` Tim Deegan
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=C8902606.1E12D%keir.fraser@eu.citrix.com \
--to=keir.fraser@eu.citrix.com \
--cc=Christoph.Egger@amd.com \
--cc=Tim.Deegan@eu.citrix.com \
--cc=eddie.dong@intel.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).