Re: [RFC] HOWTO do Linux kernel development

[Josh Boyer <jwboyer@gmail.com>]

On 11/14/05, Greg KH <gregkh@suse.de> wrote:
> On Mon, Nov 14, 2005 at 02:07:09PM -0800, Greg KH wrote:
> > So, I've been working on a document for the past week or so to help
> > alleviate a lot of these problems.
>
> Oh, the latest version can be found at:
>         http://www.kernel.org/git/?p=linux/kernel/git/gregkh/patches.git;a=blob;f=HOWTO
> as I'm keeping it in my git patch tree.

I don't know if this should go in the <TODO> section of the
development process or somewhere else within "Working with the
community", but I think something along the lines of this might be
helpful:

Leave your ego at home
---------------------------------

The goal of the kernel community is to provide the best possible
kernel there is.  When you submit a patch for acceptance, it will be
reviewed on it's technical merits and those alone.  So, what should
you be expecting?

- criticism,
- comments
- requests for change
- requests justification.

Remember, this is part of getting your patch into the kernel.  You
have to be able to take criticism and comments about your patches,
evaluate them at a technical level and either rework your patches or
provide clear and concise reasoning as to why those changes should not
be made.

What should you not do?

- expect your patch to be accepted without question
- become defensive
- ignore comments and resubmit the patch without making any changes
- explain how your project is funded by XYZ and therefore must be
awesome as it is

In a community that is looking for the best technical solution
possible, there is no place for ego.  You have to be cooperative,
polite, and willing to adapt your idea to fit within the kernel. 
Remember, being wrong is ok as long as you are willing to work toward
a solution that is right.

josh
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/