From owner-cvs-src@FreeBSD.ORG Fri Jun 15 20:32:14 2007 Return-Path: X-Original-To: cvs-src@freebsd.org Delivered-To: cvs-src@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id CF87C16A469; Fri, 15 Jun 2007 20:32:14 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from server.baldwin.cx (66-23-211-162.clients.speedfactory.net [66.23.211.162]) by mx1.freebsd.org (Postfix) with ESMTP id 5CB5913C4AE; Fri, 15 Jun 2007 20:32:14 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from localhost.corp.yahoo.com (john@localhost [127.0.0.1]) (authenticated bits=0) by server.baldwin.cx (8.13.8/8.13.8) with ESMTP id l5FKW4Pi037946; Fri, 15 Jun 2007 16:32:07 -0400 (EDT) (envelope-from jhb@freebsd.org) From: John Baldwin To: Randall Stewart Date: Fri, 15 Jun 2007 13:40:28 -0400 User-Agent: KMail/1.9.6 References: <200706142259.l5EMx4aW053356@repoman.freebsd.org> <20070615022956.GA96741@rot13.obsecurity.org> <4671FA96.1010304@cisco.com> In-Reply-To: <4671FA96.1010304@cisco.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200706151340.29868.jhb@freebsd.org> X-Greylist: Sender succeeded SMTP AUTH authentication, not delayed by milter-greylist-2.0.2 (server.baldwin.cx [127.0.0.1]); Fri, 15 Jun 2007 16:32:07 -0400 (EDT) X-Virus-Scanned: ClamAV 0.88.3/3429/Fri Jun 15 10:25:06 2007 on server.baldwin.cx X-Virus-Status: Clean X-Spam-Status: No, score=-4.4 required=4.2 tests=ALL_TRUSTED,AWL,BAYES_00 autolearn=ham version=3.1.3 X-Spam-Checker-Version: SpamAssassin 3.1.3 (2006-06-01) on server.baldwin.cx Cc: Randall Stewart , src-committers@freebsd.org, cvs-all@freebsd.org, cvs-src@freebsd.org, Kris Kennaway Subject: Re: cvs commit: src/sys/netinet sctp.h sctp_asconf.c sctp_asconf.h sctp_bsd_addr.c sctp_constants.h sctp_indata.c sctp_input.c sctp_lock_bsd.h sctp_os_bsd.h sctp_output.c sctp_pcb.c sctp_pcb.h sctp_peeloff.c sctp_sysctl.c sctp_sysctl.h sctp_timer.c sctp_uio.h ... 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: Fri, 15 Jun 2007 20:32:14 -0000 On Thursday 14 June 2007 10:33:58 pm Randall Stewart wrote: > Kris Kennaway wrote: > > On Thu, Jun 14, 2007 at 10:25:15PM -0400, Randall Stewart wrote: > > > >>Kris Kennaway wrote: > >> > >>>On Thu, Jun 14, 2007 at 10:59:04PM +0000, Randall Stewart wrote: > >>> > >>> > >>>>rrs 2007-06-14 22:59:04 UTC > >>>> > >>>>FreeBSD src repository > >>>> > >>>>Modified files: > >>>> sys/netinet sctp.h sctp_asconf.c sctp_asconf.h > >>>> sctp_bsd_addr.c sctp_constants.h > >>>> sctp_indata.c sctp_input.c > >>>> sctp_lock_bsd.h sctp_os_bsd.h > >>>> sctp_output.c sctp_pcb.c sctp_pcb.h > >>>> sctp_peeloff.c sctp_sysctl.c > >>>> sctp_sysctl.h sctp_timer.c sctp_uio.h > >>>> sctp_usrreq.c sctputil.c sctputil.h > >>>> sys/netinet6 sctp6_usrreq.c > >>>> sys/conf options > >>>>Log: > >>>>- Fix so ifn's are properly deleted when the ref count goes to 0. > >>>>- Fix so VRF's will clean themselves up when no references are around. > >>>>- Allow sctp_ifa to be passed into inpcb_bind, addr_mgmt_ep_sa to bypass > >>>> normal validation checks. > >>>>- turn auto-asconf off for subset bound sockets > >>>>- Moves all logging to use KTR. This gets rid of most > >>>> of the logging #ifdef's with a few exceptions reducing > >>>> the number of config options for SCTP. > >>> > >>> > >>>| +#ifndef SCTP_SUBSYS_KTR > >>>| +#define SCTP_SUBSYS_KTR KTR_GEN > >>>| +#endif > >>> > >>>Brief silence after previous disapproval doesn't equal approval ;-) > >>> > >>>What was wrong with the method I suggested, namely using KTR_SUBSYS if > >>>a SCTP_TRACE option is included in the kernel? That is the intended > >>>way that events local to a particular subsystem should be handled. > >>> > >>>Kris > >> > >>I asked if KTR_GEN was ok.. I can use KTR_SUBSYS.. sure.. but that > >>means I can't really run witness on my machine as I test.. since > >>witness is the only one that uses KTR_SUBSYS.. No one else uses > >>KTR_GEN.. why is it a problem using one. > > > > > > WITNESS does not use itq: > > > > #if 0 > > #define KTR_WITNESS KTR_SUBSYS > > #else > > #define KTR_WITNESS 0 > > #endif > > > > KTR_WITNESS is used for debugging of the WITNESS code itself (probably > > no-one has needed to use it for many years). It is a completely > > orthogonal purpose to yours, so they can co-exist happily. > > > > KTR_GEN is for "general events". SCTP is not a general event, it's > > localized events belonging to a particular subsystem. > > > > > >>So whats wrong with using a unused one? > > > > > > If you look at the history there was a big effort to reclaim these > > fields. Because space is at such a premium here there needs to be a > > strong reason for using up a spare field. SCTP does not strike me as > > such a reason, particularly since the alternative seems quite > > practical. > > > > Kris > Ok.. whatever you say.. I can change it very easily ;-) Also, I would use KTR_SCTP and use the same #ifdef above so it is 0 or KTR_SUBSYS depending on whether or not you want it enabled. -- John Baldwin