From patchwork Sun Mar 29 22:01:27 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luca Tettamanti X-Patchwork-Id: 15054 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 n2TM1jTg027146 for ; Sun, 29 Mar 2009 22:01:45 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754343AbZC2WBp (ORCPT ); Sun, 29 Mar 2009 18:01:45 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754320AbZC2WBp (ORCPT ); Sun, 29 Mar 2009 18:01:45 -0400 Received: from mail-bw0-f169.google.com ([209.85.218.169]:36841 "EHLO mail-bw0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753857AbZC2WBn (ORCPT ); Sun, 29 Mar 2009 18:01:43 -0400 Received: by bwz17 with SMTP id 17so1694796bwz.37 for ; Sun, 29 Mar 2009 15:01:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:received:date:from:to:cc :subject:message-id:references:mime-version:content-type :content-disposition:in-reply-to:user-agent; bh=UGbIRZPb+6fF3cphIA5/mBE+rOVpJKqZIVG6XSI9mMM=; b=R7go4Ds6GMsrUq3V9C2AJz+XPV//QL5C/Z6PhmXODItdlcvv59UthVopKaT9LLS2y0 WD42zOIWRlVFFVqQrTL92NZvfRlFOhDE4NRSWiiHUQ80qglIZ/gnl9QFMBEz57Xp8u0r bQvA4qElo+87qoW3ufnvPpXcRE+/xZGvI03o4= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=IWE67EkP9NTvrDi+WnlWUZ9buW/dyRebjHyncGY/RTLLxd9Yw34b+M8DoSBCpO33Ld bFw4E4vb8Qe+UIR4p54aYQdPQ3xO2nvphLG+QfLNEl2s+Ek9rf7AnoGOdk4MAHo5XATp fP4pNSuvHEgPQFehUfRGCbM/RpTBpQ1pkfXgs= Received: by 10.223.114.135 with SMTP id e7mr3379470faq.89.1238364100187; Sun, 29 Mar 2009 15:01:40 -0700 (PDT) Received: from dreamland.darkstar.lan (host-78-14-226-217.cust-adsl.tiscali.it [78.14.226.217]) by mx.google.com with ESMTPS id d13sm2021394fka.0.2009.03.29.15.01.39 (version=TLSv1/SSLv3 cipher=RC4-MD5); Sun, 29 Mar 2009 15:01:39 -0700 (PDT) Received: by dreamland.darkstar.lan (Postfix, from userid 1000) id E6AD0701CD; Mon, 30 Mar 2009 00:01:27 +0200 (CEST) Date: Mon, 30 Mar 2009 00:01:27 +0200 From: Luca Tettamanti To: Jean Delvare Cc: Hans de Goede , linux-kernel@vger.kernel.org, Matthew Garrett , Len Brown , Thomas Renninger , linux-acpi@vger.kernel.org, Pavel Machek Subject: Re: [PATCH] ACPI: add "auto" to acpi_enforce_resources Message-ID: <20090329220127.GA12822@dreamland.darkstar.lan> References: <20090204142015.GB3923@srcf.ucam.org> <20090210145716.105ab58b@hyperion.delvare> <20090210140829.GA25397@srcf.ucam.org> <49919E08.5050002@redhat.com> <20090210172408.400cacee@hyperion.delvare> <20090227132740.GF1482@ucw.cz> <68676e00903240539i119b6d26j516ee131d4cd6781@mail.gmail.com> <49C8DE51.2010103@redhat.com> <20090329201617.GA10303@dreamland.darkstar.lan> <20090329225501.0c1771f1@hyperion.delvare> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20090329225501.0c1771f1@hyperion.delvare> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org Il Sun, Mar 29, 2009 at 10:55:01PM +0200, Jean Delvare ha scritto: > Hi Luca, > > On Sun, 29 Mar 2009 22:16:17 +0200, Luca Tettamanti wrote: > > Il Tue, Mar 24, 2009 at 02:21:21PM +0100, Hans de Goede ha scritto: > > > On 03/24/2009 01:39 PM, Luca Tettamanti wrote: > > >> On Fri, Feb 27, 2009 at 2:27 PM, Pavel Machek wrote: > > >>> Hi! > > >>> > > >>>>> For the record we have changed the default to strict in Fedora's > > >>>>> development branch, for 2 weeks or so now, including in the recently > > >>>>> released Fedora 11 release and we've had 0 complaints so far. > > >>>> Well, if the number of affected systems is small, this is good news. > > >>>> But this is only 2 weeks and one distribution, coverage isn't > > >>>> sufficient to claim anything yet IMHO. > > >>>> > > >>>> That being said... if there's a common consensus that switching to > > >>>> strict and dealing with fallouts is the best thing to do, and I'm the > > >>>> only one objecting to this, then I am ready to admit that I was wrong > > >>>> and let you proceed. > > >>> I believe that 'enable strict, deal with fallout' is the best > > >>> long-term strategy... > > >> > > >> Hello, > > >> the merge window for .30 is now open, what are we going to do with this issue? > > >> > > > > > > I think the consensus was to make the default strict and to merge the atk0110 > > > driver, right? > > > > Ok, > > here's a patch: > > --- > > The following patch changes the default value for option "acpi_enforce_resource" > > to strict. It enforces strict resource checking - disallowing access by native > > drivers to IO ports and memory regions claimed by ACPI firmware. > > > > The patch is mainly aimed to block native hwmon drivers from touching > > monitoring chips that ACPI thinks it own. > > > > Signed-off-by: Luca Tettamanti > > --- > > Documentation/kernel-parameters.txt | 16 ++++++++++++++++ > > drivers/acpi/osl.c | 6 +++--- > > 2 files changed, 19 insertions(+), 3 deletions(-) > > > > Index: b/Documentation/kernel-parameters.txt > > =================================================================== > > --- a/Documentation/kernel-parameters.txt 2009-03-29 15:47:28.000000000 +0200 > > +++ b/Documentation/kernel-parameters.txt 2009-03-29 15:51:30.000000000 +0200 > > @@ -259,6 +259,22 @@ > > to assume that this machine's pmtimer latches its value > > and always returns good values. > > > > + acpi_enforce_resources= [ACPI] > > + { strict, lax, no } > > Other options use | as a delimited between allowed values. [...] > The driver will fail to _bind_ to the device, this doesn't mean that the > module will fail to load. In practice, PCI modules will still load > while "fake" platform modules will indeed fail to load. Many hwmon > drivers fall into the second category. [...] > Other that these minor details: > > Acked-by: Jean Delvare Hi Jean, here's a revised patch: Acked-by: Jean Delvare --- The following patch changes the default value for option "acpi_enforce_resource" to strict. It enforces strict resource checking - disallowing access by native drivers to IO ports and memory regions claimed by ACPI firmware. The patch is mainly aimed to block native hwmon drivers from touching monitoring chips that ACPI thinks it own. Signed-off-by: Luca Tettamanti --- Documentation/kernel-parameters.txt | 16 ++++++++++++++++ drivers/acpi/osl.c | 6 +++--- 2 files changed, 19 insertions(+), 3 deletions(-) Luca Index: b/Documentation/kernel-parameters.txt =================================================================== --- a/Documentation/kernel-parameters.txt 2009-03-29 23:58:11.574893074 +0200 +++ b/Documentation/kernel-parameters.txt 2009-03-29 23:58:31.582894852 +0200 @@ -259,6 +259,22 @@ to assume that this machine's pmtimer latches its value and always returns good values. + acpi_enforce_resources= [ACPI] + { strict | lax | no } + Check for resource conflicts between native drivers + and ACPI OperationRegions (SystemIO and SystemMemory + only). IO ports and memory declared in ACPI might be + used by the ACPI subsystem in arbitrary AML code and + can interfere with legacy drivers. + strict (default): access to resources claimed by ACPI + is denied; legacy drivers trying to access reserved + resources will fail to bind to device using them. + lax: access to resources claimed by ACPI is allowed; + legacy drivers trying to access reserved resources + will bind successfully but a warning message is logged. + no: ACPI OperationRegions are not marked as reserved, + no further checks are performed. + agp= [AGP] { off | try_unsupported } off: disable AGP support Index: b/drivers/acpi/osl.c =================================================================== --- a/drivers/acpi/osl.c 2009-03-29 23:58:11.942913535 +0200 +++ b/drivers/acpi/osl.c 2009-03-29 23:58:31.586892815 +0200 @@ -1070,9 +1070,9 @@ * in arbitrary AML code and can interfere with legacy drivers. * acpi_enforce_resources= can be set to: * - * - strict (2) + * - strict (default) (2) * -> further driver trying to access the resources will not load - * - lax (default) (1) + * - lax (1) * -> further driver trying to access the resources will load, but you * get a system message that something might go wrong... * @@ -1084,7 +1084,7 @@ #define ENFORCE_RESOURCES_LAX 1 #define ENFORCE_RESOURCES_NO 0 -static unsigned int acpi_enforce_resources = ENFORCE_RESOURCES_LAX; +static unsigned int acpi_enforce_resources = ENFORCE_RESOURCES_STRICT; static int __init acpi_enforce_resources_setup(char *str) {