USB: unbind all interfaces before rebinding any
authorAlan Stern <stern@rowland.harvard.edu>
Wed, 12 Mar 2014 15:30:38 +0000 (11:30 -0400)
committerJiri Slaby <jslaby@suse.cz>
Thu, 15 May 2014 07:55:34 +0000 (09:55 +0200)
commitc1ad06e8bf308f609edbb1bbec60d0ee926009bb
tree955da964813e1a543779b7f0d06d70976ba40370
parent2c66d33ded4e9431e36789c26e3b3ff114072f3a
USB: unbind all interfaces before rebinding any

commit 6aec044cc2f5670cf3b143c151c8be846499bd15 upstream.

When a driver doesn't have pre_reset, post_reset, or reset_resume
methods, the USB core unbinds that driver when its device undergoes a
reset or a reset-resume, and then rebinds it afterward.

The existing straightforward implementation can lead to problems,
because each interface gets unbound and rebound before the next
interface is handled.  If a driver claims additional interfaces, the
claim may fail because the old binding instance may still own the
additional interface when the new instance tries to claim it.

This patch fixes the problem by first unbinding all the interfaces
that are marked (i.e., their needs_binding flag is set) and then
rebinding all of them.

The patch also makes the helper functions in driver.c a little more
uniform and adjusts some out-of-date comments.

Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Reported-and-tested-by: "Poulain, Loic" <loic.poulain@intel.com>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
drivers/usb/core/driver.c
drivers/usb/core/hub.c
drivers/usb/core/usb.h