From: Myron Stowe <myron.stowe@hp.com>
To: ykzhao <yakui.zhao@intel.com>
Cc: Matthew Garrett <mjg59@srcf.ucam.org>,
"minyard@acm.org" <minyard@acm.org>,
"lenb@kernel.org" <lenb@kernel.org>,
"openipmi-developer@lists.sourceforge.net"
<openipmi-developer@lists.sourceforge.net>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
Bjorn Helgaas <bjorn.helgaas@hp.com>
Subject: Re: [PATCH -v6 2/2] IPMI: use ACPI detection mechanism firstly to detect IPMI system interface
Date: Tue, 08 Jun 2010 21:02:37 -0600 [thread overview]
Message-ID: <1276052557.2769.47.camel@zim> (raw)
In-Reply-To: <1275973808.3718.99.camel@localhost.localdomain>
On Tue, 2010-06-08 at 13:10 +0800, ykzhao wrote:
> On Tue, 2010-06-08 at 09:34 +0800, Matthew Garrett wrote:
> > On Tue, Jun 08, 2010 at 09:28:51AM +0800, ykzhao wrote:
> >
> > > Does there exist the ACPI detection mechanism on the machines you
> > > mentioned? If exists, does it detect the same IPMI interface with the
> > > PCI IPMI detection mechanism?
> >
> > What is "the same"? It's not using the same ioport space, certainly.
>
> "The same" means that they will use the same ioport space/address.
> If they use the different ioport space/address, they will be regarded as
> the different IPMI device.
ACPI namespace* should not be enumerating a device that is discoverable
via standard PCI device enumeration so if there are multiple IPMI
devices in a system, where one is enumerated via ACPI namespace and
another is enumerated via PCI they should never be "the same" literal
device.
* I specifically denoted namespace since the ACPI SPMI table, yet
another ACPI based IPMI enumeration mechanism, seems as if it could
denote "the same" literal device.
Matthew's approach, giving a PCI based IPMI device precedence due to its
inclusion of interrupt information, does seem to have merit.
Myron
>
> >
> > > If the two mechanisms will detect the same IPMI interface, I agree with
> > > what you are concerned. Do you have an idea/thought to set up the
> > > relationship between ACPI and IPMI interface? In order to enable that
> > > AML code can access the IPMI, it should know which IPMI interface will
> > > be accessed and create the corresponding user interface. If ACPI
> > > mechanism will fail to register the IPMI interface, maybe it is
> > > difficult to create the correct user interface.
> >
> > Well, right now if you change the ordering then the PCI interface will
> > never be exposed. It would be preferable to only expose the ACPI
> > interface as a user-visible device if there's no prior device - if there
> > is, I think the ideal solution would be for it to be an in-kernel only
> > device without a corresponding UI.
>
> Sorry that I don't explain it clearly. The concept of "user interface"
> in IPMI interface is only a channel that can be used to communicate with
> the IPMI controller. It has no relationship with whether the IPMI
> interface should be exposed to user space. If one driver wants to
> communicate with one IPMI interface, we should create one "user
> interface" firstly and send the corresponding IPMI message by using the
> "user interface".
>
> If one IPMI interface(controller) is already detected by PCI mechanism,
> then ACPI will fail to detect the same IPMI interface. In such case it
> is difficult for ACPI to know which IPMI interface should be accessed
> when the ACPI AML code need to communicate with the IPMI interface.
>
> thanks
> Yakui
>
> >
>
--
Myron Stowe HP Open Source Linux Lab (OSLL)
--
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
next prev parent reply other threads:[~2010-06-09 3:02 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-07 9:27 [PATCH -v6 0/2] IPMI: Install the ACPI IPMI opregion yakui.zhao
2010-06-07 9:27 ` [PATCH -v6 1/2] IPMI: Install the IPMI space handler to enable ACPI to access the BMC controller yakui.zhao
2010-06-07 9:27 ` [PATCH -v6 2/2] IPMI: use ACPI detection mechanism firstly to detect IPMI system interface yakui.zhao
2010-06-07 12:52 ` Matthew Garrett
2010-06-08 1:28 ` ykzhao
2010-06-08 1:34 ` Matthew Garrett
2010-06-08 5:10 ` ykzhao
2010-06-09 3:02 ` Myron Stowe [this message]
2010-06-09 13:08 ` Corey Minyard
2010-06-10 2:34 ` ykzhao
2010-06-10 3:12 ` Corey Minyard
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=1276052557.2769.47.camel@zim \
--to=myron.stowe@hp.com \
--cc=bjorn.helgaas@hp.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=minyard@acm.org \
--cc=mjg59@srcf.ucam.org \
--cc=openipmi-developer@lists.sourceforge.net \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).