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/
Home Products Support Community News


RE: [Xen-devel] [PATCH] Adjust time init sequence

To: 'Keir Fraser' <keir.fraser@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] Adjust time init sequence
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Thu, 11 Dec 2008 17:06:45 +0800
Accept-language: en-US
Acceptlanguage: en-US
Delivery-date: Thu, 11 Dec 2008 01:07:37 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C5668827.20145%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: <0A882F4D99BBF6449D58E61AAFD7EDD603BB491F@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <C5668827.20145%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-topic: [Xen-devel] [PATCH] Adjust time init sequence
>From: Keir Fraser [mailto:keir.fraser@xxxxxxxxxxxxx] 
>Sent: Thursday, December 11, 2008 5:05 PM
>On 11/12/2008 00:47, "Tian, Kevin" <kevin.tian@xxxxxxxxx> wrote:
>> I guess not, due to same reason as why I sent out 1st patch idle
>> vcpu state entry. The point is the current TSC value, which count
>> from power on and is translated to a dozens of seconds for elapsed
>> time upon a zero tsc stamp. :-( I didn't realize that point 
>in the start...
>Ah, because it's set up in early_time_init().
>By the way, instead of avoiding NOW() early on, could we just set
>local_tsc_stamp in early_time_init()? Then we could use NOW() when
>initialising idle VCPUs, and also early on in init_xen_time()?
>We could set stime_platform_stamp = NOW() too, so that platform time is
>kicked off following BP's time.
>I could send a patch which I find tasteful if you think this 
>could work? :-)

Sure, I think that should work too. :-)

Xen-devel mailing list