Date: Sat, 04 Oct 2008 02:21:04 +0400 From: Vladimir Grebenschikov <vova@fbsd.ru> To: Maksim Yevmenkin <maksim.yevmenkin@gmail.com> Cc: freebsd-bluetooth@freebsd.org, usb@freebsd.org Subject: Re: Bluetooth audio - crash on USB bluetooth dongle disconnect - solved Message-ID: <1223072464.4832.2.camel@localhost> In-Reply-To: <bb4a86c70810031418t5b9eef45p4d57b55f22c05715@mail.gmail.com> References: <3a386af20809261420j535680e8pf44453dbf6f84b20@mail.gmail.com> <bb4a86c70809261504v45ffe1a8oaf26670a1032e86c@mail.gmail.com> <1223034512.1842.111.camel@localhost> <bb4a86c70810030945g3d870a1eqacc85233d9698a66@mail.gmail.com> <1223067257.2362.6.camel@localhost> <bb4a86c70810031418t5b9eef45p4d57b55f22c05715@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
В Fri, 03/10/2008 в 14:18 -0700, Maksim Yevmenkin пишет:
> one thing that is different from interrupt and bulk transfers is that
> ng_ubt(4) always has multiple outstanding incoming isoc transfers.
> when device is simply pulled out, there is not much driver can do. if
> i enable debugging i can see my transfer completion routine called
> with ioerror status or something like that. so, i suspect this is a
> cleanup issue. i'm not sure who supposed to do the cleanup in this
> case driver or stack.
>
> in any case, can you verify that ubt_reset() is called when device is
> pulled out? (it should be called as part of hook disconnect).
Right guess, it does not called.
> if not - then please try to call ubt_reset() from ubt_detach() just before
> closing all the pipes.
Yes, it fixes crash, patch below.
Index: netgraph/bluetooth/drivers/ubt/ng_ubt.c
===================================================================
RCS file: /usr/ncvs/src/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c,v
retrieving revision 1.35
diff -u -r1.35 ng_ubt.c
--- netgraph/bluetooth/drivers/ubt/ng_ubt.c 29 Jul 2008 00:17:53 -0000 1.35
+++ netgraph/bluetooth/drivers/ubt/ng_ubt.c 3 Oct 2008 22:13:46 -0000
@@ -803,6 +803,9 @@
struct ubt_softc *sc = device_get_softc(self);
int i;
+ /* Reset queues and pipes */
+ ubt_reset(sc);
+
/* Destroy Netgraph node */
if (sc->sc_node != NULL) {
NG_NODE_SET_PRIVATE(sc->sc_node, NULL);
> thanks,
> max
--
Vladimir B. Grebenschikov
vova@fbsd.ru
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1223072464.4832.2.camel>