*BSD News Article 33766


Return to BSD News archive

Xref: sserve comp.os.386bsd.misc:3007 comp.os.linux.misc:20984
Path: sserve!newshost.anu.edu.au!harbinger.cc.monash.edu.au!msuinfo!netnews.upenn.edu!dsinc!newsfeed.pitt.edu!gatech!newsxfer.itd.umich.edu!uunet!uunet.ca!uunet.ca!fw.novatel.ca!sidney.novatel.ca!hpeyerl
From: hpeyerl@sidney.novatel.ca (Herb Peyerl)
Newsgroups: comp.os.386bsd.misc,comp.os.linux.misc
Subject: Re: source of TCP/IP (was I hope this wont ignite a major flame ...)
Followup-To: comp.os.386bsd.misc,comp.os.linux.misc
Date: 3 Aug 1994 15:31:25 GMT
Organization: NovAtel Communications Ltd.
Lines: 30
Message-ID: <31od8d$15l@fw.novatel.ca>
References: <3163r7$440@quagga.ru.ac.za> <CtMp4G.7Ap@calcite.rhyolite.com> <31bl91$3b9@cesdis1.gsfc.nasa.gov> <CtqrFJ.IM5@calcite.rhyolite.com> <31fd07$6pq@cesdis1.gsfc.nasa.gov>
NNTP-Posting-Host: sidney.novatel.ca
X-Newsreader: TIN [version 1.2 PL1]

Donald Becker (becker@cesdis.gsfc.nasa.gov) wrote:
: Errr, I guess you haven't read any of the Linux source.  Hint: I wasn't
: worried about at least one of those organizations.

Just to make a minor branch on this discussion...

I've tried reading Linux networking code. At least some of the device 
drivers and each of the ones I looked at gave me a brain hemorrage...

This is an example of some of the Linux device-drivers I've seen:

        short error = rx_status & 0x3C00;
        outw(inw(ioaddr + 0x0A) | 0x00C0, ioaddr + 0x0A);

As far as I can tell; Linux Ethernet device-drivers were written in
Write-Only-C.  There are no comments in the surrounding code that in any
way indicates exactly what "0x3c00", "0x0a", "0x00c0" actually mean. To
people without docs (usually these are the people who are trying to fix
the code) the above is completely meaningless.

Thank you; I'll stick to working on code that I can actually read.

Note; this is not meant to ignite a flamefest.  I just want to point out
that not everything is perfect everywhere... Yes; I admit to having written
buggy code too. But at least other people can read it.

--
hpeyerl@novatel.ca                           |  NovAtel Commnications Ltd.
hpeyerl@fsa.ca                               | <nothing I say matters anyway>
 "A sucking chest wound is nature's way of telling you to slow down."