Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 8 Sep 2010 08:42:28 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        freebsd-stable@freebsd.org, vadim_nuclight@mail.ru
Cc:        freebsd-security@freebsd.org
Subject:   Re: Policy for removing working code (Was: HEADS UP: FreeBSD 6.4 and 8.0 EoLs coming soon)
Message-ID:  <201009080842.28495.jhb@freebsd.org>
In-Reply-To: <slrni8ep2a.2h26.vadim_nuclight@kernblitz.nuclight.avtf.net>
References:  <201009011653.o81Grkm4056064@fire.js.berklix.net> <4C8704E3.5000408@FreeBSD.org> <slrni8ep2a.2h26.vadim_nuclight@kernblitz.nuclight.avtf.net>

next in thread | previous in thread | raw e-mail | index | archive | help
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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201009080842.28495.jhb>