(no subject)

[Lukas Hejtmanek]

<stern@rowland.harvard.edu>, Greg KH <greg@kroah.com>
Bcc: 
Subject: Re: [Bug #10630] USB devices plugged into dock are not discoverred
	until reload of ehci-hcd
Reply-To: 
In-Reply-To: <200805201327.34678.oliver@neukum.org>
X-echelon: NSA, CIA, CI5, MI5, FBI, KGB, BIS, Plutonium, Bin Laden, bomb

On Tue, May 20, 2008 at 01:27:34PM +0200, Oliver Neukum wrote:
> > done.
> > http://bugzilla.kernel.org/show_bug.cgi?id=10630
> 
> Aha. Thanks.
> Please recompile without CONFIG_USB_SUSPEND

Hm, without USB_SUSPEND it works. So what next, considered fixed or any
further investigation is needed?

-- 
Lukáš Hejtmánek

Re:

[Oliver Neukum]

Am Dienstag 20 Mai 2008 14:34:23 schrieb Lukas Hejtmanek:
> <stern@rowland.harvard.edu>, Greg KH <greg@kroah.com>
> Bcc: 
> Subject: Re: [Bug #10630] USB devices plugged into dock are not discoverred
> 	until reload of ehci-hcd
> Reply-To: 
> In-Reply-To: <200805201327.34678.oliver@neukum.org>
> X-echelon: NSA, CIA, CI5, MI5, FBI, KGB, BIS, Plutonium, Bin Laden, bomb
> 
> On Tue, May 20, 2008 at 01:27:34PM +0200, Oliver Neukum wrote:
> > > done.
> > > http://bugzilla.kernel.org/show_bug.cgi?id=10630
> > 
> > Aha. Thanks.
> > Please recompile without CONFIG_USB_SUSPEND
> 
> Hm, without USB_SUSPEND it works. So what next, considered fixed or any
> further investigation is needed?

It is by no means fixed!

Now we find out what exactly doesn't work. Please apply this patch
and provide "dmesg -c" before you plug in the device and after that.

	Regards
		Oliver

---

--- linux-2.6.25/drivers/usb/host/ehci-hcd.c	2008-05-20 10:07:45.585199135 +0200
+++ alt/drivers/usb/host/ehci-hcd.c	2008-05-20 11:11:53.614580823 +0200
@@ -712,11 +712,15 @@ static irqreturn_t ehci_irq (struct usb_
 		unsigned	i = HCS_N_PORTS (ehci->hcs_params);
 		pcd_status = status;
 
+		printk(KERN_ERR"Detected PCD bit set\n");
 		/* resume root hub? */
-		if (!(ehci_readl(ehci, &ehci->regs->command) & CMD_RUN))
+		if (!(ehci_readl(ehci, &ehci->regs->command) & CMD_RUN)) {
+			printk(KERN_ERR"About to resume root hub due to PCD\n");
 			usb_hcd_resume_root_hub(hcd);
+		}
 
 		while (i--) {
+			printk(KERN_ERR"Checking motherboard port %d\n", i);
 			int pstatus = ehci_readl(ehci,
 						 &ehci->regs->port_status [i]);
 
@@ -730,6 +734,7 @@ static irqreturn_t ehci_irq (struct usb_
 			 * and make khubd collect PORT_STAT_C_SUSPEND to
 			 * stop that signaling.
 			 */
+			printk(KERN_ERR"Starting timer for port %d\n", i);
 			ehci->reset_done [i] = jiffies + msecs_to_jiffies (20);
 			ehci_dbg (ehci, "port %d remote wakeup\n", i + 1);
 			mod_timer(&hcd->rh_timer, ehci->reset_done[i]);
--- linux-2.6.25/drivers/usb/core/hcd.c	2008-05-20 10:07:45.583199804 +0200
+++ alt/drivers/usb/core/hcd.c	2008-05-20 11:12:23.506597140 +0200
@@ -589,6 +589,8 @@ void usb_hcd_poll_rh_status(struct usb_h
 			hcd->poll_pending = 1;
 		}
 		spin_unlock_irqrestore(&hcd_root_hub_lock, flags);
+	} else {
+		printk(KERN_ERR"hub_status_data() returned 0\n");
 	}
 
 	/* The USB 2.0 spec says 256 ms.  This is close enough and won't

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Lukas Hejtmanek]

On Tue, May 20, 2008 at 02:40:46PM +0200, Oliver Neukum wrote:
> > > > http://bugzilla.kernel.org/show_bug.cgi?id=10630
> > Hm, without USB_SUSPEND it works. So what next, considered fixed or any
> > further investigation is needed?
> 
> It is by no means fixed!
> 
> Now we find out what exactly doesn't work. Please apply this patch
> and provide "dmesg -c" before you plug in the device and after that.

Should I use USB_SUSPEND or not?
 
-- 
Lukáš Hejtmánek

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Oliver Neukum]

Am Dienstag 20 Mai 2008 16:11:32 schrieb Lukas Hejtmanek:
> On Tue, May 20, 2008 at 02:40:46PM +0200, Oliver Neukum wrote:
> > > > > http://bugzilla.kernel.org/show_bug.cgi?id=10630
> > > Hm, without USB_SUSPEND it works. So what next, considered fixed or any
> > > further investigation is needed?
> > 
> > It is by no means fixed!
> > 
> > Now we find out what exactly doesn't work. Please apply this patch
> > and provide "dmesg -c" before you plug in the device and after that.
> 
> Should I use USB_SUSPEND or not?

Yes, use USB_SUSPEND.

	Regards
		Oliver

Re: your mail

[Alan Stern]

On Tue, 20 May 2008, Lukas Hejtmanek wrote:

> <stern@rowland.harvard.edu>, Greg KH <greg@kroah.com>
> Bcc: 
> Subject: Re: [Bug #10630] USB devices plugged into dock are not discoverred
> 	until reload of ehci-hcd
> Reply-To: 
> In-Reply-To: <200805201327.34678.oliver@neukum.org>
> X-echelon: NSA, CIA, CI5, MI5, FBI, KGB, BIS, Plutonium, Bin Laden, bomb
> 
> On Tue, May 20, 2008 at 01:27:34PM +0200, Oliver Neukum wrote:
> > > done.
> > > http://bugzilla.kernel.org/show_bug.cgi?id=10630
> > 
> > Aha. Thanks.
> > Please recompile without CONFIG_USB_SUSPEND
> 
> Hm, without USB_SUSPEND it works. So what next, considered fixed or any
> further investigation is needed?

No further investigation is needed.  I tried doing essentially the same 
thing on my system and the same problem occurred.

It is caused by the way ehci-hcd "auto-clears" the port
change-suspend feature.  This patch should fix the problem.  Please 
try it out and let us know if it works.

Alan Stern



Index: usb-2.6/drivers/usb/host/ehci.h
===================================================================
--- usb-2.6.orig/drivers/usb/host/ehci.h
+++ usb-2.6/drivers/usb/host/ehci.h
@@ -97,6 +97,8 @@ struct ehci_hcd {			/* one per controlle
 			dedicated to the companion controller */
 	unsigned long		owned_ports;		/* which ports are
 			owned by the companion during a bus suspend */
+	unsigned long		port_c_suspend;		/* which ports have
+			the change-suspend feature turned on */
 
 	/* per-HC memory pools (could be per-bus, but ...) */
 	struct dma_pool		*qh_pool;	/* qh per active urb */
Index: usb-2.6/drivers/usb/host/ehci-hub.c
===================================================================
--- usb-2.6.orig/drivers/usb/host/ehci-hub.c
+++ usb-2.6/drivers/usb/host/ehci-hub.c
@@ -609,7 +609,7 @@ static int ehci_hub_control (
 			}
 			break;
 		case USB_PORT_FEAT_C_SUSPEND:
-			/* we auto-clear this feature */
+			clear_bit(wIndex, &ehci->port_c_suspend);
 			break;
 		case USB_PORT_FEAT_POWER:
 			if (HCS_PPC (ehci->hcs_params))
@@ -688,7 +688,7 @@ static int ehci_hub_control (
 			/* resume completed? */
 			else if (time_after_eq(jiffies,
 					ehci->reset_done[wIndex])) {
-				status |= 1 << USB_PORT_FEAT_C_SUSPEND;
+				set_bit(wIndex, &ehci->port_c_suspend);
 				ehci->reset_done[wIndex] = 0;
 
 				/* stop resume signaling */
@@ -765,6 +765,8 @@ static int ehci_hub_control (
 			status |= 1 << USB_PORT_FEAT_RESET;
 		if (temp & PORT_POWER)
 			status |= 1 << USB_PORT_FEAT_POWER;
+		if (test_bit(wIndex, &ehci->port_c_suspend))
+			status |= 1 << USB_PORT_FEAT_C_SUSPEND;
 
 #ifndef	VERBOSE_DEBUG
 	if (status & ~0xffff)	/* only if wPortChange is interesting */

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Lukas Hejtmanek]

On Tue, May 20, 2008 at 11:20:10AM -0400, Alan Stern wrote:
> > On Tue, May 20, 2008 at 01:27:34PM +0200, Oliver Neukum wrote:
> > > > done.
> > > > http://bugzilla.kernel.org/show_bug.cgi?id=10630
> > > 
> > > Aha. Thanks.
> > > Please recompile without CONFIG_USB_SUSPEND
> > 
> > Hm, without USB_SUSPEND it works. So what next, considered fixed or any
> > further investigation is needed?
> 
> No further investigation is needed.  I tried doing essentially the same 
> thing on my system and the same problem occurred.
> 
> It is caused by the way ehci-hcd "auto-clears" the port
> change-suspend feature.  This patch should fix the problem.  Please 
> try it out and let us know if it works.

seems to be working, device is detected. Thanks. Please, push the patch.

-- 
Lukáš Hejtmánek

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Rafael J. Wysocki]

On Saturday, 31 of May 2008, Alan Stern wrote:
> On Sat, 31 May 2008, Greg KH wrote:
> 
> > On Thu, May 29, 2008 at 11:17:14PM +0200, Rafael J. Wysocki wrote:
> > > On Sunday, 25 of May 2008, Oliver Neukum wrote:
> > > > Am Samstag 24 Mai 2008 22:31:39 schrieb Rafael J. Wysocki:
> > > > > This message has been generated automatically as a part of a report
> > > > > of recent regressions.
> > > > > 
> > > > > The following bug entry is on the current list of known regressions
> > > > > from 2.6.25.  Please verify if it still should be listed.
> > > > 
> > > > Alan's patch fixes the issue. It has been tested and submitted.
> > > 
> > > Has it reached the Linus' tree already?
> > 
> > Yes, it should now be in there, as of last night.
> > 
> > Alan, is this correct?
> 
> Confirmed; the patch is now in Linus's tree:
> 
> 	Commit d1f114d12bb4db3147e1b1342ae31083c5a79c84

Thanks, closed.

Rafael

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Alan Stern]

On Sat, 31 May 2008, Greg KH wrote:

> On Thu, May 29, 2008 at 11:17:14PM +0200, Rafael J. Wysocki wrote:
> > On Sunday, 25 of May 2008, Oliver Neukum wrote:
> > > Am Samstag 24 Mai 2008 22:31:39 schrieb Rafael J. Wysocki:
> > > > This message has been generated automatically as a part of a report
> > > > of recent regressions.
> > > > 
> > > > The following bug entry is on the current list of known regressions
> > > > from 2.6.25.  Please verify if it still should be listed.
> > > 
> > > Alan's patch fixes the issue. It has been tested and submitted.
> > 
> > Has it reached the Linus' tree already?
> 
> Yes, it should now be in there, as of last night.
> 
> Alan, is this correct?

Confirmed; the patch is now in Linus's tree:

	Commit d1f114d12bb4db3147e1b1342ae31083c5a79c84

Alan Stern

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Greg KH]

On Thu, May 29, 2008 at 11:17:14PM +0200, Rafael J. Wysocki wrote:
> On Sunday, 25 of May 2008, Oliver Neukum wrote:
> > Am Samstag 24 Mai 2008 22:31:39 schrieb Rafael J. Wysocki:
> > > This message has been generated automatically as a part of a report
> > > of recent regressions.
> > > 
> > > The following bug entry is on the current list of known regressions
> > > from 2.6.25.  Please verify if it still should be listed.
> > 
> > Alan's patch fixes the issue. It has been tested and submitted.
> 
> Has it reached the Linus' tree already?

Yes, it should now be in there, as of last night.

Alan, is this correct?

thanks,

greg k-h

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Rafael J. Wysocki]

On Sunday, 25 of May 2008, Oliver Neukum wrote:
> Am Samstag 24 Mai 2008 22:31:39 schrieb Rafael J. Wysocki:
> > This message has been generated automatically as a part of a report
> > of recent regressions.
> > 
> > The following bug entry is on the current list of known regressions
> > from 2.6.25.  Please verify if it still should be listed.
> 
> Alan's patch fixes the issue. It has been tested and submitted.

Has it reached the Linus' tree already?

Rafael

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Oliver Neukum]

Am Samstag 24 Mai 2008 22:31:39 schrieb Rafael J. Wysocki:
> This message has been generated automatically as a part of a report
> of recent regressions.
> 
> The following bug entry is on the current list of known regressions
> from 2.6.25.  Please verify if it still should be listed.

Alan's patch fixes the issue. It has been tested and submitted.

	Regards
		Oliver

[Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Rafael J. Wysocki]

This message has been generated automatically as a part of a report
of recent regressions.

The following bug entry is on the current list of known regressions
from 2.6.25.  Please verify if it still should be listed.


Bug-Entry	: http://bugzilla.kernel.org/show_bug.cgi?id=10630
Subject		: USB devices plugged into dock are not discoverred until reload of ehci-hcd
Submitter	: Lukas Hejtmanek <xhejtman@ics.muni.cz>
Date		: 2008-05-05 10:02 (20 days old)
References	: http://lkml.org/lkml/2008/5/5/77
		  http://lkml.org/lkml/2008/5/19/44
Handled-By	: Oliver Neukum <oliver@neukum.org>
		  Alan Stern <stern@rowland.harvard.edu>
Patch		: http://lkml.org/lkml/2008/5/20/285

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Oliver Neukum]

