public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Myron Stowe <myron.stowe@hp.com>
To: Bela Lubkin <blubkin@vmware.com>
Cc: "minyard@acm.org" <minyard@acm.org>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"openipmi-developer@lists.sourceforge.net"
	<openipmi-developer@lists.sourceforge.net>,
	"lenb@kernel.org" <lenb@kernel.org>
Subject: RE: [Openipmi-developer] [PATCH 2/4] ipmi: Remove SPMI table based device discovery method
Date: Fri, 05 Mar 2010 10:13:50 -0700	[thread overview]
Message-ID: <1267809230.2411.114.camel@zim> (raw)
In-Reply-To: <E0AA796A7E3F4C4E8F1065D15FE478610176ABB8AD@EXCH-MBX-4.vmware.com>

On Fri, 2010-03-05 at 04:58 -0800, Bela Lubkin wrote:
> Myron Stowe wrote:
> 
> > ACPI in new systems does tend to present a lot of challenges. 
> >  My job is
> > new platform enablement focused which more often than not 
> > translates to
> > "helping discover and fix BIOS/ACPI issues" so I definitely 
> > understand.
> > 
> > You also astutely point out that ACPI's static tables are 
> > available for
> > use by an OS - or OSPM in ACPI speak - much earlier than its 
> > namespace.
> > This is true but it is also our belief that no one uses it except HP.
> 
> Are you watching what goes on in embedded systems or only
> the PC / server marketplace?

No, I do not watch the embedded system space.  Would such folks tend to
track openipmi-developer@lists.sourceforge.net (at least with respect to
IPMI)?  If not, are you aware of a specific mail reflector that I should
be including in this discussion?

> 
> > I see how one's life could be easier relying on the BIOS setting up
> > static tables such as either the SMBIOS or SPMI.  That way the BIOS is
> > taking the dynamic aspects into account relieving the OS, or 
> > OS/platform
> > bring up engineer, from such.  In such a case, what benefits does the
> > SPMI table provide as opposed to the SMBIOS table?
> 
> It gives you twice as many chances to find a sufficiently
> non-buggy table to get on with your work ;-}
> 
> > > I would be more comfortable if you kept this code, perhaps
> > > suppressed under a "CONFIG_IPMI_SPMI" config option.
> > 
> > I just don't see enough remaining value in keeping the SPMI table
> > capability.  It seems to be redundant, and if truly so, just adds
> > unnecessary code, complexity, and maintenance aspects to the 
> > driver.  We
> > typically don't keep code in the kernel just for bring up efforts.
> 
> Well, I'm not doing any such bring-up work so I will
> step out of the discussion here.  But I'll leave you
> with a bit of a curse: if you remove this code, I
> predict that the person it will come back to bite is
> you...

I hope I'm not coming off as closed minded and causing you to disengage
as most every concern you have raised has been valid and needful of
consideration.  As mean as putting a jinx on me was (I'm mostly kidding
there), you are correct there too: it will be me that it comes back to
bite.  I still feel that this is worth pursuing as no progress is ever
made without some risk but I do want discussions to occur so that I can
learn, preferably beforehand, if I am wrong.

Myron
> 
> >Bela<


-- 
Myron Stowe                             HP Open Source Linux Lab (OSLL)


  parent reply	other threads:[~2010-03-05 17:13 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-04  3:44 [PATCH 0/4] ipmi: remove SPMI and update core driver with dev_printk Myron Stowe
2010-03-04  3:44 ` [PATCH 1/4] ipmi: Raise precedence of PNP based discovery mechanisms (ACPI, PCI) Myron Stowe
2010-03-04  3:44 ` [PATCH 2/4] ipmi: Remove SPMI table based device discovery method Myron Stowe
2010-03-04  7:56   ` [Openipmi-developer] " Bela Lubkin
2010-03-04 18:47     ` Myron Stowe
2010-03-05 12:58       ` Bela Lubkin
2010-03-05 16:06         ` Bjorn Helgaas
2010-03-05 17:13         ` Myron Stowe [this message]
2010-03-04  3:44 ` [PATCH 3/4] ipmi: Convert tracking of the ACPI device pointer to a PNP device Myron Stowe
2010-03-04  9:20   ` ykzhao
2010-03-04 20:48     ` Myron Stowe
2010-03-05  1:46       ` ykzhao
2010-03-05 16:41         ` Myron Stowe
2010-03-05 19:34           ` Bjorn Helgaas
2010-03-04  3:44 ` [PATCH 4/4] ipmi: Update driver to use 'dev_printk()' and its constructs Myron Stowe
2010-03-04 14:07 ` [PATCH 0/4] ipmi: remove SPMI and update core driver with dev_printk Corey Minyard
2010-03-05 16:31   ` Myron Stowe
2010-03-09 23:16   ` Myron Stowe
2010-03-10 14:55     ` [SPAM] - Re: [PATCH 0/4] ipmi: remove SPMI and update core driver with dev_printk - Email found in subject Andy Cress
2010-03-10 15:20       ` Corey Minyard
2010-03-10 17:55         ` [Openipmi-developer] " Myron Stowe
2010-03-11 17:10           ` Bela Lubkin

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=1267809230.2411.114.camel@zim \
    --to=myron.stowe@hp.com \
    --cc=blubkin@vmware.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=minyard@acm.org \
    --cc=openipmi-developer@lists.sourceforge.net \
    /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