Hi Mark,
On Thu, 25 Oct 2007 08:04:38 -0400, Mark M. Hoffman wrote:
> Hi Thomas:
>
> I recently told someone in private that ACPI vs. hwmon conflicts are the
> biggest open problems for the hwmon subsystem. Thank you (and Jean) for
> doing this.
>
> * Thomas Renninger <trenn@xxxxxxx> [2007-10-24 16:31:59 +0200]:
> > Hi,
> >
> > it seems Len's test tree and Linus tree diverged a bit, at least with
> > this patch set things do not apply cleanly.
> >
> > Therefore I post these for discussion whether and in which kernel tree
> > they should end up before doing work for nothing.
> > If they are still a candidate for 2.6.24 (rather unintrusive), pls tell
> > me whether and when I should base them against Len's test/release branch
> > or whatever other tree.
> > If not, it would be great if they can be included into the -mm tree and
> > I can rebase them against this one.
>
> Andrew has already picked this series; I vote for extended time in -mm. On the
> hwmon side, there is almost guaranteed to be fallout from this that may take
> time to resolve.
Of course, otherwise we wouldn't have done it ;)
> > A boot parameter acpi_enforce_resources=strict/lax/no is provided, which
> > is default set to lax:
> > - strict: let conflicting drivers fail to load with an error message
> > - lax: let conflicting driver work normal with a warning message
> > - no: no functional change at all
> > Depending on the feedback and the kind of interferences we see, this
> > should be set to strict at later time.
>
> As long as it's in -mm, you may as well default to =strict right away. This
> will force people to report. Open the floodgates; I hope I don't drown.
Good point. Here's a patch. Andrew, can you please apply this on top of
the other patches? Thanks.
Subject: Enforce ACPI resource conflict checks
In -mm, enforce ACPI resource conflict checks, so that users will report
to us.
This patch is NOT meant to go to Linus at this point.
Signed-off-by: Jean Delvare <khali@xxxxxxxxxxxx>
---
drivers/acpi/osl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- linux-2.6.24-rc1.orig/drivers/acpi/osl.c 2007-10-24 10:01:16.000000000 +0200
+++ linux-2.6.24-rc1/drivers/acpi/osl.c 2007-10-25 17:13:58.000000000 +0200
@@ -1076,7 +1076,7 @@ __setup("acpi_wake_gpes_always_on", acpi
#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)
{
--
Jean Delvare
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
