cvs commit: src/sys/net radix.c

Brian Fundakowski Feldman green at FreeBSD.org
Wed Apr 21 23:43:25 PDT 2004


Darren Reed <darrenr at hub.freebsd.org> wrote:
> On Thu, Apr 22, 2004 at 02:59:38AM +0000, Max Laier wrote:
> > > In the end, if someone decides that the code needs to be restructured
> > > for better performance in SMP hardware, then that's what'll happen and
> > > changes like these will have little significance at that point in time.
> > 
> > I don't think so. It's much easier to restructure code you can actually read 
> > and understand. And honest, when one sees the current radix code (etc.), the 
> > first thing that comes to mind is "run!". Same applies to IPv6/KAME-code, btw 
> > - and I didn't see anybody to work on the locking there and I know a reason 
> > why.
> 
> Now now, don't be mean :)
> 
> But don't forget that at least the way the radix code "was" had the
> backing of a number of chapters in a book that explain(ed/s) it all
> and so it has an advantage over IPv6/KAME...

We're not Unix; we're FreeBSD+KAME+abazillionotherthings, and whatever 
advantages Stevens' historical Unix had in the way of book documentation, 
you can count on it being out of date when it comes to what the code 
actually looks like, whether Luigi is "cleaning it up" or not.

-- 
Brian Fundakowski Feldman                           \'[ FreeBSD ]''''''''''\
  <> green at FreeBSD.org                               \  The Power to Serve! \
 Opinions expressed are my own.                       \,,,,,,,,,,,,,,,,,,,,,,\




More information about the cvs-src mailing list