From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Subject: Re: Re: fix ACPI poweroff in 2.4.25-rc2 Date: Mon, 8 Mar 2004 19:23:50 +0100 Sender: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Message-ID: <20040308182350.GC484@elf.ucw.cz> References: <20040308163452.GE5352@openzaurus.ucw.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Errors-To: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: Marcelo Tosatti Cc: Bjorn Helgaas , willy-tRhHnHjXm5DYtjvyW6yDsg@public.gmane.org, acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-acpi@vger.kernel.org Hi! > > > The recent 2.4 patch breaks the build for ia64 because we don't > > > set CONFIG_ACPI_SLEEP, and acpi_system_save_state() is only > > > defined under #ifdef CONFIG_ACPI_SLEEP. Can you add the following > > > patch to deal with this? > > > > Original patch should be reverted. Having different > > behaviour of poweroff depending on ACPI_SLEEP would > > be wrong. > > Hi Pavel, > > Original patches fixes a real problem, so its not as easy as "lets revert > it", yes? Well, it is. Real problem was that someone's machine was not able to suspend with APIC enabled (IIRC). He found that calling acpi_system_save_state() accidentally fixes it for him (probably AML disables APIC or something like that). Now, right fix is to make linux shutdown APIC (etc) prior to powerdown. Calling random functions is not the way to go. Pavel -- When do you have a heart between your knees? [Johanka's followup: and *two* hearts?] ------------------------------------------------------- This SF.Net email is sponsored by: IBM Linux Tutorials Free Linux tutorial presented by Daniel Robbins, President and CEO of GenToo technologies. Learn everything from fundamentals to system administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click