Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
- From: "Felipe Balbi" <felipebalbi@xxxxxxxxxxxxxxxxxxxxx>
- Date: Fri, 3 Aug 2007 08:34:06 -0400
On 8/3/07, Rogan Dawes <lists@xxxxxxxxxxxx> wrote:
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 aIf both options have drawbacks, IMHO we follow the standard, which
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".
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.
Even though we should be following what the specs says and find other
ways of threating the "messy" devices. A sysfs entry for
enabling/disabling autosuspend and even to add devices to blacklist
seems quite nice to me.
FWIW.
Rogan
-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
linux-usb-devel@xxxxxxxxxxxxxxxxxxxxx
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
--
Best Regards,
Felipe Balbi
felipebalbi@xxxxxxxxxxxxxxxxxxxxx
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- References:
- Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
- From: Alan Stern
- Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
- From: David Brownell
- Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
- From: Matthew Garrett
- Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
- From: Oliver Neukum
- Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
- From: Matthew Garrett
- Re: [PATCH] USB: Only enable autosuspend by default on certain device classes
- From: Rogan Dawes
- Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
- Prev by Date: Re: [PATCH] USB: Only enable autosuspend by default on certain device classes
- Next by Date: Re: [PATCH] update checkpatch.pl to version 0.08
- Previous by thread: Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
- Next by thread: Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
- Index(es):
Relevant Pages
|