All of lore.kernel.org
 help / color / mirror / Atom feed
From: Len Brown <lenb@kernel.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Tear <tarrqt@yahoo.com>,
	mingo@redhat.com, akpm@linux-foundation.org,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [RFC][PATCH] IO-APIC blacklist
Date: Sat, 2 Jun 2007 19:13:26 -0400	[thread overview]
Message-ID: <200706021913.27351.lenb@kernel.org> (raw)
In-Reply-To: <alpine.LFD.0.98.0706021400330.23741@woody.linux-foundation.org>

On Saturday 02 June 2007 17:28, Linus Torvalds wrote:

> Finally, I wonder why that particular box is marked with an "acpi=ht" 
> blacklisting in the first place. Rather than add a new blacklist, it migth 
> be better to remove an old (and perhaps incorrect) one.
> 
> That blacklist entry is _ancient_. It's entirely possible that it's just 
> bogus: we've had so many ACPI fixes since it was added, that it's quite 
> possible that the blacklist entry itself is bogus, and is the result of 
> some old ACPI bug that triggered on that entry.
> 
> The Dell GX240 entry was added by commit 68e4ad79294 in the historic Linux 
> archive:
> 
>     Author: Len Brown <len.brown@intel.com>
>     Date:   Sat Aug 9 15:00:59 2003 -0400
> 
>     ACPI from 2.4:
>     build: add ACPI_HT, delete ACPI_HT_ONLY
>     boot: add acpi={force, off, ht}; delete "noht", "acpismp="
>     add DMI blacklist from UnitedLinux
>
> and since it sounds like the machine _works_ with ACPI on, my real 
> preference would be to just remove the black-list entry.
> 
> In fact, I thought that patch already existed in the -mm tree?

Yes, Tear sent the correct patch a while back, akpm picked it up, and i slurped
it into my tree just last night for 2.6.23:

http://git.kernel.org/?p=linux/kernel/git/lenb/linux-acpi-2.6.git;a=commit;h=4d2fafd17a325b3f4f5f9edb1211bc7f4c311269

> Len - do you have any archives back from 2003 and earlier to indicate why 
> the Dell GX240 was blacklisted?

Yep, it came to us in this Cset on 2003-08-09
http://linux.bkbits.net:8080/linux-2.6.11-stable/?PAGE=cset&REV=3f35b56btNYYpQfjOuQmiUdjSGFtWg

A young, impressionable maintainer, I'd been working on Linux for about 2 months.
I sync'd the workarounds from 2.4 into 2.5, and this particular one came
from the "UnitedLinux" tree.

My theory at the time was that SuSE had been most successful deploying ACPI,
and so upstream should benefit from their workarounds.
I later realized that this was a strategic mistake, as distro workarounds
often paper-over real bugs by addressing the symptom only.
A bunch of these entries were removed over time, and these days I add
DMI workarounds only when we are sure we have the root cause.

cheers,
-Len

  parent reply	other threads:[~2007-06-02 23:13 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-02 14:10 [RFC][PATCH] IO-APIC blacklist Tear
2007-06-02 14:39 ` Heikki Orsila
2007-06-02 16:39 ` Linus Torvalds
2007-06-02 20:32   ` Tear
2007-06-02 21:28     ` Linus Torvalds
2007-06-02 22:54       ` Andrew Morton
2007-06-02 23:13       ` Len Brown [this message]
2007-06-02 23:33       ` Len Brown
2007-06-02 23:53       ` Tear
2007-06-03  4:16         ` Len Brown
2007-06-03 10:24           ` Tear
2007-06-03 11:25           ` Tear
2007-06-02 17:46 ` Len Brown
2007-06-02 20:39   ` Tear
2007-06-02 20:55     ` Linus Torvalds
2007-06-02 23:48       ` Tear

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=200706021913.27351.lenb@kernel.org \
    --to=lenb@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=tarrqt@yahoo.com \
    --cc=torvalds@linux-foundation.org \
    /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.