Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 21 Aug 2005 14:25:00 -0700
From:      Maksim Yevmenkin <maksim.yevmenkin@gmail.com>
To:        Pawel Jakub Dawidek <pjd@freebsd.org>
Cc:        cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/netgraph/bluetooth/l2cap ng_l2cap_misc.c
Message-ID:  <bb4a86c705082114253fda9ee2@mail.gmail.com>
In-Reply-To: <20050821205152.GB1037@garage.freebsd.pl>
References:  <200508211916.j7LJGR72062875@repoman.freebsd.org> <bb4a86c705082113341d46203a@mail.gmail.com> <20050821205152.GB1037@garage.freebsd.pl>

next in thread | previous in thread | raw e-mail | index | archive | help
Pawel,

> +> first, let me thank you for looking into the problem. i just want to
> +> ask you a couple of questions. did you manage to reproduce the
> +> problem? does your patch fix it? i would like to see the code path
> +> that triggers this panic.
>=20
> I was able to reproduce it by doing:
>=20
>         # l2ping -f -a somehost -s 65531
>         ^C
>         # /etc/rc.bluetooth stop ubt0

great, thanks!

> +> in the future, i would really appreciate if you submit your patches
> +> for review before committing them.
>=20
> Sure, sorry about that. I assumed noone is interested, because noone
> offered help to submitter.

actually, i was looking into this. unfortunately my emails did not
make it to the list because someone broke our local mail system here.
one of the smtp hosts we use does not have dns entry and my mail is
rejected :( also i was very busy with real work for the last two
weeks, so i did not have any spare time to look into this.

please take no offence, but i'd like to be involved when bluetooth
code is changed. while i appreciate your work here, i'd like to see a
slightly different patch. in particular, i do think think that
ng_l2cap_free_con() should call ng_l2cap_xxx_untimeout() routines.=20
imo, ng_l2cap_free_con()  should panic if it detects that callout is
pending. also L2CAP_CON_AUTO_DISCON_TIMO and NG_L2CAP_CON_LP_TIMO are
mutually exclusive. both use the same callout handle, so you can not
have both at the same time.

having said all that, i think your patch should stay for now. please
do not MFC it just yet. i will look into this first thing tomorrow
(monday).

thanks for your work again!

max



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bb4a86c705082114253fda9ee2>