All of lore.kernel.org
 help / color / mirror / Atom feed
From: Robert Hancock <hancockrwd@gmail.com>
To: "Moore, Robert" <robert.moore@intel.com>
Cc: Thomas Renninger <trenn@suse.de>, Jean Delvare <jdelvare@suse.de>,
	"Lin, Ming M" <ming.m.lin@intel.com>,
	"Zhao, Yakui" <yakui.zhao@intel.com>,
	linux-acpi <linux-acpi@vger.kernel.org>,
	Len Brown <lenb@kernel.org>
Subject: Re: [PATCH] ACPICA: Revert "ACPICA: Remove obsolete   acpi_os_validate_address interface"
Date: Fri, 03 Jul 2009 19:29:23 -0600	[thread overview]
Message-ID: <4A4EB073.1000907@gmail.com> (raw)
In-Reply-To: <4911F71203A09E4D9981D27F9D8308582EA5E8FC@orsmsx503.amr.corp.intel.com>

On 07/02/2009 09:49 AM, Moore, Robert wrote:
>> -----Original Message-----
>> From: Jean Delvare [mailto:jdelvare@suse.de]
> ...
>> Native Linux drivers trying to access devices which ACPI also wants
>> to access. Most frequently these are SMBus master drivers or hardware
>> monitoring drivers (for Super I/O chips) but virtually any other
>> device type could be affected, due to the fact that ACPI I/O
>> resources do not show up in /proc/ioports and /proc/iomem.
>
>
> I have some concern with this. The implication is that there is a serious hole in ACPI that can only be solved by monitoring exactly what the AML code is doing, because access to resources may conflict with device drivers. I have to think that if this were truly the case, the operating system vendors and the ACPI contributors would have pushed a fix for this problem into the ACPI specification by now (in the 13 years since the first release of ACPI).
>
> The SMBus driver issue is very interesting. I have not heard of any conflicts between AML code and SMBus drivers on other operating systems, nor have we received any other complaints about removing the acpi_os_validate_address interface. I wonder if the Linux SMBus driver is doing something that it should not be doing.
>
> I'd like to see a concrete example of the problem. If someone can send me a DSDT that contains such an example, I would appreciate it. I want to see the exact ASL code that is causing the problem along with the driver code that conflicts with it. This information will help me understand the problem in relation to the ACPICA code, and I can also present this to other ACPI and BIOS experts for their opinions.
>
> As far as an immediate fix for the problem, I suggest that you think about using acpi_walk_namespace to retrieve all objects of type Region (or field). It seems a waste to build an entire list of objects that duplicates information that already exists in the namespace.

My guess is that exactly the same thing will happen if you install 
add-on hardware monitoring software on Windows that loads drivers that 
access SMBus hardware directly on such systems where ACPI accesses the 
same devices. The ACPI code and the native driver will likely stomp on 
each other's hardware accesses. Likely on Linux it's more frequent that 
this kind of native hardware monitoring driver gets used (in some setups 
it seems they will auto-load by default..)

  reply	other threads:[~2009-07-04  1:28 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-01  2:29 [PATCH] ACPICA: Revert "ACPICA: Remove obsolete acpi_os_validate_address interface" Lin Ming
2009-07-01  8:56 ` Thomas Renninger
2009-07-01  9:23   ` Lin Ming
2009-07-01  9:35     ` Thomas Renninger
2009-07-01 15:29       ` Moore, Robert
2009-07-01 21:19         ` Thomas Renninger
2009-07-01 22:07           ` Moore, Robert
2009-07-02  8:20             ` Jean Delvare
2009-07-02  8:30           ` Jean Delvare
2009-07-02  2:03 ` Len Brown
2009-07-02  6:27   ` Lin Ming
2009-07-02  6:42     ` Moore, Robert
2009-07-02 10:15       ` Thomas Renninger
2009-07-02 10:12     ` Thomas Renninger
2009-07-03  1:30       ` Lin Ming
2009-07-13 15:36     ` Thomas Renninger
2009-07-14  2:28       ` Lin Ming
2009-07-17 15:02         ` Thomas Renninger
2009-07-02 10:22   ` Thomas Renninger
2009-07-02 15:49     ` Moore, Robert
2009-07-04  1:29       ` Robert Hancock [this message]
2009-08-30 13:43         ` Jean Delvare

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A4EB073.1000907@gmail.com \
    --to=hancockrwd@gmail.com \
    --cc=jdelvare@suse.de \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=ming.m.lin@intel.com \
    --cc=robert.moore@intel.com \
    --cc=trenn@suse.de \
    --cc=yakui.zhao@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.