提交 d01f42a2 编写于 作者: E Eric Lescouet 提交者: Greg Kroah-Hartman

staging: usbip: Fix deadlock

When detaching a port from the client side (usbip --detach 0),
the event thread, on the server side, is going to deadlock.
The "eh" server thread is getting USBIP_EH_RESET event and calls:
  -> stub_device_reset() -> usb_reset_device()
the USB framework is then calling back _in the same "eh" thread_ :
  -> stub_disconnect() -> usbip_stop_eh() -> wait_for_completion()
the "eh" thread is being asleep forever, waiting for its own completion.
This patch checks if "eh" is the current thread, in usbip_stop_eh().
Signed-off-by: NEric Lescouet <eric@lescouet.org>
Cc: stable <stable@kernel.org>
Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
上级 c0087580
...@@ -117,6 +117,9 @@ void usbip_stop_eh(struct usbip_device *ud) ...@@ -117,6 +117,9 @@ void usbip_stop_eh(struct usbip_device *ud)
{ {
struct usbip_task *eh = &ud->eh; struct usbip_task *eh = &ud->eh;
if (eh->thread == current)
return; /* do not wait for myself */
wait_for_completion(&eh->thread_done); wait_for_completion(&eh->thread_done);
usbip_dbg_eh("usbip_eh has finished\n"); usbip_dbg_eh("usbip_eh has finished\n");
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册