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: "Tian, Kevin" <kevin.tian@xxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Adjust time init sequence
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Thu, 11 Dec 2008 09:04:39 +0000
Cc:
Delivery-date: Thu, 11 Dec 2008 01:05:10 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <0A882F4D99BBF6449D58E61AAFD7EDD603BB491F@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AclayLVCCL6aQHvlSCiWfWMkyOK68AABWP7wABXIhpAAARtcGAAAAhtQABFzgAU=
Thread-topic: [Xen-devel] [PATCH] Adjust time init sequence
User-agent: Microsoft-Entourage/12.14.0.081024
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? :-)

 -- Keir



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