From: Neil Brown <neilb@suse.de>
To: Mike Snitzer <snitzer@redhat.com>
Cc: linux-raid@vger.kernel.org, linux-kernel@vger.kernel.org,
dm-devel@redhat.com
Subject: Re: reproducer for DM on MD flush deadlock? (was: Re: [PULL REQUEST] md bug fixes)
Date: Sat, 18 Dec 2010 08:01:58 +1100 [thread overview]
Message-ID: <20101218080158.58e17c21@notabene.brown> (raw)
In-Reply-To: <AANLkTi=H3-GrEZVWzmHyHvyx93HU6x=t_n6TBqCiEwxX@mail.gmail.com>
On Fri, 17 Dec 2010 13:13:47 -0500 Mike Snitzer <snitzer@redhat.com> wrote:
> On Tue, Dec 14, 2010 at 2:22 AM, Neil Brown <neilb@suse.de> wrote:
> >
> >
> > Hi Linus,
> > here are a few bug fixes for md.
> > Some of the patches are actually clean-up rather than bug-fix,
> > but I that make the bugfix simpler to review.
> >
> > Thanks,
> > NeilBrown
> >
> >
> > The following changes since commit 6313e3c21743cc88bb5bd8aa72948ee1e83937b6:
> >
> > Merge branches 'x86-fixes-for-linus', 'perf-fixes-for-linus' and 'sched-fixes-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip (2010-12-08 06:40:59 -0800)
> >
> > are available in the git repository at:
> >
> > git://neil.brown.name/md/ for-linus
> >
> > NeilBrown (5):
> > md: remove handling of flush_pending in md_submit_flush_data
> > md: move code in to submit_flushes.
> > md: fix possible deadlock in handling flush requests.
>
> Hi Neil,
>
> Thanks for fixing this DM on MD flush issue. But my attempts to
> reproduce it have been unsuccessful.
>
> I've tried ext4 w/ barriers to a DM device above a 2 member MD RAID1.
> The DM device has a table with 2 linear targets to the same md0
> device:
>
> # dmsetup table
> multiple_targets: 0 24576 linear 9:0 2048
> multiple_targets: 24576 49152 linear 9:0 26624
>
> No amount of IO with flushes has enabled me to hit a deadlock (in
> md_flush_request, md_write_start, etc).
>
> Do you have a simple reproducer for this issue?
No. I think the issue is very sensitive to the exact placement of the border
between the two dm targets. You need to be able to produce a flush request
that crosses that border.
So to reproduce it I would:
Create an ext4 filesystem of some known size.
Impose some simple easily reproducible load and use e.g. blktrace to gets a
log of the flush requests.
Choose on such request that is larger than a sector and note it's location
Create a DM device of the same size with two targets on md devices where the
first target ends in the middle of where the flush request was
Repeat the above sequence on the dm device. That should result in a flush
request overlapping both targets and thus triggering the issue.
NeilBrown
prev parent reply other threads:[~2010-12-17 21:01 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-17 18:13 reproducer for DM on MD flush deadlock? (was: Re: [PULL REQUEST] md bug fixes) Mike Snitzer
2010-12-17 21:01 ` Neil Brown [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=20101218080158.58e17c21@notabene.brown \
--to=neilb@suse.de \
--cc=dm-devel@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-raid@vger.kernel.org \
--cc=snitzer@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;
as well as URLs for NNTP newsgroup(s).