From owner-cvs-src@FreeBSD.ORG Sun Aug 21 21:25:02 2005 Return-Path: X-Original-To: cvs-src@freebsd.org Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 13A5B16A41F for ; Sun, 21 Aug 2005 21:25:02 +0000 (GMT) (envelope-from maksim.yevmenkin@gmail.com) Received: from rproxy.gmail.com (rproxy.gmail.com [64.233.170.206]) by mx1.FreeBSD.org (Postfix) with ESMTP id CFC8B43D48 for ; Sun, 21 Aug 2005 21:25:00 +0000 (GMT) (envelope-from maksim.yevmenkin@gmail.com) Received: by rproxy.gmail.com with SMTP id i8so839590rne for ; Sun, 21 Aug 2005 14:25:00 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=rPg+LEumwDoRV2WSj4ykCUicqTWMC/spRF5pQq6+8kZF0qLAoYHYIoPTmRqht2SMwG8afw2a6HxcAr58vtZCkh+oML58Hi8+ZT29yQtGeMZ2+z6XIDABxaT2L9JhDyvYUaLEKc6J67QnZxhhpdCgpJWgNmW6WyD0Ow4NTy2/ZiI= Received: by 10.38.181.6 with SMTP id d6mr477253rnf; Sun, 21 Aug 2005 14:25:00 -0700 (PDT) Received: by 10.38.208.5 with HTTP; Sun, 21 Aug 2005 14:25:00 -0700 (PDT) Message-ID: Date: Sun, 21 Aug 2005 14:25:00 -0700 From: Maksim Yevmenkin To: Pawel Jakub Dawidek In-Reply-To: <20050821205152.GB1037@garage.freebsd.pl> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline References: <200508211916.j7LJGR72062875@repoman.freebsd.org> <20050821205152.GB1037@garage.freebsd.pl> 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 X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 21 Aug 2005 21:25:02 -0000 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