Am Dienstag 20 Mai 2008 13:17:01 schrieb Lukas Hejtmanek:
> On Tue, May 20, 2008 at 12:57:59PM +0200, Oliver Neukum wrote:
> > > The following bug entry is on the current list of known regressions
> > > from 2.6.25.  Please verify if it still should be listed.
> > 
> > Can you please also post your .config with which this problem happens?
> 
> done.
> http://bugzilla.kernel.org/show_bug.cgi?id=10630

Aha. Thanks.
Please recompile without CONFIG_USB_SUSPEND

	Regards
		Oliver

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Lukas Hejtmanek]

On Tue, May 20, 2008 at 12:57:59PM +0200, Oliver Neukum wrote:
> > The following bug entry is on the current list of known regressions
> > from 2.6.25.  Please verify if it still should be listed.
> 
> Can you please also post your .config with which this problem happens?

done.
http://bugzilla.kernel.org/show_bug.cgi?id=10630

-- 
Lukáš Hejtmánek

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Oliver Neukum]

Am Sonntag 18 Mai 2008 13:13:55 schrieb Rafael J. Wysocki:
> This message has been generated automatically as a part of a report
> of recent regressions.
> 
> The following bug entry is on the current list of known regressions
> from 2.6.25.  Please verify if it still should be listed.

