Re: [patch 0/6] AMD C1E aware idle support
- From: "Rafael J. Wysocki" <rjw@xxxxxxx>
- Date: Thu, 12 Jun 2008 14:31:23 +0200
On Thursday, 12 of June 2008, Thomas Gleixner wrote:
AMD CPUs with C1E support are currently excluded from high resolution
timers and NOHZ support. The reason is that C1E is a BIOS controlled
C3 power state which switches off TSC and the local APIC timer. The
ACPI C-State control manages the TSC/local APIC timer wreckage, but
this does not include the C1 based ("halt" instruction) C1E mode. The
BIOS/SMM controlled C1E state works on most systems even without
enabling ACPI C-State control.
The fact that a system has C1E support enabled is advertised in a MSR,
but the time during boot when the C1E bit is set by the BIOS varies:
1) Boot CPU has already C1E bit set
2) Secondary CPU sets C1E bit
3) C1E bit is set after ACPI C-State query
Case #1 and #2 are covered by the current implementation, but case #3
results in a complete system lockup due to missing timer
interrupts. The current solution is to disable the local APIC timer
and use the PIT in broadcast mode. This restricts the C1E enabled
systems to periodic timer mode.
The following patch series implements a C1E aware idle function which
also covers the late C1E enablement (case #3):
The function is selected during boot for CPUs which have possibly C1E
support. The function checks the MSR which contains the C1E active
bits before executing the halt instruction. When one of the C1E active
bits is set, it makes the system C1E aware by enabling the timer
broadcast mechanism for all CPUs. For high resolution timer and/or
nohz enabled systems it calls the oneshot timer broadcast mechanism
before executing the halt instruction. This is the same mechanism
which is used in the ACPI C-State control for C2/C3 power states.
On my C1E affected X2 box these patches reduce the wakeups/sec down to
20 according to powertop.
The patches work fine on systems which are not affected by the dreaded
ATI chipset timer wreckage. On those which have the problem, the box
needs help from the keyboard to continue working.
The x86 changes for .27 contain a complete overhaul of the affected
code, but this is out of scope for this patchset.
For those who are interested to test those patches on top of 2.6.26-rc
I extracted a patch and added it to the c1e series. It's available
from:
http://www.kernel.org/pub/linux/kernel/people/tglx/c1e/2.6.26-rc5-c1e-patches.tar.gz
or
http://www.kernel.org/pub/linux/kernel/people/tglx/c1e/2.6.26-rc5-c1e-patches.tar.bz2
I guess the x86-next-apic-timer-fixes.patch is a combination of all patches in
this series. Is that correct?
Thanks,
Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- Follow-Ups:
- Re: [patch 0/6] AMD C1E aware idle support
- From: Thomas Gleixner
- Re: [patch 0/6] AMD C1E aware idle support
- From: Ingo Molnar
- Re: [patch 0/6] AMD C1E aware idle support
- References:
- [patch 0/6] AMD C1E aware idle support
- From: Thomas Gleixner
- [patch 0/6] AMD C1E aware idle support
- Prev by Date: [PATCH 12/23] checkpatch: check spacing for square brackets
- Next by Date: Re: [PATCH] ext2: Use page_mkwrite vma_operations to get mmap write notification.
- Previous by thread: Re: [patch 6/6] x86: add c1e aware idle function
- Next by thread: Re: [patch 0/6] AMD C1E aware idle support
- Index(es):
Relevant Pages
|
