Re: [Xen-devel] [PATCH][discuss] io_apic disable_8254_timer

[Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>]

On 2 Mar 2006, at 21:29, Woller, Thomas wrote:

> This patch only adds "disable_8254_timer" and leaves the default as
> enabled - ie. There is no "enable_8254_timer" in this patch.  The linux
> solution provides more detailed determination of whether to
> enable/disable using timer_over_8254, including determining if ATI
> chipset is present.  Most of which is most likely not necessary with 
> xen
> hv at this time.
>
> The added code in this patch could be ifdef 64bit, as the linux 
> solution
> is only implemented in the x86_64 io_apic.c code.
> Tom
Actually this gives me an excuse to pick a scab and get rid of the 
'missed tick' logic that relies on that code altogether. So I just 
deleted the line instead of making it conditional. I'm pretty sure that 
won't break anything.
 Thanks,
 Keir


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