Patches and reviews

Patches and reviews

[Nathan Scott]

Hi all,

We have an internal process here (@sgi) where all changes are
reviewed by someone else in the team before we merge them into
our local tree.  This is good and effective, but it obviously
isn't visible to people outside SGI.  Christoph has suggested
we try to change this, so we plan to start sending out patches
here before they are merged into our local tree, in the hope
that others outside SGI will also take some time and cast a
critical eye over changes before they go in.

So, this is just a heads up that there will now be occasional
mail coming to the list with "review: ..." in the Subject line
and you're encouraged to try spot the bugs I'll "purposefully"
inject from time to time, just to check you're awake. ;-)  A
response like "Beaauutiful code, Tim" is also helpful (recall
that our internal process requires an ACK).

I also now plan to start CC'ing the list on the mails I send to
Linus and Andrew, requesting our XFS updates be merged into the
mainline kernel.  Hopefully this will also increase visibility
everyone has as to what's going on in XFS with each new kernel.

If you have other suggestions, particularly of things that are
not going to increase my current workload at all, I'm all ears.

cheers.

-- 
Nathan