WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

RE: [Xen-devel] [PATCH] replace rdtsc emulation-vs-native xen boot optio

To: Ian Pratt <Ian.Pratt@xxxxxxxxxxxxx>, Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>, Jeremy Fitzhardinge <jeremy@xxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] replace rdtsc emulation-vs-native xen boot option with per-domain (hypervisor part)
From: "Zhang, Xiantao" <xiantao.zhang@xxxxxxxxx>
Date: Tue, 29 Sep 2009 09:42:35 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "Xen-Devel \(E-mail\)" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Keir Fraser <Keir.Fraser@xxxxxxxxxxxxx>
Delivery-date: Mon, 28 Sep 2009 18:43:45 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4FA716B1526C7C4DB0375C6DADBC4EA342A68E5A00@xxxxxxxxxxxxxxxxxxxxxxxxx>
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: <4AC1424D.8070008@xxxxxxxx> <5f6198f2-3f74-4bdb-ad9a-d9fd6151a97f@default> <4FA716B1526C7C4DB0375C6DADBC4EA342A68E5A00@xxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcpAlJwrr/3+pG6cQAaufIDtXghWHwABEfsgAAJuYPA=
Thread-topic: [Xen-devel] [PATCH] replace rdtsc emulation-vs-native xen boot option with per-domain (hypervisor part)
Ian Pratt wrote:
>>> Well, it shouldn't be enabled by default.  That slows down all rdtsc
>>> operations for the benefit of very niche applications.  The Xen
>>> clocksource assumes that rdtsc is fast, unemulated and in need of
>>> correction. 
>>> 
>>> If someone really needs an artificial tsc, then they can enable the
>>> option for themselves.
>> 
>> While I understand and sympathize (and the idealistic side of me even
>> agrees with) your position, let me argue your point "they can enable
>> the option for themselves" by mangling a famous movie quote:
>> 
>>       "In a cloud, no-one can hear rdtsc scream"
> 
> If we had some actual bug reports detailing failures of real-world
> applications then the decision would be easy. 
> I realise it could be hard to pin strange application behaviour down
> to TSC issues, but I can't readily spot any bugs on the XenServer or
> xen.org bug trackers that might even be candidates. 

Maybe Dan can give more reasons. I think the TSC skew issue should be thre key 
one. But IMO, the TSC skew issue maybe not that important. In real-world apps, 
rdtsc is mostly read by the kernel syscall gettimeofday, and kernel can 
eliminate the skew before returning to user app through returning the value 
"plus 1 on last_tsc". Even if apps use "raw rdtsc" instruction to get timestamp 
in rare cases, I think the issue also can be solved by apps' ownself through a 
simple check.  Therefore, I still can't figure out the suasive reasons why 
adopts emulation way for the tsc read so far. 
Dan, 
    Any other necessary reasons to introduce it except fixing skew issue 
between cpus?
Xiantao
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>