From: Linus Torvalds <torvalds@linux-foundation.org>
To: Bernd Schmidt <bernds_cb1@t-online.de>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Blackfin Architecture Team
<uclinux-dist-devel@blackfin.uclinux.org>,
David Howells <dhowells@redhat.com>,
stable@kernel.org
Subject: Re: Fix for shared flat binary format in 2.6.30
Date: Mon, 22 Jun 2009 11:44:08 -0700 (PDT) [thread overview]
Message-ID: <alpine.LFD.2.01.0906221137580.3240@localhost.localdomain> (raw)
In-Reply-To: <4A3F5EAE.7020402@t-online.de>
On Mon, 22 Jun 2009, Bernd Schmidt wrote:
>
> This fixes a crash in 2.6.30 with shared flat binaries. prepare_binfmt now
> requires a cred pointer to be set up, which isn't done in binfmt_flat.c.
This has so many bugs in patch submission that I literally almost gave up
despite it being a trivial patch.
Attachements, wrong separators between explanation, wrong patch format
(hint: use "-p1" or some sane patch system that gets things right
_without_ crap), you name it. Very irritating.
I'm generally not interested in cleaning up after people who can't bother
to try to make patches in the right format. If you can't bother to spend
the small amount of time to make sure the patch is properly formatted, why
should I bother to apply it?
It literally looks like you used SVN to generate that sh*t-for-brains
patch. If you have to use svn, at least learn to make it generate proper
patches.
(How? I don't know. Maybe "svn diff -p1" works? Why would _anybody_ ever
use SVN for the kernel anyway? The "S" stands for "Substandard").
I applied this patch, but I want to just vent my frustration with it.
Linus
next prev parent reply other threads:[~2009-06-22 18:46 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-22 10:36 Bernd Schmidt
2009-06-22 18:44 ` Linus Torvalds [this message]
2009-06-22 18:56 ` Linus Torvalds
2009-06-22 19:35 ` [PATCH] FLAT: fix uninitialized ptr with shared libs Mike Frysinger
2009-06-22 20:48 ` Linus Torvalds
2009-06-22 21:00 ` [Uclinux-dist-devel] " Mike Frysinger
2009-06-23 10:14 ` Bernd Schmidt
2009-06-29 23:44 ` [stable] " Greg KH
2009-06-22 20:39 ` Fix for shared flat binary format in 2.6.30 John Stoffel
2009-06-22 21:06 ` Linus Torvalds
2009-06-23 1:59 ` John Stoffel
2009-06-23 2:44 ` Linus Torvalds
2009-06-25 7:12 ` Junio C Hamano
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=alpine.LFD.2.01.0906221137580.3240@localhost.localdomain \
--to=torvalds@linux-foundation.org \
--cc=bernds_cb1@t-online.de \
--cc=dhowells@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=stable@kernel.org \
--cc=uclinux-dist-devel@blackfin.uclinux.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
Powered by JetHome