From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759230AbXFWNXX (ORCPT ); Sat, 23 Jun 2007 09:23:23 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756807AbXFWNV7 (ORCPT ); Sat, 23 Jun 2007 09:21:59 -0400 Received: from www.osadl.org ([213.239.205.134]:59194 "EHLO mail.tglx.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1756179AbXFWNV5 (ORCPT ); Sat, 23 Jun 2007 09:21:57 -0400 Message-Id: <20070623124030.215035893@inhelltoy.tec.linutronix.de> References: <20070623124005.931747831@inhelltoy.tec.linutronix.de> User-Agent: quilt/0.46-1 Date: Sat, 23 Jun 2007 13:32:27 -0000 From: Thomas Gleixner To: Andrew Morton Cc: Andi Kleen , Ingo Molnar , Arjan van de Ven , Venkatesh Pallipadi , John Stultz , Chris Wright , LKML , "Udo A. Steinberg" , Len Brown Subject: [patch -mm 02/28] ACPI: Move timer broadcast and pmtimer access before C3 arbiter shutdown Content-Disposition: inline; filename=acpi-move-timer-broadcast-before-arb-dis.patch Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org From: Udo A. Steinberg The chipset doc for IHC4 tells us: 1.In general, software should not attempt any non-posted accesses during arbiter disable except to the ICH4's power management registers. This implies that interrupt handlers for any unmasked hardware interrupts and SMI/NMI should check ARB_DIS status before reading from ICH devices. So it's not a good idea to access ICH devices after arbiter shut down. Signed-off-by: Udo A. Steinberg Signed-off-by: Thomas Gleixner Cc: Len Brown --- drivers/acpi/processor_idle.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) Index: linux-2.6.22-rc4-mm/drivers/acpi/processor_idle.c =================================================================== --- linux-2.6.22-rc4-mm.orig/drivers/acpi/processor_idle.c 2007-06-23 14:38:57.000000000 +0200 +++ linux-2.6.22-rc4-mm/drivers/acpi/processor_idle.c 2007-06-23 14:38:57.000000000 +0200 @@ -978,6 +978,12 @@ static int acpi_idle_enter_c3(struct cpu return 0; } + /* + * Must be done before busmaster disable as we might need to + * access HPET ! + */ + acpi_state_timer_broadcast(pr, cx, 1); + /* disable bus master */ if (pr->flags.bm_check) { spin_lock(&c3_lock); @@ -997,7 +1003,6 @@ static int acpi_idle_enter_c3(struct cpu /* Get start time (ticks) */ t1 = inl(acpi_gbl_FADT.xpm_timer_block.address); - acpi_state_timer_broadcast(pr, cx, 1); acpi_idle_do_entry(cx); t2 = inl(acpi_gbl_FADT.xpm_timer_block.address); --