From: Alex Elsayed <eternaleye+usenet@gmail.com>
To: linux-btrfs@vger.kernel.org
Cc: linux-bcache@vger.kernel.org
Subject: Re: [RFC PATCH 0/7] bcache: md conversion
Date: Fri, 18 May 2012 19:35:11 -0700 [thread overview]
Message-ID: <jp70t0$4ru$2@dough.gmane.org> (raw)
In-Reply-To: 20120511194327.25770.79292.stgit@dwillia2-linux.jf.intel.com
Dan Williams wrote:
> The consensus from LSF was that bcache need not invent a new interface
> when md and dm can both do the job. As mentioned in patch 7 this series
> aims to be a minimal conversion. Other refactoring items like
> deprecating register_lock for mddev->reconfig_mutex are deferred.
>
> This supports assembly of an already established cache array:
>
> mdadm -A /dev/md/bcache /dev/sd[ab]
>
> ...will create the /dev/md/bcache container and a subarray representing
> the cache volume. "Flash-only", or backing-device only volumes were not
> tested. "Create" support and hot-add/hot-remove come later.
>
> Note:
> * When attempting to test with small loopback devices (100MB), assembly
> soft locks in bcache_journal_read(). That hang went away with larger
> devices, so there seems to be minimum component device size that needs
> to be considered in the tooling.
Is there any plan to separate the on-disk layout (per-device headers, etc)
from the logic for the purpose of reuse? I can think of at least one case
where this would be extremely useful: integration in BtrFS.
BtrFS already has its own methods for making sure a group of devices are all
present when the filesystem is mounted, so it doesn't really need the
formatting of the backing device bcache does to prevent it from being
mounted solo. Putting bcache under BtrFS would be silly in the same way as
putting it under a raid array, but bcache can't be put on top of BtrFS.
Logically, in looking at BtrFS' architecture, a cache would likely fit best
at the 'block group' level, which IIUC would be roughly equivalent to the
recommended 'over raid, under lvm' method of using bcache.
prev parent reply other threads:[~2012-05-19 2:35 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-11 19:46 [RFC PATCH 0/7] bcache: md conversion Dan Williams
[not found] ` <20120511194327.25770.79292.stgit-p8uTFz9XbKgaePuBGzJMJzMJUdESFZ8XQQ4Iyu8u01E@public.gmane.org>
2012-05-11 19:46 ` [RFC PATCH 1/7] bcache: compile fix Dan Williams
2012-05-11 19:46 ` [RFC PATCH 2/7] bcache: disable lockdep, enable CONFIG_BCACHE=m Dan Williams
2012-05-11 19:46 ` [RFC PATCH 3/7] bcache: drop select COMPACTION Dan Williams
2012-05-11 19:46 ` [RFC PATCH 4/7] bcache: fix symlink removal Dan Williams
2012-05-11 19:46 ` [RFC PATCH 5/7] bcache: move to drivers/md/ Dan Williams
2012-05-11 19:46 ` [RFC PATCH 6/7] bcache: uplevel allocation of 'cached_dev' and 'cache' Dan Williams
2012-05-11 19:46 ` [RFC PATCH 7/7] md: add bcache personality Dan Williams
2012-05-18 16:52 ` Doug Ledford
2012-05-18 16:57 ` Dan Williams
2012-05-11 19:52 ` [RFC PATCH 0/7] bcache: md conversion Joseph Glanville
2012-05-14 23:15 ` Kent Overstreet
2012-05-19 2:35 ` Alex Elsayed [this message]
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='jp70t0$4ru$2@dough.gmane.org' \
--to=eternaleye+usenet@gmail.com \
--cc=linux-bcache@vger.kernel.org \
--cc=linux-btrfs@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;
as well as URLs for NNTP newsgroup(s).