lkml.org 
[lkml]   [2009]   [Sep]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC][PATCH] SCHED_EDF scheduling class
On 09/23/2009 03:40 AM, Daniel Walker wrote:
> I am trying to make the kernel (or community) better, but my motivation
> is purely my own (as it often is with people on this list) ..
>
> I don't see the effects your indicating, discouraging contributions .. I
> think if anything I'm helping people to get to a place where they can
> contribute in a comfortable way without worrying about things like
> checkpatch .. I think the problem most submitters have on this list is
> that no one tells them they are doing something wrong so they don't
> correct it. I clearly recall submitting my first several patches
> copy-and-pasted which got either ignored , or caused a maintainer lots
> of pain .. Had I been told earlier that it was a problem I would have
> corrected it immediately (and been happy to have been notified about
> it)..
>
>

Look at the context. Here we have a new scheduling class introduced out
of the blue. The discussion should be around whether it makes sense; if
it does, whether the design is good or not; if it is, whether the code
is maintainable or not, if that passes, whether there are bugs in the
code or not. Until we've decided that Linux needs this scheduling class
and that this patchset is well designed and written, spelling errors are
just a nuisance and distract people from the important issues.

> discouraging contributions is more something that happens when you get
> the responses I got earlier in this thread..
>

That's probably intentional. Whitespace fixes have their place but not
at this stage in a patch's lifecycle.

--
error compiling committee.c: too many arguments to function



\
 
 \ /
  Last update: 2009-09-23 13:51    [W:0.811 / U:0.480 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site