Re: [RFC][PATCH] i386 x86-64 Eliminate Local APIC timer interrupt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Venkatesh Pallipadi wrote:
Background: Local APIC timer stops functioning when CPU is in C3 state. As a
result the local APIC timer interrupt will fire at uncertain times, depending
on how long we spend in C3 state. And this has two side effects
* Idle balancing will not happen as we expect it to.
* Kernel statistics for idle time will not be proper (as we get less LAPIC
  interrupts when we are idle). This can result in confusing other parts of
  kernel (like ondemand cpufreq governor) which depends on this idle stats.


Proposed Fix: Attached is a prototype patch, that tries to eliminate the dependency on local APIC timer for update_process_times(). The patch gets rid of Local APIC timer altogether. We use the timer interrupt (IRQ 0) configured in broadcast mode in IOAPIC instead (Doesn't work with 8259). As changing anything related to basic timer interrupt is a little bit risky, I have a boot parameter currently ("useapictimer") to switch back to original local APIC timer way of doing things.

This may seem like a overkill to solve this particular problem. But, I feel
it simplifies things and will have other advantages:
* Should help dynamick tick as one has to change only global timer interrupt freq with varying jiffies. * Reduces one interrupt per jiffy. * One less interrupt source to worry about.


Sorry I missed this when it came out, but, I think a better way to handle this is to use an IPI. In the non-VST case it can be to all but self, while when a given cpu is sleeping we can not send it to that cpu. The advantages are: 1) The broadcast has a race/ contention on the xtime lock. The read lock is needed by all and the write lock taken by one. The IPI should be sent AFTER the xtime write unlock.
2) It is easy to prune VST sleeping cpus from the list of all to wake.




--
George Anzinger   [email protected]
High-res-timers:  http://sourceforge.net/projects/high-res-timers/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux