[linux-usb-devel] usb suspend backtraces.

2007-07-15 Thread Dave Jones
Hi folks, Given the potential power savings USB_SUSPEND offers, we'd like to turn this on in Fedora kernels, however there still seems to be quite a few devices out there that seem to have issues. (Notably printers it seems). For our development kernels, we're just shipping stuff as-is, so that

Re: [linux-usb-devel] usb suspend backtraces.

2007-07-15 Thread David Brownell
On Sunday 15 July 2007, Dave Jones wrote: > Before I dig deeper, are there other side-effects of changing > this default ? Other than spending more power than necessary? :) I don't think there should be, but Alan's most on top of these particular issues. The main thing I'd worry about is root

Re: [linux-usb-devel] usb suspend backtraces.

2007-07-15 Thread Dave Jones
On Sun, Jul 15, 2007 at 08:07:01PM -0700, David Brownell wrote: > On Sunday 15 July 2007, Dave Jones wrote: > > > Before I dig deeper, are there other side-effects of changing > > this default ? > > Other than spending more power than necessary? :) > > I don't think there should be, but

Re: [linux-usb-devel] usb suspend backtraces.

2007-07-15 Thread David Brownell
On Sunday 15 July 2007, Dave Jones wrote: > On Sun, Jul 15, 2007 at 08:07:01PM -0700, David Brownell wrote: > > On Sunday 15 July 2007, Dave Jones wrote: > > > > > Before I dig deeper, are there other side-effects of changing > > > this default ? > > > > Other than spending more power than

Re: [linux-usb-devel] usb suspend backtraces.

2007-07-16 Thread Alan Stern
On Sun, 15 Jul 2007, Dave Jones wrote: > Hi folks, >Given the potential power savings USB_SUSPEND offers, we'd > like to turn this on in Fedora kernels, however there still > seems to be quite a few devices out there that seem to have issues. > (Notably printers it seems). Scanners too. Plus

Re: [linux-usb-devel] usb suspend backtraces.

2007-07-16 Thread Dave Jones
On Mon, Jul 16, 2007 at 10:38:48AM -0400, Alan Stern wrote: > > -static int usb_autosuspend_delay = 2; /* Default delay value, > > +static int usb_autosuspend_delay = 0; /* Default delay value, > > * in seconds */ > > module_

Re: [linux-usb-devel] usb suspend backtraces.

2007-07-16 Thread Alan Stern
On Mon, 16 Jul 2007, Dave Jones wrote: > > > in usb_autopm_do_device() > > > I'm suspecting logic is underflowing here causing this. > > > > Yes, that's what it means. I'm puzzled as to how this could occur; let > > me try some tests. What types of devices give rise to these warnings? >

Re: [linux-usb-devel] usb suspend backtraces.

2007-07-16 Thread Dave Jones
On Mon, Jul 16, 2007 at 03:22:17PM -0400, Alan Stern wrote: > > I'm not sure how that 5-1.4 relates to the above map. > > > > oh, there's one other case, where it prints the same thing for usb 4-2: > > All right, it turns out there really is a minor bug that shows up only > with the autos

Re: [linux-usb-devel] usb suspend backtraces.

2007-07-16 Thread Alan Stern
On Mon, 16 Jul 2007, Dave Jones wrote: > On Mon, Jul 16, 2007 at 03:22:17PM -0400, Alan Stern wrote: > > > > I'm not sure how that 5-1.4 relates to the above map. > > > > > > oh, there's one other case, where it prints the same thing for usb 4-2: > > > > All right, it turns out there reall