From patchwork Tue Sep 8 13:31:46 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jean Delvare X-Patchwork-Id: 46214 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n88DVoeU030737 for ; Tue, 8 Sep 2009 13:31:51 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752918AbZIHNbq (ORCPT ); Tue, 8 Sep 2009 09:31:46 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753544AbZIHNbq (ORCPT ); Tue, 8 Sep 2009 09:31:46 -0400 Received: from cantor2.suse.de ([195.135.220.15]:56985 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752918AbZIHNbo (ORCPT ); Tue, 8 Sep 2009 09:31:44 -0400 Received: from relay2.suse.de (relay-ext.suse.de [195.135.221.8]) by mx2.suse.de (Postfix) with ESMTP id C5DDA86445; Tue, 8 Sep 2009 15:31:46 +0200 (CEST) From: Jean Delvare Organization: SuSE Linux To: linux-acpi@vger.kernel.org Subject: [PATCH v2] ACPI: Clarify resource conflict message Date: Tue, 8 Sep 2009 15:31:46 +0200 User-Agent: KMail/1.9.6 (enterprise 20070904.708012) Cc: Zhang Rui , Len Brown , Thomas Renninger , Alan Jenkins MIME-Version: 1.0 Content-Disposition: inline Message-Id: <200909081531.46716.jdelvare@suse.de> Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org From: Jean Delvare Subject: ACPI: Clarify resource conflict message The message "ACPI: Device needs an ACPI driver" is misleading. The device _may_ need an ACPI driver, if the BIOS implemented a custom API for the device in question (which, AFAIK, can't be checked.) If not, then either a generic ACPI driver may be used (for example "thermal"), or nothing can be done (other than a white list). I propose to reword the message to: ACPI: If an ACPI driver is available for this device, you should use it instead of the native driver which I think is more correct. Comments and suggestions welcome. I also added a message warning about possible problems and system instability when users pass acpi_enforce_resources=lax, as suggested by Len. Signed-off-by: Jean Delvare Cc: Zhang Rui Cc: Len Brown Cc: Thomas Renninger Cc: Alan Jenkins --- Len, if you are satisfied with the wording, any chance to have this patch in 2.6.31? Thanks. drivers/acpi/osl.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) --- linux-2.6.31-rc9.orig/drivers/acpi/osl.c 2009-09-08 09:46:51.000000000 +0200 +++ linux-2.6.31-rc9/drivers/acpi/osl.c 2009-09-08 13:56:53.000000000 +0200 @@ -1182,7 +1182,13 @@ int acpi_check_resource_conflict(struct res_list_elem->name, (long long) res_list_elem->start, (long long) res_list_elem->end); - printk(KERN_INFO "ACPI: Device needs an ACPI driver\n"); + if (acpi_enforce_resources == ENFORCE_RESOURCES_LAX) + printk(KERN_NOTICE "ACPI: This conflict may" + " cause random problems and system" + " instability\n"); + printk(KERN_INFO "ACPI: If an ACPI driver is available" + " for this device, you should use it instead of" + " the native driver\n"); } if (acpi_enforce_resources == ENFORCE_RESOURCES_STRICT) return -EBUSY;