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] Adjust time init sequence

To: 'Keir Fraser' <keir.fraser@xxxxxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] Adjust time init sequence
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Sat, 13 Dec 2008 00:40:32 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 12 Dec 2008 08:40:58 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C567EF98.2028C%keir.fraser@xxxxxxxxxxxxx>
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: <494246AE.76E4.0078.0@xxxxxxxxxx> <C567EF98.2028C%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AclcRb2WC58pgPn0g0urUZDnthS3kgAL+vxg
Thread-topic: [Xen-devel] [PATCH] Adjust time init sequence
>From: Keir Fraser [mailto:keir.fraser@xxxxxxxxxxxxx] 
>Sent: Friday, December 12, 2008 6:38 PM
>
>On 12/12/2008 10:10, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:
>
>>> How about rdtscll(t->local_tsc_stamp) at the top of 
>init_xen_time(), and
>>> remove the one I added to early_time_init()? That would 
>allow NOW() usage at
>>> least in init_xen_time(), and be later than the TSC reset.
>> 
>> Perhaps it should be considered to switch to the non-resetting
>> synchronization logic in x86-64 Linux up to 2.6.20 (according to the
>> comments there derived from ia64), or even the current version that
>> doesn't re-write the TSC at all?
>
>Quite agreeable, albeit a bigger patch. ;-)
>

Current Linux doesn't re-write TSC but marking it as unstable
and then may use alternative source if available. For Xen, tsc is 
one basic wheel to drive both pv and hvm time virtualization. By
not re-writing TSCs on those platforms with unsynchrozed TSC,
it may cause time issue since currently hvm guest doesn't adjust
guest TSC offset across vcpu migration. Then if above change is
pursued, first we'd better add guest TSC offset adjustment to 
avoid cause regression on those platforms?

Also do you know the reason why Linux doesn't re-write TSC
nowadays? I recalled some thread seen before that such re-write
may cause time warp on some platforms, but not sure about the
detail. Did we ever see similar issue caused by re-write in Xen
before?

Thanks
Kevin
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel