kernelnewbies.kernelnewbies.org archive mirror
 help / color / mirror / Atom feed
From: xerofoify@gmail.com (Nick Krause)
To: kernelnewbies@lists.kernelnewbies.org
Subject: Bad Patches and Issues with other devolopers
Date: Tue, 5 Aug 2014 16:20:15 -0400	[thread overview]
Message-ID: <CAPDOMVh8OZTuhe6Ag-pJhtKa8qFC8HGPCWNbwgfCSjyXXXsxOA@mail.gmail.com> (raw)
In-Reply-To: <12636.1407268471@turing-police.cc.vt.edu>

On Tue, Aug 5, 2014 at 3:54 PM,  <Valdis.Kletnieks@vt.edu> wrote:
> On Tue, 05 Aug 2014 13:42:58 -0400, Nick Krause said:
>> I have sent out just ten bad patches and the developers seem very
>> annoyed with me and
>
> Let's face it - if you've sent ten bad patches in a row without getting
> one right, you're doing something wrong.  And although total noob coders
> scale very well (there seems to be a never-ending supply of them), maintainers
> don't scale well at all - and they have a *huge* workload to review a lot of
> patches every release cycle.
>
> I can't think of a single maintainer that isn't willing to provide advice.
>
> I also can't think of a single maintainer who *doesn't* get torqued off
> massively when V2 of a patch, or another patch, comes in from the same
> person and it's obvious the advice wasn't listened to.  They don't have
> time for that sort of foolishness.
>
>> think I am trolling. If someone on this list can find a way for me to
>> improve my relationship
>> with them and let me continue my work here that would be great.
>
> First and foremost, when senior kernel developers give you specific advice,
> *listen to it*.  If somebody like Ted T'so tells you that it's unacceptable to
> send patches that aren't compile-tested, *you should never be sending another
> patch that didn't compile clean*.  Period. End Of Discussion.
>
> In fact, you should strive higher - don't submit a patch unless you are
> (a) booted onto the patched kernel, (b) verify it by checking uname -r, and
> (c) have done testing that your patch actually fixed the issue you were
> patching without breaking anything.
>
> Running around willy-nilly submitting patches all over the tree doesn't
> inspire confidence in your patches - especially after you've hit multiple
> subsystems and been told "This is wrong and you obviously (a) don't understand
> the subsystem and (b) didn't bother figuring it out".
>
> Also, you may want to sit down for a few days, and think long and hard
> about *why* you're so desperate to submit kernel patches.  Do you have a
> good reason to devote the time?  Or is it just ego-stroking?  (Personally,
> I've been around since the 2.5.47 or so kernel - and I'm only doing it
> because I have a Dell laptop on my desk and a quarter acre of servers across
> the hall, and lots of users on our campus - and every good bug report I file
> against linux-next means a crappy bug report from a user after the release
> escapes)
>
I want to help and improve the code plus get a code doing kernel development.
I understand now and am not going to waste time anymore, I am going to make
sure all my patches are tested correctly and to the best of my ability first.
Regards NIck

  reply	other threads:[~2014-08-05 20:20 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-05 17:42 Bad Patches and Issues with other devolopers Nick Krause
2014-08-05 17:56 ` Kristofer Hallin
2014-08-05 17:59 ` Mandeep Sandhu
2014-08-05 18:04   ` Nick Krause
2014-08-05 18:28     ` Anuz Pratap Singh Tomar
2014-08-05 18:25 ` Sudip Mukherjee
2014-08-05 19:52   ` Rohan Puri
2014-08-05 19:54 ` Valdis.Kletnieks at vt.edu
2014-08-05 20:20   ` Nick Krause [this message]
     [not found]     ` <alpine.LFD.2.11.1408051702460.26301@localhost>
2014-08-05 21:35       ` Nick Krause
2014-08-05 22:49         ` Greg Freemyer
2014-08-05 23:43           ` Nick Krause
2014-08-06  9:48             ` Anuz Pratap Singh Tomar
2014-08-06 10:02               ` Pramod Gurav
2014-08-06 10:30                 ` Anuz Pratap Singh Tomar
2014-08-06 12:47                   ` Nick Krause
2014-08-06 13:25                     ` Nick Krause
2014-08-06 13:28                       ` Kristofer Hallin
2014-08-06 13:32                         ` Nick Krause
2014-08-06 13:45                           ` Sudip Mukherjee
2014-08-06 13:46                           ` Anuz Pratap Singh Tomar
2014-08-06 13:56                             ` Nick Krause
2014-08-06 13:59                               ` Andev
2014-08-06 14:07                                 ` Nick Krause
2014-08-06 14:22                                   ` Nick Krause
2014-08-06 14:30                                     ` Kristofer Hallin
2014-08-06 13:35                       ` Robert P. J. Day
2014-08-06 16:31                         ` Josh Carlson
2014-08-06 16:47                           ` Nick Krause
2014-08-06 17:11                             ` Greg Freemyer
2014-08-06 17:16                               ` Nick Krause
2014-08-06 16:52                           ` Mandeep Sandhu
2014-08-06 16:58                             ` Nick Krause
2014-08-06 16:58               ` Valdis.Kletnieks at vt.edu
2014-08-06 17:03                 ` Nick Krause
2014-08-06 17:24                 ` Robert P. J. Day
2014-08-06 18:05                   ` Nick Krause
2014-08-06 18:08                     ` Nick Krause
2014-08-06 18:30                     ` Lidza Louina
2014-08-06 18:33                       ` Nick Krause
2014-08-06 18:42                         ` Lidza Louina
2014-08-06 18:45                           ` Nick Krause
2014-08-06 18:53                             ` Kristofer Hallin
2014-08-06 18:56                               ` Nick Krause
2014-08-06 20:45                               ` StephanT

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=CAPDOMVh8OZTuhe6Ag-pJhtKa8qFC8HGPCWNbwgfCSjyXXXsxOA@mail.gmail.com \
    --to=xerofoify@gmail.com \
    --cc=kernelnewbies@lists.kernelnewbies.org \
    /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).