From: Neil Brown <neilb@suse.de>
To: heinzm@redhat.com
Cc: Christoph Hellwig <hch@infradead.org>,
Dan Williams <dan.j.williams@intel.com>,
device-mapper development <dm-devel@redhat.com>,
Ed Ciechanowski <ed.ciechanowski@intel.com>
Subject: Re: [PATCH 1/6] dm raid45 target: export region hash functions and add a needed one
Date: Wed, 17 Jun 2009 08:46:28 +1000 [thread overview]
Message-ID: <19000.8388.899607.749942@notabene.brown> (raw)
In-Reply-To: message from Heinz Mauelshagen on Tuesday June 16
On Tuesday June 16, heinzm@redhat.com wrote:
>
> That being said: once the future work on a unified virtual block device
> infrastructure is production ready, we're open to use that.
>
I was kind-a hoping that you (and others) would be involved in
developing this unified infrastructure, rather than just waiting for
it.
I think a great first step would be to allow md/raid5 to be used
directly as a dm target, thus turning dm-raid5 into a shim layer over
md/raid5. The process of doing this would very likely highlight a
lot of the issues we would need to address in creating a unified
framework.
I will try to find time to review your dm-raid5 code with a view to
understanding how it plugs in to dm, and then how the md/raid5 engine
can be used by dm-raid5.
Part of this will be disconnecting the md/raid5 code from any specific
knowledge of a gendisk and a request_queue as I suppose a dm-target
doesn't own any of these. Also I would probably want the mddev not be
have to be on the "all_mddevs" list, as we would not want a 'dm' raid5
to appear in /proc/mdstat.
NeilBrown
next prev parent reply other threads:[~2009-06-16 22:46 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-15 17:21 [PATCH 1/6] dm raid45 target: export region hash functions and add a needed one heinzm
2009-06-16 14:09 ` Christoph Hellwig
2009-06-16 14:51 ` Heinz Mauelshagen
2009-06-16 17:55 ` Dan Williams
2009-06-16 19:11 ` Heinz Mauelshagen
2009-06-16 19:48 ` Dan Williams
2009-06-16 22:46 ` Neil Brown [this message]
2009-06-18 16:08 ` Jonathan Brassow
2009-06-19 1:43 ` Neil Brown
2009-06-19 10:33 ` Heinz Mauelshagen
2009-06-21 0:32 ` Dan Williams
2009-06-21 12:06 ` Neil Brown
2009-06-22 12:25 ` Neil Brown
2009-06-22 19:10 ` Heinz Mauelshagen
2009-07-02 12:52 ` Heinz Mauelshagen
2009-07-06 3:21 ` Neil Brown
2009-07-07 18:38 ` Doug Ledford
2009-07-10 15:23 ` Heinz Mauelshagen
2009-07-11 12:44 ` Doug Ledford
2009-07-12 2:56 ` Dan Williams
2009-07-08 18:56 ` Heinz Mauelshagen
2009-06-18 16:39 ` Jonathan Brassow
2009-06-18 20:01 ` Heinz Mauelshagen
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=19000.8388.899607.749942@notabene.brown \
--to=neilb@suse.de \
--cc=dan.j.williams@intel.com \
--cc=dm-devel@redhat.com \
--cc=ed.ciechanowski@intel.com \
--cc=hch@infradead.org \
--cc=heinzm@redhat.com \
/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