From mboxrd@z Thu Jan 1 00:00:00 1970 From: yakui_zhao Subject: Re: [PATCH]: ACPI: Skip the power state check in power transition Date: Wed, 13 May 2009 11:13:04 +0800 Message-ID: <1242184384.3773.241.camel@localhost.localdomain> References: <1242106060.3773.216.camel@localhost.localdomain> <200905120857.40058.bjorn.helgaas@hp.com> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: Received: from mga02.intel.com ([134.134.136.20]:21946 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758570AbZEMDKp (ORCPT ); Tue, 12 May 2009 23:10:45 -0400 In-Reply-To: <200905120857.40058.bjorn.helgaas@hp.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Bjorn Helgaas Cc: "lenb@kernel.org" , "linux-acpi@vger.kernel.org" On Tue, 2009-05-12 at 22:57 +0800, Bjorn Helgaas wrote: > On Monday 11 May 2009 11:27:40 pm yakui_zhao wrote: > > From: Zhao Yakui > > > > Skip the power state check in course of power transition by changing the > > default value of acpi_power_nocheck to 1. If so, it is unnecessary to add the > > boot option of "acpi.power_nocheck=1" or add it into DMI power check table to > > skip the power state check. > > > > Of course the power state check still can be enabled by adding the boot option > > of "acpi.power_nocheck=0". > > What is the value of keeping the "acpi.power_nocheck" option at all? > > If we can get along without it, it'd be nice to just remove the > whole thing. Yes. If we delete the course of the power state check in power transition, it will be unnecessary to add the boot option. In fact this object is defined in ACPI spec. And we had better follow that. IMO Linux ACPI does the right thing. The boot option of "acpi.power_nocheck" is only to make Linux be compatible with windows. At the same time if we want to make the power state check more strict, we will have to re-add the source-code. So IMO it will be better that the power state check in power transition is controlled by one module parameter. Best regards. Yakui. > > Bjorn > > > Index: linux-2.6/drivers/acpi/power.c > > =================================================================== > > --- linux-2.6.orig/drivers/acpi/power.c 2009-04-16 16:10:24.000000000 +0800 > > +++ linux-2.6/drivers/acpi/power.c 2009-05-12 13:12:27.000000000 +0800 > > @@ -54,7 +54,7 @@ > > #define ACPI_POWER_RESOURCE_STATE_ON 0x01 > > #define ACPI_POWER_RESOURCE_STATE_UNKNOWN 0xFF > > > > -int acpi_power_nocheck; > > +int acpi_power_nocheck = 1; > > module_param_named(power_nocheck, acpi_power_nocheck, bool, 000); > > > > static int acpi_power_add(struct acpi_device *device); > > > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > >