|
xen-devel
[Xen-devel] RE: One (possible) x86 get_user_pages bug
|
To: |
'Avi Kivity' <avi@xxxxxxxxxx>, 'Jeremy Fitzhardinge' <jeremy@xxxxxxxx> |
|
Subject: |
[Xen-devel] RE: One (possible) x86 get_user_pages bug |
|
From: |
Kaushik Barde <kbarde@xxxxxxxxxx> |
|
Date: |
Sun, 30 Jan 2011 14:21:11 -0800 |
|
Cc: |
xen-devel@xxxxxxxxxxxxxxxxxxx, 'Kenneth Lee' <liguozhu@xxxxxxxxxx>, 'Nick Piggin' <npiggin@xxxxxxxxx>, 'Marcelo Tosatti' <mtosatti@xxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, 'Jan Beulich' <JBeulich@xxxxxxxxxx>, wangzhenguo@xxxxxxxxxx, 'Xiaowei Yang' <xiaowei.yang@xxxxxxxxxx>, 'linqaingmin' <linqiangmin@xxxxxxxxxx>, fanhenglong@xxxxxxxxxx, 'Wu Fengguang' <fengguang.wu@xxxxxxxxx>, 'Peter Zijlstra' <a.p.zijlstra@xxxxxxxxx> |
|
Delivery-date: |
Sun, 30 Jan 2011 14:22:34 -0800 |
|
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxxx |
|
In-reply-to: |
<4D456139.4090508@xxxxxxxxxx> |
|
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: |
<4D416D9A.9010603@xxxxxxxxxx> <4D419416020000780002ECB7@xxxxxxxxxxxxxxxxxx> <4D41B90D.5000305@xxxxxxxx> <4D456139.4090508@xxxxxxxxxx> |
|
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
|
Thread-index: |
AcvAfeg92q3NfNbiSnmuJgUEgTqz7gASdFRg |
I agree i.e. deviation from underlying arch consideration is not a good
idea.
Also, agreed, hypervisor knows which page entries are ready for TLB flush
across vCPUs.
But, using above knowledge, along with TLB flush based on IPI is a better
solution. Its ability to synchronize it with pCPU based IPI and TLB flush
across vCPU. is key.
IPIs themselves should be in few hundred uSecs in terms latency. Also, why
should pCPU be in sleep state for active vCPU scheduled page workload?
-Kaushik
-----Original Message-----
From: Avi Kivity [mailto:avi@xxxxxxxxxx]
Sent: Sunday, January 30, 2011 5:02 AM
To: Jeremy Fitzhardinge
Cc: Jan Beulich; Xiaowei Yang; Nick Piggin; Peter Zijlstra;
fanhenglong@xxxxxxxxxx; Kaushik Barde; Kenneth Lee; linqaingmin;
wangzhenguo@xxxxxxxxxx; Wu Fengguang; xen-devel@xxxxxxxxxxxxxxxxxxx;
linux-kernel@xxxxxxxxxxxxxxx; Marcelo Tosatti
Subject: Re: One (possible) x86 get_user_pages bug
On 01/27/2011 08:27 PM, Jeremy Fitzhardinge wrote:
> And even just considering virtualization, having non-IPI-based tlb
> shootdown is a measurable performance win, since a hypervisor can
> optimise away a cross-VCPU shootdown if it knows no physical TLB
> contains the target VCPU's entries. I can imagine the KVM folks could
> get some benefit from that as well.
It's nice to avoid the IPI (and waking up a cpu if it happens to be
asleep) but I think the risk of deviating too much from the baremetal
arch is too large, as demonstrated by this bug.
(well, async page faults is a counterexample, I wonder if/when it will
bite us)
--
error compiling committee.c: too many arguments to function
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
Home