cvs commit: src/sys/netgraph/bluetooth/l2cap ng_l2cap_misc.c

Pawel Jakub Dawidek pjd at FreeBSD.org
Sun Aug 21 21:36:21 GMT 2005


On Sun, Aug 21, 2005 at 02:25:00PM -0700, Maksim Yevmenkin wrote:
+> > +> in the future, i would really appreciate if you submit your patches
+> > +> for review before committing them.
+> > 
+> > 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. 
+> 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.

I saw it uses the same callout handle, but it also can cope when I call it
twice. And in this case those two flags were set, so if they are mutually
exclusive, something is wrong here and you may want to verify this as well.

+> 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).

Ok, fell free to commit a better fix.
I'm glad you can work on this, as you know the code best.
Thanks.

-- 
Pawel Jakub Dawidek                       http://www.wheel.pl
pjd at FreeBSD.org                           http://www.FreeBSD.org
FreeBSD committer                         Am I Evil? Yes, I Am!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-all/attachments/20050821/f97a0ff3/attachment.bin


More information about the cvs-all mailing list