From: Greg KH <greg@kroah.com>
To: "Grover, Andrew" <andrew.grover@intel.com>
Cc: "KOCHI, Takayoshi" <t-kouchi@mvf.biglobe.ne.jp>,
"Lee, Jung-Ik" <jung-ik.lee@intel.com>,
linux-kernel@vger.kernel.org
Subject: Re: bare pci configuration access functions ?
Date: Thu, 31 Oct 2002 18:45:04 -0800 [thread overview]
Message-ID: <20021101024504.GC13031@kroah.com> (raw)
In-Reply-To: <EDC461A30AC4D511ADE10002A5072CAD04C7A495@orsmsx119.jf.intel.com>
On Thu, Oct 31, 2002 at 06:07:31PM -0800, Grover, Andrew wrote:
> > From: Greg KH [mailto:greg@kroah.com]
> > Nice, thanks for pointing that out. But what about the fact that I
> > think we can now start optimizing certain parts of the "generic"
> > code to play nicer with Linux?
>
> It is much much more important that ACPI be *correct* than fast or small.
I agree. I was just thinking that ACPI was correct already, and we
could move on toward making it fast and small. Sorry for thinking that
:)
> I'm used to ACPI ranting from all quarters, you know that ;-) but let me
> just say this:
>
> - ACPI is not performance-critical
> - ACPI will never be simple and elegant, even if you made it Linux-specific
> - Portability enhances correctness and maximizes developer productivity
> - Read my lips, no new taxes!
>
> (dunno where that last one came from ;-)
You already said 3 other lies, so a fourth one rounded them all out? :)
(sorry, couldn't help myself. For the readers in the peanut gallery,
I consider Andy a friend, this was not a personal attack, just a chance
to make a joke.)
To address the above:
> - ACPI is not performance-critical
But it can't hurt in both stack size, and execution speed to fix obvious
things that cause it to slow down. And if ACPI is too slow, booting
takes longer, and getting ACPI events to other places start taking
unacceptable amounts of times. Not that this is happening right now :)
> - ACPI will never be simple and elegant, even if you made it Linux-specific
Heh, you said it, I didn't.
> - Portability enhances correctness and maximizes developer productivity
Only if the developers are being forced to work on multiple platforms.
For the majority of Linux kernel developers, luckily we do not have to
do this. For your group, I understand the constraints, and am willing
to live with it, in order to get a working ACPI implementation. Beggars
can't be choosy :)
thanks,
greg k-h
next prev parent reply other threads:[~2002-11-01 2:41 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-11-01 2:07 bare pci configuration access functions ? Grover, Andrew
2002-11-01 2:45 ` Greg KH [this message]
-- strict thread matches above, loose matches on Subject: below --
2002-10-31 22:00 Grover, Andrew
2002-10-31 22:11 ` Greg KH
2002-10-31 22:50 ` Scott Murray
2002-10-31 23:46 ` Greg KH
2002-11-01 0:23 ` Scott Murray
2002-10-31 23:37 ` KOCHI, Takayoshi
2002-10-31 23:54 ` Greg KH
2002-11-01 0:23 ` KOCHI, Takayoshi
2002-11-01 1:13 ` Greg KH
2002-10-31 21:07 Grover, Andrew
2002-10-31 21:23 ` Greg KH
2002-10-31 23:40 ` Kai Germaschewski
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=20021101024504.GC13031@kroah.com \
--to=greg@kroah.com \
--cc=andrew.grover@intel.com \
--cc=jung-ik.lee@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=t-kouchi@mvf.biglobe.ne.jp \
/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.