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] Re: [PATCH] Align periodic vpts

To: 'Dan Magenheimer' <dan.magenheimer@xxxxxxxxxx>, Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, "Wei, Gang" <gang.wei@xxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] Re: [PATCH] Align periodic vpts
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Wed, 11 Feb 2009 23:24:53 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc:
Delivery-date: Wed, 11 Feb 2009 07:25:18 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <71f42a19-14bd-40fb-b45b-a98eba19f410@default>
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: <C5B86599.2905%keir.fraser@xxxxxxxxxxxxx> <71f42a19-14bd-40fb-b45b-a98eba19f410@default>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcmMWRTWZXFgV78TSz+gfFUkXHCBQgAAz2mg
Thread-topic: [Xen-devel] Re: [PATCH] Align periodic vpts
>From: Dan Magenheimer
>Sent: Wednesday, February 11, 2009 10:56 PM
>
>> > + * CAUTION:
>> > + * While vlapic timer ticking too close to the pit.  We 
>> saw a userspace
>> > + * application getting the wrong answer because long CPU 
>> bound sequences
>> > + * appeared to run with zero CPU time. This only showed up 
>> with old Linux
>> > + * kernels (IIRC, it was with Red Hat 3 U8). So this 
>> option may cause a
>> > + * regression in this case.
>> > + */
>> > +static int opt_align_periodic_vpt = 0;
>> > +boolean_param("align_periodic_vpt", opt_align_periodic_vpt);
>> > +
>> 
>> Presumably there are common cases where not aligning vlapic too has
>> significant power overheads? Personally I'm not sure I care 
>> too much about a
>> minor regression on RH3, if this patch is worthwhile at all 
>I think it
>> should be always on and at most have a domain config option. 
>> I think a boot
>> option will never ever be used.
>
>Given the wide variety of guests, and clocksource defaults/choices
>in those guests, I'm leery about this change, especially turning it
>on by default.  The consequences of guest clocks losing time or gaining
>time or appearing to go backwards are significant and the potential
>problems go well beyond "a minor regression on RH3" and IMHO are
>much more impactful for customers than saving a watt or two.

I'm not sure why you count this feature as the cause for guest time
inaccuracy. This patch only shifts 1st expiration of periodical timer,
and later expirations are all exactly as expected relative to previous
one.

>
>It is difficult in a simple test environment to reproduce the
>problem unless you know what you are looking for.
>Virtual Iron had a rather extensive set of test cases and I'd
>like to see those run before this is turned on by default.
>

VI's issue is not about time inaccuracy or performance, which is
about accounting.

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