[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.LFD.2.00.1005130724280.3711@i5.linux-foundation.org>
Date: Thu, 13 May 2010 07:35:02 -0700 (PDT)
From: Linus Torvalds <torvalds@...ux-foundation.org>
To: Dmitry Torokhov <dmitry.torokhov@...il.com>
cc: Andrew Morton <akpm@...ux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
linux-input@...r.kernel.org, Bastien Nocera <hadess@...ess.net>
Subject: Re: [git pull] Input updates for 2.6.34-rc6
On Thu, 13 May 2010, Dmitry Torokhov wrote:
>
> Bastien Nocera (1):
> Input: i8042 - do not try to probe ports on Intel Apple Macs
I pulled, but I skipped the last commit, because I think this one is
fundamentally _wrong_.
It is _not_ maintainable to create random tables of exceptions ("DMI
tables"), and it's actively _wrong_ to do for something like this where we
not only have historically worked perfectly well, and this apparently
tries to hide some other bug (the commit says "could potentially lock
up/hang/wait for timeout for long periods of time").
We should fix the problems instead of hiding them for specific machines.
Does anybody really think that Apple machines are the only ones with no
legacy keyboard? Hello? Does anybody seriously think that it's ok to add
entries to DMI tables for random new machines coming out?
So I think that commit was (a) totally inappropriate to send at this point
in the late -rc series _anyway_ (it sure as hell isn't a refression fix),
and that makes me wonder about the other ones. But (b) I don't think I
want to ever see anything like that during a merge window either, because
it's quite seriously the wrong thing to do.
What are the _actual_ problems on legacy-free machines? And keep in mind
that I ask that exactly because I actually _have_ two Apple Mac Mini's in
my household, and have never seen any problems with keyboard/mouse
handling.
So if somebody saw "could potentially lock up/hang/wait" issues, then
dangit, say what those issues are, AND LET'S FIX THEM! And not like this,
trying to hide them for some particular machines, rather than fixing the
actual underlying detection bug.
Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists