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] xend: fix rtc_timeoffset when localtime=1

To: Kouya Shimura <kouya@xxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xend: fix rtc_timeoffset when localtime=1
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Wed, 1 Apr 2009 17:04:26 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 01 Apr 2009 09:04:55 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <7ky6uk6c8y.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxx>
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>
Newsgroups: chiark.mail.xen.devel
References: <7ky6uk6c8y.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Kouya Shimura writes ("[Xen-devel] [PATCH] xend: fix rtc_timeoffset when 
localtime=1"):
> The rtc_timeoffset parameter in VM config is ignored when localtime=1.
> Also it is not preserved across reboot.
> 
> NOTE:
> This patch changes the meaning of the xenstore /vm/<uuid>/rtc/timeoffset
> from utc offset to local offset. (it's OK for the ioemu)

Are you sure that this is an appropriate change ?  I'm not sure I
follow exactly and have only skimread your patch.  Can you explain why
it is right to change the meaning of a xenstore value like this ?

Ian.

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