public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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

  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