Re: [PATCH v7 12/16] usb: synchronize port poweroff and khubd

2014-04-28 Thread Alan Stern
On Fri, 25 Apr 2014, Dan Williams wrote: > >> @@ -4892,11 +4897,17 @@ static void hub_events(void) > >> > >> /* deal with port status changes */ > >> for (i = 1; i <= hdev->maxchild; i++) { > >> + struct usb_port *port_dev = hub->ports[i - 1]; > >> +

Re: [PATCH v7 12/16] usb: synchronize port poweroff and khubd

2014-04-28 Thread Dan Williams
On Mon, Apr 28, 2014 at 11:25 AM, Alan Stern wrote: > On Fri, 25 Apr 2014, Dan Williams wrote: > >> >> @@ -4892,11 +4897,17 @@ static void hub_events(void) >> >> >> >> /* deal with port status changes */ >> >> for (i = 1; i <= hdev->maxchild; i++) { >> >> +

Re: [PATCH v7 12/16] usb: synchronize port poweroff and khubd

2014-04-25 Thread Dan Williams
On Wed, Mar 26, 2014 at 1:10 PM, Alan Stern wrote: > On Wed, 19 Mar 2014, Dan Williams wrote: > >> If a port is powered-off, or in the process of being powered-off, prevent >> khubd from operating on it. Otherwise, the following sequence of events >> leading to an unintended disconnect may occur:

Re: [PATCH v7 12/16] usb: synchronize port poweroff and khubd

2014-03-26 Thread Alan Stern
On Wed, 19 Mar 2014, Dan Williams wrote: > If a port is powered-off, or in the process of being powered-off, prevent > khubd from operating on it. Otherwise, the following sequence of events > leading to an unintended disconnect may occur: > > Events: > (0) > (1) hub 2-2:1.0: hub_resume > (2) h