Can you please also post your .config with which this problem happens?

	Regards
		Oliver

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Greg KH]

On Mon, May 19, 2008 at 12:46:26PM +0200, Lukas Hejtmanek wrote:
> On Mon, May 19, 2008 at 12:39:13PM +0200, Oliver Neukum wrote:
> > > ls -F /sys/module/ehci_hcd/
> > > drivers/  holders/  initstate  notes/  parameters/  refcnt  sections/
> > 
> > Sorry, I was refering to the registers file as documented in
> > Documentation/usb/ehci.txt
> 
> I've checked that but still have no idea how to obtain the registers.

mount -t debugfs /sys/kernel/debug
cd /sys/debug/ehci/
cd <DEVICE PCI ID>
cat registers > file_to_emai_to_oliver.txt


Hope this helps,

greg k-h

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Lukas Hejtmanek]

On Mon, May 19, 2008 at 10:02:43AM +0200, Oliver Neukum wrote:
> Can you compile with USB_DEBUG and attach the "register" file from
> /sys/modules/ehci_hcd to see whether an interrupt is pending?

# gzip -dc /proc/config.gz | grep DEBUG | grep USB
CONFIG_USB_DEBUG=y
# CONFIG_USB_STORAGE_DEBUG is not set
# CONFIG_USB_SERIAL_DEBUG is not set

