All of lore.kernel.org
 help / color / mirror / Atom feed
From: Heiko Carstens <heiko.carstens@de.ibm.com>
To: Andrew Morton <akpm@linux-foundation.org>,
	Alasdair G Kergon <agk@redhat.com>,
	dm-devel@redhat.com, Neil Brown <neilb@suse.de>
Cc: linux-kernel@vger.kernel.org
Subject: Re: -mm merge plans for 2.6.21 -- md-dm-reduce-stack-usage-with-stacked-block-devices.patch
Date: Sat, 10 Feb 2007 10:58:58 +0100	[thread overview]
Message-ID: <20070210095858.GA8145@osiris.boeblingen.de.ibm.com> (raw)
In-Reply-To: <20070208150710.1324f6b4.akpm@linux-foundation.org>

On Thu, Feb 08, 2007 at 03:07:10PM -0800, Andrew Morton wrote:
> drivers-mdc-use-array_size-macro-when-appropriate.patch
> md-dm-reduce-stack-usage-with-stacked-block-devices.patch
>  
>  -> neilb
>  
>  (The second one is getting idiotic.  When are we going to fix this??)

Since it was me who asked to have the stack usage reduced and Neil was
kind enough to fix this, I'm wondering what the state of dm is.

I think more than a year ago we had this:

Alasdair G Kergon <agk@redhat.com> said:

 I can see nothing wrong with this in principle.

 For device-mapper at the moment though it's essential that, while the bio
 mappings may now get delayed, they still get processed in exactly
 the same order as they were passed to generic_make_request().

 My main concern is whether the timing changes implicit in this patch
 will make the rare data-corrupting races in the existing snapshot code
 more likely. (I'm working on a fix for these races, but the unfinished
 patch is already several hundred lines long.)

 It would be helpful if some people on this mailing list would test
 this patch in various scenarios and report back.

Alasdair, is dm already fixed or is there any chance that it will
ever get fixed?

  parent reply	other threads:[~2007-02-10 10:00 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-08 23:07 -mm merge plans for 2.6.21 Andrew Morton
2007-02-08 23:12 ` Roland Dreier
2007-02-08 23:29 ` Jan Engelhardt
2007-02-08 23:44   ` Andrew Morton
2007-02-09 15:02     ` Thomas Gleixner
2007-02-09 10:57   ` Frederik Deweerdt
2007-02-09 11:24     ` Arjan van de Ven
2007-02-09 11:39       ` Andrew Morton
2007-02-09 12:32         ` Arjan van de Ven
2007-02-09 14:05           ` deweerdt
2007-02-09 13:04         ` Andi Kleen
2007-02-09 12:27           ` Jan Engelhardt
2007-02-10 11:42       ` Arnd Bergmann
2007-02-10 14:19         ` Frederik Deweerdt
2007-02-08 23:34 ` Kyle McMartin
2007-02-08 23:53   ` Andrew Morton
2007-02-09  0:55     ` Paul Mackerras
2007-02-09  1:00       ` Andrew Morton
2007-02-09  5:32 ` Bharata B Rao
2007-02-09  8:26   ` Sébastien Dugué
2007-02-09  9:05     ` Andrew Morton
2007-02-09 10:10       ` Sébastien Dugué
2007-02-09  9:54 ` Lenar Lõhmus
2007-02-09 10:12   ` Andrew Morton
2007-02-09 12:48     ` James
2007-02-09 12:59     ` Lenar Lõhmus
2007-02-09 17:35 ` David Woodhouse
2007-02-09 21:45   ` Andrew Morton
2007-02-09 21:49     ` Russell King
2007-02-09 21:53       ` David Woodhouse
2007-02-09 22:03         ` Russell King
2007-02-09 22:12           ` David Woodhouse
2007-02-09 22:42             ` David Woodhouse
2007-02-10  2:05           ` Oleg Verych
2007-02-09 22:00       ` Andrew Morton
2007-02-09 22:06         ` Russell King
2007-02-09 21:59     ` David Woodhouse
2007-02-09 22:50       ` Davide Libenzi
2007-02-10 10:22         ` Heiko Carstens
2007-02-10 10:32           ` David Woodhouse
2007-02-10 21:34             ` Ralf Baechle
2007-02-11  4:53               ` Davide Libenzi
2007-02-11 15:33               ` David Woodhouse
2007-02-11 16:09                 ` Ralf Baechle
2007-02-11 16:14               ` Heiko Carstens
2007-02-11 16:34                 ` Davide Libenzi
2007-02-11 18:01                 ` Ralf Baechle
2007-02-10 21:05           ` Ralf Baechle
2007-02-11 10:37             ` Andi Kleen
2007-02-10 13:03   ` Andi Kleen
2007-02-09 19:37 ` Alan
2007-02-09 21:51   ` Andrew Morton
2007-02-10  1:15     ` Carl-Daniel Hailfinger
2007-02-10  1:29       ` Andrew Morton
2007-02-10 13:06   ` Andi Kleen
2007-02-10 13:48     ` Alan
2007-02-10 14:43       ` Andi Kleen
2007-02-12 20:56     ` Doug Thompson
2007-02-12 21:46       ` Andi Kleen
2007-02-12 22:45         ` Doug Thompson
2007-02-09 22:18 ` Guillaume Chazarain
2007-02-10  9:58 ` Heiko Carstens [this message]
2007-02-10 22:35   ` -mm merge plans for 2.6.21 -- md-dm-reduce-stack-usage-with-stacked-block-devices.patch Alasdair G Kergon
2007-02-10 22:35     ` Alasdair G Kergon
2007-02-11  0:31 ` -mm merge plans for 2.6.21 Dave Jones

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=20070210095858.GA8145@osiris.boeblingen.de.ibm.com \
    --to=heiko.carstens@de.ibm.com \
    --cc=agk@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=dm-devel@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=neilb@suse.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.