Date: Tue, 15 Nov 2005 09:33:07 -0800
From: Greg KH <gregkh@suse.de>
To: Andi Kleen <ak@suse.de>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [RFC] HOWTO do Linux kernel development
Message-ID: <20051115173307.GB13707@suse.de>
References: <20051114220709.GA5234@kroah.com> <p73veyu2crf.fsf@verdi.suse.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <p73veyu2crf.fsf@verdi.suse.de>
User-Agent: Mutt/1.5.11
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1947
Lines: 48

On Tue, Nov 15, 2005 at 03:42:44PM +0100, Andi Kleen wrote:
> Greg KH <gregkh@suse.de> writes:
> 
> > The kernel is written using GNU C and the GNU toolchain. While it
> > adheres to the ISO C99 (??) standard, it uses a number of extensions
> 
> C89 - The few left over gcc 2.95 users are blocking modern C constructs.
> Even without that it would be a C99 subset, e.g. arbitary long long divisions 
> or floating point are not supported.
> 
> Also the kernel is a freestanding C environment, so parts are not supported.

Thanks, I've modified it to mention this.

> > Also realize that it is not acceptable to send patches for inclusion
> > that are unfinished and will be "fixed up later."
> 
> I'm not sure I fully agree on that. I conflicts with the "merge early, merge
> often" imperative.  IMHO it's ok to submit patches that are not perfect,
> but improve something or make a incremental cleanup step, as long as the
> problems are not severe and the patch by itself is a clear improvement. Of course
> this is handled on a case by case basis.

Yeah, it is a case-by-case, but generally we want to know up front what
the final result is going to be.

> > Justify your change
> > -------------------
> > 
> > Along with breaking up your patches, it is very important for you to let
> > the Linux community know why they should add this change.  New features
> > must be justified as being needed and useful.
> 
> My request is that each patch should carry a meaningful changelog.
> That should tell why and a rough (doesn't need to be detailed) overview how
> the change is done.

Others privately commented on this too, and I've added a section
describing it.

Thanks for the comments.

greg k-h
-
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/