From owner-svn-src-all@FreeBSD.ORG Fri Jan 8 13:47:16 2010 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id CFDF3106566C; Fri, 8 Jan 2010 13:47:16 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from cyrus.watson.org (cyrus.watson.org [65.122.17.42]) by mx1.freebsd.org (Postfix) with ESMTP id A02A88FC14; Fri, 8 Jan 2010 13:47:16 +0000 (UTC) Received: from bigwig.baldwin.cx (66.111.2.69.static.nyinternet.net [66.111.2.69]) by cyrus.watson.org (Postfix) with ESMTPSA id 5A2DA46B23; Fri, 8 Jan 2010 08:47:16 -0500 (EST) Received: from jhbbsd.localnet (smtp.hudson-trading.com [209.249.190.9]) by bigwig.baldwin.cx (Postfix) with ESMTPA id 60D7F8A01F; Fri, 8 Jan 2010 08:47:15 -0500 (EST) From: John Baldwin To: Luigi Rizzo Date: Fri, 8 Jan 2010 08:12:21 -0500 User-Agent: KMail/1.12.1 (FreeBSD/7.2-CBSD-20091103; KDE/4.3.1; amd64; ; ) References: <201001041825.o04IPcXb043347@svn.freebsd.org> <20100107203536.GB8230@rincewind.paeps.cx> <20100107214334.GA35184@onelab2.iet.unipi.it> In-Reply-To: <20100107214334.GA35184@onelab2.iet.unipi.it> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201001080812.21124.jhb@freebsd.org> X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.0.1 (bigwig.baldwin.cx); Fri, 08 Jan 2010 08:47:15 -0500 (EST) X-Virus-Scanned: clamav-milter 0.95.1 at bigwig.baldwin.cx X-Virus-Status: Clean X-Spam-Status: No, score=-2.5 required=4.2 tests=AWL,BAYES_00 autolearn=ham version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on bigwig.baldwin.cx Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Michael Tuexen Subject: Re: moving sctp to a separate directory ? (Re: svn commit: r201523 - head/sys/netinet) X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 08 Jan 2010 13:47:16 -0000 On Thursday 07 January 2010 4:43:34 pm Luigi Rizzo wrote: > > What do you do with udp, for instance? Compared to tcp and sctp, it's > > trivial in terms of code, but it's an upper layer protocol from the > > perspective of netinet/netinet6 - do we put it in its own directory too? > > Also note that this won't only cause churn for people who have patches against > > or (out-of-tree) branches from netinet/, but also in other kernel subsystems > > which rely on tcp -- nfs, for instance. > > + i find the concern about churn in external patchsets a bit weak, first of > all because this is bound to happen unless we stop all development, > and secondly because this kind of file moving or splitting happens > once every 10-15 years which is well beyond the lifetime of a patchset. Having the files rename is entirely different from merging changes. At least for svn and p4 I believe that merging a rename into a branch is not smart enough to merge your local changes into the new files. Instead it involves a big manual fixup. Also, the 10-15 years thing is completely non-relevant. What is relevant is if you are working on a project in a branch and someone renames files before you have finished your branch and merged it up to HEAD. For example, assume that someone else renamed the ipfw files in HEAD next week. That would create an utter mess for you to resolve in your current ipfw3 branch. Moving TCP would create similar a headache, except much more widespread since TCP is one of the most widely worked-on subsystems. FWIW, I do think it would be cleaner to have netinet more split up perhaps, but I do not think it is worth the pain that would be involved. -- John Baldwin