|
|
|
|
|
|
|
|
|
|
xen-devel
[Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation
To: |
"Ingo Molnar" <mingo@xxxxxxx>, "Jeremy Fitzhardinge" <jeremy@xxxxxxxx> |
Subject: |
[Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation |
From: |
"Jan Beulich" <JBeulich@xxxxxxxxxx> |
Date: |
Tue, 19 May 2009 11:22:02 +0100 |
Cc: |
Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, the arch/x86 maintainers <x86@xxxxxxxxxx>, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>, "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>, "H. Peter Anvin" <hpa@xxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> |
Delivery-date: |
Tue, 19 May 2009 03:22:27 -0700 |
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxxx |
In-reply-to: |
<20090519095918.GA11790@xxxxxxx> |
List-help: |
<mailto:xen-devel-request@lists.xensource.com?subject=help> |
List-id: |
Xen developer discussion <xen-devel.lists.xensource.com> |
List-post: |
<mailto:xen-devel@lists.xensource.com> |
List-subscribe: |
<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe> |
List-unsubscribe: |
<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> |
References: |
<4A0ADBA2.2020300@xxxxxxxx> <20090515182757.GA19256@xxxxxxx> <4A0DCC11.10307@xxxxxxxx> <m1my9ex818.fsf@xxxxxxxxxxxxxxxxx> <4A0DFF78.6000501@xxxxxxxx> <20090515202250.0f1218ef@jbarnes-g45> <m1iqk1k708.fsf@xxxxxxxxxxxxxxxxx> <4A10EAC4.9070701@xxxxxxxx> <20090518085902.GE10687@xxxxxxx> <4A11A3F8.1010202@xxxxxxxx> <20090519095918.GA11790@xxxxxxx> |
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
>>> Ingo Molnar <mingo@xxxxxxx> 19.05.09 11:59 >>>
>Exactly what is 'bizarre' about using the API defined by the _CPU_
>already, without adding any ad-hoc hypecall? Catch the dom0 WRMSRs,
>filter out the MTRR indices - that's it.
But that is *not* the same as using the hypercalls: The hypercall tells Xen
"Change all CPUs' MTRRs with the indicated index to the indicated value",
while the MSR write says "Change the MTRR with the given index on the
physical CPU the current virtual CPU happens to run on to the given value".
A write-base/write-mask pair may happen to get interrupted (preempted)
by the hypervisor, and hence the two writes may happen on different
pCPU-s. Teaching the hypervisor to (correctly!) guess what the guest
meant in that situation isn't trivial, as then it needs to handle all possible
situations (and it can never know whether Dom0 really intended to do
something that may look bogus/inconsistent at the first glance). This is
even more so considering that Linux is not the only OS capable of
running as Dom0.
An apparently relatively simple solution - latch the writes and commit them
only when the global view became consistent again - isn't possible,
since Dom0 may not have a vCPU running on each individual pCPU.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
<Prev in Thread] |
Current Thread |
[Next in Thread>
|
- [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, (continued)
- [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Jeremy Fitzhardinge
- [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Ingo Molnar
- [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Jan Beulich
- [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Jeremy Fitzhardinge
- [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Ingo Molnar
- [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation,
Jan Beulich <=
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Ingo Molnar
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Gerd Hoffmann
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Ingo Molnar
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Alan Cox
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Ingo Molnar
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Gerd Hoffmann
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Ingo Molnar
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Gerd Hoffmann
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Ingo Molnar
- Re: [Xen-devel] Re: [GIT PULL] xen /proc/mtrr implementation, Gerd Hoffmann
|
|
|
|
|