From: "David P. Reed" <dpreed@reed.com>
To: Rene Herman <rene.herman@keyaccess.nl>
Cc: "H. Peter Anvin" <hpa@zytor.com>,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@redhat.com>,
Alan Cox <alan@lxorguk.ukuu.org.uk>,
Dmitry Torokhov <dtor_core@ameritech.net>,
linux-kernel@vger.kernel.org
Subject: Re: [linux-kernel] Re: [PATCH] x86: use explicit timing delay for pit accesses in kernel and pcspkr driver
Date: Wed, 20 Feb 2008 15:13:41 -0500 [thread overview]
Message-ID: <47BC89F5.90305@reed.com> (raw)
In-Reply-To: <47BC5EE3.5090101@keyaccess.nl>
Actually, disparaging things as "one idiotic system" doesn't seem like a
long-term thoughtful process - it's not even accurate. There are more
such systems that are running code today than the total number of 486
systems ever manufactured. The production rate is $1M/month.
a) ENE chips are "documented" to receive port 80, and also it is the
case that modern chipsets will happily diagnose writes to non-existent
ports as MCE's. Using side effects that depend on non-existent ports
just creates a brittle failure mode down the road. And it's not just
post ACPI "initialization". The pcspkr use of port 80 caused solid
freezes if you typed "tab" to complete a command line and there were
more than one choice, leading to beeps.
b) sad to say, Linux is not what hardware vendors use as the system that
their BIOSes MUST work with. That's Windows, and Windows, whether we
like it or not does not require hardware vendors to stay away from port 80.
IMHO, calling something "idiotic" is hardly evidence-based decision
making. Maybe you love to hate Microsoft, but until Intel writes an
architecture standard that says explicitly that a "standard PC" must not
use port 80 for any peripheral, the port 80 thing is folklore, and one
that is solely Linux-defined.
Rene Herman wrote:
> On 20-02-08 18:05, H. Peter Anvin wrote:
>
>> Rene Herman wrote:
>>>
>>> _Something_ like this would seem to be the only remaining option. It
>>> seems fairly unuseful to #ifdef around that switch statement for
>>> kernels without support for the earlier families, but if you insist...
>>>
>>
>> "Only remaining option" other than the one we've had all along. Even
>> on the one idiotic set of systems which break, it only breaks
>> post-ACPI intialization, IIRC.
>
> Linus vetoed the DMI switch.
>
> Rene.
>
next prev parent reply other threads:[~2008-02-20 20:14 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-18 18:58 [PATCH] x86: use explicit timing delay for pit accesses in kernel and pcspkr driver David P. Reed
2008-02-18 20:17 ` Alan Cox
2008-02-18 20:38 ` Rene Herman
2008-02-18 20:43 ` H. Peter Anvin
2008-02-18 21:04 ` Rene Herman
2008-02-18 21:05 ` Rene Herman
2008-02-18 21:44 ` H. Peter Anvin
2008-02-18 21:59 ` Rene Herman
2008-02-18 22:01 ` H. Peter Anvin
2008-02-18 22:07 ` Rene Herman
2008-02-18 22:32 ` Rene Herman
2008-02-18 22:44 ` H. Peter Anvin
2008-02-20 12:06 ` Rene Herman
2008-02-20 17:05 ` H. Peter Anvin
2008-02-20 17:09 ` Rene Herman
2008-02-20 20:13 ` David P. Reed [this message]
2008-02-21 6:21 ` [linux-kernel] " Rene Herman
2008-02-18 22:43 ` H. Peter Anvin
2008-02-19 9:46 ` Ingo Molnar
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=47BC89F5.90305@reed.com \
--to=dpreed@reed.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=dtor_core@ameritech.net \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=rene.herman@keyaccess.nl \
--cc=tglx@linutronix.de \
/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