linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

      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).