From: "Li, Yan" <yanli@infradead.org>
To: Matthew Garrett <mjg59@srcf.ucam.org>, yan.i.li@intel.com
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
"linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
"greg@kroah.com" <greg@kroah.com>
Subject: Re: [PATCH] More i8042-reset quirks for MSI Wind-clone netbooks
Date: Thu, 18 Jun 2009 08:08:09 +0800 [thread overview]
Message-ID: <20090618000809.GB16835@thyme.bj.intel.com> (raw)
In-Reply-To: <20090617025101.GA27595@srcf.ucam.org>
On Wed, Jun 17, 2009 at 03:51:01AM +0100, Matthew Garrett wrote:
> On Wed, Jun 17, 2009 at 09:01:40AM +0800, Li, Yan I wrote:
> > This seems a change too aggressive for me. Do we have a good reason
> > for taking this risk?
>
> It's generally much easier to find regressions (people complain) than it
> is to find things that have never worked (people just assume Linux is
> broken).
That's true. But we are not sure how many regressions we'll meet and
whether the efforts devoted to handle them is worthy. (How to handle
regressions? Perhaps, ironically, we'll need another 'whitelist' for
them!)
> > Of course if we found the "actual problem" we'd conjure up a better
> > fix. But before that, I'd prefer the conservative way.
>
> Does stock Windows work on the machine? I think this really ought to be
> a pretty obvious minimal test before adding quirks to the kernel.
Does this matter? Does whether Windows fail or not affect our
decision here? (Worse that I have no "stock Windows XP" for
testing. All I have are those companion Windows Recovery CDs that
include all drivers).
--
Li, Yan
next prev parent reply other threads:[~2009-06-18 3:36 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-15 14:16 [PATCH] More i8042-reset quirks for MSI Wind-clone netbooks Yan Li
2009-06-15 14:34 ` Matthew Garrett
2009-06-15 14:47 ` Li, Yan
2009-06-16 15:42 ` Matthew Garrett
2009-06-16 16:36 ` Alan Cox
2009-06-16 16:43 ` Matthew Garrett
2009-06-16 17:15 ` Alan Cox
2009-06-16 17:17 ` Matthew Garrett
2009-06-17 1:01 ` Li, Yan I
2009-06-17 2:51 ` Matthew Garrett
2009-06-18 0:08 ` Li, Yan [this message]
2009-06-18 15:42 ` Matthew Garrett
2009-06-19 7:16 ` Li, Yan I
2009-06-19 12:37 ` Matthew Garrett
2009-06-22 23:35 ` Andrew Morton
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=20090618000809.GB16835@thyme.bj.intel.com \
--to=yanli@infradead.org \
--cc=akpm@linux-foundation.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=greg@kroah.com \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
--cc=yan.i.li@intel.com \
/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;
as well as URLs for NNTP newsgroup(s).