xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Cooper <andrew.cooper3@citrix.com>
To: James Harper <james.harper@bendigoit.com.au>
Cc: "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: XENMAPSPACE_vlapic vs XENMAPSPACE_vlapic_compat
Date: Wed, 2 Oct 2013 00:29:11 +0100	[thread overview]
Message-ID: <524B5AC7.5070002@citrix.com> (raw)
In-Reply-To: <6035A0D088A63A46850C3988ED045A4B66597E25@BITCOM1.int.sbss.com.au>

On 01/10/2013 23:39, James Harper wrote:
> GPLPV tries to map vlapic for TPR acceleration. Under what circumstances/versions are XENMAPSPACE_vlapic and XENMAPSPACE_vlapic_compat supported or not supported?
>
> Thanks
>
> James

Short answer: not at all.
Long answer: will continue to work on XenServer for the forseeable
future for compatibility with our older windows drivers.

Both of them were gross XenServer hacks to solve the WinXP TPR
performance problem on hardware without vTPR, and never upstreamed.

XENMAPSPACE_vlapic_compat (defined as 3) became a binary incompatibility
with Xen 4.2 with the introduction of XENMAPSPACE_gmfn_range

XENMAPSPACE_vlapic (defined as 0x80000000) is luckily a long way away
from being a binary incompatibility.


With the lapic fastpath in commit
5d43891bf4002b754cd90d83e91d9190e8c8b9d0, there is less of a need for
the vlapic mapping anyway.  Furthermore, allowing a guest a RW mapping
of its own hypervisor vlapic page is a hard sell for the security
concious, although I did spend quite a long time investigating the
possible vulnerabilities and came to the conclusion that the worst a
guest could do was monkey with its own interrupt injection.

As these are just performance tweaks for WinXP (which is almost out of
extended support) on older hardware only, there are no plans to upstream
the patches. 

~Andrew

  reply	other threads:[~2013-10-01 23:29 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-01 22:39 XENMAPSPACE_vlapic vs XENMAPSPACE_vlapic_compat James Harper
2013-10-01 23:29 ` Andrew Cooper [this message]
2013-10-03  8:04   ` James Harper

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=524B5AC7.5070002@citrix.com \
    --to=andrew.cooper3@citrix.com \
    --cc=james.harper@bendigoit.com.au \
    --cc=xen-devel@lists.xen.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).