The Unix Heritage Society mailing list
 help / color / mirror / Atom feed
From: Larry McVoy <lm@mcvoy.com>
To: Theodore Ts'o <tytso@mit.edu>
Cc: "tuhs@minnie.tuhs.org" <tuhs@minnie.tuhs.org>
Subject: Re: [TUHS] CMU Mach sources?
Date: Mon, 24 Jun 2019 17:45:23 -0700	[thread overview]
Message-ID: <20190625004523.GB7655@mcvoy.com> (raw)
In-Reply-To: <20190625003120.GA28608@mit.edu>

On Mon, Jun 24, 2019 at 08:31:21PM -0400, Theodore Ts'o wrote:
> On Mon, Jun 24, 2019 at 05:06:30PM -0700, Larry McVoy wrote:
> > All interesting points but messy code is messy code.  I had a bunch of the
> > FreeBSD folks over here for a BBQ a couple days ago (they want you at the
> > next one Clem).  We got to talking about Mach and someone told me that in
> > the FreeBSD tree the Mach code was gone through and 60% of was deleted and
> > it still worked.  It just seems like the Mach folks wanted to try this,
> > and that, and then next thing and never went back to clean up the mess.
> 
> Welcome to academic/research code.  :-)

Like I said, I can point anyone at code I wrote as a grad student that 
while I'm not proud of the style, it has style and it is clean.  Just
because you are a grad student that doesn't excuse messy code.  If you
write messy code then you're a bad hire.

> I'm reminded of a description of the Coda File System by Peter Braam;
> he said that it was irretrivably tainted by a dozen Ph.D. students
> working on their thesis.  Naturally, once they had done the necessary
> work for them to get their doctorate, any interest in doing the
> necessary code cleanup for their various experimental efforts
> evaporated.  

Yeah, like I said, bad hires.  People who are good coders take pride
in their work.  They put in the extra time to clean it up.  That's 
why SunOS 4.x was a nice code base, everyone pulled their weight 
to make it be so.  I get that that is unusual but it is super nice
when it happens.  

And I wasn't trying to belittle the Mach effort, I'm impressed with
what it does.  I am most definitely belittling the people who did it.
Not because of what they accomplished, that's cool, but they didn't care
enough to clean it up.  That sucks.  And that means they suck as 
professional programmers.

I'm a canoe guy and any canoe guy knows that the ultimate insult is
"I wouldn't want him in my boat."  Well, I wouldn't want the Mach
people, for all their talent and accomplishments, on my team.  I
like people who get the job done, all the way done.  Code is clean,
the docs cover the code, the test cases are there.  Done done.

I just don't buy that academic/research code needs to be bad.  If
the people doing it are people you'd want to hire, they get it 
done done.  I get that I'm describing a unicorn but I was one,
and I'm not that great.  Doesn't seem so much to ask that people
give a shit and do it right.

  reply	other threads:[~2019-06-25  0:45 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-23  4:38 Chris Hanson
2019-06-23  5:15 ` Larry McVoy
2019-06-23  8:52   ` Andrew Warkentin
2019-06-23 13:39   ` Jon Forrest
2019-06-23 13:59     ` arnold
2019-06-23 14:03     ` Jason Stevens
2019-06-23  8:04 ` Jason Stevens
2019-06-23 14:54   ` Henry Bent
2019-06-23 21:52     ` Clem Cole
2019-06-25  0:06       ` Larry McVoy
2019-06-25  0:31         ` Theodore Ts'o
2019-06-25  0:45           ` Larry McVoy [this message]
2019-06-25  0:55             ` Kurt H Maier
2019-06-25  4:18               ` Larry McVoy
2019-06-26 23:19                 ` [TUHS] Craft vs Research (Re: " Bakul Shah
2019-06-27  0:16                   ` tuhs
2019-06-27 17:06                     ` Clem Cole
2019-06-25  1:00             ` [TUHS] " Richard Salz
2019-06-25  8:00               ` Kevin Bowling
2019-06-25 12:11                 ` Arthur Krewat
2019-06-25 12:17                   ` Arthur Krewat
2019-06-26  2:45               ` Kurt H Maier
2019-06-26  2:56                 ` Larry McVoy
2019-06-26 15:11                   ` Theodore Ts'o
2019-06-26 17:44                     ` Larry McVoy
2019-06-26 18:01                       ` arnold
2019-06-26 18:18                         ` Warner Losh
2019-06-26 19:22                       ` Chris Hanson
2019-06-26 19:32                         ` Ben Greenfield via TUHS
2019-06-26 20:21                           ` Larry McVoy
2019-06-27  0:22                             ` Chris Hanson
2019-06-27  1:02                               ` Larry McVoy
2019-06-27  1:26                                 ` Chris Hanson
2019-06-27  4:01                             ` Lyndon Nerenberg
2019-06-27 10:34                               ` Ben Greenfield via TUHS
2019-06-27 10:59                                 ` arnold
2019-06-27 11:13                                   ` Ben Greenfield via TUHS
2019-06-27 11:39                                     ` arnold
2019-06-27 14:58                                     ` Warner Losh
2019-06-27 17:25                                       ` Larry McVoy
2019-06-26 19:30                       ` Dennis Boone
2019-06-26 19:25                     ` Adam Thornton
2019-06-23  8:27 ` Kevin Bowling
2019-06-25  3:07 ` Gregg Levine
2019-06-25  8:15   ` Kevin Bowling
2019-06-25 18:18   ` Chris Hanson
2019-06-25 20:23     ` Gregg Levine
2019-06-26  1:04       ` Jason Stevens
2019-06-26  0:53     ` Jason Stevens
2019-06-25  7:49 ` Jason Stevens
2019-06-25  7:59   ` Andreas Grapentin
2019-06-23 22:08 Noel Chiappa
2019-06-23 23:54 ` Theodore Ts'o
2019-06-24 17:04   ` Jason Stevens
2019-07-01 13:20 Jason Stevens

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190625004523.GB7655@mcvoy.com \
    --to=lm@mcvoy.com \
    --cc=tuhs@minnie.tuhs.org \
    --cc=tytso@mit.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).