which one is the file you want?

ls -F /sys/module/ehci_hcd/
drivers/  holders/  initstate  notes/  parameters/  refcnt  sections/

-- 
Lukáš Hejtmánek

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Lukas Hejtmanek]

On Mon, May 19, 2008 at 12:39:13PM +0200, Oliver Neukum wrote:
> > ls -F /sys/module/ehci_hcd/
> > drivers/  holders/  initstate  notes/  parameters/  refcnt  sections/
> 
> Sorry, I was refering to the registers file as documented in
> Documentation/usb/ehci.txt

I've checked that but still have no idea how to obtain the registers.

-- 
Lukáš Hejtmánek

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Oliver Neukum]

Am Montag 19 Mai 2008 12:51:50 schrieb Lukas Hejtmanek:
> On Mon, May 19, 2008 at 10:02:43AM +0200, Oliver Neukum wrote:
> > Can you compile with USB_DEBUG and attach the "register" file from
> > /sys/modules/ehci_hcd to see whether an interrupt is pending?
> 
> # gzip -dc /proc/config.gz | grep DEBUG | grep USB
> CONFIG_USB_DEBUG=y
> # CONFIG_USB_STORAGE_DEBUG is not set
> # CONFIG_USB_SERIAL_DEBUG is not set
> 
> which one is the file you want?
> 
> ls -F /sys/module/ehci_hcd/
> drivers/  holders/  initstate  notes/  parameters/  refcnt  sections/

