From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andi Kleen Subject: ACPI boot fix for 2.6.15 Date: Tue, 13 Dec 2005 06:28:34 +0100 Message-ID: <20051213052834.GI23384@wotan.suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline Sender: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: torvalds-3NddpPZAyC0@public.gmane.org, akpm-3NddpPZAyC0@public.gmane.org, len.brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, trenn-l3A5Bk7waGM@public.gmane.org, acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-acpi@vger.kernel.org Hallo, I have the following patch in my x86-64 tree because it was needed to boot without acpi=off on several of the machines here. Without it the ACPI device setup code would always create a loop in the ACPI device tree and eventually some other code walking it would hang the machine. The patch has been around for a long time - it was originally done by Thomas Renninger - and has even been shipped in SUSE 10.0 so it was well tested (which was 2.6.13 based, there the problem only hit occasionally, in 2.6.14+ it seems to happen more frequently for some reason) I think it was queued in the ACPI trees too and acked by the ACPI poeople. I know Len skipped the ACPI update for .15 for some reason, but could we perhaps merge this safe patch. It would help my test machines at least :) ---- From: trenn-l3A5Bk7waGM@public.gmane.org Avoid endless loops in ACPI device tree. Cc: Len.Brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org Signed-off-by: Andi Kleen Index: linux/drivers/acpi/scan.c =================================================================== --- linux.orig/drivers/acpi/scan.c +++ linux/drivers/acpi/scan.c @@ -1110,7 +1110,7 @@ acpi_add_single_object(struct acpi_devic * * TBD: Assumes LDM provides driver hot-plug capability. */ - result = acpi_bus_find_driver(device); + acpi_bus_find_driver(device); end: if (!result) ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click