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
next prev parent 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).