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 2/4] CPUIDLE: Avoid remnant LAPIC timer intrwhile

To: "Wei, Gang" <gang.wei@xxxxxxxxx>, "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH 2/4] CPUIDLE: Avoid remnant LAPIC timer intrwhile force hpetbroadcast
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Wed, 10 Sep 2008 22:51:31 +0800
Cc:
Delivery-date: Wed, 10 Sep 2008 07:51:57 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <8FED46E8A9CA574792FC7AACAC38FE7701ACD18DCE@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>
References: <8FED46E8A9CA574792FC7AACAC38FE7701ACA5C4B4@xxxxxxxxxxxxxxxxxxxxxxxxxxxx><C4ED5CDE.26F1B%keir.fraser@xxxxxxxxxxxxx> <8FED46E8A9CA574792FC7AACAC38FE7701ACD18DCE@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AckSVfqaFjPrXqrARTKdgjxn6iTeJQA2DOERAAhrdiAAAN6s4A==
Thread-topic: [Xen-devel] [PATCH 2/4] CPUIDLE: Avoid remnant LAPIC timer intrwhile force hpetbroadcast
>From: Wei, Gang
>Sent: 2008年9月10日 22:25
>
>> It's not clear to me the disable/enable_LAPIC_timer() work 
>is worthwhile for
>> the few timer interrupts it is likely to avoid. The other 
>bit of the patch
>> is a good bugfix though. I've taken just the latter part.
>
>Thanks for accept most of these patches. As to 
>disable/enable_LAPIC_timer(), I add them because some 
>platforms require the LAPIC timer intr being disabled before 
>entering C3, otherwise there may be some unexpected things. As 
>a reference, Linux kernel always do so.
>

Also this is conceptually clearer, as at given time we only
want one clock source to drive timers. It'd make little sense
to have APIC timer freely counting down to trigger spurious
interrupt when HPET/PIT already plays the role before APIC 
timer is reprogrammed upon C3 exit. :-)

Thanks,
Kevin

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