From owner-freebsd-stable@FreeBSD.ORG Wed Sep 8 12:42:37 2010 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2A0D210656E1; Wed, 8 Sep 2010 12:42:37 +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 EB82B8FC1F; Wed, 8 Sep 2010 12:42:36 +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 902E646B8E; Wed, 8 Sep 2010 08:42:36 -0400 (EDT) Received: from jhbbsd.localnet (smtp.hudson-trading.com [209.249.190.9]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id A06AD8A050; Wed, 8 Sep 2010 08:42:34 -0400 (EDT) From: John Baldwin To: freebsd-stable@freebsd.org, vadim_nuclight@mail.ru Date: Wed, 8 Sep 2010 08:42:28 -0400 User-Agent: KMail/1.13.5 (FreeBSD/7.3-CBSD-20100819; KDE/4.4.5; amd64; ; ) References: <201009011653.o81Grkm4056064@fire.js.berklix.net> <4C8704E3.5000408@FreeBSD.org> In-Reply-To: MIME-Version: 1.0 Message-Id: <201009080842.28495.jhb@freebsd.org> Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.0.1 (bigwig.baldwin.cx); Wed, 08 Sep 2010 08:42:35 -0400 (EDT) X-Virus-Scanned: clamav-milter 0.95.1 at bigwig.baldwin.cx X-Virus-Status: Clean X-Spam-Status: No, score=-2.6 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: freebsd-security@freebsd.org Subject: Re: Policy for removing working code (Was: HEADS UP: FreeBSD 6.4 and 8.0 EoLs coming soon) X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 08 Sep 2010 12:42:37 -0000 On Wednesday, September 08, 2010 6:24:11 am Vadim Goncharov wrote: > > Because the original I4B code didn't > > work without the Giant lock, and because no one stepped forward to fix > > that, the code had to be removed. > > No. The code needn't removal, the stack should be modified to be fast without > I4B and slow for those who wish to compile it with I4B anf Giant. Then slowness > is their problem, not of the Project. No, that would require maintaining two network stacks, not just one. The shims to allow unlocked code to run were not trivial. The choices were this: 1) Moving forward on work to allow the network stack to scale on SMP systems (e.g. modern x86 multi-core servers) and support higher rate protocols such as 10GB, 40GB, and 100GB. 2) Stop all progress on making the network stack scale on SMP. I'm sorry, but 2) just isn't feasible. Not if FreeBSD is to continue to be a modern, relevant system. Also, despite your claims to the contrary, there _was_ adequate notice: http://lists.freebsd.org/pipermail/freebsd-current/2007-June/072977.html This was also documented in the release notes for 7.0: http://www.freebsd.org/releases/7.0R/relnotes.html If you wish to help work on ISDN support, I suggest you offer to test hps@' ISDN stack. hps@ recently became a committer so I think there is a very good chance his code will be brought into the tree. We do have a policy for removing code in that it only gets removed if no one is able to maintain it and/or test patches for it. I locked several of the remaining NIC drivers during the push to remove Giant and a few of them were removed from the system because no one had the hardware around to test the patches to add locking (think of really old ISA NICs that only do 10Mbps). Even in that case, the code will always live on in the source code control repository's history. That means it can always be resurrected if someone shows up who will maintain it and keep it up to date. At this point I think this thread has reached the end of its usefulness. -- John Baldwin