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:32:28 +0100 Sender: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Message-ID: <20040308183228.GD484@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 On Po 08-03-04 15:15:58, Marcelo Tosatti wrote: > > > 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? To put it simply: there's broken machine where calling acpi_system_save_state() with invalid argument fixes it by accident. It can do random stuff on any other machine... 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