linux-kernel - Re: 2.6.26-rc1 regression since 2.6.25

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Pine.LNX.4.44L0.0806301318570.2138-100000@iolanthe.rowland.org>
Date:	Mon, 30 Jun 2008 13:21:00 -0400 (EDT)
From:	Alan Stern <stern@...land.harvard.edu>
To:	Lukas Hejtmanek <xhejtman@....muni.cz>
cc:	Greg KH <greg@...ah.com>,
	Kernel development list <linux-kernel@...r.kernel.org>,
	USB list <linux-usb@...r.kernel.org>
Subject: Re: 2.6.26-rc1 regression since 2.6.25 - problem in 2.6.26-rc8 again

On Mon, 30 Jun 2008, Lukas Hejtmanek wrote:

> On Fri, Jun 27, 2008 at 04:35:59PM -0400, Alan Stern wrote:
> > Do exactly the same thing under both operating system versions, and 
> > then we should have suitably comparable logs.
> 
> all you wanted is attached including dmesgs.

Ah, good.  I see the problem now.

To tell the truth, this has been fixed in the development tree for so 
long that I had forgotten about it.  Unfortunately those fixes won't 
get into the regular kernel until 2.6.27.  Until then we need another 
fix.

The patch below should take care of the problem.  Let me know how it 
works.

Alan Stern



Index: 2.6.26-rc8/drivers/usb/core/hub.c
===================================================================
--- 2.6.26-rc8.orig/drivers/usb/core/hub.c
+++ 2.6.26-rc8/drivers/usb/core/hub.c
@@ -713,18 +713,11 @@ static void hub_restart(struct usb_hub *
 		}
 
 		/* Was the power session lost while we were suspended? */
-		switch (type) {
-		case HUB_RESET_RESUME:
-			portstatus = 0;
-			portchange = USB_PORT_STAT_C_CONNECTION;
-			break;
+		status = hub_port_status(hub, port1, &portstatus, &portchange);
 
-		case HUB_RESET:
-		case HUB_RESUME:
-			status = hub_port_status(hub, port1,
-					&portstatus, &portchange);
-			break;
-		}
+		/* If the device is gone, khubd will handle it later */
+		if (status == 0 && !(portstatus & USB_PORT_STAT_CONNECTION))
+			continue;
 
 		/* For "USB_PERSIST"-enabled children we must
 		 * mark the child device for reset-resume and

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/