From: Rogan Dawes <lists@dawes.za.net>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: Oliver Neukum <oliver@neukum.org>,
David Brownell <david-b@pacbell.net>, Greg KH <gregkh@suse.de>,
Alan Stern <stern@rowland.harvard.edu>,
linux-usb-devel@lists.sourceforge.net,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] USB: Only enable autosuspend by default on certain device classes
Date: Fri, 03 Aug 2007 14:26:43 +0200 [thread overview]
Message-ID: <46B31F03.80908@dawes.za.net> (raw)
In-Reply-To: <20070803120418.GB16802@srcf.ucam.org>
Matthew Garrett wrote:
> On Fri, Aug 03, 2007 at 01:44:02PM +0200, Oliver Neukum wrote:
>> Am Freitag 03 August 2007 schrieb Matthew Garrett:
>>> It's certainly possible to do that, but it's also possible to have a
>>> userspace solution that whitelists devices. The question is whether the
>>> default kernel behaviour should be "Save power, but potentially break
>>> some of my devices" or "Don't break my devices, but use some more
>>> powre".
>> If both options have drawbacks, IMHO we follow the standard, which
>> says that devices must support suspension.
>
> Except that lots of hardware doesn't follow the standard in this
> respect, otherwise we wouldn't be having this discussion. Personally, I
> think "Will break an unknown number of devices" is a significantly
> larger drawback than "Will consume a small quantity of additional
> power".
>
I guess the question could be phrased:
Which one is more likely to conclude at some point?
That is, if we blacklist by default, we consume that additional power
indefinitely, because it is unlikely that people will report "my machine
uses 200mW more than I think it should", and thus we are unlikely to
build up knowledge of exactly which devices/classes should be blacklisted.
Compare that to:
"My USB printer broke, guess I'd better report it to LKML".
The first option is unlikely to ever reach a satisfactory conclusion,
whereas the second one is quite likely to flush out the guilty parties
within a relatively short time.
FWIW.
Rogan
next prev parent reply other threads:[~2007-08-03 12:32 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-02 23:56 [PATCH] USB: Only enable autosuspend by default on certain device classes Matthew Garrett
2007-08-03 1:15 ` Greg KH
2007-08-03 1:47 ` Matthew Garrett
2007-08-03 2:46 ` [linux-usb-devel] " Alan Stern
2007-08-03 6:01 ` David Brownell
2007-08-03 11:28 ` Matthew Garrett
2007-08-03 11:44 ` Oliver Neukum
2007-08-03 12:04 ` Matthew Garrett
2007-08-03 12:26 ` Rogan Dawes [this message]
2007-08-03 12:32 ` Matthew Garrett
2007-08-03 14:01 ` David Brownell
2007-08-03 14:09 ` Matthew Garrett
2007-08-03 14:28 ` Alan Stern
2007-08-03 14:33 ` Matthew Garrett
2007-08-03 14:41 ` Alan Stern
2007-08-03 15:10 ` Matthew Garrett
2007-08-03 14:43 ` Jiri Kosina
2007-08-03 14:53 ` [linux-usb-devel] " Dave Jones
2007-08-03 14:58 ` Jiri Kosina
2007-08-03 15:24 ` David Brownell
2007-08-03 15:29 ` Jiri Kosina
2007-08-03 17:47 ` Alan Stern
2007-08-03 15:25 ` Adam Kropelin
2007-08-03 15:31 ` Jiri Kosina
2007-08-03 15:48 ` Dave Jones
2007-08-03 16:49 ` David Brownell
2007-08-03 16:55 ` Adam Kropelin
2007-08-03 15:06 ` Matthew Garrett
2007-08-03 14:37 ` [linux-usb-devel] " David Brownell
2007-08-03 15:03 ` Matthew Garrett
2007-08-03 16:08 ` Oliver Neukum
2007-08-03 17:49 ` Alan Stern
2007-08-03 20:03 ` Dave Jones
2007-08-03 16:29 ` David Brownell
2007-08-03 16:50 ` Matthew Garrett
2007-08-03 17:49 ` Greg KH
2007-08-03 17:44 ` Greg KH
2007-08-03 17:48 ` Matthew Garrett
2007-08-03 19:29 ` Chuck Ebbert
2007-08-03 19:56 ` Pete Zaitcev
2007-08-07 9:14 ` Amit Kucheria
2007-08-03 20:12 ` [linux-usb-devel] " Dave Jones
2007-08-03 21:17 ` Alan Stern
2007-08-03 21:31 ` Dave Jones
2007-08-03 21:33 ` Matthew Garrett
2007-08-03 12:34 ` Felipe Balbi
2007-08-03 6:06 ` David Brownell
2007-08-03 14:22 ` Dave Jones
2007-08-03 14:52 ` David Brownell
2007-08-03 7:57 ` Oliver Neukum
2007-08-03 14:24 ` Dave Jones
2007-08-03 14:32 ` Oliver Neukum
2007-08-03 14:36 ` [linux-usb-devel] [PATCH] USB: Only enable autosuspend by?default " Dave Jones
2007-08-03 19:34 ` [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default " Pete Zaitcev
2007-08-03 19:45 ` Dave Jones
2007-08-03 20:04 ` Pete Zaitcev
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=46B31F03.80908@dawes.za.net \
--to=lists@dawes.za.net \
--cc=david-b@pacbell.net \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb-devel@lists.sourceforge.net \
--cc=mjg59@srcf.ucam.org \
--cc=oliver@neukum.org \
--cc=stern@rowland.harvard.edu \
/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