public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Russell King <rmk@arm.linux.org.uk>
To: Arjan van de Ven <arjanv@redhat.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [LARGE patch 23/124] sets sent over and over again Re: [PATCH] ext2/3 updates for 2.5.44 (1/11): Default mount options in superblock
Date: Sun, 20 Oct 2002 11:31:35 +0100	[thread overview]
Message-ID: <20021020113135.A25278@flint.arm.linux.org.uk> (raw)
In-Reply-To: <1035108575.3130.10.camel@localhost.localdomain>; from arjanv@redhat.com on Sun, Oct 20, 2002 at 12:09:35PM +0200

On Sun, Oct 20, 2002 at 12:09:35PM +0200, Arjan van de Ven wrote:
> I hereby politely ask EVERYONE who wants to (re)posts large patchsets,
> to at minimum try to follow something like the following politeness
> guidelines
> 
> 1) Make it ONE thread. Do this by cc or bcc'ing yourself on the mails
>    and use the reply feature of your mailer to reply each next number of
>    the set to the previous one. This allows people that use mail/news
>    readers that can do threading to properly sort it. This is not hard,
>    and I consider it the least you can do for the people that read lklm.

It would be nice if someone scripted this - then people will be much more
likely to follow it.  It should be relatively trivial to script; you
just need to generate the message id's and add the relevant headers.

I'd like to question the appropriateness of such a blanket rule.  I agree
that it is appropriate for patches that are all part of the same area of
the kernel (eg, ext2fs, ext3fs, trace toolkits, etc)

However, is it appropriate to make one thread of a small set of unrelated
patches that touch different, unrelated parts of the kernel?

If all you want to do is delete them, I agree it does.  However, that
doesn't help the sender, who's reason for sending them is to get comments
from the community.

For instance, one of my patches - the rdunzip one.  It would be _really_
nice to get some feedback on it; it isn't perfect, because the behaviour
of gunzip is inherently undeterministic when given bad input data.  The
only real solution IMHO is setjmp/longjmp, which I think would suck in
the kernel.  I would have expected _this_ to attract some comments from
people like you.  Maybe you feel that setjmp/longjmp is an approprate
solution.  Unfortunately, I don't know that because no one has replied
to tell me so.

Maybe very few people look at them, I don't know.  If that is the case,
I might as well send them directly to Linus and bypass lkml altogether.

-- 
Russell King (rmk@arm.linux.org.uk)                The developer of ARM Linux
             http://www.arm.linux.org.uk/personal/aboutme.html


  parent reply	other threads:[~2002-10-20 10:25 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-10-20  9:35 [PATCH] ext2/3 updates for 2.5.44 (1/11): Default mount options in superblock tytso
2002-10-20 10:09 ` [LARGE patch 23/124] sets sent over and over again " Arjan van de Ven
2002-10-20 10:13   ` Jens Axboe
2002-10-20 10:31   ` Russell King [this message]
2002-10-20 10:35     ` Arjan van de Ven
2002-10-20 11:07     ` longjmp/setjmp in kernel Keith Owens
2002-10-20 12:39   ` [LARGE patch 23/124] sets sent over and over again Re: [PATCH] ext2/3 updates for 2.5.44 (1/11): Default mount options in superblock Nicholas Wourms
2002-10-20 16:44   ` Richard Gooch

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=20021020113135.A25278@flint.arm.linux.org.uk \
    --to=rmk@arm.linux.org.uk \
    --cc=arjanv@redhat.com \
    --cc=linux-kernel@vger.kernel.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