On Thu, Jan 29, 2009 at 11:30 AM, Thomas Renninger <trenn@xxxxxxx> wrote: > On Sunday 25 January 2009 22:05:20 Luca Tettamanti wrote: >> (This is an improved version of my earlier patch, I've reworked deviced >> detection to simply check for the desired HID) >> >> The following patch adds "auto" option to "acpi_enforce_resource"; this >> value is also the new default. >> "auto" enforces strict resource checking - disallowing access by native >> drivers to IO ports and memory regions claimed by ACPI firmware - when a >> device with a known ACPI driver is found (currently only ASUS ATK0110 is >> checked), and reverts to lax checking otherwise. >> >> The patch is mainly aimed to block native hwmon drivers from touching >> monitoring chips that ACPI thinks it own. > > Having this in the core ACPI code is ugly. I think we all agree :-) > Either this should be more generic (instead of hardcoded "ATK0110" device, > use a list to add further specific thermal ACPI drivers). Hmm, maybe it's > only ASUS violating the spec? ASUS it's not actually violating any spec... > Thinkpads seem to read out extra thermal > data from EC and shouldn't interfere with hwmon drivers. The point is that there is only 1 physical sensor, which both ACPI and a native driver want to drive; transaction on SMBus to read the sensor are usually in the form "select bank" + "read" and the sequence is *not* atomic. In ASUS case the IO ports are correctly reserved by the firmware, but (historically) this wasn't taken into account. Aside from ASUS the problem is generic since there are two drivers poking at the same hardware; for example there are reports of native drivers interfering with normal TZ polling (see [1]). The EC does not make any difference, since a native driver would speak directly to the monitoring chip, effectively by-passing the EC. Now, in principle "strict" is the correct behaviour for the resource checking code, but it would break many working setups leaving the users without any kind of hw monitoring. The "auto" hack is meant to force the users to migrate to the ACPI driver... > Are there other known, model specific ACPI devices, > accessing thermal IOs directly which could interfere with hwmon, then it > might be worth it. Right now "auto" is ASUS-specific because *I* know that there any many different native drivers that works on various boards (and I wrote the the ACPI driver...); At least eeepc and (some?) thinkpads have ACPI hwmon capabilities but I don't know whether there are native drivers available or not (but they could be "blacklisted" preemptively like ASUS ATK). Luca [1] http://www.lm-sensors.org/ticket/2072 and: http://thread.gmane.org/gmane.linux.drivers.sensors/12359 -- 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