Sorry, I was refering to the registers file as documented in
Documentation/usb/ehci.txt

	Regards
		Oliver

Re: [Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Oliver Neukum]

Am Sonntag 18 Mai 2008 13:13:55 schrieb Rafael J. Wysocki:
> This message has been generated automatically as a part of a report
> of recent regressions.
> 
> The following bug entry is on the current list of known regressions
> from 2.6.25.  Please verify if it still should be listed.
> 
> 
> Bug-Entry	: http://bugzilla.kernel.org/show_bug.cgi?id=10630
> Subject		: USB devices plugged into dock are not discoverred until reload of ehci-hcd
> Submitter	: Lukas Hejtmanek <xhejtman@ics.muni.cz>
> Date		: 2008-05-05 10:02 (14 days old)
> References	: http://lkml.org/lkml/2008/5/5/77

Can you compile with USB_DEBUG and attach the "register" file from
/sys/modules/ehci_hcd to see whether an interrupt is pending?

	Regards
		Oliver

[Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Rafael J. Wysocki]

This message has been generated automatically as a part of a report
of recent regressions.

The following bug entry is on the current list of known regressions
from 2.6.25.  Please verify if it still should be listed.


Bug-Entry	: http://bugzilla.kernel.org/show_bug.cgi?id=10630
Subject		: USB devices plugged into dock are not discoverred until reload of ehci-hcd
Submitter	: Lukas Hejtmanek <xhejtman@ics.muni.cz>
Date		: 2008-05-05 10:02 (14 days old)
References	: http://lkml.org/lkml/2008/5/5/77

[Bug #10630] USB devices plugged into dock are not discoverred until reload of ehci-hcd

[Rafael J. Wysocki]

This message has been generated automatically as a part of a report
of recent regressions.

The following bug entry is on the current list of known regressions
from 2.6.25.  Please verify if it still should be listed.


Bug-Entry	: http://bugzilla.kernel.org/show_bug.cgi?id=10630
Subject		: USB devices plugged into dock are not discoverred until reload of ehci-hcd
Submitter	: Lukas Hejtmanek <xhejtman@ics.muni.cz>
Date		: 2008-05-05 10:02 (7 days old)
References	: http://lkml.org/lkml/2008/5/5/77