mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@digeo.com>
To: Andries.Brouwer@cwi.nl
Cc: linux-kernel@vger.kernel.org, torvalds@osdl.org
Subject: Re: [PATCH] cryptoloop
Date: Wed, 2 Jul 2003 12:58:22 -0700	[thread overview]
Message-ID: <20030702125822.329463f5.akpm@digeo.com> (raw)
In-Reply-To: <UTC200307021942.h62Jg1L20419.aeb@smtp.cwi.nl>

Andries.Brouwer@cwi.nl wrote:
>
> (Usually I plan a track and submit individual steps.
> When they get applied I continue.
> If not, there is not need to waste time on the rest.)

As someone who is on the receiving end of this process I must say that it
is not comfortable.

It really helps to be able to see the whole thing laid out all at the same
time.  So we can see that it works, so that we can see that the end result
is the right one, etc.

Whereas receiving a long drawn out trickle of patches is quite confusing. 
One doesn't know where it is leading, nor where it will end, nor when to
get down and actually start testing it, etc.  Nor whether this ongoing
churn is stomping on someone else's development effort.

And there is the risk that you get halfway through and then suddenly "no
way, we don't want to do that".  Then what?  Argue?  Revert?


So for everyone except the guy who's writing the code it is best to have
all the work in place and reviewable at the same time.

For the author, yes, there is a risk that more code than necessary will be
tossed away.  We can minimise that by discussing things beforehand, getting
understanding and agreement from the relevant people on the intended
direction.  I think we have done that for cryptoloop.  We still have not
really done it for 64-bit dev_t.



  reply	other threads:[~2003-07-02 19:49 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-07-02 19:42 Andries.Brouwer
2003-07-02 19:58 ` Andrew Morton [this message]
  -- strict thread matches above, loose matches on Subject: below --
2003-07-04 13:21 Andries.Brouwer
2003-07-04 13:28 ` Christoph Hellwig
2003-07-04 11:08 Andries.Brouwer
2003-07-04 12:13 ` Christoph Hellwig
2003-07-03 16:25 Andries.Brouwer
2003-07-03 16:31 ` Christoph Hellwig
2003-07-02 22:57 Andries.Brouwer
2003-07-02 22:27 Andries.Brouwer
2003-07-02 21:00 Andries.Brouwer
2003-07-02 21:06 ` Greg KH
2003-07-02 21:31 ` Andrew Morton
2003-07-03 16:23 ` Christoph Hellwig
2003-07-02 18:44 Andries.Brouwer
2003-07-02 19:02 ` Andrew Morton
2003-07-02 19:16   ` Linus Torvalds
2003-07-02 19:20     ` Andrew Morton
2003-07-02 19:31       ` Linus Torvalds
2003-07-03 11:21 ` Jari Ruusu
2003-07-03 15:20   ` Andrew Morton
2003-07-03 17:29     ` Jari Ruusu
2003-07-03 17:38       ` Chris Friesen
2003-07-04  7:43         ` Jari Ruusu
2003-07-04  8:44           ` Andrew Morton
2003-07-04  9:41           ` Christoph Hellwig
2003-07-05  8:41             ` Jari Ruusu
2003-07-05  8:58               ` Andrew Morton
2003-07-05  9:00                 ` Andre Hedrick
2003-07-05  9:10               ` Andre Hedrick
2003-07-05 17:16               ` James Morris
2003-07-05 17:20               ` Linus Torvalds
2003-07-08 12:43               ` Christoph Hellwig
2003-07-04  9:39       ` Christoph Hellwig
2003-07-03 16:20 ` Christoph Hellwig
2003-07-02 15:21 Andries.Brouwer
2003-07-02 17:16 ` Andrew Morton
2003-07-03 15:44 ` Christoph Hellwig

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=20030702125822.329463f5.akpm@digeo.com \
    --to=akpm@digeo.com \
    --cc=Andries.Brouwer@cwi.nl \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@osdl.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

all inboxes | Powered by JetHome®