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: Fri, 12 Dec 2008 09:08:30 +0000
Cc:
Delivery-date: Fri, 12 Dec 2008 01:08:51 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <0A882F4D99BBF6449D58E61AAFD7EDD603BB493B@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: AclayLVCCL6aQHvlSCiWfWMkyOK68AABWP7wABXIhpAAARtcGAAAAhtQABFzgAUACKfNpAAdvNggAAwIYV4=
Thread-topic: [Xen-devel] [PATCH] Adjust time init sequence
User-agent: Microsoft-Entourage/12.14.0.081024
On 12/12/2008 03:40, "Tian, Kevin" <kevin.tian@xxxxxxxxx> wrote:

> A temp workaround is to move rdtscll(t->local_tsc_stamp) into
> calibrate_tsc_bp, which gives a sane NOW() before percpu time
> init. But then the purpose behind is ambiguous... why would we
> want to count system time from this point? If we can't count
> system time starting from power on, it looks clearer to tag system
> time as 0 when initializing wc_sec/wc_nsec by do_settime.
> Actually the starting point of xen system time is not that critical
> since it's mostly used by relative progress. Then my previous
> updated patch can be used? :-)

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.

 -- Keir



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